[JASS] GetEnumUnit() isnt working

[SlayerII]

function Fly takes nothing returns nothing
    local unit u = GetEnumUnit()
    local real rx = LoadReal( udg_hashTable, GetHandleId(u), StringHash("X"))
    local real ry = LoadReal( udg_hashTable, GetHandleId(u), StringHash("Y"))
    local real time = LoadReal( udg_hashTable, GetHandleId(u), StringHash("t"))
    if (time>0) then 
    call SetUnitX(u,GetUnitX(u)+rx)
    call SetUnitY(u,GetUnitY(u)+ry)
    endif
    call DisplayTextToForce( GetPlayersAll(), "Work")
    set u = null
    call KillUnit(u)
endfunction

function Trig_Fly_Actions takes nothing returns nothing
    call ForGroup( udg_misslegroup, function Fly )
endfunction
//===========================================================================
function InitTrig_Fly takes nothing returns nothing
    set gg_trg_Fly = CreateTrigger(  )
    call TriggerRegisterTimerEventPeriodic( gg_trg_Fly, 0.03 )
    call TriggerAddAction( gg_trg_Fly, function Trig_Fly_Actions )
endfunction

call DisplayTextToForce( GetPlayersAll(), "Work") is working
but the rest of the trigger8whoch is related to "u") isnt working.
Pls help

 

[Ironside]

First of all, why making it JASS if you use user defined globals?
And EnumUnit() does nothing because your group is always empty as i see

 

[SlayerII]

i filled the group with a other trigger and as long it is empty call

DisplayTextToForce( GetPlayersAll(), "Work")

isnt working(which should be like that)

 

[Anachron]

The group has to be empty. Else you are simply doing it wrong (lol).

 

[SlayerII]

?? what? i just wanna now wy getenumunit isnt working

 

[Anachron]

It works, but its quite simple: The group you used seems to be empty or having dead units, so no unit can be affected by it.

 

[Dr Super Good]

Your testing does not concluded that GetEnumUnit is not working, it purly concludes that the hashtable recall is failing.

function Fly takes nothing returns nothing
    local unit u = GetEnumUnit()
    local real rx = LoadReal( udg_hashTable, GetHandleId(u), StringHash("X")) //Imagine this fails and rx becomes 0.
    local real ry = LoadReal( udg_hashTable, GetHandleId(u), StringHash("Y")) //Imagine this fails and ry becomes 0.
    local real time = LoadReal( udg_hashTable, GetHandleId(u), StringHash("t")) //Imagine this fails and time becomes 0.
    if (time>0) then //Will not work unless time is greater than 0, so obviously nothing will happen if the hashtable fails to work. 
        call SetUnitX(u,GetUnitX(u)+rx) //If hashtable load fails you move the unit to where it is already.
        call SetUnitY(u,GetUnitY(u)+ry) //If hashtable load fails you move the unit to where it is already.
    endif
    call DisplayTextToForce( GetPlayersAll(), "Work") //This means shit all for a test.
    set u = null
    call KillUnit(u) //Um, this is usless and should be deleted as you are removing null which makes shit all sense.   Invert this line with the line above as you are after killing the unit and then nulling the local which held it.
endfunction

function Trig_Fly_Actions takes nothing returns nothing
    call ForGroup( udg_misslegroup, function Fly )
endfunction
//===========================================================================
function InitTrig_Fly takes nothing returns nothing
    set gg_trg_Fly = CreateTrigger(  )
    call TriggerRegisterTimerEventPeriodic( gg_trg_Fly, 0.03 )
    call TriggerAddAction( gg_trg_Fly, function Trig_Fly_Actions )
endfunction

Thus in conclusion, your GetEnumUnit() is working perfectly, everything else is not.
Make sure the hashtable you reference is actually created, afterall null can not store values (make sure you initialize a hashtable to the global).
Also make sure the recall makes sense of the values (they are the same type as was attached and in the same indexes).

 

[SlayerII]

no its not empty it is filled with a other trigger...

as long the other trigger doesnt fire this trigger doesnt do anything, after the other one fired this lione suddenly is working:
DisplayTextToForce( GetPlayersAll(), "Work")
But not the FIRST one! which is the most important one

function Trig_Balls_Conditions takes nothing returns boolean
    if ( GetSpellAbilityId() == 'A000' ) then
        return true
    endif
    return false
endfunction

function Trig_Balls_Actions takes nothing returns nothing
    //variables
        local unit u = GetTriggerUnit()
        local location l = GetSpellTargetLoc()
        local unit u2 = CreateUnitAtLoc(GetOwningPlayer(u),'h000',l,0)
        local real array r
        local real rx
        local real ry
        local real rr
        set r[0] = GetUnitX(u)
        set r[1] = GetUnitY(u)
        set r[2] = GetUnitX(u2)
        set r[3] = GetUnitY(u2)
        set rx = r[2]-r[0]
        set ry = r[3]-r[1]
        set rr = SquareRoot(rx*rx+ry*ry)
        set rx = rx/rr
        set ry = ry/rr
    //code
        call SetUnitX (u2,r[0])
        call SetUnitY (u2,r[1])
        call SetUnitX (u2,r[0]+rx*500)
        call SetUnitY (u2,r[1]+ry*500)
        call GroupAddUnit(udg_misslegroup,u2)
        call SaveReal( udg_hashTable, GetHandleId(u2), StringHash("X"),rx*3)
        call SaveReal( udg_hashTable, GetHandleId(u2), StringHash("Y"),ry*3)
        call SaveReal( udg_hashTable, GetHandleId(u2), StringHash("t"),2.1)
    //leaks
        call RemoveLocation(l)
        set u= null
        set u2= null
        set l = null
endfunction

//===========================================================================
function InitTrig_Balls takes nothing returns nothing
    set gg_trg_Balls = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_Balls, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddCondition( gg_trg_Balls, Condition( function Trig_Balls_Conditions ) )
    call TriggerAddAction( gg_trg_Balls, function Trig_Balls_Actions )
endfunction

This is the other trigger

 

[Dr Super Good]

Again, I believe the entire fault here is udg_hashTable is null. You need to make sure that some time during initialization of your map udg_hashTable is set to a new hashtable object or nothing will work (like now).

 

[SlayerII]

function Trig_Hash_Actions takes nothing returns nothing
    call InitHashtableBJ(  )
    set udg_hashTable = GetLastCreatedHashtableBJ()
endfunction

//===========================================================================
function InitTrig_Hash takes nothing returns nothing
    set gg_trg_Hash = CreateTrigger(  )
    call TriggerAddAction( gg_trg_Hash, function Trig_Hash_Actions )
endfunction

thats my hasch init trigger

 

[Dr Super Good]

Are you sure it is even run?

 

[Anachron]

Maybe he activated the "Runs on map initialisation"
Else, I pretty think

1.) Either the force has no valid units.
2.) The hashtable is wrong.
3.) You are doing it wrong.

 

[SlayerII]

Oo it wasnt running
now it works
pretty thanks to you both