[GUI]Lightning Counter v7.1

[Moderator]

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

11:52, 20th Jan 2014
Maker: The dummy unit type needs to be set in a variable
and since moving unit instantly issues stop order,
the spell can almost lock the attacked unit in place
if there are many enemies attacking

 

[Cokemonkey11]

you should make the caster also use bladestorm, and spawn mirror image

 

[Haley]

about adding mirror image, yes, i was wanted to, but how?
is creating dummy and order him to cast mirror image will work?

 

[Ofel]

Yes, it works...

REVIEW|Lightning Counter|v7.0

Resource Status: Approved Rating: 3/5

GENERAL	None
PROS	MUI Leakless Simple (not really)
CONS	None
SUGGESTIONS	Add more idea

[/TD][/tr]
If you have any question about this review, PM or VM me Review template by Doomlord

 

[deathismyfriend]

Hidden

Yes, it works...

REVIEW|Lightning Counter|v7.0

Resource Status: Approved Rating: 3/5

GENERAL	None
PROS	MUI Leakless Simple (not really)
CONS	None
SUGGESTIONS	Add more idea

[/TD][/tr]
If you have any question about this review, PM or VM me Review template by Doomlord

You should put approvable not approved.

@TO
The pic looks interesting but it does seem a little too simple.
Also why is it version 7.0 already ?

 

[Haley]

Number 7 is my favorite number...

 

[Maker]

I think it would be best to use one permanent dummy since a unit can be attacked very often and therefore you create lots of dummy units. Creating a dummy is a heavy operation.

Or when a unit is attacked, create a dummy if one does not exist already. Make it be removed 10 seconds after the attack. If the unit is attacked again during that 10 seconds, reset the time.

 

[BPower]

• Instead of creating a dummy over and over again, you should do it once on map init.
  
  • Custom script: set udg_LC_Dummy = CreateUnit(Player(15), 'u000', 0,0,0)
  udg is the prefix for GUI generated variables in you case LC_Dummy, the set keyword is the same as set variable. Player(15) is neutral passive, 'u000' is the raw-code of your dummy and 0,0,0 are x-coordinate, y-coordinate and facing.
• • Unit - Move LC_Dummy instantly to LC_TempLoc
  • Unit - Change ownership of LC_Dummy to LC_Owner and Retain color
  Instead of creating the dummy and add an expiration timer to it, you can now just move it to the casters location and change the ownership.

So far the spell is leakless and MUI. The coding style is rather simple has been common years ago.

 

[Haley]

i'll update this tomorrow... so sleepy... zzzzzz

 

[Daffa]

I prefer using DDS than

• Unit - A Unit is attacked

That event can be abused

 

[Haley]

@Daffa
but the caster can get damaged, and the following benefit will gone: Avoid attack/damage

 

[Maker]

I prefer using DDS than

• Unit - A Unit is attacked

That event can be abused

In this case using a DDS would change the functionality of the ability. With A unit is attacked event, the counter is preemptive. It is not likely that the attacker would use the attack-stop method to get the event to fire more often, it would be disadvantageous to the attacking unit.

 

[Haley]

Updated...

 

[BPower]

Terraindeformations are handles which won't be destroyed once they are finished. They exist permanently in data and can't be removed GUI or in JASS.
Warstomp does create such a terraindeformation, hence you should choose another spell for the dummy.

The coding is ok, but I don't see any practical use for such a spell. To me it feels more like disadvantage playing the attacked unit, because it gets uncontrollable once you are under attack.
i.e. Try to run away from the gryphons in your demo map. In most maps you are not always in attack mode .

You could move the ability level check into the Trigger condition.

 

[Haley]

I don't see any practical use for such a spell. To me it feels more like disadvantage playing the attacked unit, because it gets uncontrollable once you are under attack.
i.e. Try to run away from the gryphons in your demo map. In most maps you are not always in attack mode .

so how to cover this? the best way? and not too hard...