Illusion Strike

[Moderator]

02:13, 1st Dec 2009
TriggerHappy:

The coding is horrible. You could have used a loop and it leaks a lot.

The spell is too simple to be approved anyways.

 

[kola]

your spell isnt in MUI neither MPI..
also.. u use Waits and u setted them to 0.05 sec... minimum a wait can do is 0.27 sec so ur 0.05 sec waits will do 0.27 sec.
You also leak points.

Go read some tuts and remake ur spell... this wont be approved for now.

 

[Maker]

• Unit - Create 1 IllusionDummy for (Owner of Illusion_Caster[1]) at ((Position of Illusion_Target[1]) offset by 300.00 towards 360.00 degrees) facing Default building facing degrees

^That creates two location leaks, Position of Illusion_Target[1] and the offset location.

 

[Marsal]

1. what is MUI?
2. the waits just needed for the spell's effects it's looks better when the illusions aren't show up at the same moment

 

[MortAr]

MUI (Multi-Unit-Instanceable) - More than one unit per player can cast the same spell without conflicts.
MPI (Multi-Player-Instanceable) - At least one unit per player can cast the same spell without conflicts.

Waits are not acceptable, just remove them.

 

[Marsal]

oooooh thanks
and whats the problem with the Illusion_Target[1] and offset?

 

[MortAr]

• ((Position of Illusion_Target[1])

This leaks, store the position into an veriable and use it

Same for the offset, store it into veriable in the begining of the spell.

 

[Marsal]

okay
fixing...

 

[kola]

the Waits are bad because they make ur spell not MUI.. so u need to use a Loop Trigger.

 

[krisserz]

1st
2nd

Read them (the second one may be harder to understand, but I hope you will get it)

 

[Marsal]

i made it into MPI but i don't know what to do with the target leaks
can someone give me a good example?

 

[hell gate]

• Set LeakPoint = Poisition of (TriggeringUnit)
• Unit - Create 1 ... at LeakPoint facting 0 degrees
• CustomScript - call RemoveLocation(udg_LeakPoint)

 

[Marsal]

i can't really make out what shall i do with the target location leaks :S

 

[kola]

you have to set the Point with a variable. after u can use the varialbe.
dont set a point on a variable that is aleardy used except if u Destroy it with a custom script

 

[Marsal]

so i must change the target unit variable into a target point variable?

 

[kola]

U need to set the Point in a Variable Before using it for anything

 

[Marsal]

but why isn't good the targeted unit of ability being cast as traget point?

 

[MortAr]

but why isn't good the targeted unit of ability being cast as traget point?

It leaks in memory, causing future lag, since it cannot be destroyed.

 

[Marsal]

tried to make it but i can't
it doen's works :S

 

[baassee]

but why isn't good the targeted unit of ability being cast as traget point?

it creates a point that takes RAM memory, then when you use it again it is with 99.999% chance that it wont be on the same spot and then it overwrites itself, creating a new point and take more memory. Then you will have loads of points that just got wasted and takes space on the memory, that's why you set it to a point, use it and then remove it afterwards.

 

[Marsal]

ok but i can't make it :S

 

[krisserz]

Look, create variable - location (POINT (this is a name)). First set it to something like

• Set POINT = Poisition of (There_Can_Be_Your_Unit_In_Variable)

Point is your variable

Then you use it into action, like

• Unit - Create 1 your_unit at POINT facting 0 degrees

And then to remove leak of location you simply do

• Custom script: call RemoveLocation (udg_POINT)

 

[Marsal]

but if i do it on that way the dummys won't be in a circle

 

[krisserz]

Aw, I opened the map and I think you should use math and loops instead of spamming triggers...

EDIT: Ok, I fixed it a little bit, but I didn`t have time to make it MUI.
Here`s script

code

• Illusion Strike
  • Events
    • Unit - A unit Begins ...
  • Conditions
    • (Ability being cast) Equal to Illusion Strike
  • Actions
    • Set Illusion_Target[1] = (Target unit of ability being cast)
    • Set Illusion_Caster[(Player number of (Owner of (Casting unit)))] = (Casting unit)
    • Set dgr = 0.00
    • Set point_1 = (Position of Illusion_Target[1])
    • For each (Integer A) from 1 to 8, do (Actions)
      • Loop - Actions
        • Set point_2 = (point_1 offset by 300.00 towards dgr degrees)
        • Unit - Create 1 IllusionDummy for (Owner of Illusion_Caster[(Player number of (Owner of (Casting unit)))]) at point_2 facing Default orientacja budynku degrees
        • Unit - Make (Last created unit) face Illusion_Target[1] over 0.00 seconds
        • Unit - Add a 2.00 second General expiration timer to (Last created unit)
        • Unit - Add Strike to (Last created unit)
        • Unit - Set level of Strike for (Last created unit) to (Level of Illusion Strike for Illusion_Caster[1])
        • Unit - Order (Last created unit) to Undead - ... (Position of Illusion_Target[1])
        • Set dgr = (dgr + 45.00)
        • Custom script: call RemoveLocation (udg_point_2)
    • Custom script: call RemoveLocation (udg_point_1)

Variables I used are
location - point_1
location - pont_2
real - dgr

 

[Marsal]

ty but i just needed it in mpi i created this spell for a hero arena

 

[baassee]

ty but i just needed it in mpi i created this spell for a hero arena

you need it to be MUI for uploading at hive ...

 

[hell gate]

Aw, I opened the map and I think you should use math and loops instead of spamming triggers...

EDIT: Ok, I fixed it a little bit, but I didn`t have time to make it MUI.
Here`s script

code

• Illusion Strike
  • Events
    • Unit - A unit Begins ...
  • Conditions
    • (Ability being cast) Equal to Illusion Strike
  • Actions
    • Set Illusion_Target[1] = (Target unit of ability being cast)
    • Set Illusion_Caster[(Player number of (Owner of (Casting unit)))] = (Casting unit)
    • Set dgr = 0.00
    • Set point_1 = (Position of Illusion_Target[1])
    • For each (Integer A) from 1 to 8, do (Actions)
      • Loop - Actions
        • Set point_2 = (point_1 offset by 300.00 towards dgr degrees)
        • Unit - Create 1 IllusionDummy for (Owner of Illusion_Caster[(Player number of (Owner of (Casting unit)))]) at point_2 facing Default orientacja budynku degrees
        • Unit - Make (Last created unit) face Illusion_Target[1] over 0.00 seconds
        • Unit - Add a 2.00 second General expiration timer to (Last created unit)
        • Unit - Add Strike to (Last created unit)
        • Unit - Set level of Strike for (Last created unit) to (Level of Illusion Strike for Illusion_Caster[1])
        • Unit - Order (Last created unit) to Undead - ... (Position of Illusion_Target[1])
        • Set dgr = (dgr + 45.00)
        • Custom script: call RemoveLocation (udg_point_2)
    • Custom script: call RemoveLocation (udg_point_1)

Variables I used are
location - point_1
location - pont_2
real - dgr

LOL

• Unit - Order (Last created unit) to Undead - ... (Position of Illusion_Target[1])

this also leak

 

[krisserz]

Ahh yes, I was doing that fast and I may missed something...
Thx hell gate