Engine crashes on .delete(); function.

#1 Your deleting an object while the engine is using it... your deleting AlienRifle while Torque is calling your onCollision call. Instead do this...

function AlienRifle::OnCollision(%this, %col, %obj)
{
if(%col = %obj.client.player)
{
%this.schedule( 10, delete );
$bullets++;
}
}

... which will delay the deletion of the rifle till the next tick.

#2 Hm,
i think deleting an object inside a script OnEvent method via %this.delet() is okay,
so if this fixes things something else may be weird..

#3 For some reason, it still crashes, thanks for the idea anyway. Do you think it may be something else?

#4 What is the assert? There are numerous asserts in SimObject::deleteObject, which is what you are calling in both cases.

#5 Well even if I use AlienRifle.delete(); the engine still crashes. I'm not sure what the assert is. Here is the whole code:

datablock ItemData(AlienRifle)
{
category = "Weapons";
shapeFile = "~/data/shapes/stuff/AlienRifle.dts";
mass = 2;
friction = 2;
};

function AlienRifle::OnCollision(%this, %col, %obj)
{
if(%col = %obj.client.player)
{
%this.delete();
$bullets++;
}
}

... that's it.

#6 Youre deleting the datablock. is that really what you want to do?

#7 ^ What he said.

#8 Oh...wait... no I don't. How would I delete just the object that I collide with?

#9 I'm not really sure but I'm guessing you want to pass the object along as a parameter in OnCollision(). Maybe %obj is that object already?

#10 Firstly, your OnCollison callback has a logic error. '=' is the assignment operator, so your if block will always evaluate to true unless the value of %obj.client.player is 0, or "".

Next, the function signature for the OnCollision callback is:

OnCollision( %this, %obj, %col )

%obj will be the object which has collided with %col, which has datablock %this. Knowing this, your function should look like the following:

function AlienRifle::OnCollision( %this, %obj, %col )
{
   if( %obj.client.player /* has a rifle already [your code does not do this check] */ )
   {
      %col.delete();
      $bullets++;
   }
}

That being said, you don't want to implement this functionality this way at all. Since this code gets executed on the server, $bullets will be constantly overwritten any time any player collides with an alien rifle. I highly recomend looking at 'starter.fps' and the scripts in 'server/scripts'.

#11 Thanks for the help. The code works perfect, and I understand what those mean now.

#12 Cool, glad it worked for you!