[vJASS] about selections

[machris13]

scope Selection initializer onInit

    private function select takes nothing returns nothing
        
    
        local integer index = GetPlayerId(GetTriggerPlayer())
        
    if DONT[index] == false then
        set PlayerTarget[index] = GetTriggerUnit()
        set DONT[index] = true
        call SelectUnitForPlayerSingle(gg_unit_Hmkg_0000, GetTriggerPlayer())
        set DONT[index] = false
    endif
    
    endfunction
    
    private function onInit takes nothing returns nothing
        local trigger t = CreateTrigger()
        local integer index = 0
        call TriggerAddAction(t, function select)
        loop
            call TriggerRegisterPlayerSelectionEventBJ(t, Player(index), true)
            set index = index + 1
            exitwhen index == bj_MAX_PLAYERS
        endloop
    endfunction

endscope

i was wondering why if i select another unit, it still says that PlayerTarget is gg_unit_Hmkg_0000..

i have this:

scope Globals initializer onInit

    globals
        boolean array DONT
        constant string INV_TAR = "Invalid Target!"
        constant string NO_MAN = "Not Enought Mana!"
        constant string TOO_FAR = "Target Too Far Away!"
        constant string TOO_NEA = "Target Too Near!"
        constant string NO_TAR = "No Target!"
        constant string MAX_HP = "Your Target's HP is full!"
    endglobals

    function strings takes nothing returns nothing
        local integer index = 0
        debug call BJDebugMsg( "Globals initialized" )
        loop
            set DONT[index] = false
            set index = index + 1
            exitwhen index == bj_MAX_PLAYERS
        endloop
    endfunction

    private function onInit takes nothing returns nothing
        local trigger t = CreateTrigger()
        call TriggerAddAction(t, function strings)
        call TriggerRegisterTimerEventSingle(t, 1.00)
    endfunction

endscope

and:

debug call BJDebugMsg(GetUnitName(PlayerTarget[index]))

what i want is when i select a unit, the unit becomes the

PlayerTarget[index]

, after that the selection will go back to the hero.. please help me

i know its quite confusing but please ask me questions dealing this.

 

[Adiktuz]

try changing the scope Globals into a library...

and its better if you post the whole code containing the function call for showing the name of the target...

not sure but maybe the select action still interferes with the code...

this is not related to prob, but use Conditions as its faster than actions... and also if DONT[index] == false is just the same as if not DONT[index] ...

 

[machris13]

i think it would be better if i post my map

 

[Adiktuz]

I don't have time to open a map...

 

[machris13]

as always

if you have time dude..its just a tester map nothing large

 

[Pharaoh_]

Where is the condition to prevent the script from counting Mountain King as the unit under selection? You issue a selection of the Mountain king and the select function fires again. Use GetTriggerUnit() != gg_unit_Hmkg_0000 instead of those booleans, that replace their previous values really fast.