Spinning Shurikens

Hi everyone

Im trying to make a trigger enchanced spell, wich would create a shuriken to circle around the caster. And because i want the caster to be able to create more than one, im trying to use Vexorians local variable trick.
The trigger looks like this:
Shuriken Shield
Someone please tell me whats wrong?

note: if i make the trigger without the use of the local variables it works, but i have to use them, since i want this spell to be multiinstanceable.

Next time you post a trigger that was copied using the Copy As text feature, use the [trigger] tag instead of [code]

You won't get smooth movement by 0.10 wait probably :) hmm

I am not sure how W3 works with angles greater than 360*. In your trigger it will go until 1500 degrees.
And you should nullify unit variables too.

And are you sure you gave Premanent Immolation to shuriken unit?

ok, first, im not sure how to nullify variables, could you give me the costum script for that? And yes i did give the Premanent Immolation (Shuriken) to the shuriken. But how come if the degress is over 360, then you say it might not work, but it does with the non local variables?
I changed the code to this now:
now it chrashes the game?!

NOTE: the spell still works just perfect if i remove my costum scripts, so i think they are the problem.

EDIT: After reading Vexorians post again, i found out the problem myself. The problem is that loops dont support local variables!
But what else could i do to make his ability work? And be multi instanceable

Loops should support local variables, since they don't create a seperate function for the loop actions, like the if-then-else does for conditions. However, you have other problems, you wait inside an Integer A loop, which is bad because Integer a is global and can be altered by other triggers during the wait. Also, warcraft is incapable of waiting for such a short time, the wait action is inaccurate, for such small wait, you must use timers or triggers with periodic events. That's how I would do this spell, I would just put all the shuriken into a unit group and then, in a periodic trigger, move all the shuriken around the hero.

thanks alot Anitarf, i might do that. Was thinking something like that aswell. But one problem. How can i move the units around the caster, in another trigger, without assigning the caster to a variable. Because if i do that, it wouldnt be multi instanceable?

Have a unit array and a unit group array. Into the unit array go the heroes, and into the unit groups go the shuriken groups. When creating the shuriken, you just need to figure out into which group it goes (the one that has the array index under which the hero is stored in the unit array).

For determining array indexes, it depends on what degree of multi-instanceability you want to have; if it's one hero per player, you can easily use player numbers as the array indexes when storing the hero or the shuriken he creates. For full multi-instanceablility, you have to loop through the array until you find an empty index to store the hero, and afterwards loop through it every time a shuriken is cast until you find the index under which the casting hero is stored.

i only think im gonna need one hero per player, so i guess that would work. Thanks again.

the local variable trick only works for one local vaiable at a time for some odd reasons.

locals work well in loops. Not in Unit Group - Pick every unit in group / also pick every item/destructable/player thise use another function

I forgot that blizzard tells in triggers that Waits shouldn't be used within loops.

The gamr already handles shit like that, if you offset it by 390 for example, it will go 390 degrees from the static point of 0 degrees, so on that front he needs not worry.

When using that trick for 2 or more locals it causes undefined code apparentally and many weird things occur

Oh, well, thanks alot for the replies everyone. I made it work now.