[Trigger] Coded Right?

[Tharky]

• Meteora Level 4
  • Events
    • Unit - A unit Is attacked
  • Conditions
    • ((Attacking unit) is A Hero) Equal to True
    • ((Attacking unit) has buff Meteora (Level 4)) Equal to True
  • Actions
    • Set MeteoraPos = (Position of (Triggering unit))
    • Set Roll = (Random integer number between 1 and 100)
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Roll Less than or equal to 10
      • Then - Actions
        • Special Effect - Create a special effect at MeteoraPos using Units\Demon\Infernal\InfernalBirth.mdl
        • Special Effect - Destroy (Last created special effect)
        • Unit - Cause (Attacking unit) to damage circular area after 0.00 seconds of radius 400.00 at MeteoraPos, dealing 200.00 damage of attack type Chaos and damage type Fire
      • Else - Actions
        • Do nothing
    • Set Roll = 0
    • Custom script: call RemoveLocation(udg_MeteoraPos)

Is there a way to make it smaller??

Am I posted correct place?

 

[M4stah]

You will have to use a damage detection system instead of A unti is attacked. Check the spells section. Also you can directly check for a random number instead of setting a variable first

 

[Dr Super Good]

Do nothing is not needed. It does nothing but waste time.
Instead of 1 to 100 and less than or equal to 10, try 0 to 9 and equal to 0. Will produce the same result (10% chance) but overall deal with smaller numbers so should be more efficent.

I would also advise a damage dectection system, as currently by stopping the unit repetitivly, a player could cause this spell to execute far more than desired by getting the unit to trigger the event but not actually completing the attack (and starting attack cooldown).

 

[UreDe4D]

you can leave out the "Do nothing", just make it blank, and you don't need the "set roll = 0" ; but apart from that, why you think its too long? i guess you have 4 triggers of that kind and you are asking if its possible to make it up with a single one right?

 

[Tharky]

you can leave out the "Do nothing", just make it blank, and you don't need the "set roll = 0" ; but apart from that, why you think its too long? i guess you have 4 triggers of that kind and you are asking if its possible to make it up with a single one right?

Actually yes. I get ONLY 8 triggers for just one spell. Um ok i will request this spell in requests section then.