Rapid Slash v 1.0

[Moderator]

14th Jan 2012
Bribe: The indexing is really weird. You don't need a "has" variable because there is no reason to iterate over dead indices. Check out Hanky's Dynamic Indexing which is the best GUI indexing out there.

Looking at the comments I can see that Maker has already mentioned this. His system works just fine in its current state so maybe the version you tried last March was buggy (I do recall he said he fixed some bugs in the past few months).

However I happen to know that your indexing works (it simply could be "better"). So I will approve this with a 3/5 rating for now.

 

[Maker]

You could use proper dynamic indexing, where "dead" instances are removed and you don't need the Has boolean.
Improve the learn tooltip, you should list the damage of every level there. Look at standard Blizzard abilities.
Change the learned icon position.
Ordering the unit to constantly stop causes the stop and attack icons to flash which doesn't look professional.
You could try to make the spell support being able to abort it by ordering the unit to move. You move the unit instantly periodically so the unit can't be ordered to do anything else. Maybe that is instended, but I like being able to abort my spells.
Maybe you could change the colour of the dummies to the owner of the caster.

Good job with the ability, triggering looks ok.

Somehow this reminds me of my Lightning Speed Laceration spell.

 

[Marsal]

You could use proper dynamic indexing, where "dead" instances are removed and you don't need the Has boolean.

Well if you can tell me what the hell is the problem of this spell then I will try it(again).
http://www.hiveworkshop.com/forums/1873973-post6.html
That was the first time(and the last as well) when I've tried to make a spell with other's index recycling system. This one was made with Hanky's template and it often crashed.

Improve the learn tooltip, you should list the damage of every level there. Look at standard Blizzard abilities.
Change the learned icon position.

Okay.

Ordering the unit to constantly stop causes the stop and attack icons to flash which doesn't look professional.

Then how could I make it without stunning the target? If I just change the movement speed of the target to nearly zero and order a dummy unit to cast a spell on the target that disables the attack is more complicated and I have to use another one dummy and spell.

You could try to make the spell support being able to abort it by ordering the unit to move. You move the unit instantly periodically so the unit can't be ordered to do anything else. Maybe that is instended, but I like being able to abort my spells.

I'll try it.

Maybe you could change the colour of the dummies to the owner of the caster.

I've changed it to neutral because it looked kinda strange for that hero. The glow was too bright.

Good job with the ability, triggering looks ok.

Then why won't you approve it? T~T

Somehow this reminds me of my Lightning Speed Laceration spell.

Well, this type of spell can be found in almost every Hero Arena map.

 

[Jay the Editor]

You should check only after you reduce the integer; move the block into it.

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • FIndexS Equal to 0
  • Then - Actions
    • Trigger - Turn off (This trigger)
  • Else - Actions

If I'm not mistaken, it automatically creates the unit group after you set it. You don't need to use native to create it anymore.

• Custom script: set udg_FFreezeGroup[udg_TempInt] = CreateGroup()
• Set FFreezeGroup[TempInt] = (Units within 750.00 of FCastPoint[TempInt] matching ((((Matching unit) is alive) Equal to True) and ((Matching unit) Not equal to FCaster[TempInt])))

Well, this type of spell can be found in almost every Hero Arena map.

Lightning Speed Laceration spell is one of Maker's spell. lol.

EDIT:

Then how could I make it without stunning the target? If I just change the movement speed of the target to nearly zero and order a dummy unit to cast a spell on the target that disables the attack is more complicated and I have to use another one dummy and spell.

My suggestion would be to create a dummy and order the dummy unit to stun the target. In the end of the duration, remove the buff.

 

[Marsal]

You should check only after you reduce the integer; move the block into it.

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • FIndexS Equal to 0
  • Then - Actions
    • Trigger - Turn off (This trigger)
  • Else - Actions

If I'm not mistaken, it automatically creates the unit group after you set it. You don't need to use native to create it anymore.

• Custom script: set udg_FFreezeGroup[udg_TempInt] = CreateGroup()
• Set FFreezeGroup[TempInt] = (Units within 750.00 of FCastPoint[TempInt] matching ((((Matching unit) is alive) Equal to True) and ((Matching unit) Not equal to FCaster[TempInt])))

Lightning Speed Laceration spell is one of Maker's spell. lol.

EDIT:
My suggestion would be to create a dummy and order the dummy unit to stun the target. In the end of the duration, remove the buff.

Read again my comment -.-"
Let me emphasize the keywords. Almost every HERO ARENA have a spell of THIS TYPE.

 

[Jay the Editor]

My comments are mentioning about the Hanky's indexing system that you used which you posted for Maker.

 

[Grey Knight]

Can you Add Death Animation on Trigger
if you Use Rapid Slash on the Target his Animation is Death

Like his >< Slash,>< Slash,>< Slash,>< Slash,>< Slash,>< Slash