[Solved] What's the use of ForGroup

[mckill2009]

SOLVED!
I was wondering about something...What's the use of ForGroup if there is GroupEnumUnitsInRange?...

Which do you preffer and faster?...

function MR_FILTER takes nothing returns boolean
    if IsUnitType(GetFilterUnit(), UNIT_TYPE_HERO)==false then
        call KillUnit(GetFilterUnit())
    endif
    return false
endfunction

function TEST takes nothing returns nothing
    call GroupEnumUnitsInRange(bj_lastCreatedGroup, GetUnitX(u), GetUnitY(u), 500, Filter(function MR_FILTER))
endfunction

function MR_FILTER takes nothing returns nothing
    if IsUnitType(GetEnumUnit(), UNIT_TYPE_HERO)==false then
        call KillUnit(GetEnumUnit())
    endif
endfunction

function TEST takes nothing returns nothing
    call GroupEnumUnitsInRange(bj_lastCreatedGroup, GetUnitX(u), GetUnitY(u), 500, null)
    call ForGroup(bj_lastCreatedGroup, function MR_FILTER)
endfunction

 

[Maker]

Use ForGroup if you want to loop through a group you have created previously.

If you add/remove units from a group with some other method for example, not with GroupEnumUnitsInRange.

The first method is better for this case.

 

[mckill2009]

Now that makes sense, GroupEnumUnitsInRange cant recall a group while ForGroup can...wow thanks, it clears up a lot!...

 

[Troll-Brain]

Now that makes sense, GroupEnumUnitsInRange cant recall a group while ForGroup can...wow thanks, it clears up a lot!...

Actually it clears the group and adds each filtered unit each time you return true on the boolexpr call.
If you don't need to keep units during time, you can always return false, use a static group (as the GROUP_ENUM of the library GroupUtils) and code directly inside the boolexpr of the GroupEnum.

EDIT : Just like you did in fact

 

[Adiktuz]

basically if you will only loop actions once for a new group, just use GroupEnum, but if you will have more than one function to be run by the same group or if the group is previously made, then use ForGroup... ^.^