deletestring (serverside)

[Com013]

commands.rtf
deletestring list,index; deletes string list entry at position 'index'

It is not working correctly. I said it should delete index 1, but it removed index 0.
Now I have to use this:

NPC Code:

---

      //    deletestring clientr.hats,hatnum;



          setstring temp,;

          for (i=0; i<sarraylen(clientr.hats); i++) {

            if (i != hatnum) addstring temp,#I(clientr.hats,i);

          }

          setstring clientr.hats,#s(temp);

          setstring temp,;

---

I don't know whether it is working clientside/offline.

[Tyhm]

...you're starting at the highest index and working down, right?
Otherwise you're gonna skip every other index...
Delete 0
Index 1 becomes Index 0, 2 becomes 1, etc.
Delete 1 (formerly index 2...)
Delete 2 (formerly index 4...)

[Com013]

Originally posted by Tyhm
...you're starting at the highest index and working down, right?
Otherwise you're gonna skip every other index...
Delete 0
Index 1 becomes Index 0, 2 becomes 1, etc.
Delete 1 (formerly index 2...)
Delete 2 (formerly index 4...)

Sorry, what are you talking about?
Do you mean that I used deletestring the wrong way? I didn't even use it in a loop.
Are you talking about the script I am using as replacement? I just copied every thing else into another string.

[Admins]

I am using deletestring for many things and its working, can you post some code example?

[Com013]

I don't want to publish it here.
It is on Classic Developement, Weapons, -System/Trading.

I have put "//" in front of the deletestring command.
The following six lines are what fixed it. (I rescripted the deletestring command).

[CheeToS2]

Originally posted by Com013
I don't want to publish it here.
It is on Classic Developement, Weapons, -System/Trading.

I have put "//" in front of the deletestring command.
The following six lines are what fixed it. (I rescripted the deletestring command).

nice 6 lines :o

[Com013]

Originally posted by CheeToS2


nice 6 lines :o

What do you mean? I published those six lines in my start post of this thread.

[Admins]

Arg you are using 'temp' ? that will send a lot of data to the player x-x And it still doesnt tell me what should be wrong. A code example of the bug would be good

[CheeToS2]

Originally posted by Com013

What do you mean? I published those six lines in my start post of this thread.

Oh.. you said the following 6 lines.. but they didn't follow that text.. so

[Loriel]

Originally posted by Stefan
Arg you are using 'temp' ? that will send a lot of data to the player x-x

Well, you could always code the server so it does not send data to the client which it cannot read/edit anyways.

[Com013]

Originally posted by Stefan
Arg you are using 'temp' ? that will send a lot of data to the player x-x And it still doesnt tell me what should be wrong. A code example of the bug would be good

Ok, now I am using this.temp, but we are going off-topic here.
Tyhm told me sometime that only LATs are allowed to see the scripts used on Classic, so I won't post it here, but, like I wrote before, you can look at it on Classic Developement, it's in the -System/Trading weapon.
(I tried sending it to you using a PM, but it was 10 times too huge for a forum PM).

However, this is a very short version of it:

NPC Code:

---

if (actionserverside) {

  hatnum = strtofloat(#p(0));

  putnpc2 playerx,playery+2,{join hats;};

  with (npcs[npcscount-1]) {

    this.hat = strtofloat(#I(clientr.hats,hatnum));

    deletestring clientr.hats,hatnum; // <- always delets #0

    callnpc -1,updategani;

  }

}

//#CLIENTSIDE

if (weaponfired) triggeraction 0,0,serverside,DeleteTest,2;

---

Originally posted by CheeToS2
Oh.. you said the following 6 lines.. but they didn't follow that text

No, I meant the following six lines in the script and not in my post.

[Loriel]

Originally posted by Com013

NPC Code:

---

if (actionserverside) {

  hatnum = strtofloat(#p(0));

  putnpc2 playerx,playery+2,{join hats;};

  with (npcs[npcscount-1]) {

    this.hat = strtofloat(#I(clientr.hats,hatnum));

    deletestring clientr.hats,hatnum; // <- always delets #0

    callnpc -1,updategani;

  }

}

//#CLIENTSIDE

if (weaponfired) triggeraction 0,0,serverside,DeleteTest,2;

---

Variables like those are local to a certain NPC.

[Com013]

Originally posted by Loriel

Variables like those are local to a certain NPC.

They don't start with this., so NO!

And:

NPC Code:

---

  with (npcs[npcscount-1]) {

    this.hat = strtofloat(#I(clientr.hats,hatnum)); // <- this works

    deletestring clientr.hats,hatnum; // <- this doesn't

  }

---

[Kaimetsu]

Personally I just wouldn't use a string array, though it depends on how many of each hat you want to allow people to have. If you use ascii() and #K() then you can compact numbers over a fairly large range into a single byte, and then just use a normal string with one character per hat.

[Admins]

Variables like hatnum are local, belonging to the current script, so you can use hatnum like you have written, and it works. You have problems when calling functions of joined classes, because those don't know of the local variables of the other script.