Lightning Orb v1.0

[Moderator]

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

Required changes:

• Add importing instructions into the map file
• Add hotkeys
• Learned icon position should be changed
• Try to avoid spamming the cast order
• Lightning Knock shouldn't be on all the time

 

[Magtheridon96]

Almost perfect!

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • ((Playable map area) contains TempPoint2) Equal to False
  • Then - Actions
    • Custom script: call RemoveLocation(udg_TempPoint2)
    • Set TempPoint2 = (TempPoint1 offset by LightningSpeed towards ((Facing of TempUnit) - 180.00) degrees)
    • Unit - Move TempUnit instantly to TempPoint1, facing TempPoint2
  • Else - Actions

You don't have to check if the point is inside the playable map area

The user would be expected to have some library like BoundSentinel implemented (Yes, there are Jass versions)

Also, you should cache (Facing of TempUnit) there since you're repeating it twice ^.^

Other than that, this looks like a simple, but nifty GUI spell

 

[maddeem]

If I didn't know that existed I'm sure other GUIers wouldn't know that existed either. Therefore I'm going to keep the bounds check just as a precaution. (A map crash can be very troubling for a mapper)

Also, putting the facing into a variable would not increase efficiency for a sufficient amount of time. Because it is only used twice when it is near map bounds.

N yeah this spell is indeed nifty. It surprised me at how well it worked. But when I tried it for the first time I didn't know it would circle around it's target. (Didn't realize that units who cast spells face that direction therefore altering the path of the lightning orb) It created a nice unintentional effect

 

[Bribe]

SetUnitPositionLoc does not cause an out-of-bounds check.

 

[maddeem]

Whats your point?

 

[Bribe]

I meant to say it does not cause a crash, because it does it's own bounds check.