Picking all enemy units in a single action?

[klemen]

I have no idea, hot to pick all enemy units in a single action, like: Pick every unit owned by enemy of xxx player in rect and do actions.


Now, I use:

1. Pick player group (enemies of xxxx player)
........If/then/else
................Owned of picked unit equal to picked player
........................then
...............................pick every unit in unit group and do action


I dunno if this even works... Any more simple ways to do it (and wrorking..)?

 

[spiwn]

• Player Group - Pick every player in (All enemies of (Triggering player)) and do (Actions)
  • Loop - Actions
    • Set Abadon = (Units owned by (Picked player))
    • Unit Group - Pick every unit in Abadon and do (Actions)
      • Loop - Actions
        • Do Nothing

 

[Emm-A-]

I think this is better, because spiwns solution leaks a player group:

• Set Unitgroup = (Units in (Playable map area) matching (((Matching unit) belongs to an enemy of Player X (Colour)) Equal to True))
• Unit Group - Pick every unit in Unitgroup and do (Actions)
  • Loop - Actions
    • -------- //Your Actions here, after the UnitGroup Loop, just destroy it to prefent it leaking =) --------
• Custom script: call DestroyGroup(udg_Unitgroup)
• Custom script: set udg_Unitgroup=null

 

[klemen]

Ok, I tried a little different thingie, by removing all allied units from unit group, so I basicaly have only enemies.

Should this work?

 

[spiwn]

That is correct, it leaks - easily fixable.
But it is faster.
P.s. You do not need the Custom script: set udg_Unitgroup=null
Unitgroup is a global variable, so you do not need to null it.
Edit:
Oh klemen that will not work.

 

[klemen]

Oky, if it won't work after the test, I use second.


Thanks for the help.

 

[spiwn]

You lack experience in triggering, which I doubt you will earn very fast.
The trigger you posted will not do anything. If no one else post a working trigger, I will, but later.

 

[HINDYhat]

Why do you GUI guys keep using DestroyGroup? I mean, the Blizzard dudes added the bj_wantDestroyGroup global JUST for this. So instead of destroying the group AFTER the enum, just do this right before the enum:

set bj_wantDestroyGroup = true

 

[spiwn]

Here is a sample map. It does what the trigger you posted does(next time use trigger tags). I hope that is what you wanted it to do
P.s. Sample map so that you can see the variables or just copy it.