Advice about data storage for aggro system

[Embrace_It]

Hi guys,

I'm making an aggro system, like in WoW.

I need some kind of system that can store the data.
I need to store the following:

- The unit registered to the system
- The five units on the above unit's aggro list
- The five units' aggro (integers)

I was considering using AIDS, but then I read this in the documentation:

- You cannot use array members.

Please suggest an alternative I'm thinking hashtables, but I'm having a hard time figured how to save it. Maybe something like:

call SaveUnit(HT, GetHandleId(registered), 0, registered)
call SaveUnit(HT, GetHandleId(registered), 1, unit1)
...
call SaveInteger(HT, GetHandleId(registered), 1, aggro1) // &lt;-- collision? I figure blizz would take this into account <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite7" alt=":p" title="Stick Out Tongue    :p" loading="lazy" data-shortname=":p" />

 

[Jesus4Lyf]

>- You cannot use array members.
I would suggest AIDS:

struct FiveUnits
    private unit array data[5]
    method operator [] takes integer index returns unit
        return this.data[index]
    endmethod
    method operator []= takes integer index, unit whichUnit returns nothing
        set this.data[index]=whichUnit
    endmethod
endstruct
struct MyAIDSDataStruct extends array
    private FiveUnits units // now functions same as private unit array units[5]
    //! runtextmacro AIDS()
    private method AIDS_onCreate takes nothing returns nothing
        set this.units=FiveUnits.create()
    endmethod
    private method AIDS_onDestroy takes nothing returns nothing
        call this.units.destroy()
    endmethod
endstruct

It's not really true that you can't use array members...

By the way, the alternative is you can use a wrapper. It will still be faster than hashtables off memory:

struct MyUnitData
    private unit unit
    private unit array units[5]
    private unit array aggro[5]
    
    static method create takes unit u returns thistype
        local thistype this=thistype.allocate()
        set this.unit=u
        return this
    endmethod
endstruct
struct MyUnitDataWrapper extends array
    MyUnitData data
    //! runtextmacro AIDS()
    private method AIDS_onCreate takes nothing returns nothing
        set this.data=MyUnitData.create(this.unit)
    endmethod
    private method AIDS_onDestroy takes nothing returns nothing
        call this.data.destroy()
    endmethod
endstruct
function GetData takes unit u returns MyUnitData
    return MyUnitDataWrapper<u>.data // inlines to Array[Native(u)]
endfunction
</u>

 

[Nestharus]

Or you can use AIDS in conjunction with a 2 stacks.

That'd be best you were wanting a dynamic number (not exactly 5 each time) .

Although, you could want 5 each time, in which case Jesus4Lyf's method is best ; )

 

[Jesus4Lyf]

>Or you can use AIDS in conjunction with a 2 stacks.
Not stacks, linked lists. But the nice thing would be making them a linked list means each node would be a struct, meaning each node could contain a unit, the aggro value, and anything else to attach...
I'd do it. But my point with my post was actually AIDS structs can have array members.

 

[Embrace_It]

It's not really true that you can't use array members...

You should probably edit your documentation then For the rest of us that aren't that advanced in vJass, this is missleading, although I understand now :thup:

AIDS it is then, thanks again Jesus4Lyf and Nestharus!

EDIT: Gotta spread some rep first. You help me all too often