Jump to content

Script commands for offline characters


Euphy

Recommended Posts


  • Group:  Members
  • Topic Count:  72
  • Topics Per Day:  0.02
  • Content Count:  2997
  • Reputation:   1130
  • Joined:  05/27/12
  • Last Seen:  

If it isn't too much work, it'd be useful to have a few more character commands that work regardless of online/offline status. It's both annoying and inefficient (script-wise) to keep checking whether to attachrid or do an SQL query, since attachrid doesn't work for offline characters (obviously) and SQL queries don't update immediately for online characters. And when setting character/account variables, there's the additional check whether to do an SQL update or insertion. Altogether it'd just be a lot more convenient to have these commands, or something similar:

  • Like setd, set a character or account variable for a specified character/account ID. This would work without attachrid(), and regardless of online/offline status. The variable prefix should automatically determine whether the supplied ID is character or account.
    * setc <character ID>,"<variable name>",<value>;
    * setc <account ID>,"<variable name>",<value>;


  • Like getd, and same as above.
    * getc(<character ID>,"<variable name>")
    * getc(<account ID>,"<variable name>")


  • Also, extend rid2name() and getcharid() to work for offline characters.

Opinions?

  • Upvote 7
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  75
  • Topics Per Day:  0.02
  • Content Count:  2223
  • Reputation:   593
  • Joined:  10/26/11
  • Last Seen:  

I think these would be very useful!

But would it require more char-server code too? because character saving happens at the char-server level.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  1
  • Topics Per Day:  0.00
  • Content Count:  71
  • Reputation:   25
  • Joined:  11/23/11
  • Last Seen:  

getcharid is some of the ways to know if a character is online or not, it would make problems with some scripts.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  72
  • Topics Per Day:  0.02
  • Content Count:  2997
  • Reputation:   1130
  • Joined:  05/27/12
  • Last Seen:  

@Omnipotent: That's what we have this command for.

*isloggedin(<account id>{,<char id>})

(I'm aware a few people used getcharid() to check online status, but that shouldn't be the reason offline implementation is held back.)

  • Upvote 1
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  14
  • Topics Per Day:  0.00
  • Content Count:  351
  • Reputation:   52
  • Joined:  11/15/11
  • Last Seen:  

I support this.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  11
  • Topics Per Day:  0.00
  • Content Count:  427
  • Reputation:   123
  • Joined:  11/17/11
  • Last Seen:  

Since we dropped TXT, I think it is not too hard to make it possible to modify offline player's variables, or read them.

In fact, you can already do that with query_sql, however built in support would be much more elegant of course.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  72
  • Topics Per Day:  0.02
  • Content Count:  2997
  • Reputation:   1130
  • Joined:  05/27/12
  • Last Seen:  

@xazax: The problem with doing SQL queries is that variables aren't updated right away. There's always some delay, and from what I've tested it ranges anywhere from a few seconds to minutes.

I didn't realize we had an "@set" command, though. Wouldn't at least half the code for these commands be identical to the atcommand?

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  9
  • Topics Per Day:  0.00
  • Content Count:  379
  • Reputation:   304
  • Joined:  11/10/11
  • Last Seen:  

getcharid is some of the ways to know if a character is online or not, it would make problems with some scripts.

They should use playerattached(). In memory getcharid() throw an error if no player attached to the script

No opinion about an offline mode.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  1
  • Topics Per Day:  0.00
  • Content Count:  71
  • Reputation:   25
  • Joined:  11/23/11
  • Last Seen:  

I'm supporting this too, I just wanted to mark out that detail.

@KeyWorld:

But I mean getting the status, and if he's connected the id of a character who is not attached at that moment to the script.

>if(getcharid(0,"john")) he's online, so attachrid and stuff.

@Euphy:

I'm agree on this, only was a detail for some old scripts, but the change required is minimal.

Edited by Omnipotent
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  6
  • Topics Per Day:  0.00
  • Content Count:  112
  • Reputation:   89
  • Joined:  11/12/11
  • Last Seen:  

I like the idea.

I want to start working on it after work today.

I'm probably going to make it so if the command is used and the offline character is actually online, it will switch to the regular set routines. It'll be like a safer version of set.

Also, I don't think character ID and account ID as an parameter in the same command would be a feasible idea. A sufficiently large enough character database could have IDs that are the same numbers as the account IDs, and that would mess up the logic. I would probably have to separate those into separate commands.

Edited by sketchyphoenix
  • Upvote 2
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  72
  • Topics Per Day:  0.02
  • Content Count:  2997
  • Reputation:   1130
  • Joined:  05/27/12
  • Last Seen:  

You're amazing. Thank you~

Couldn't you only use one command, though? Because variable types are defined by prefixes, can't you read the variable type first before processing the ID? I really don't know C at all, so correct me if I'm wrong.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  6
  • Topics Per Day:  0.00
  • Content Count:  112
  • Reputation:   89
  • Joined:  11/12/11
  • Last Seen:  

The problem is that only the parser in the script engine recognizes variable affixes and once it leaves the txt file and goes into memory, those get stripped from the variable name before being stored..if i remember correctly. If not, then oh well. That's no impact on anything, I still have to set aside several commands to differentiate between char ID and account ID - and if the server is assuming these players are offline, it can't assume that the calling NPC has a player attached, so it needs that information beforehand.

So I'll need 4 commands. One that alters an entry for a character variable.. One that alters an entry for an account variable. And 2 more to retrieve those. If the character or account is logged in, then the command will simply act as set would, instead of altering the database directly.

  • Upvote 1
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  72
  • Topics Per Day:  0.02
  • Content Count:  2997
  • Reputation:   1130
  • Joined:  05/27/12
  • Last Seen:  

Alright, that makes sense. At least it'll prevent any confusion. Thanks a lot. c:

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  10
  • Topics Per Day:  0.00
  • Content Count:  392
  • Reputation:   47
  • Joined:  11/18/11
  • Last Seen:  

+1 great idea.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  169
  • Topics Per Day:  0.04
  • Content Count:  1260
  • Reputation:   750
  • Joined:  11/19/11
  • Last Seen:  

The problem is that only the parser in the script engine recognizes variable affixes and once it leaves the txt file and goes into memory, those get stripped from the variable name before being stored..if i remember correctly.

you should be able to identify both smoothly without the need of another 2 commands imo, you can take a look on script.c::set_reg() for reference

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  9
  • Topics Per Day:  0.00
  • Content Count:  303
  • Reputation:   101
  • Joined:  11/13/11
  • Last Seen:  

This can slow down the script VM(mainly for getc), the script execution will need to be suspended and wait a response from char-server, so this command should be used wisely.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  6
  • Topics Per Day:  0.00
  • Content Count:  112
  • Reputation:   89
  • Joined:  11/12/11
  • Last Seen:  

Finally started on it today.

Saving is done. Loading in progress.

  • Upvote 1
Link to comment
Share on other sites

  • 2 years later...

  • Group:  Members
  • Topic Count:  53
  • Topics Per Day:  0.01
  • Content Count:  411
  • Reputation:   259
  • Joined:  04/25/12
  • Last Seen:  

This was updated? I tried to make a getcharvariable but I have problem with the sql delay to save on global reg.

 

@edit

 

Never mind, i maked this with script

 

https://rathena.org/board/topic/101748-function-getcharvar/

Edited by Zell
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...