Unit selection in region

[D0W3R]

whats wrong with these triggers? I cant get them working...

• Untitled Trigger 001
  • Events
    • Player - Player 1 (Red) Selects a unit
  • Conditions
    • RoundOn Equal to True
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Unit-type of (Triggering unit)) Equal to Footman
      • Then - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (2a <gen> contains (Triggering unit)) Equal to True
          • Then - Actions
            • Unit Group - Pick every unit in (Units in 3a <gen>) and do (Actions)
              • Loop - Actions
                • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                  • If - Conditions
                    • (Number of units in (Units in 3a <gen>)) Equal to 0
                  • Then - Actions
                    • Unit - Create 1 Footman for Player 1 (Red) at (Center of 3a <gen>) facing Default building facing degrees
                  • Else - Actions
          • Else - Actions
      • Else - Actions

 

[Adiktuz]

please say what's not working...

THIS: if the unit group loop runs, there is no way thay the number of units in 3a is equal to 0... the create unit will never run... just remove the group loop, its useless...

here's an edited code, also fixed leaks

• Events
  • Player - Player 1 (Red) Selects a unit
• Conditions
  • RoundOn Equal to True
• Actions
  • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    • If - Conditions
      • (Unit-type of (Triggering unit)) Equal to Footman
    • Then - Actions
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • (2a <gen> contains (Triggering unit)) Equal to True
        • Then - Actions
          • Set TempGroup = Units in 3a <gen>
          • Set TempLoc = Center of 3a <gen>
          • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            • If - Conditions
              • (Number of units in TempGroup) Equal to 0
            • Then - Actions
              • Unit - Create 1 Footman for Player 1 (Red) at (TempLoc) facing Default building facing degrees
            • Else - Actions
          • Custom script: call RemoveLocation(udg_TempLoc)
          • Custom script: call DestroyGroup(udg_TempGroup)
          • Else - Actions

 

[D0W3R]

Ok it's working, but I can't destroy (udg_TempGroup)
I get error:
Line 367: Invalid argument type (real)

 

[GangSpear]

This is my idea,xD:

• First
• Events:
• Unit - A unit enters Region 014 <gen>
• Conditions:
• (Unit-type of (Entering unit)) Equal to Footman
• Actions:
• Trigger - Turn on Untitled Trigger 006 <gen>
• Trigger - Turn off (This trigger)

Next trigger(create a new trigger first)

• Events:
• Player - Player 1 (Red) Selects a unit
• Conditions:
• (Unit-type of (Triggering unit)) Equal to Footman
• Actions:
• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • (Region 014 <gen> contains (Triggering unit)) Equal to True
  • Then - Actions
    • Unit - Create 1 Footman for Player 1 (Red) at (Center of Region 014 <gen>) facing Default building facing degrees
    • Trigger - Turn off (This trigger)
  • Else - Actions

I think no problem == xD

 

[Adiktuz]

Ok it's working, but I can't destroy (udg_TempGroup)
I get error:
Line 367: Invalid argument type (real)

tempgroup is a group variable...

 

[D0W3R]

GangSpear you see I can't create 2 trigger for one unit becouse If Ill make for every unit 2triggers it duebles my work, from +200trigger it will need +400triggers this sucks.

 

[D0W3R]

tempgroup is a group variable...

For tempGroup I used units group but when I click test map he writes that TempGroup is real...

 

[Adiktuz]

For tempGroup I used units group but when I click test map he writes that TempGroup is real...

first dont double post...

try to save it first and try it using wc3 rather than test map...

or can you upload map, I'll see what's wrong in what you're doing...

 

[D0W3R]

OK. heres the map: http://www.hiveworkshop.com/forums/pastebin_data/7v8zb3/_files/Chess.w3x

 

[Adiktuz]

OK. heres the map: http://www.hiveworkshop.com/forums/pastebin_data/7v8zb3/_files/Chess.w3x

the problem is TempLoc not TempGroup... you set it to real... it should be Point... and use TempLoc in the Create Unit...

• Unit - Create 1 Footman for Player 1 (Red) at (TempLoc) facing Default building facing degrees

 

[D0W3R]

Im dumb... Thanks to you Now I'll can to create other things )