Is there any difference between these 2 triggers?

[JFAMAP]

I know how to use that but i just wanted to make sure about it 100% percent.
Both of the triggers are doing the same thing right?

 

[Antares]

 

[JFAMAP]

Ok i just wanted to make it 100% that means using CustomScript 2 times at then and else or using 1 times at very bottom of the trigger doesn't make a difference

 

[Pyrogasm]

It is almost always preferable to clean up leaking objects in the same scope that they are created/assigned a value. Since the group was not created/generated inside the If block it’s best to destroy it after the if block. One also wants to reduce duplicate code in both halves of the If-block, so putting it in both is unnecessary.

 

[Uncle]

Unrelated: You're leaking a Point when you do: (Center of CurrentTarget[...])

The fix:

• Set Variable TempPoint = (Center of CurrentTarget[...])
• Unit - Order YourUnit to Attack-Move to TempPoint
• Custom script: call RemoveLocation(udg_TempPoint)

The same goes for PreviousTarget[].

And like Pyro said about duplicate code, you can separate this Point from the If Then Else so you don't have to do the same thing twice.

Edit: On second thought, you wouldn't want to separate it since it gets Set to two different places.