[Trigger] Trigger Help

[Gallin]

this is the trigger for changing the color of a unit and moving it to another region heres the trigger
Untitled Trigger 001

• Events
  • Time - Elapsed game time is 5.00 seconds
  • Conditions
    • (Unit-type of (Triggering unit)) Equal to Red Blood Cell
  • Actions
    • Unit Group - Pick every unit in (Units in right foot <gen>) and do (Actions)
      • Loop - Actions
    • Unit - Change color of (Triggering unit) to (Color of Player 2 (Blue))
    • Unit - Order (Picked unit) to Move To (Center of right ankel to right hip <gen>)

 

[Xelephant]

Hmm what you intended to do with this trigger? Moving Units from one region to another an recolor them?

 

[Gallin]

yes i am trying to do that

 

[redmarine]

• Move Blue Blood Cells :D
  • Events
    • Time - Elapsed game time is 5.00 seconds
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in right foot <gen>) and do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Unit-type of (Picked unit)) Equal to Blood Cell
          • Then - Actions
            • Unit - Change color of (Picked unit) to (Color of Player 2 (Blue))
            • Unit - Order (Picked unit) to Move To (Center of right ankel to right hip <gen>)
          • Else - Actions

Done. Enjoy.

 

[Gallin]

ok thxs alot this will work to change color back and for them to move i jsut have to chnage the rgions aorund?? oo and another tihng that says game time Elapsed is 5 seocnds i want this is go on over and over

 

[Gallin]

this isnt working

• right foot to ankel
  • Events
    • Time - Every 2.00 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in right foot <gen>) and do (Actions)
      • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Unit-type of (Picked unit)) Equal to Red Blood Cell
      • Then - Actions
        • Unit - Change color of (Picked unit) to (Color of Player 2 (Blue))
        • Unit - Order (Picked unit) to Move To (Center of right ankel to right hip <gen>)
      • Else - Actions

 

[Maddawg101]

Your problem is that the actions aren't lining up correctly.
Here's what you have:

• right foot to ankel
  • Events
    • Time - Every 2.00 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in right foot <gen>) and do (Actions)
      • Loop - Actions
    • **If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Unit-type of (Picked unit)) Equal to Red Blood Cell
      • Then - Actions
        • Unit - Change color of (Picked unit) to (Color of Player 2 (Blue))
        • Unit - Order (Picked unit) to Move To (Center of right ankel to right hip <gen>)
      • Else - Actions

Check out the **

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)

It is not being looped because it is not under the Loop - Actions category.
Although it is below the trigger it is not within that part of the trigger.
I hope that makes sense yea?

 

[redmarine]

Maddawg101 if I put it in the loop will it stop functioning?

 

[Maddawg101]

• Move Blue Blood Cells Correctly
  • Unit Group - Pick every unit in (Units in right foot <gen>) and do (Actions)
    • Loop - Actions
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)

• Move Blue Blood Cells Almost
  • Unit Group - Pick every unit in (Units in right foot <gen>) and do (Actions)
    • Loop - Actions
  • If (All Conditions are True) then do (Then Actions) else do (Else Actions)

There's the comparison. Redmarine you had it in the loop so it would work correctly, Gallin was off by just a line when he was looking at yours so that's why his wouldn't loop or work.

So here's what you want Gallin

• Move Blue Blood Cells Correctly
  • Events
    • Time - Every 2.00 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in right foot <gen>) and do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Unit-type of (Picked unit)) Equal to Blood Cell
          • Then - Actions
            • Unit - Change color of (Picked unit) to (Color of Player 2 (Blue))
            • Unit - Order (Picked unit) to Move To (Center of right ankel to right hip <gen>)
          • Else - Actions

 

[Gallin]

im not seeing the differnce could u maybe put it in a map and give it to me??

 

[GhostWolf]

Put your If/Then/Else inside the group (pick) actions...

 

[Gallin]

how do u do that?????? i tohught i did do that

 

[Ghan_04]

The code needs to be indented under where it says Loop - Actions.
What you have is that the unit group loop loops through, but does nothing, then, after it's finished, it goes down to the next action in list (the if statement) and executes it once.

You need to make it such that the unit group loop executes that if statement in the loop itself.

 

[Maddawg101]

• Move Blue Blood Cells Correctly
  • Unit Group - Pick every unit in (Units in right foot <gen>) and do (Actions)
    • Loop - Actions
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)

• Move Blue Blood Cells Almost
  • Unit Group - Pick every unit in (Units in right foot <gen>) and do (Actions)
    • Loop - Actions
  • If (All Conditions are True) then do (Then Actions) else do (Else Actions)

Here's the difference for you via the triggers, as said before the IF part needs to be within the Loop -Actions.

 

[Gallin]

would this work

• right foot to ankel
• Events
  • Time - Every 2.00 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in (right foot)) and do (If ((Unit-type of (Picked unit)) Equal to Red Blood Cell) then do (Unit - Order (Picked unit) to Move To (Center of right ankel to right hip <gen>)) else do (Do nothing))

 

[Ghan_04]

Click and hold down on the If function.
Then move it up little by little.
You should see a gray bar show up as you move it upward.
Maneuver this gray bar so that it is indented under the Loop - Actions heading.
Then release the mouse.

See if you can get it.
(The bar may be black or so. It's not necessarily uniform.)

 

[redmarine]

Just type (Ctrl + X or C) on your if/the/else function. Highlight the loop and press (Ctrl + V). Voila! that should take care of it.
And no! if you try to do that within the group trigger it most be matching unit. Not picking.

 

[GhostWolf]

It would work just find but it would leak a group.

Add this custom script before your pick line - "set bj_wantDestroyGroup = true".

By the way, nice region names