[Trigger] Question about unit group

[klemen]

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • AstroJumpLevel Equal to 1
  • Then - Actions
    • Unit Group - Pick every unit in (Units within 400.00 of (Position of (Casting unit))) and do (Actions)
      • Loop - Actions
        • Unit Group - Add (Picked unit) to AstroJumpUnitGroup
        • Unit Group - Remove (Casting unit) from AstroJumpUnitGroup
        • Advanced - Slide (Picked unit) using (Facing of (Casting unit)) degrees angle with a starting velocity of 20.00 and kill destructables in the way True
        • Unit - Create 1 DummyAstroJump for (Owner of (Casting unit)) at (Position of (Casting unit)) facing Default building facing (270.0) degrees
        • Unit - Add a 3.50 second Generic expiration timer to (Last created unit)
        • Unit - Order (Last created unit) to Follow (Casting unit)
  • Else - Actions

It's simple. I add all units within 400 AOE of casting unit to unit group AstroJumpUnitGroup and then I remove Caster from unit group so it won't slide.

The problem is in Slide action. I must select all units in AstroJumpUnitGroup to slide, not Picked unit. The problem is, that I can't find function that would select all units in unit group. I only found "Find first unit in unit group" and "Random unit in unit group".

I need to select all from unit group of them, not only 1.


+rep to the person who helps me
Ty in advance

 

[Dynasti]

There is no such trigger action in GUI as

• Advanced - Slide (slippery unit) using (smoke) degrees angle with a starting velocity of (lolznumber) and kill destructables in the way Maybe

 

[klemen]

Advanced-Slide action exists in WEU.

But I am not asking just for this action, but for all Unit-actions.

 

[Avator]

Seems pretty simple to me.

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • AstroJumpLevel Equal to 1
  • Then - Actions
    • Unit Group - Pick every unit in (Units within 400.00 of (Position of (Casting unit))) and do (Actions)
      • Loop - Actions
        • Unit Group - Add (Picked unit) to AstroJumpUnitGroup
    • Unit Group - Remove (Casting unit) from AstroJumpUnitGroup
    • Unit Group - Pick every unit in AstroJumpUnitGroup and do (Actions)
      • Loop - Actions
        • Advanced - Slide (Picked unit) using (Facing of (Casting unit)) degrees angle with a starting velocity of 20.00 and kill destructables in the way True
    • Unit - Create 1 DummyAstroJump for (Owner of (Casting unit)) at (Position of (Casting unit)) facing Default building facing (270.0) degrees
    • Unit - Add a 3.50 second Generic expiration timer to (Last created unit)
    • Unit - Order (Last created unit) to Follow (Casting unit)
  • Else - Actions

 

[Kingz]

That trigger leaks he needs 2 more variables:

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
• If - Conditions
• AstroJumpLevel Equal to 1
• Then - Actions
  • Set Position1 = Positions of (Casting unit)
  • Set Unit Group1 = Units within 400 of Position1
  • Unit Group - Pick every unit in Unit Group1 and do (Actions)
  • Loop - Actions
  • Unit Group - Add (Picked unit) to AstroJumpUnitGroup
  • Unit Group - Remove (Casting unit) from AstroJumpUnitGroup
  • Unit Group - Pick every unit in AstroJumpUnitGroup and do (Actions)
    • Loop - Actions
      • Advanced - Slide (Picked unit) using (Facing of (Casting unit)) degrees angle with a starting velocity of 20.00 and kill destructables in the way True
      • Unit - Create 1 DummyAstroJump for (Owner of (Casting unit)) at Position1 facing Default building facing (270.0) degrees
      • Unit - Add a 3.50 second Generic expiration timer to (Last created unit)
      • Unit - Order (Last created unit) to Follow (Casting unit)
• Custom script: call DestroyGroup(udg_Unit Group1)
• Custom script: call RemoveLocation(udg_Position1)
• Else - Actions

 

[Avator]

I've said this in a couple of other threads too: I'm not the anti-leak king. I help finding a solution to the problem, and here the problem is not solving leaks, it's ordering a Unit Group. People should know to make their triggers leakless, and if they don't know how to they should ask someone for help with that or ask them to do it for them.

Sorry I have to post this, but this is about the 2nd or 3rd time somebody posts that my trigger isn't good because it leaks, while I see 90% of the trigger solutions given on this forum causing leaks and nobody complaining about those.
So as I said before: If you need help with making triggers leakless, ask specificly for help or somebody to do it for you. If not asked, people can assume they don't need help there.

 

[Kingz]

Take it easy man i only added 4 lines in the trigger which were missing so it becomes leakless. It is still your trigger only modified a bit, so i realy don't see the problem here i just added that so he doesn't have some problems with memory leaks.

 

[PurplePoot]

Why would you loop through a group to dump it into another group, just to loop through that second group? It's twice as slow and twice as leaky.