Titans Retribution

[Mellstorm]

It does a little different, it's more like what I need. Perhaps you could do this Diablo lightning stream Diablo 2-like spells

 

[Arthourias]

It does a little different, it's more like what I need. Perhaps you could do this Diablo lightning stream Diablo 2-like spells

Are you waiting for this from 2009??? Omg. It's too easy to make that. Im alittle busy these days but I will make it for you. Please remind me to do that.

 

[Wrda]

Help_TTT looks pointless. It is 0 at the first cast and then is set to 1 in the middle of nowhere.

• Unit Group - Pick every unit in (Units in (Region centered at ((Position of Hero_TTT) offset by (((Real(Cast_TTT_2)) x 50.00) x -1.00) towards (Facing_TTT + 120.00) degrees.) with size (300.00, ((Real((Cast_TTT_2 + 1))) x 100.00)))) and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • ((Owner of (Picked unit)) is an enemy of (Owner of Hero_TTT).) Equal to True
      • Then - Actions
        • Unit - Cause Hero_TTT to damage (Picked unit), dealing ((Real(DMG_TTT)) / 2.00) damage of attack type Spells and damage type Normal
      • Else - Actions

I don't understand why you decided to sometimes pick units within a region and other times within X range.
Lots of special effect leaks, unit groups and point leaks. Destroy them imediately after creating (the thunderclap ones), since you only want the their death animations.
You should use your own integer loops, using only integer A and B might interfer in other users' triggers and are usually unsafe as they can get overwritten.
Waits are almost always avoided due to their inaccuracy on low duration amounts.
Most spells (if not most) require to be MUI, as in, able to be cast by multiple units without bugs.

You should structure the spell in the following ways:
1. A configurable trigger, where the user sets up his desirable values that affect the spell overall.
2. A cast trigger, when the unit casts the ability, turns periodic trigger on if it's the first index of the spell.
3. A periodic trigger, where the spell has effects per interval, it can also have stages or phases. Includes deindexing when the spell ends.
4. You also might have a cancel trigger, depending on the spell.

Memory Leaks

C0Memory leaks For a quick lookup on most important actions, read Things That Leak. Introduction Object Leaks Reference Leaks Miscellaneous Conclusion C1Introduction If your computer's memory keeps occupied with stuff you already lost access to, it's called memory leak. It slows down the...

www.hiveworkshop.com

Visualize: Dynamic Indexing

Contents Introduction MUI Dynamic Indexing Comparison Credits A Introduction Dynamic indexing is a technique used to ensure that your spell or code can be ran multiple times without any MUI issues. At first, it is incredibly daunting to look at. This tutorial aims to show it in a better...

www.hiveworkshop.com

These tutorials should help you on that.

This needs quite a lot of rework at the moment.

 

[Arthourias]

Help_TTT fixes a bug you can remove then you will see the bugs happening.
In warcraft 3 region width and height changes with facing of casting hero and in some directions the region is smaller and vice versa. so every weird thing you see have a reason.
As you said I destroyed a special effect right after creating it, the reason is I need it's death animation.
The reason of small leaks in unit groups is 'region width and height' problem it can be fixed by using multiple small areas for unit groups instead of it.
I realized it too late, but I will fix it as soon as I can.
Thanks for review.

 

[Wrda]

Unfortunately regions don't natively rotate, so some units will get hit while others don't. If you do wish to still use regions, I'd just remind you to be aware to remove the regions after using them with RemoveRect(udg_rectvariable).
I saw the bug and I managed to fix that. You were using Integer A in parts where it wasn't being used at all, so it became Cast_TTT[0], which is 0.
This also had to be changed:

• ((NumberofCast_TTT / 2) + 1) Greater than Cast_TTT_3

Looping with Integer A, B, or your own variable already increments the counter automatically, no need to "set Loop integer A.
Cast_TTT doesn't even need to be array at all too.

When I said destroying special effects immediately after creating I meant right after creation, with no delay.
I also forgot to mention that terrain deformations always cause permanent memory leaks, it's unavoidable, if it does really need to be part of the spell, I would use it with caution, or better, sparingly.

This should also help with replacing the waits.

PS: I attached the map without that bug.