Problem With Destroy()

[Struggler]

I'm havin troubles getting this script to delete itself after the player touchs it.. Can someone tell me whats worng with it?

PHP Code:

function onCreated(){
  setshape( 1, 32, 32);
  setimg( "beer.png");
}
function onActionServerside(){
  if(params[0] == "destroy"){
    destroy();
  }
}
//#CLIENTSIDE
function onPlayerTouchsme(){
  setimg( "beer.png");
  setshape( 1, 32, 32);
  player.hearts += 5;
  player.bombs += 5;
  player.darts += 5;
  triggerserver("gui",this.name,"destroy");
  destroy();
} 


[Stryke]

triggerserver(..) only works on weapons.

Use this:

PHP Code:

triggeraction(this.x + .5, this.y + .5, "Trigger"); 


And to receive:

PHP Code:

function onCreated() {
  setshape(1, 32, 32);
}

function onActionTrigger() {
  this.destroy();
} 


[Struggler]

okay, now i gots this, still doesnt work:

PHP Code:

function onCreated(){
  setimg( "beer.png");
  setshape(1, 32, 32);
}
function onActionTrigger() { 
  destroy(); 
}
//#CLIENTSIDE
function onPlayerTouchsme(){
  setimg( "beer.png");
  setshape( 1, 32, 32);
  player.hearts += 5;
  player.bombs += 5;
  player.darts += 5;
  triggeraction(this.x + .5, this.y + .5, "Trigger");  
} 


[salesman]

Try adding a null parameter to the trigger

triggeraction(this.x + .5, this.y + .5, "Trigger", null);

I'm not sure why, but it seems to work

[Tigairius]

Originally Posted by salesman

Try adding a null parameter to the trigger

triggeraction(this.x + .5, this.y + .5, "Trigger", null);

I'm not sure why, but it seems to work

Yes, triggeraction expects 4 params, in the new version it's only 3

[cbk1994]

I assume this is in a level NPC? In that case, you should just use

PHP Code:

function onCreated(){
  setshape( 1, 32, 32);
  setimg( "beer.png");
}
//#CLIENTSIDE
function onPlayerTouchsme(){
  setimg( "beer.png");
  setshape( 1, 32, 32);
  player.hearts += 5;
  player.bombs += 5;
  player.darts += 5;
  hide();
} 


[Gambet]

Originally Posted by cbk1994

I assume this is in a level NPC? In that case, you should just use

PHP Code:

function onCreated(){
  setshape( 1, 32, 32);
  setimg( "beer.png");
}
//#CLIENTSIDE
function onPlayerTouchsme(){
  setimg( "beer.png");
  setshape( 1, 32, 32);
  player.hearts += 5;
  player.bombs += 5;
  player.darts += 5;
  hide();
} 


That would just hide the NPC but the NPC itself would still be there. If you use destroy, the NPC itself is destroyed.

It all depends on how he wants to use it, but hide is not the same as destroy.

[Pelikano]

Using hide() is not very good, since it still exists, making it just use CPU it shouldn't.

[zokemon]

Originally Posted by Pelikano

Using hide() is not very good, since it still exists, making it just use CPU it shouldn't.

Well, memory actually. CPU would only be used if the script was actually doing something (it might use some minimal CPU but I mean MINIMAL)

[Pelikano]

Originally Posted by zokemon

Well, memory actually. CPU would only be used if the script was actually doing something (it might use some minimal CPU but I mean MINIMAL)

whatever, it just uses something it shouldn't

[cbk1994]

Originally Posted by Gambet

That would just hide the NPC but the NPC itself would still be there. If you use destroy, the NPC itself is destroyed.

It all depends on how he wants to use it, but hide is not the same as destroy.

Destroying would just make it come back when they reentered the level if it's done clientside, like I think he was trying to.

[Chompy]

Originally Posted by cbk1994

Destroying would just make it come back when they reentered the level if it's done clientside, like I think he was trying to.

He's using a class.

[Pelikano]

Originally Posted by cbk1994

Destroying would just make it come back when they reentered the level if it's done clientside, like I think he was trying to.

Also if you tried to read his post, atleast tried, you'd see that he's doing it serverside.