removeweapon()

[Chompy]

Well, there's something weird about it..

lets see

PHP Code:

function onActionPlayerOnline() {
  temp.weps = player.weapons;
  temp.i = 0;
  for(temp.wep : weps) {
    echo(++ i SPC wep.name);
  }
} 


(In this case it echoed 28 weapons)

This echoes all the player's weapons on login. However.. If I try to remove all weapons on login, something weird happens..

PHP Code:

function onActionPlayerOnline() {
  temp.weps = player.weapons;
  temp.i = 0;
  for(temp.wep : weps) {
    removeweapon(wep.name);
    echo(++ i SPC wep.name);
  }
} 


It only echoes 14 weapons and removes those that were echoed..

So to get all weapons removed you have to reconnect like.. >5 times o.o;;

Why is this? Anyone knows?

[cbk1994]

Happened on Utopia too, is very annoying.

[DustyPorViva]

There are a lot of weapons that are hidden that are given to the player by the client... stuff like the server list, player list, client RC. I dunno if this is what you're talking about, I'm having a hard time understanding your post.

I don't think they can be removed.

Also, your second loop isn't accurate. You should run a loop that removes weapons, then run a separate loop that echos the weapons.

[Chompy]

Originally Posted by DustyPorViva

There are a lot of weapons that are hidden that are given to the player by the client... stuff like the server list, player list, client RC. I dunno if this is what you're talking about, I'm having a hard time understanding your post.

I don't think they can be removed.

Also, your second loop isn't accurate. You should run a loop that removes weapons, then run a separate loop that echos the weapons.

I want to remove all weapons the player has in "Weapons" when I open their attributes :o (Oh, and edited post to add a missing word xD)

[Admins]

When you call player.removeweapon() then its modifying player.weapons[], that's why the loop is only removing 14. Possible solution would be

PHP Code:

for (i=player.weapons.size()-1; i>=0; i--)
  player.removeweapon(player.weapons[i].name); 


(might be confusing but player.weapons is a special intelligent object which lets you print the player weapon list as string even when there are weapons which don't really exist, and by using weapons[] you can still directly access the weapon npc)

[xXziroXx]

I do it like this on Mythic, work's like a charm:

PHP Code:

  temp.wsize = player.weapons.size();
  for (temp.i = 0; temp.i < temp.wsize; temp.i++) {
    if (player.weapons[i].name != name) {
      removeweapon(player.weapons[i].name);
      if (temp.wsize != player.weapons.size()) {
        i--;
        temp.wsize = player.weapons.size();
      }
    }
  } 


[DrakilorP2P]

Wouldn't this work?

PHP Code:

for (temp.i=0; temp.i<player.weapons.size(); temp.i++)
  player.removeweapon(player.weapons[0].name); 


[Inverness]

Originally Posted by DrakilorP2P

Wouldn't this work?

PHP Code:

for (temp.i=0; temp.i<player.weapons.size(); temp.i++)
  player.removeweapon(player.weapons[0].name); 


Skipping over Stefan's posts usually isn't a smart thing to do. If you don't understand it then just think really hard about how arrays work.

[DrakilorP2P]

Originally Posted by Inverness

Skipping over Stefan's posts usually isn't a smart thing to do. If you don't understand it then just think really hard about how arrays work.

Just noticed my previous post is a really misleading way of deleting half the list.

Anyway, similar method, different language but guaranteed to work:

PHP Code:

>>> foo = [1, "bar", False, ["baz", "crux"], 9]
>>> cruux = len(foo)
>>> for i in range(cruux):
...     print "Foo: " + str(foo[0])
...     foo.remove(foo[0])
... 
Foo: 1
Foo: bar
Foo: False
Foo: ['baz', 'crux']
Foo: 9
>>> foo
[]
>>> 


All assuming the precise functioning of this "special intelligent object" did not elude me.

[Inverness]

Originally Posted by DrakilorP2P

Anyway, similar method, different language but guaranteed to work:

No it wouldn't.

If you want to delete stuff in an array part by part you need to start at the end so when you delete the element and the array shrinks in size you're not missing anything. When you're deleting a part from an array everything after it is decreasing in index so the for loop misses the element that replaced the one it just deleted while it moves on to the next highest index.

Also in your Python code the 'cruux' variable is unnecessary. You're taking the length from the 'foo' array and creating a new array from it but not actually using variable 'i', so you should just do for i in foo:

Edit:

Originally Posted by DrakilorP2P

Just noticed my previous post is a really misleading way of deleting half the list.

It appears I was the one who was mislead when reading your post. Since I just noticed it was only deleting index 0 and not index i. If done like that it should work fine I guess.

[Dan]

Drakilors method would work too. He's using [0], not [temp.i], so it would just delete all first weapons in the list untill nothing's left.

[zokemon]

Originally Posted by Dan

Drakilors method would work too. He's using [0], not [temp.i], so it would just delete all first weapons in the list untill nothing's left.

But like Stefan said, player.weapons is a special object in that it can give you a string list which could include weapons that don't actually exist. If you try to removeweapon() a weapon in that list that isn't an actual weapon, the weapon won't be removed from the list thus causing an infinite loop.

[Dan]

Originally Posted by zokemon

But like Stefan said, player.weapons is a special object in that it can give you a string list which could include weapons that don't actually exist. If you try to removeweapon() a weapon in that list that isn't an actual weapon, the weapon won't be removed from the list thus causing an infinite loop.

Indeed you're right. Perhaps there should come some built in check to see if a weapon in someone's weaponlist still exists.

[Inverness]

Originally Posted by Dan

Indeed you're right. Perhaps there should come some built in check to see if a weapon in someone's weaponlist still exists.

player.weapons.index() ?

[zokemon]

Originally Posted by Inverness

player.weapons.index() ?

Fairly costly to be doing array searching on each iteration.