Jump to content

  •  

Photo

Script commands for offline characters

source

  • Please log in to reply
16 replies to this topic

#1 Euphy

Euphy

    Tyranitar

  • Managers
  • 2740 posts

User's Awards

     

Posted 09 June 2012 - 12:34 PM

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?
  • 4

#2 Brian

Brian

    Forum Administrator

  • Administrators
  • 2075 posts
  • Gender: Male
  • Location: California

User's Awards

        

Posted 09 June 2012 - 02:01 PM

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.
  • 0

#3 Omnipotent

Omnipotent

    Drops

  • Members
  • PipPip
  • 46 posts
  • Gender: Male
  • Location: Heaven.

Posted 10 June 2012 - 12:35 PM

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

#4 Euphy

Euphy

    Tyranitar

  • Managers
  • 2740 posts

User's Awards

     

Posted 10 June 2012 - 01:58 PM

@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.)
  • 1

#5 Nameless2you

Nameless2you

    Hurling

  • Members
  • PipPipPipPipPip
  • 294 posts
  • Gender: Not Telling
  • Location: localhost

Posted 11 June 2012 - 02:11 PM

I support this.
  • 0

#6 xazax

xazax

    Stapo

  • Community Contributors
  • 284 posts
  • Gender: Male
  • Location: Hungary

Posted 14 June 2012 - 08:49 AM

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.
  • 0

#7 Euphy

Euphy

    Tyranitar

  • Managers
  • 2740 posts

User's Awards

     

Posted 14 June 2012 - 09:15 AM

@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?
  • 0

#8 KeyWorld

KeyWorld

    Stapo

  • Community Contributors
  • 378 posts
  • Gender: Male
  • Location: Paris, France
  • Github:vthibault

User's Awards

     

Posted 14 June 2012 - 09:18 AM

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.
  • 0

#9 Omnipotent

Omnipotent

    Drops

  • Members
  • PipPip
  • 46 posts
  • Gender: Male
  • Location: Heaven.

Posted 18 June 2012 - 08:25 PM

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, 18 June 2012 - 08:26 PM.

  • 0

#10 sketchyphoenix

sketchyphoenix

    ┬─┬ノ( º _ ºノ)

  • Community Contributors
  • 107 posts
  • Gender: Male

Posted 20 June 2012 - 12:27 PM

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, 20 June 2012 - 12:41 PM.

  • 2

#11 Euphy

Euphy

    Tyranitar

  • Managers
  • 2740 posts

User's Awards

     

Posted 20 June 2012 - 01:46 PM

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.
  • 0

#12 sketchyphoenix

sketchyphoenix

    ┬─┬ノ( º _ ºノ)

  • Community Contributors
  • 107 posts
  • Gender: Male

Posted 20 June 2012 - 02:25 PM

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.
  • 1

#13 Euphy

Euphy

    Tyranitar

  • Managers
  • 2740 posts

User's Awards

     

Posted 20 June 2012 - 02:42 PM

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

#14 Imp

Imp

    Martin

  • Community Contributors
  • 295 posts
  • Gender: Male
  • Location: Mexico

Posted 20 June 2012 - 04:12 PM

+1 great idea.
  • 0

#15 Ind

Ind

    Hercules

  • Community Contributors
  • 847 posts
  • Gender: Not Telling

User's Awards

     

Posted 20 June 2012 - 05:42 PM

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
  • 0

#16 GreenBox

GreenBox

    bogs pls

  • Community Contributors
  • 194 posts
  • Gender: Male
  • Location: Brazil

Posted 22 June 2012 - 11:57 AM

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.
  • 0

#17 sketchyphoenix

sketchyphoenix

    ┬─┬ノ( º _ ºノ)

  • Community Contributors
  • 107 posts
  • Gender: Male

Posted 25 June 2012 - 05:57 PM

Finally started on it today.

Saving is done. Loading in progress.
  • 1





Also tagged with one or more of these keywords: source

0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users

determination Hosted by SourceForge.net