[Solved] i have a return problem

[deathismyfriend]

i have a unit enters region event and it should fire when player 0 enters this rect. problem is it doesnt return true can anyone tell me y. here is the code

return reg[GetPlayerId( GetOwningPlayer( GetTriggerUnit()))] == GetTriggeringRegion()

 

[Maker]

Is the region created?

 

[deathismyfriend]

sry shouldve posted that here is the function that the reg is created in and that the condition is added in.

        private static method setRects takes nothing returns nothing
            local trigger t1 = CreateTrigger()
            local integer L = 0
            loop
                exitwhen L > 12
                set reg[L] = CreateRegion()  
                set L = L + 1
            endloop
            set L = 0
            call RegionAddRect( reg[0], gg_rct_urRectName)
            call RegionAddRect( reg[1], gg_rct_urRectName1)
            call RegionAddRect( reg[2], gg_rct_urRectName2)
            loop
                exitwhen L > 11
                call TriggerRegisterEnterRegion( t1, reg[L], function thistype.enterReg)
                set L = L + 1
            endloop
            call TriggerRegisterEnterRegion( t1, reg[12], null)
            call TriggerAddCondition( t1, function thistype.inRect)
            set t1 = null
        endmethod

this also returns false which i didnt think was possible lol

IsUnitInRegion( GetTriggeringRegion(), GetTriggerUnit())

 

[OmagnaT]

i have a unit enters region event and it should fire when player 0 enters this rect. problem is it doesnt return true can anyone tell me y. here is the code

return reg[GetPlayerId( GetOwningPlayer( GetTriggerUnit()))] == GetTriggeringRegion()

Is this condition even neccessary? I dunno you're probably gonna have to post up the rest of the trigger but this

[GetPlayerId( GetOwningPlayer( GetTriggerUnit()))]

should be like this by default. If the above was your purpose, then yea, moar info

[GetPlayerId( GetOwningPlayer( GetTriggerUnit()))+1]

 

[deathismyfriend]

this is for gui users

[GetPlayerId( GetOwningPlayer( GetTriggerUnit()))+1]

 

[Maker]

IsUnitInRegion can return false for the region that is entered. I'm not sure why that is, maybe due to collision size.

Do you need to detect the entered region?

 

[deathismyfriend]

yes i need to check the region something like this

return reg[GetPlayerId( GetOwningPlayer( GetTriggerUnit()))] == GetTriggeringRegion()

 

[Maker]

You could use a hashtable and save an integer (id) for the region.

 

[deathismyfriend]

tried that still the same problem

this wont work either

return IsUnitInRegion( reg[GetPlayerId( GetOwningPlayer( GetTriggerUnit()))], GetTriggerUnit())

 

[Maker]

Look how I did it in this system (yeah the triggers are mine): http://www.hiveworkshop.com/forums/spells-569/portal-system-1-0-a-229816/

 

[deathismyfriend]

ooo i see u used the handleIds i just used the region one ill give it a try

i think i did something wrong can u find the problem ?
heres were i save it

set regionTable[GetHandleId(reg[L])] = GetHandleId(reg[L])

heres the loading

local integer p = GetPlayerId( GetOwningPlayer( GetTriggerUnit()))
return regionTable[GetHandleId(reg)] == GetHandleId( GetTriggeringRegion())

this returns 0 ?????

call BJDebugMsg( I2S( GetHandleId( GetTriggeringRegion())))

heres the map so u can see what i mean

 

[Maker]

When you register the event, leave the boolexpr to null. Use the enterReg when you register the condition.

 

[deathismyfriend]

sweet works great thanks a lot maker cant give u rep now tho sry

 

[Zwiebelchen]

I'm pretty sure preplaced rects are not initialized before any library/scope initialization. So unless you postpone the creation of your region, your region will be empty.

 

[deathismyfriend]

maker helped me get it to work. thx anyways zwiebelchen
this solved it
When you register the event, leave the boolexpr to null. Use the enterReg when you register the condition.

 

[Zwiebelchen]

maker helped me get it to work. thx anyways zwiebelchen
this solved it
When you register the event, leave the boolexpr to null. Use the enterReg when you register the condition.

Yeah and this is because of the thing I wrote.
You can not register a preplaced rect to the event because it isn't created at the moment you register the event.
Instead of registering the event at library init, register it after a 0 second timer and it should be fine.

 

[deathismyfriend]

hmm thats odd but understandable. i might try that tomorrow just to see what happens. thanks for that