[Trigger] Probluem : About SetunitX/Y

[tadekza20]

I WEAK TO ENGLISH

• Events
• Conditions
  • Multiple ConditionsAnd - All (Conditions) are true
    • Conditions
      • (Triggering unit) Equal to (==) BMT_ATKED_UNIT
      • (Damage source) Equal to (==) BMT_ATKING_UNIT
• Actions
  • Trigger - Turn off (This trigger)
  • Set BMT_Target_Location = (Position of (Triggering unit))
  • Set BMT_Blink_Location = (BMT_Target_Location offset by 100.00 towards (Random angle) degrees)
  • Custom script: call SetUnitX ( GetEventDamageSource(), GetLocationX(udg_BMT_Blink_Location))
  • Custom script: call SetUnitY ( GetEventDamageSource(), GetLocationY(udg_BMT_Blink_Location))
  • Special Effect - Destroy (Create a special effect using Abilities\Spells\NightElf\Blink\BlinkCaster.mdl at BMT_Blink_Location)
  • Custom script: call RemoveLocation (udg_BMT_Target_Location)
  • Custom script: call RemoveLocation (udg_BMT_Blink_Location)
  • Unit - Order (Damage source) to Right-Click (Triggering unit)
  • Wait BMT_CD[(Level of Blademaster [Blademaster Ultimath] for (Damage source))] seconds
  • Trigger - Turn on (This trigger)

That is my Trigger
setunitx/y with variable
-----------------------------------------------------------------------


My Question

• Set BMT_Target_Location = (Position of (Triggering unit))
  • Set BMT_Blink_Location = (BMT_Target_Location offset by 100.00 towards (Random angle) degrees)
  • Custom script: call SetUnitX ( GetEventDamageSource(), GetLocationX(udg_BMT_Blink_Location))
  • Custom script: call SetUnitY ( GetEventDamageSource(), GetLocationY(udg_BMT_Blink_Location))

I have know that is a leak point for a few day
(Set Variables then SetunitX with last setted variables)

How to SetUnitX likes that Trigger without set variable

I would like to fix this , but i can't do it TT

 

[Spellbound]

As far as I can tell, your locations are not leaking. If you want to calculate polar offset without using locations, you'll have to use some simple trigonometry.

 

[Spartipilo]

No leaks since you remove both Locations. However you can turn this to more efficient jass (since you're already using SetUnitX/Y

Also, find a workaround to that "wait", these are never good.

 

[Wietlol]

I would use a unit indexer and a global game-time timer.
Then you can store the moment that your unit would be able to use the ability again as game-time and you can check if the current game-time is higher than that value.

1, It will make this MUI.
2, It will be 100% accurate.
3, It will be fun to do so.

I also recommend you to use a DDS.