SphereCommunity

how can i add to the NEXT open spot?

GSLAIN01 - GSLAIN250




it keeps adding to GSLAIN0


thx

If Endif
For Endfor

english please?


top part works, bottom does not

always overwrites to GSLAIN0

[EVENTS e_cao_cao]
on=@deathcorpse
ref1=<attacker.max>//who ever did most damage
for x 1 250
IF (STRCMPI("<ref1.tag.gslain<LOCAL.X>>","Cao Cao")==0)
ref1.sysmessage @0482 You have already defeated General Cao Cao...
else
if <ref1.isplayer>//if max damage doner is player.
ref1.tag.gslain<dlocal._for> Cao Cao
ref1.sysmessage @0481 Congratulations! You have defeated General Cao Cao!
endif
ENDFOR -----------------> close if before for

i did that already and its the same...

still saves to gslain0

if glsain01
and gslain02 are taken, put it at gslain03
ect...

get what im saying?



i tried this and it keeps saving to GSLAIN1

i want to to find the next empty one

try to use elseif

nope...

this added it 250 times =/

what a mess... try this.

[EVENTS e_cao_cao]
on=@deathcorpse
ref1=<attacker.max>
for x 1 250
IF (STRCMP("<ref1.tag.gslain<dLOCAL.X>>","Cao Cao")==0)
ref1.sysmessage @0482 You have already defeated General Cao Cao...
return 1
elseif (<ref1.isplayer> && <ISEMPTY <ref1.tag.gslain<dLOCAL.X>>>)
ref1.tag.gslain<dLOCAL.X> Cao Cao
ref1.sysmessage @0481 Congratulations! You have defeated General Cao Cao!
return 1
endif
ENDFOR

ISEMPTY!!!!???

nice...

---

BEN IS MY HERO... again...

thank you

you must use "return 0/1" to stop the loop, otherwise it will continue looping until 250 and will set 250 tags
and also use the right return 0/1, because on @DeathCorpse the return 1 will prevent the corpse creation, and return 0 will allow the corpse creation

and LOCAL.X (hex) is not the same as dLOCAL.X (dec)
tag.gslain<LOCAL.X> = tag.gslain01
tag.gslain<dLOCAL.X> = tag.gslain1

so if you set the value on tag.gslain1, you wont find the tag value if you search it on tag.gslain01

Ben already post a working example with these things fixed

PS: just a code optimization tip, avoid make checks inside loops which doesn't really need to be inside the loop. If you check the <REF1.ISPLAYER> value inside the loop, sphere will try to get the <REF1.ISPLAYER> result 1x ~ 250x, but you just need check it 1x and not 250x. So you can check it just a single time before start the loop