more efficient

[envenger]

Which one of the two is more is more efficient of the two?

1. Checking if a unit has a particular buff or not

like

function Trig_Untitled_Trigger_002_Actions takes nothing returns nothing
if (GetUnitAbilityLevel(GetTriggerUnit(), 'B000') > 0) then
call DisplayTextToForce( GetPlayersAll(), "ok" )
endif

or checking a unit array of 50 and checking is a unit it there or not using loop.

I heard checking unit buff is very less efficient so asking...

 

[mckill2009]

as you can see buff checking is like ability level check so its fine, its also work like GetSpellAbilityId()=='blah'

 

[envenger]

as you can see buff checking is like ability level check so its fine, its also work like GetSpellAbilityId()=='blah'

ok thanks....

 

[-Kobas-]

Whatever method you use you will end with checking some integer comparasion.
GetUnitAbilityLevel(GetTriggerUnit(), 'B000') > 0)
Will be better than loop in any way, after all you don't need to call some functions/natives 50+ times.
Code is at least 3 or 4 lines shorter so I suggest first one.

 

[envenger]

Actually i got to do both in yes condition.

Like if unit has the buff.(yes condition)

then search in the array for the unit.

If unit doesn't has the buff.(no condition)

then don't search in the array.

And there is about 95% chance the unit won't have the buff.