so I decided to learn Jass finally

[TheDamien]

Will that execute the function 'group_stuff'?

Yes, but the function will be executed as a group filter, which means it must return a boolean (use false in this case) and you must use GetFilterUnit() instead of GetEnumUnit().

 

[Vestras]

Negative. Filter(...) is a conditions, so that function would be a conditions, for example if I wanted enemy units in the group.

 

[TheDamien]

Both Filter and Condition work, as far as I know. I personally use Condition.

 

[duyen]

So that's not what I want <_<

 

[Vestras]

When you have declared the group, just do call ForGroup(...)

 

[Larcenist]

An alternative is the FirstOfGroup() loop.

function Substitute takes nothing returns boolean
    return true
endfunction

function blablabla...

local group g = CreateGroup()
local real x = GetUnitX(caster)
local real y = GetUnitY(caster)
local unit u

call GroupEnumUnitsInRange(g, x, y, AoE, Condition(function Substitute))

loop
    set u = FirstOfGroup(g)
    exitwhen u == null
    if IsUnitAlly(u, GetOwningPlayer(caster)) == true and IsUnitType(u, UNIT_TYPE_STRUCTURE) != true and GetWidgetLife(u) > .405 then
        //Do your actions
    endif
    call GroupRemoveUnit(g, u)
endloop

 

[TheDamien]

I suggest just using the GroupEnum filter function. ForGroup and FirstOfGroup loops are unnecessary in this situation.

 

[duyen]

It's all good now.

Thx for help.

 

[Flare]

Hmmm, just noticed this

set hlife = 0
set glife = 0
set gx = 0
set gy = 0

You don't need to do that

1) You're always setting the value of the variable before using it, so it doesn't really matter

2) Since those are local, each time the function runs, a new instance of that variable is generated (and it starts out completely blank) so there's no problem with the variables having incorrect values unless you specifically give it the wrong value

 

[Romek]

function Substitute takes nothing returns boolean
    return true
endfunction
// Other stuff
call GroupEnumUnitsInRange(g, x, y, AoE, Condition(function Substitute))

You can use null for a boolexpr argument.

call GroupEnumUnitsInRange(g, x, y, AoE, null)

This would have the same effect.

 

[Flare]

That leaks Wouldv'e been silly not to pass null to the boolexpr argument otherwise ^_^