[Solved] unit group problem

[Nirds]

how to make a unit group take units only belonging to the enemy of my caster?

 

[Mercious]

GroupEnumUnitsInRange

With a filter funtion that does
return IsPlayerEnemy(GetOwningPlayer(GetFilterUnit()), GetOwningPlayer(YourUnit))

Alternatively you just get all units in range and then check for that condition when you loop through the group.

It's on my list to benchmark what is more efficient - I guess it depends on the setting. If there are a lot of non-enemies in range then you will have to remove a lot units from the resulting group, which is likely slower than the filter function calls.

 

[Nirds]

thanks problem solved!

 

[Anitarf]

It's on my list to benchmark what is more efficient - I guess it depends on the setting. If there are a lot of non-enemies in range then you will have to remove a lot units from the resulting group, which is likely slower than the filter function calls.

The filter function calls will always be slower, since they grow in proportion to the number of units enumed as well.

 

[Mercious]

The filter function calls will always be slower, since they grow in proportion to the number of units enumed as well.

Sure, but you forget about additional function-calls, that arise from you having to remove more units of the resulting groups when you loop through it.

That is, assuming that you use the FirstOfGroup() -> GrouRemoveUnit() iteration method.

But i guess even if not, with other iteration methods it is the same. More units in the resulting group -> more work.

 

[Anitarf]

The point is, adding (with the GroupEnum) and then removing the unit from the group (by calling GroupRemoveUnit, along with the FirstOfGroup call) is less costly than evaluating a filter function. So even if the area contains only units that don't pass the filter, it will still be faster to use a null filter function and then filter the units in the FirstOfGroup loop.

 

[Mercious]

How can you be so certain of this? Its not deducible by logic itself, so it must be something else.

Did you / someone benchmark it already?

 

[Anitarf]

Yes.

 

[Mercious]

Cool, then it's certain, thanks.