Redemption Edge v1.0 by BetaGod

[Moderator]

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

23:43, 17th Jul 2012
Magtheridon96:

Comments

• You're leaking groups with your current configuration. You should destroy the group at the end of each sell cast.
• RE_Location and RE_Location2 don't have to be arrays. You can use normal variables.
• Owner of caster -> Triggering player
• The constants should not be stored in arrays, you should make a configuration trigger and use constant variables.
• Your current indexing method is pretty outdated. Click here to learn more about the best indexing method there is.
• You can totally forget about checking if the group is null.

 

[BeTaGod-]

Thanks a lot Maker & Adiktuz

 

[Tank-Commander]

you should use 0.03 instead of 0.02 seconds on that loop

 

[mckill2009]

- you really dont need this >>> if udg_RE_Group[udg_RE_Times] == null then
- RE_Location & RE_Location2 desont need to be arrayed
- You need to put a condition if caster is alive since the caster will hurt the enemy units
- RE_Player >>> TriggerPlayer
- RESFX1-4 and other configurables should be separated so that it will not run on every cast
- You really dont need to index all, like speed, aoe, abilitylevel, unless you are using it in the loop
- 125 AOE is small, make it a little higher
- The damage is too fast, you should multiply it according to your loop, in this case >>> Set RE_Power[RE_Times] = (((Real(RE_AbilityLevel[RE_Times])) x 75.00) + 60.00) x 0.02
- Use Hanky's indexing system, it's better

 

[BeTaGod-]

you should use 0.03 instead of 0.02 seconds on that loop

omg, really? i though there was no difference!

- you really dont need this >>> if udg_RE_Group[udg_RE_Times] == null then
- RE_Location & RE_Location2 desont need to be arrayed
- You need to put a condition if caster is alive since the caster will hurt the enemy units
- RE_Player >>> TriggerPlayer
- RESFX1-4 and other configurables should be separated so that it will not run on every cast
- You really dont need to index all, like speed, aoe, abilitylevel, unless you are using it in the loop
- 125 AOE is small, make it a little higher
- The damage is too fast, you should multiply it according to your loop, in this case >>> Set RE_Power[RE_Times] = (((Real(RE_AbilityLevel[RE_Times])) x 75.00) + 60.00) x 0.02
- Use Hanky's indexing system, it's better

Vey usefull review. Thanks a lot, i will work on this changes for the next update!

 

[Blackair]

thank you betagod its really Great , i wanna use it in my map + credit and Rep
5/5

but i try to learn the spells instead of copy/paste them
can you explain what Re_skip is it?

 

[Blackair]

you could use hashtables instead of Arrays the hashtables make your spell completely MUI
in mean time dont damage units in loop trigger cuz it will damage the units as loop action
like 100dmg each 0.02 time!
periodic times like 0.01 and 0.02 make your map so laggy please fix it