Missile [Blizzlike Spell]

[Moderator]

12th Dec 2015
IcemanBo: For too long time as NeedsFix. Rejected.

03:05, 19th Jun 2011
Maker: Read post #3 of this thread.

 

[Jazztastic]

This spell needs to support multiple levels.

 

[Maker]

Comments

You have location leaks. Position if x offset by y uses two locations. You need to set position of x into a varaible and the offset point into another variable.

You also have a unit group leak and a region leak.

The visibility modifier action doesn't belong to the spell, remove it from the init trigger.

 

[Magtheridon96]

Holy **** >.<
No wonder GUI hurts Nestharus' eyes

Location Leaks:

• (Position of Missile_dummy[Missile_index[2]])
• (Position of Missile_picked)

Here, you can simply convert the real to a string instead of from a real to an integer to a string

• String((Integer(Missile_damage[Missile_index[2]])))))

Also:

• Unit Group - Pick every unit in (Units within 100.00 of (Position of Missile_dummy[Missile_index[2]]) matching (((((Matching unit) is A structure) Equal to False) and (((Matching unit) is Magic Immune) Equal to False)) and ((((Matching unit) is alive) Equal to True) and (((Matching unit) be and do (Actions)

This leaks.
Add a custom script set bj_wantDestroyGroup = true before it to destroy the group


One other thing:

• Floating Text - Create floating text that reads ((- + (String((Integer(Missile_damage[Missile_index[2]]))))) + !) at (Position of Missile_picked) with Z offset 0.00, using font size 10.00, color (100.00%, 0.00%, 0.00%), and 0.00% transparency
• Set Missile_floatingtext = (Last created floating text)
• Floating Text - Change Missile_floatingtext: Disable permanence
• Floating Text - Set the velocity of Missile_floatingtext to 90.00 towards 90.00 degrees
• Floating Text - Change the fading age of Missile_floatingtext to 2.00 seconds
• Floating Text - Change the lifespan of Missile_floatingtext to 3.00 seconds

These settings should be configurable

I hope this helps, as I don't like to review GUI spells/systems anymore, so you're lucky I'm even here xD


edit

LOL Maker, you beat me xD


edit

One other thing:

• Set Missile_index[0] = (Missile_index[0] - 1)
  • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
  • Missile_index[0] Equal to 0
  • Then - Actions
  • Set Missile_index[1] = 0
  • Trigger - Turn off (This trigger)

This entire block should be removed from the if/then/else. It will ALWAYS return true anyway


edit

Maker: Read post #3 of this thread.

Post #4's good too