Charge-Slash 2.0v

[Moderator]

10 Feb 2012
Bribe: Looks alright, thanks for sharing

Approved 3.75/5

Older reviews

• Leak
  • Game - Display to (All players controlled by a ((Owner of CCaster[CArray]) controller) player) the text: ...
• You should do the distance check as early as possible in the cast trigger to avoid useless actions
• Don't use arrays for the locations
• Some locations are not correctly removed in the looping trigger
• Add importing instructions into the map file
• The looping trigger is never turned off

The caster can get stuck on deep water and can charge through cliffs and buildings. Maybe these things are intended, but getting stuck on water could be an issue.

 

[JollyD]

You are cruel ))

 

[KhaosMachine]

Nice ability,

 

[CoLd Bon3]

Thanks!

 

[Maker]

It seems to me that you don't destroy the force if "CWait[CArray] Less than or equal to 0.00" is true.

The ability is quite well done, the effects are subtle and fitting.

 

[CoLd Bon3]

@Maker - i thought i should not destroy the force because i don't use it when CWait is less than or equal to 0 and i have never used it before in the trigger. :? I have used it only when i have to erm... send a game message to that force (when the hero is too close to his target (in the cast trigger) and when there is a cliff or deep water (later in the trigger)). Please, tell me if i'm wrong so that i can update the map spell.

Thank you!

 

[Palooo3]

Finally found impressive GUI charge, Thank you

 

[Maker]

@Maker - i thought i should not destroy the force because i don't use it when CWait is less than or equal to 0 and i have never used it before in the trigger. :? I have used it only when i have to erm... send a game message to that force (when the hero is too close to his target (in the cast trigger) and when there is a cliff or deep water (later in the trigger)). Please, tell me if i'm wrong so that i can update the map spell.

The group is not removed here, thus it leaks:

Trigger

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
• If - Conditions
• CWait[CArray] Less than or equal to 0.00
• Then - Actions
• Unit - Cause CCaster[CArray] to damage CTarget[CArray], dealing CDamage[CArray] damage of attack type Spells and damage type Normal
• Special Effect - Create a special effect attached to the chest of CTarget[CArray] using Abilities\Spells\Other\Stampede\StampedeMissileDeath.mdl
• Special Effect - Destroy (Last created special effect)
• Animation - Change CCaster[CArray]'s animation speed to 100.00% of its original speed
• Unit - Turn collision for CCaster[CArray] On
• Unit - Order CCaster[CArray] to Attack CTarget[CArray]
• Set ID[CLoop] = ID[ID2]
• Set ID[ID2] = CArray
• Set ID2 = (ID2 - 1)
• Set CLoop = (CLoop - 1)
• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
• If - Conditions
• ID2 Equal to 0
• Then - Actions
• Trigger - Turn off (This trigger)

 

[RoyalLich]

Nice model name for your dummy dick in a box
It looks nice, and I like how neat you made the triggers
its easy to find things, and learn how to make it. ty

 

[Drazhar]

Very nice ability, only problem i've had with it is that it isn't blocked by pathing blockers? Both ground and air.

 

[Slash_Raptor]

How do you use this? I started coding the triggers and what not, but when I got to the part where it says "(Ability being cast) Equal to Charge", it stumped me. There is no "Charge" variable. There's a "Charge Gold and Lumber" variable, but that's not it. It almost seems like you have to create the the spell itself first before writing the triggers, but how can you do that if there's no already existing spell to base it off of?

 

[Mr. Reez]

How do you use this? I started coding the triggers and what not, but when I got to the part where it says "(Ability being cast) Equal to Charge", it stumped me. There is no "Charge" variable. There's a "Charge Gold and Lumber" variable, but that's not it. It almost seems like you have to create the the spell itself first before writing the triggers, but how can you do that if there's no already existing spell to base it off of?

You are supposed to copy from the test map, buffs (when there are buffs) then spells, then Dummy units, then you tick the "Automatically create unknown variable while pasting trigger data" (found in File - Preferences) and finally you can either write it line by line, or just copy the whole trigger then configure it in that order. (I think)

 

[BUP]

@Cold Bon3
Great Idea with the acceleration!!