[JASS] "Destroying" Indexers

[Ciebron]

I want to destroy a struct which is saved in a indexer, but i have 2 places it can get destroyed. 1. Inside the loop and 2. from a method.

globals
     private timer Timer = CreateTimer()
     private ASDF array A
     private integer Index = 0
endglobals

struct ASDF

integer INDEX

    static method create takes nothing returns thistype
        local thistype this = thistype.allocate()

        //struff
        set this.INDEX = Index        

        if Index == 0 then
             call TimerStart(//blaH)
        endif
        

        set A[Index] = this
        set Index = Index + 1
    endmethod

    method Destroy takes nothing returns nothing
          //Here is the question, will this break the indexer?
          //Is there are better way, then how?
          call this.destroy()
 
          set Index = Index - 1
          set A[this.INDEX] = A[Index]

          if Index == 0 then
               call PauseTimer(Timer)
          endif
    endmethod

endstruct

 

[Pharaoh_]

Delete it through the method, but after the "setA[this.INDEX] = A[Index]".
If you don't destroy it at that point, it should be destroyed/-recycled, when Timer gets recycled.

 

[Ciebron]

Im not talking about the Timer im talking about the indexer!

This; http://www.hiveworkshop.com/forums/...ls-280/vjass-indexer-tutorial-structs-114223/

What i want to know is if this is the best way too remove a instance from the indexer, and if it wont break it.

 

[maskedpoptart]

The best way to do this is using the onDestroy instance method, which executes right before the instance is destroyed. Simply call .destroy() and the indexer will do its stuff as well as recycle the struct index. Here you go:

globals
     private timer Timer = CreateTimer()
     private ASDF array A
     private integer Index = 0
endglobals

struct ASDF

    integer INDEX

    static method create takes nothing returns thistype
        local thistype this = thistype.allocate()

        //struff
        set this.INDEX = Index

        if Index == 0 then
             call TimerStart(//blaH)
        endif


        set A[Index] = this
        set Index = Index + 1
        return this
    endmethod

    method onDestroy takes nothing returns nothing
          set Index = Index - 1
          set A[this.INDEX] = A[Index]
          set A[this.INDEX].INDEX = this.INDEX

          if Index == 0 then
               call PauseTimer(Timer)
          endif
    endmethod

endstruct