[Trigger] I need help with a trigger :(

[1337_s0u1]

Well, basically I have preset units on the map for each player and I want to remove the units owned by the player if there is no player in that slot. How should I do this in a leak free way?

 

[syltman]

Try this: Events:Map initialization
Conditions: none
Actions:
Player Group - Pick every player in (All players) and do (Actions)
Loop - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
((Picked player) slot status) Equal to Is playing
Then - Actions
Unit Group - Pick every unit in (Units owned by (Picked player) and do (Unit - remove (Picked unit))
Else - Actions
Do nothing

 

[Bob27]

If you want to have it leak free then you should put this in front of the unit group trigger.

• Custom script: set bj_wantDestroyGroup = true

 

[raft_pl]

Unit Group - Pick every unit in (Units owned by Player 1 (Red)) and do (Unit - Hide (Picked unit))

Its much better to remove unit than to hide it, cause hiding just makes the unit kinda "invisible", noncontrollable and paused, but the unit still exists and takes the memory.
And you forgot to change from "Player 1 (Red)" to "(Picked player)".

 

[1337_s0u1]

Didn't work, nothing happened.

 

[Just_Spectating]

its units owned by picked player not player 1, also try using game time is 1 second for the event.

 

[1337_s0u1]

Yeah I used picked player for the last part. I'll try it with the 1 second event.

 

[Daedin]

Well, the way guy there has it will remove the unit if the player is playing. So change it up to this:

RemovePlayerUnit
    Events
        Time - Elapsed game time is 1.00 seconds
    Conditions
        Or - Any (Conditions) are true
            Conditions
                (Player 1 (Red) slot status) Equal to Has left the game
                (Player 1 (Red) slot status) Equal to Is unused
    Actions
        Unit Group - Pick every unit in (Units owned by Player 1 (Red)) and do (Actions)
            Loop - Actions
                Unit - Remove (Picked unit) from the game

Of course you would have to do this for each player, and it isint leak free (Im a noob when it comes to fixing leaks) but it gets the job done rather nicely.

 

[humpadumpa]

@ Syltman:
This doesn't work, because of

• If - Conditions
  • ((Picked player) slot status) Equal to Is playing

It will only trigger if every player is playing.

@leetsoul
You can do it like this:

• Events
  • Time - Elapsed game time is 1.00 seconds
• Conditions
• Actions
  • Player Group - Pick every player in (All players matching (((Matching player) slot status) Not equal to Is playing)) and do (Actions)
    • Loop - Actions
      • Unit Group - Pick every unit in (Units in (Playable map area) owned by (Picked player)) and do (Actions)
        • Loop - Actions
          • Unit - Remove (Picked unit) from the game

 

[Daedin]

If you do it that way hump, it will only remove them if they arent playing, and because battle.net classifies wether or not someone is playing by where they are when game starts (while map is loading) it wont remove players that drop or get disconnected before map is done loading, my way makes sure both ways get removed.

 

[The-Weeh.]

Try this: Events:Map initialization
Conditions: none
Actions:
Player Group - Pick every player in (All players) and do (Actions)
Loop - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
((Picked player) slot status) Equal to Is playing
Then - Actions
Unit Group - Pick every unit in (Units owned by (Picked player) and do (Unit - remove (Picked unit))
Else - Actions
Do nothing

Can u tell me how to loop

 

[humpadumpa]

• Events
  • Map initialization
• Conditions
• Actions
  • Player Group - Pick every player in (All players) and do (Actions)
    • Loop - Actions
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • ((Picked player) slot status) Equal to Is playing
        • Then - Actions
          • Unit Group - Pick every unit in (Units owned by (Picked player) and do (Unit - remove (Picked unit))
        • Else - Actions
          • Do nothing

The loop is from 'Player Group - Pick every player in player group'

For loops: here's GhostWolf's tutorial about loops and arrays: http://www.hiveworkshop.com/forums/showthread.php?t=45561