Discussion A way to detect if an handle is invalid

Troll-Brain

You can change this now in User CP.
Reaction score
85
Maybe most of you already know but you can't store a "null" handle in an hashtable, neither an invalid handle, for example an handle which was destroyed but not setted to null.

I've just realized recently we can use that for safety checks.

JASS:
library CheckHandles initializer onInit

globals
    private hashtable HashT
endglobals

function IsGroupValid takes group whichGroup returns boolean
    return SaveGroupHandle(HashT,0,0,whichGroup)
endfunction

private function onInit takes nothing returns nothing
    set HashT = InitHashtable()
endfunction

endlibrary


Note that i'm perfectly aware about the priority of module initializer.
And yes, i could initialize the hashtable in the global declaration.
And ofc it might have better ways for specifics handles, like for units (GetUnitTypeId(<unit>) != 0)

What you think about that ?
A such library may exist ?

If it's really obvious, then amen, at least it wasn't for me :p
 

Troll-Brain

You can change this now in User CP.
Reaction score
85
JASS:
function IsHandleValid takes handle h returns boolean
    return GetHandleId(h) != 0
endfunction

Try?

That was a nice idea but it doesn't work, it displays "true".

JASS:
library CheckHandles initializer onInit

globals
    private hashtable HashT
endglobals

function IsGroupValid takes group whichGroup returns boolean
    return SaveGroupHandle(HashT,0,0,whichGroup)
endfunction

function IsHandleValid takes handle h returns boolean
    return GetHandleId(h) != 0
endfunction

private function onInit takes nothing returns nothing
    local group g = CreateGroup()
    call DestroyGroup(g)
    set HashT = InitHashtable()
    call TriggerSleepAction(1.)
    if IsHandleValid(g) then
        call BJDebugMsg(&quot;true&quot;)
    else
        call BJDebugMsg(&quot;false&quot;)
    endif
    
endfunction

endlibrary


Probably because the id isn't recycled (g isn't null), it returns a valid integer.

EDIT :

This works properly :

JASS:
function IsAgentValid takes agent whichAgent returns boolean
    return SaveAgentHandle(HashT,0,0,whichAgent)
endfunction


But i would a prefer a specific function for each type though.
 

Darthfett

Aerospace/Cybersecurity Software Engineer
Reaction score
615
What purpose would this serve?

JASS:
globals
    integer i = 0
endglobals

function DoSomethingCallback takes nothing returns nothing
    set i = i + 1
endfunction

function DoSomething takes group g returns integer
    set i = 0
    //if g != null then
    if IsAgentValid(g) then
        call ForGroup(g,function DoSomethingCallback)
        return 5 / i //without IsAgentValid, this will divide by 0.
    endif
    return 5
endfunction

function Test takes nothing returns nothing
    local group g = CreateGroup()
    call DestroyGroup(g)
    call DoSomething(g) //Oops!  Someone passed a non-null, but non-existant group!
endfunction
 

Troll-Brain

You can change this now in User CP.
Reaction score
85
It returns false if the agent is null or is a ghost one like in my example showed before or your's, and true only if he is valid.
 

kingkingyyk3

Visitor (Welcome to the Jungle, Baby!)
Reaction score
216
JASS:

function isHandleValid takes handle whichHandle returns boolean // you guys should use more descriptive arguement names. Even in examples. :/
     return whichHandle != null
endfunction


?

A variable with destroyed group does not have the value of null.
 

Troll-Brain

You can change this now in User CP.
Reaction score
85
Exactly, without benchmark it's pretty obvious that IsAgentValid is lightning fast, probably only 2 time slower than a variable setting.
I already use it instead of comparing to null (if i don't already need to store the argument in an hashtable ofc).

@Lyerae : So what, should i add the comment, "return true if handle is valid, and false if it is not", seems really obvious :p
 

Lyerae

I keep popping up on this site from time to time.
Reaction score
105
No, but [ljass]integer index[/ljass] is better than [ljass]integer i[/ljass] (as an example).
 

Troll-Brain

You can change this now in User CP.
Reaction score
85
No, but [ljass]integer index[/ljass] is better than [ljass]integer i[/ljass] (as an example).
Stop whining really.

I assume you wanted to say that since long time ago, but here it's pretty pointless, choice an other thread :D

Back on topic, i see that's not obvious so, at least for all persons which have posted here, i will make a such library then.
But in an other hand, it should be used for personal codes only, since it's easy to inline it, hmm maybe as a sample ?
 

Darthfett

Aerospace/Cybersecurity Software Engineer
Reaction score
615
No, but [ljass]integer index[/ljass] is better than [ljass]integer i[/ljass] (as an example).

Seriously? :nuts:

'i' is a standard name known to be used for an index, or iterator. Just the name of the function is enough to know what it's going to take and what it's going to do.

The code is not complicated, and sometimes simple code is the best.
 

Troll-Brain

You can change this now in User CP.
Reaction score
85
I just want to say that in Darthfett's example, it will divide by 0 also if the group is empty or at least full of ghost units only.
 
General chit-chat
Help Users
  • No one is chatting at the moment.

      The Helper Discord

      Members online

      No members online now.

      Affiliates

      Hive Workshop NUON Dome World Editor Tutorials

      Network Sponsors

      Apex Steel Pipe - Buys and sells Steel Pipe.
      Top