Creating units with loop

[216199]

A very simple trigger, yet it does not work. I don't understand.

Begin 
    Events
        Map initialization
    Conditions
    Actions
        For each (Integer A) from 1 to 12, do (Actions)
            Loop - Actions
                If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                    If - Conditions
                        (PlayerNumber[(Integer A)] controller) Equal to User
                        (PlayerNumber[(Integer A)] slot status) Equal to Is playing
                    Then - Actions
                        Unit - Create 1 Peasant for PlayerNumber[(Integer A)] at (Center of start <gen>) facing 0.00 degrees
                    Else - Actions
                        Do nothing

 

[1337D00D]

The other players are used and controlled by a non-computer player, right?
And PlayerNumber[] has been set to a player?

Did it only create a peasant for you?

If it entirely does not work, try this trigger:

Blah
    Events
        Map initialization
    Conditions
    Actions
        For each (Integer A) from 1 to 12, do (Actions)
            Loop - Actions
                If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                    If - Conditions
                        ((Player((Integer A))) slot status) Equal to Is playing
                        ((Player((Integer A))) controller) Equal to User
                    Then - Actions
                        Unit - Create 1 Peasant for (Player((Integer A))) at (Center of (start <gen>)) facing Default building facing degrees
                    Else - Actions
                        Do nothing

 

[elmstfreddie]

Yeah, you don't need a player variable. You just use the function or w/e it's called Player Index.

 

[216199]

Yes, when I use that function there are no problems, but I don't see a reason why it shouldn't work with a variable.

 

[elmstfreddie]

You gotta set each array on the Player variable to the player the array number represents in a separate trigger first.