[Trigger] Problems with when a player...

[DUTY OF RS]

Hello,

So in my new map, dael'barun world defence. It is a pure 3 players online game.
So my problem is, when a player is leaving the game, he is not leaving his buildings. His building is still on the map with his units also. i used this trigger but it didnt work.

• Remove Teal
  • Events
    • Player - Player 3 (Teal) leaves the game
  • Conditions
  • Actions
    • Custom script: set bj_wantDestroyGroup = true
    • Unit Group - Pick every unit in (Units owned by (Triggering player)) and do (Actions)
      • Loop - Actions
        • Game - Display to (All players) the text: ((Name of (Triggering player)) + |cFFFF0000 has left the game now we need to build us more time to get stronger!|r)
        • Wait 1.00 seconds
        • Unit - Remove (Picked unit) from the game

Hope some one can help!

 

[sethmachine]

Hmm wish I could help.

It might be your wait which is throwing it off? Try taking the "Wait 1.00 seconds" out and see if it works. It could be bugging "Unit - Remove (Picked unit) from the game."

Also you should not put the text message inside the "Unit Group Loop - Actions." Otherwise it will spam the players for each unit that gets removed.

 

[Nichilus]

yeah, what you want should be:
wait 1 sec
display game message
custom script
unit group actions

The reason is what is in the loop block repeats for each picked unit, so currently for each picked unit you write the game message and then you try to remove it (which I'm not sure if even works properly with the wait inside the loop)

 

[DUTY OF RS]

• Remove Pink Copy
  • Events
    • Player - Player 8 (Pink) leaves the game
  • Conditions
  • Actions
    • Wait 1.00 seconds
    • Game - Display to (All players) the text: ((Name of (Triggering player)) + |cFFFF0000 has left the game now we need to build us more time to get stronger!|r)
    • Custom script: set bj_wantDestroyGroup = true
    • Unit Group - Pick every unit in (Units owned by (Triggering player)) and do (Actions)
      • Loop - Actions

Like that? I feel it might look wrong

 

[Nichilus]

yes like that.
Actions are executed sequentially, meaning they are executed one by one from the upper-most action to the bottom-most one.
Loops (cycles) are part of code that repeats numerous times (the number of iteration is usually set by number or they loop endlessly until specific condition is met).

If you have a piece of trigger with loop that looks like this:

A
B
Loop
    C
    D
    E
    F
(loop block end)
G
H

where A-G are some actions and this loop iterates 4 times, then the actions are executed as AB CDEF CDEF CDEF CDEF GH. Notice how only CDEF actions inside the loop block are repeated and the order in which they are executed.

Unit group action is a loop which repeats X times where X is number of units in the unit group. The only difference from other loops is that each iteration of this unit group loop points at different unit inside this group.

So looking now at what you have and at what you had, it should be obvious why I wrote you should change the order of some actions.

 

[DUTY OF RS]

Ahh ok, thank you it will help me next time to

EDIT:

It dont works in-game.
When the player leave's the game, he/she's building & units is still in the game.

 

[sethmachine]

Can you post the trigger that your currently using which isn't working?

 

[Chaosy]

try this.

• Unit Group - Pick every unit in (Units owned by Player 1 (Red)) and do (Actions)
  • Loop - Actions
    • Custom script: local unit udg_GUI_unit_variable = GetEnumUnit()
    • Wait 1.00 seconds
    • Unit - Remove GUI_unit_variable_name from the game

 

[Nichilus]

It dont works in-game.
When the player leave's the game, he/she's building & units is still in the game.

You're not testing it by defeating computer, right?

Does the message display properly? If there is no message, the trigger didn't even fire.

 

[Vinz]

• Remove Pink Copy
  • Events
    • Player - Player 8 (Pink) leaves the game
  • Conditions
  • Actions
    • Wait 1.00 seconds
    • Game - Display to (All players) the text: ((Name of (Triggering player)) + |cFFFF0000 has left the game now we need to build us more time to get stronger!|r)
    • Custom script: set bj_wantDestroyGroup = true
    • Unit Group - Pick every unit in (Units owned by (Triggering player)) and do (Actions)
      • Loop - Actions

Like that? I feel it might look wrong

You can try adding a remove unit action.

 

[Nichilus]

You can try adding a remove unit action.

Wait, I thought that is stating the obvious

 

[Vinz]

But sometimes the obvious is not stated. D:

Or maybe he should just update us with the code.

 

[DUTY OF RS]

This one didn't work:

• Remove Teal
  • Events
    • Player - Player 3 (Teal) leaves the game
  • Conditions
  • Actions
    • Wait 1.00 seconds
    • Game - Display to (All players) the text: ((Name of (Triggering player)) + |cFFFF0000 has left the game now we need to build us more time to get stronger!|r)
    • Custom script: set bj_wantDestroyGroup = true
    • Unit Group - Pick every unit in (Units owned by (Triggering player)) and do (Actions)
      • Loop - Actions
    • Unit - Remove (Triggering unit) from the game

So, should i try Chaosy's trigger?
Tho, i have tried many now so hope some one got a solution!

 

[Vinz]

Picked Unit and put it in the loop.

 

[DUTY OF RS]

Like this?

• Remove Teal
  • Events
    • Player - Player 3 (Teal) leaves the game
  • Conditions
  • Actions
    • Wait 1.00 seconds
    • Game - Display to (All players) the text: ((Name of (Triggering player)) + |cFFFF0000 has left the game now we need to build us more time to get stronger!|r)
    • Custom script: set bj_wantDestroyGroup = true
    • Unit Group - Pick every unit in (Units owned by (Triggering player)) and do (Actions)
      • Loop - Actions
        • Unit - Remove (Picked unit) from the game

 

[sethmachine]

That looks better. Does it work?

 

[DUTY OF RS]

I will make a game on bnet to check now.
Now i hoped i never said this but i hope one player is leaving..
So i can test it ^^

EDIT:

It worked! Thanks.