[Trigger] Backpack hashtable not switching items.

[Flitty]

This is basically my first attempt at using hashtables as it seems like the best way to handle it. I've probably done something fundamentally wrong here but it seems like it should work to me. it saves the players inventory when they cast open as 1-6 of that players index and removes the items. it then loads 6+ 1-6 where backpack items are stored and gives them to the hero. on closing it saves them and switches back.

the abilities switch and the items are removed but they don't seem to be saved correctly because they don't come back when close runs.

• Testing Init
  • Events
    • Map initialization
  • Conditions
  • Actions
    • Hashtable - Create a hashtable
    • Set Backpack = (Last created hashtable)

• Backpack
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Open Backpack
  • Actions
    • Set playerNum = (Player number of (Owner of (Triggering unit)))
    • Unit - Remove Open Backpack from (Triggering unit)
    • Unit - Add Close Backpack to (Triggering unit)
    • For each (Integer loopA) from 1 to 6, do (Actions)
      • Loop - Actions
        • Hashtable - Save Handle Of(Item carried by (Triggering unit) in slot loopA) as loopA of playerNum in Backpack
        • Item - Remove (Item carried by (Triggering unit) in slot loopA)
        • Hero - Give (Load (loopA + 6) of playerNum in Backpack) to (Triggering unit)

• Backpack Copy
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Close Backpack
  • Actions
    • Set playerNum = (Player number of (Owner of (Triggering unit)))
    • Unit - Remove Close Backpack from (Triggering unit)
    • Unit - Add Open Backpack to (Triggering unit)
    • For each (Integer loopA) from 1 to 6, do (Actions)
      • Loop - Actions
        • Hashtable - Save Handle Of(Item carried by (Triggering unit) in slot loopA) as (loopA + 6) of playerNum in Backpack
        • Item - Remove (Item carried by (Triggering unit) in slot loopA)
        • Hero - Give (Load loopA of playerNum in Backpack) to (Triggering unit)

 

[Bribe]

You are saving the handle of an item and then remove the item. I think you meant to save the item type of the item or hide the item instead of remove it.

 

[Wietlol]

You can make a separate unit and hide that unit and give the items to that unit instead of removing the items.

"Item - Remove <item>" removes it from the game and not only from the unit.
You should use "Hero - Give <item> to <unit>" where the unit is your dummy unit.
Or just drop the item and hide the item itself... depending on your preference.

@Bribe
There are actually many ways that I can name that he wont use but still make your approach glitch your map.
Better to keep the item than making a new one... basically for the same reasons why "Unit - Replace" is bad.

 

[Bribe]

My recommendation included hiding the item instead of removing it if he didn't want to re-create the item each time. Would that not work?

 

[Ardenian]

Another approach would be to save the item type of each slot and create them after the old ones are removed. However, this is more performance-intense.

 

[Wietlol]

My recommendation included hiding the item instead of removing it if he didn't want to re-create the item each time. Would that not work?

Sorry, missed that part.