[Halt]

When I call ArtilleryExplosionTimer more than once in the same block it only executes the timer once. I don't understand why?

	CreateDataTimer(RandomTime, ArtilleryExplosionTimer, datapack);
	CreateDataTimer(RandomTime2, ArtilleryExplosionTimer, datapack);

I only get one explosion output. Which is whats in ArtilleryExplosionTimer, an explosion.

[Chaosxk]

Your probably getting an error in your logs.

CreateDataTimer will automatically close the Datapack when finishing, so the 2nd CreateDataTimer Datapack is no longer valid.

Edit: forgot it createdatatimer creates a new datapack, read below...

[Halt]

So each CreateDataTimer needs its own datapack?

Or can I reuse a datapack?

[WildCard65]

CreateDataTimer creates the datapack for you, all you need to do is fill it with data.

[Halt]

Yes but say I want different data each time I use CreateDataTimer. The timer is creating explosions and I want the explosions at random vectors. The datapack only holds one vector all the explosions will hit in the same place.

[WildCard65]

Originally Posted by Halt Yes but say I want different data each time I use CreateDataTimer. The timer is creating explosions and I want the explosions at random vectors. The datapack only holds one vector all the explosions will hit in the same place.

Everytime you call "CreateDataTimer", it creates a new (and empty) DataPack handle that you must fill (after the execution of the stock), not only that, but the DataPack the stock creates is also passed as the value to the "data" param of "CreateTimer" as well as the flag for auto-closing data handles is OR'd into your flags.

Edit: To answer your question, your more then likely erroring on reading an empty DataPack.

[Halt]

I'm literally using exactly what the guys on AlliedModders told me to use, and it compiles fine. No error logs.

Spoiler

 //explosion command
public Action:ArtillaryStrike(Client, Args)
{
	float RandomTime = GetRandomFloat(3.0, 8.5);

	float PlayerAimVector[3];
	Handle datapack;
	GetAimPos(Client, PlayerAimVector);
	CreateDataTimer(RandomTime, ArtilleryExplosionTimer, datapack);
	CreateDataTimer(RandomTime, ArtilleryExplosionTimer, datapack); //Dilvers entity to the same vector as the first timer
	CreateTimer(RandomTime, DistanceSounds);
	WritePackFloat(datapack, PlayerAimVector[0]);
	WritePackFloat(datapack, PlayerAimVector[1]);
	WritePackFloat(datapack, PlayerAimVector[2]);
	return Plugin_Handled;
}

//Where explosion comes from
public Action:ArtilleryExplosionTimer(Handle:timer, any:datapack)
{
	//ResetPack(datapack);
	float ArtilleryExplosionVector[3];
	ArtilleryExplosionVector[0] = ReadPackFloat(datapack);
	ArtilleryExplosionVector[1] = ReadPackFloat(datapack);
	ArtilleryExplosionVector[2] = ReadPackFloat(datapack);
	for (int i = 0; i < 2; i++)
	{
		ArtilleryExplosionVector[i] += GetRandomFloat(-250.0, 250.0);
	}
	
	new Explosion = CreateEntityByName("tf_generic_bomb");
	DispatchKeyValue(Explosion, "explode_particle", "fireSmokeExplosion_trackb");
	DispatchKeyValue(Explosion, "sound", "weapons/explode1.wav") //Far sound b4 impact air_burster_explode1.wav
	DispatchSpawn(Explosion);
	TeleportEntity(Explosion, ArtilleryExplosionVector, NULL_VECTOR, NULL_VECTOR);
	AcceptEntityInput(Explosion, "detonate");
	CloseHandle(datapack);
}

[Miu]

errors when you RUN IT on the SERVER not when u compile it

[Halt]

This is what I found in my error logs.

Spoiler

L 05/24/2017 - 12:47:55: [SM] Plugin "artyplugin.smx" encountered error 23: Native detected error
L 05/24/2017 - 12:47:55: [SM] Invalid data handle a7f800cc (error 3) passed during timer end with TIMER_DATA_HNDL_CLOSE
L 05/24/2017 - 12:47:55: [SM] Unable to call function "ArtilleryExplosionTimer" due to above error(s).
L 05/24/2017 - 12:47:56: [SM] Exception reported: DataPack operation is out of bounds.
L 05/24/2017 - 12:47:56: [SM] Blaming: artyplugin.smx
L 05/24/2017 - 12:47:56: [SM] Call stack trace:
L 05/24/2017 - 12:47:56: [SM] [0] ReadPackFloat
L 05/24/2017 - 12:47:56: [SM] [1] Line 187, C:\Users\Crusty03\Desktop\Dev\Programming\Sou rceMod 1.8\scripting\artyplugin.sp::ArtilleryExplosi onTimer

[WildCard65]

Originally Posted by WildCard65 Edit: To answer your question, your more then likely erroring on reading an empty DataPack.

Yeap, definitely your problem, I repeat, CreateDatatTimer CREATES A DataPack HANDLE FOR YOU TO FILL!

example:

PHP Code:

DataPack myPack;
CreateDataTimer(5.0, MyTimer, myPack);
myPack.WriteFloat(5.0); // Write a float to the datapack the timer WILL receive.