[General] How to make an array of boolexpr?

[sethmachine]

Hi,

For some reason it seems to be the case you can't have arrays of code.

But nothing is said about arrays of boolexpr. If I make a boolexpr, how do I put it into an array or other data structure?

 

[edo494]

boolexpr array bArray

 

[WaterKnight]

native SaveBooleanExprHandle (hashtable table, integer parentKey, integer childKey, boolexpr whichBoolexpr) returns boolean

 

[edo494]

or even better,
Table.boolexpr[index] = Filter(function f)

 

[Malhorne]

Just wondering, when do you need boolexpr array oO ?

 

[edo494]

dynamic trigger callbacks with O(1) insertion and average O(log n), worst case O(n) deletion

 

[Malhorne]

Ah you want to do dynamic triggers ?
I never used it and I don't see the real point.

 

[sethmachine]

I need them for dynamic quest making and other things.

But can you tell me why I can't have both a filter and a condition??

Edit: Nevermind. My logic was right, but I didn't know that you have to call "GetFilterUnit()."

I searched all night for a goddarn tutorial on filters and couldn't find any. If I had just known that I'd have solved this problem ages ago. All works now >_>

        call TriggerRegisterEnterRegion(t, goalRegion, playerRegionFilters[pid])
        //call TriggerAddCondition(t, playerRegionFilters[pid])
        call TriggerAddCondition(t, Condition(function regionEnterMain))

here is my filter function

function p1RegionFilter takes nothing returns boolean
    local unit u = GetTriggerUnit()
    local player p = GetOwningPlayer(u)
    if p == players[0] then
        return true
    endif
    return false
endfunction
...
    set playerRegionFilters[0] = Filter(function p1RegionFilter)

 

[WaterKnight]

There is no event response in event filters.

 

[Malhorne]

^He is right.

function p1RegionFilter takes nothing returns boolean
    local unit u = GetTriggerUnit()
    local player p = GetOwningPlayer(u)
    if p == players[0] then
        return true
    endif
    return false
endfunction

->
return GetOwningPlayer(GetTriggerUnit()) == player[0]
Btw in the function you wrote you leak ^^

 

[edo494]

what data type is playerRegionFilters?

 

[Malhorne]

I don't know sorry

 

[sethmachine]

Here is the version of the filter that works. It has to be true before the condition holds, and yes it's inside an array of boolexpr too.

I didn't fix any leaks because I was in a rush to get functionality. First get something to work, THEN optimize it in my opinion...

function p1RegionFilter takes nothing returns boolean
    local unit u = GetFilterUnit()
    local player p = GetOwningPlayer(u)
    if p == players[0] then
       set u = null
       set p = null
       return true
    endif
    set u = null
    set p = null
    return false
endfunction

 

[Malhorne]

Still return GetOwningPlayer(GetFilterUnit()) == players[0] is more efficient

 

[sethmachine]

Still return GetOwningPlayer(GetFilterUnit()) == players[0] is more efficient

Noted, made the change, thanks.

 

[Malhorne]

No problem !