• 🏆 Texturing Contest #33 is OPEN! Contestants must re-texture a SD unit model found in-game (Warcraft 3 Classic), recreating the unit into a peaceful NPC version. 🔗Click here to enter!
  • It's time for the first HD Modeling Contest of 2024. Join the theme discussion for Hive's HD Modeling Contest #6! Click here to post your idea!

Lightning Nova

This bundle is marked as useful / simple. Simplicity is bliss, low effort and/or may contain minor bugs.
  • Like
Reactions: Losam
this spell creates a Lightning Nova Around Caster Damage everyone Around
Contents

Lightning Nova (Map)

Reviews
12th Dec 2015 IcemanBo: Too long as NeedsFix. Rejected. Lightning Nova | Reviewed by Maker | 30th May 2013 NEEDS FIX You are not using K_Group destroy it Temp group is destroyed but no created again The death type of the...

Moderator

M

Moderator

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


Lightning Nova | Reviewed by Maker | 30th May 2013
NEEDS FIX


126248-albums6177-picture66522.png


  • You are not using K_Group destroy it
  • Temp group is destroyed but no created again
  • The death type of the dummies should be Can't raise, does not decay
  • The dummies give too much visibiliy
  • The units picked in 225 range function doesn't do anything
    Does it belong here?
  • Currently pathing blockers and things like that interrupt the
    movement of the dummies
126248-albums6177-picture66523.png


  • You could use a variable for owner of triggering unit
    to avoid repeated function calls
  • Remove the AND from the unit group pick
    Place the conditions directly under the If - Conditions
  • The dummies do not need Invulnerability ability, they have Locust
  • It would be better if the caster dealt the damage
[tr]
 
Level 29
Joined
Oct 24, 2012
Messages
6,543
Plz show us the triggers.

I took a look at the trigger u have.
This can be made more efficient.
Anything u use twice or more should be stored into a variable for efficiency and speed.
Get rid of the and condition. only time u need to use the and condition is inside an ir condition.
You leak a location in the very beginning.
When u use ur unit group loop the TargetLoc is null so it will bug up.
I think thats all i just took a quick look at it.
This spell is also GUI not JASS
 
Last edited:
Top