[JASS] Filtering Units

[Garfield1337]

I'd like to know how to use filters for

GroupEnumUnitsInRangeOfLoc

For example,i want to pick all units that are heroes.

 

[Bribe]

Don't use "in range of loc". Use "in range".

function filterOnlyHeroes takes nothing returns boolean
    if IsUnitType(GetFilterUnit(), UNIT_TYPE_HERO) then
        // do things
    endif
    return false
endfunction
//
call GroupEnumUnitsInRange(bj_lastCreatedGroup, 0.0, 0.0, 500.0, Filter(function filterOnlyHeroes))

 

[Garfield1337]

I just want to assign group,not pick each unit and do actions,so would this work?

function MyFunction takes nothing returns nothing
    local group g
    call GroupEnumUnitsInRange(g, 0.0, 0.0, 500.0, Filter(IsUnitType(GetFilterUnit()),UNIT_TYPE_HERO)

 

[TriggerHappy]

No, it has to be a function callback. Use the way bribe described.
You also need to create the group too (CreateGroup()).

 

[Garfield1337]

Ok,thanks.
Just 1 thing,why is it 'return false' at Bribe's example? Shouldn't it be:

function filterOnlyHeroes takes nothing returns boolean
    return IsUnitType(GetFilterUnit(), UNIT_TYPE_HERO)
endfunction

Off-topic: Internal error strikes again

 

[D4RK_G4ND4LF]

Ok,thanks.
Just 1 thing,why is it 'return false' at Bribe's example? Shouldn't it be:

function filterOnlyHeroes takes nothing returns boolean
    return IsUnitType(GetFilterUnit(), UNIT_TYPE_HERO)
endfunction

Off-topic: Internal error strikes again

he also wrote "do things"
if you want to use all units only once it is much faster to let the filter function return false and do all actions in the filter
if you want to use the units more then once your soultion would be right of course
for the first solution it would be better to use a global group 'cause it always leaks a little bit if you destroy a group and the group won't have any units in it anyway since you always return false

 

[Garfield1337]

EDIT: Nevermind,i got it. Thanks all

 

[Garfield1337]

Instead of making a new thread,i thought of asking here again.

Don't use "in range of loc". Use "in range".

function filterOnlyHeroes takes nothing returns boolean
    if IsUnitType(GetFilterUnit(), UNIT_TYPE_HERO) then
        // do things
    endif
    return false
endfunction
//
call GroupEnumUnitsInRange(bj_lastCreatedGroup, 0.0, 0.0, 500.0, Filter(function filterOnlyHeroes))

If i use global group,do i have to destroy it afterwards to prevent leak? Since in this case,it will be empty as everything will be done in filter.

 

[Frotty]

That would be the kinda same thing as a local.
You just create the group once and at the end of your actions you just clear the group. (call ClearGroup(group g))

 

[Bribe]

Use bj_lastCreatedGroup. Never destroy it, and never create it. It is a perpetual group that is always preserved by the game since one of the latest patches.