[Trigger] Problem with texts!

[reason 3]

I have this trigger in my map,but for some reason(s) it bugs.It shows the displayed message X times for X players...don't know why.Example there are 5 players in game,it will show in screen
Single Player Detected - Arena Disabled
Single Player Detected - Arena Disabled
MultiPlayer Detected -Arena Enabled
MultiPlayer Detected -Arena Enabled
MultiPlayer Detected -Arena Enabled

This is what exactly what it shows,why is this?I want it to just show Single player if there is only 1 player and Multiplayer if there are 2 or more players in the map.

• Turn Arena On
  • 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
            • And - All (Conditions) are true
              • Conditions
                • ((Picked player) controller) Equal to User
                • (Player number of (Picked player)) Greater than or equal to 4
          • Then - Actions
            • Trigger - Turn on Arena <gen>
            • Trigger - Turn on Arena2 <gen>
            • Trigger - Turn on SetHeroGroups <gen>
            • Trigger - Turn on Display Arena <gen>
            • Game - Display to (All players) the text: |c00ff0303Multiplay...
          • Else - Actions
            • Game - Display to (All players) the text: |c00ff0303Single pl...

 

[Arnadath]

Cause every time it checks a player, it writes it to All players. I assume you better put in a variable all the players owned by humans, and after that check if two at least are playing so it is multiplayer and then announce it

 

[reason 3]

Hmm...you mean instead of using the "Player Group - Pick every player in (All players) and do (Actions)" command to use a variable with all players added?

 

[Arnadath]

For every player that is a human player, add +1 to an integer. In the end check the value if it is 1 or greater than 1

 

[Reaper2008]

Why use the "if all conditions are true"? Just put them there (in the if/then/else).

• Turn Arena On
• 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) controller) Equal to User
        • (Player number of (Picked player)) Greater than or equal to 4
      • Then - Actions
        • Set Players = Players + 1
      • Else
  • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    • If - Conditions
      • Players Greater than or Equal to 2
    • Then - Actions
      • Trigger - Turn on Arena <gen>
      • Trigger - Turn on Arena2 <gen>
      • Trigger - Turn on SetHeroGroups <gen>
      • Trigger - Turn on Display Arena <gen>
      • Game - Display to (All players) the text: |c00ff0303Multiplay...
    • Else - Actions
      • Game - Display to (All players) the text: |c00ff0303Single pl...

 

[Arnadath]

Excactly what i said ^^

 

[Reaper2008]

Yup! But I posted the triggers too (simple 2min edit ).

 

[Arnadath]

Yup! But I posted the triggers too (simple 2min edit ).

I didnt have any Map editor (pc suxz hell yea)

 

[reason 3]

Gonna try it,but anyway you both deserve +rep for your answers thx!
So this should be like it?

• Turn Arena On Copy
  • 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
            • And - All (Conditions) are true
              • Conditions
                • ((Picked player) controller) Equal to User
                • (Player number of (Picked player)) Greater than or equal to 4
          • Then - Actions
            • Set PlayersArena = (PlayersArena + 1)
          • Else - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • PlayersArena Greater than or equal to 2
      • Then - Actions
        • Trigger - Turn on Arena <gen>
        • Trigger - Turn on Arena2 <gen>
        • Trigger - Turn on SetHeroGroups <gen>
        • Trigger - Turn on Display Arena <gen>
        • Game - Display to (All players) the text: |c00ff0303Multiplay...
      • Else - Actions
        • Game - Display to (All players) the text: |c00ff0303Single pl...

With PlayersArena an Integer variable.

 

[eleljrk]

Dont have it inside the "Player Group".. Of some reason it multiple the post by 3 instead of 1. It works, guaranteed.

 

[reason 3]

like this?

• Turn Arena On Copy
  • 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
        • And - All (Conditions) are true
          • Conditions
            • ((Picked player) controller) Equal to User
            • (Player number of (Picked player)) Greater than or equal to 4
      • Then - Actions
        • Set PlayersArena = (PlayersArena + 1)
      • Else - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • PlayersArena Greater than or equal to 2
      • Then - Actions
        • Trigger - Turn on Arena <gen>
        • Trigger - Turn on Arena2 <gen>
        • Trigger - Turn on SetHeroGroups <gen>
        • Trigger - Turn on Display Arena <gen>
        • Game - Display to (All players) the text: |c00ff0303Multiplay...
      • Else - Actions
        • Game - Display to (All players) the text: |c00ff0303Single pl...

 

[Adiktuz]

if you move the first if-then action outside the player group then you wont have picked player.....

you can remove the "And-All conditions are true" because basically the normal If-conditions checks everything.....

 

[reason 3]

Ok,i think this one should be fine then...i got confused with all these

• Turn Arena On Copy2
  • 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) controller) Equal to User
            • (Player number of (Picked player)) Greater than or equal to 4
          • Then - Actions
            • Set PlayersArena = (PlayersArena + 1)
          • Else - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • PlayersArena Greater than or equal to 2
      • Then - Actions
        • Trigger - Turn on Arena <gen>
        • Trigger - Turn on Arena2 <gen>
        • Trigger - Turn on SetHeroGroups <gen>
        • Trigger - Turn on Display Arena <gen>
        • Game - Display to (All players) the text: |c00ff0303Multiplay...
      • Else - Actions
        • Game - Display to (All players) the text: |c00ff0303Single pl...

So is this one ok?

 

[Reaper2008]

Yup, it is fine.

 

[reason 3]

ok,thx...+rep Reaper2008.