Why i cannot set random 1 unit as TempUnit from TempGroup?

[JFAMAP]

• Unit Group - Pick every unit in (Units in (Playable map area) owned by (Player((Integer A)))) and do (Actions)
  • Loop - Actions
    • Multiple FunctionsIf (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Multiple ConditionsOr - Any (Conditions) are true
          • Conditions
            • (Unit-type of (Picked unit)) Equal to (==) TestStructure1
            • (Unit-type of (Picked unit)) Equal to (==) TestStructure2
            • (Unit-type of (Picked unit)) Equal to (==) TestStructure3
            • (Unit-type of (Picked unit)) Equal to (==) TestStructure4
      • Then - Actions
        • Unit Group - Add (Picked unit) to TempGroup
        • Game - Display to (All players) for 2.00 seconds the text: (picked + (Name of (Picked unit)))
        • Set TempUnit = (Random unit from TempGroup)
        • Game - Display to (All players) for 1.00 seconds the text: (Name of TempUnit)

I can see the name of Picked Unit but it says (null) when i do this.
Set TempUnit = (Random unit from TempGroup)
Why i cannot get random unit from TempGroup?

 

[Chaosium]

I think the issue is in your If, you're checking whether an unit is 1 AND 2 AND 3 AND 4, so no unit can ever fit that description unless you add an "or" in the conditions

 

[JFAMAP]

When i use TempUnit as PickedUnit probably it picks last unit from condition right? I just want to pick random 1 unit from these list.
I am bit of confused in here. How can i do that? I want to pick any of these units within condition.

 

[Chaosium]

Nevermind I'm a dummy, I didn't see the "or" in the conditions, my bad <.<

 

[JFAMAP]

No problem. If i will set TempUnit at Picked Unit, which one will be the picked unit? Last one right?

 

[Uncle]

You probably want to do something like this:

• Unit Group - Pick every unit in (Units in (Playable map area) owned by (Player((Integer A)))) and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Or - Any (Conditions) are true
          • Conditions
            • (Unit-type of (Picked unit)) Equal to TestStructure1
            • (Unit-type of (Picked unit)) Equal to TestStructure2
            • (Unit-type of (Picked unit)) Equal to TestStructure3
            • (Unit-type of (Picked unit)) Equal to TestStructure4
      • Then - Actions
        • Unit Group - Add (Picked unit) to TempGroup
        • Game - Display to (All players) for 2.00 seconds the text: (picked + (Name of (Picked unit)))
      • Else - Actions
• Set TempUnit = (Random unit from TempGroup)
• Game - Display to (All players) for 1.00 seconds the text: (Name of TempUnit)

So you fill TempGroup with all of the different TestStructures. Then after that's finished you try to get one at random.

If TempGroup is empty then TempUnit will be set to "No unit", so you may want to check for that:

• Set TempUnit = (Random unit from TempGroup)
• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • TempUnit Not equal to No unit
  • Then - Actions
    • -------- We found a random unit --------
  • Else - Actions
    • -------- We could NOT find a random unit --------