[JASS] Picking highest HP target in range?

[sarastrasza]

As the title says, how do I pick the highest HP unit in range?
thanks beforehand


//Sarastrasza

 

[Maker]

local real r = 0
local unit u = no unit

Create a unit group, pick all units inside the radius (matching conditions).
loop through units
if health of GetEnumUnit() > r
then
set u = GetEnumUnit()
set r = healt of GetEnumUnit()
else

_______________

Then you have the unit stored in u.

 

[sarastrasza]

this is what i got, is it anywhere near right?

        loop
            set First = FirstOfGroup(Hate)
                if ((GetWidgetLife(First)) >= hp) then
                set HpHigh = First
                set hp = (GetWidgetLife(First))
                call GroupRemoveUnit(Hate, First)
                elseif ((GetWidgetLife(First)) < hp) then
                call GroupRemoveUnit(Hate, First)
                endif 
        exitwhen (First == null)
        endloop

 

[bReathl3sS]

the code will work, but for some performance just make

loop
    set First = FirstOfGroup(Hate)
    exitwhen First == null
    if GetUnitState(First, UNIT_STATE_LIFE) >= hp then
        set HpHigh = First
        set hp = (GetWidgetLife(First))
    endif
    call GroupRemoveUnit(Hate, First)
endloop

this way you exit the loop directly if there is no unit and you also don't need to check if the unit has less hp than the one that was picked because ithat's the thing you don't need. and whether the if returns true or false: in both cases you remove the unit of the group, so you can just do that after the if.
you also may have recognized that i used GetUnitState(). i never tested GetWidgetLife(), so i used sth. i know.

greetings

 

[sarastrasza]

okay, thanks the both of you +rep