Picking and Removing Units

[GoGoTauren]

In my map I'm having a lot of trouble removing a lot of locusted units with

• Round
  • Events
    • Player - Player 1 (Red) types a chat message containing -round as An exact match
  • Conditions
  • Actions
    • Set tempgroup = (Units in (Playable map area))
    • Unit Group - Pick every unit in tempgroup and do (Actions)
      • Loop - Actions
        • Unit - Remove (Picked unit) from the game
        • Game - Display to (All players) the text: (String((Number of units in tempgroup)))
        • Game - Display to (All players) the text: derp
    • Custom script: call DestroyGroup(udg_tempgroup)
    • For each (Integer A) from 1 to 4, 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
          • Then - Actions
            • Unit - Create 1 Engine for (Player((Integer A))) at ((Player((Integer A))) start location) facing (Center of (Playable map area))
            • Set Engine[(Integer A)] = (Last created unit)
          • Else - Actions

I am aware of the center of playable map area leak. I, honestly, do not care.


The other units get removed by this fine, but my mass of locust units does not. Do you guys have any tips? Do you want me to upload the map..?

 

[Dr. Boom]

Moin moin =)

Well locust units are unpickable at all but you can do this as far as I know:

1) Whenever you create a unit with locust, directly add them into another unit group [LastCreatedUnit]
Now you can pick all units in this unitgroup and "Kill" them (Remove unit is a leak btw)

2) When the locust units are already on the map, remove the locust ability from them (Object Editor)
Now you make at initialization a function where you pick up all units which should have locust, add them into the unitgroup and use this custom script:

• Custom script: call UnitAddAbility(GetEnumUnit(),'Aloc')

Now the units have locust again, are added in this unitgroup and can be removed from there.

Also "(Units in (Playable map area))" isn't a leak here, because you store it in a point but "((Player((Integer A))) start location)" is a leak

 

[GoGoTauren]

Moinmoin

Thanks for the reply, but the changes I've made dont seem to work.

EDIT: something I deemed insignificant but changed anyway after posting that seemed to fix it. Thanks for the help.

 

[Maker]

Well locust units are unpickable at all...

Both of these will display the number of units, even if they have locust

• Game - Display to Player Group - Player 1 (Red) the text: (String((Number of units in (Units owned by Player 12 (Brown)))))
• Game - Display to Player Group - Player 1 (Red) the text: (String((Number of units in (Units of type Footman))))

 

[Bribe]

Both of these will display the number of units, even if they have locust

• Game - Display to Player Group - Player 1 (Red) the text: (String((Number of units in (Units owned by Player 12 (Brown)))))
• Game - Display to Player Group - Player 1 (Red) the text: (String((Number of units in (Units of type Footman))))

Leaks two unit groups.

 

[Maker]

Leaks two unit groups.

It's just a demonstration to show which functions pick units with locust

 

[Bribe]

I know, but it's fun to joke like that

 

[Maker]

For a brief moment I though you were serious

 

[Bribe]

I'm never serious, I just like pretending to be. Dry humour is my speciality.

 

[defskull]

Well, we CAN pick Locust unit, by adding them to a Unit Group and use the "Unit Group - Pick Every Unit In Unit Group" action

 

[Miss_Foxy]

Or making it have more health/mana than other units would usually have and use the "Real - Unit Max Life Is Equal To" condition.