How to swap armors without dropping items?

[scotthmccoy]

I'm trying to duplicate the effect of picking up an armor from my inventory and dropping it into the armor slot using code. Ideally I'd like to make an armor wardrobe mod for quickly changing between sets of armor that have equipment in them for specialized tasks (fighting, building, moving quickly, etc).

Both of these code snippets swap in the "top-most" power-armor-mk2 in my inventory, but temporarily shrink the size of my inventory, typically dropping a bunch of stuff on the ground:

Code: Select all

/c wornArmor = game.player.get_inventory(defines.inventory.character_armor)[1]
newArmor = game.player.get_main_inventory().find_item_stack("power-armor-mk2")
newArmor.swap_stack(wornArmor)

Code: Select all

/c wornArmor = game.player.get_inventory(defines.inventory.character_armor)[1]
newArmor = game.player.get_main_inventory().find_item_stack("power-armor-mk2")
wornArmor.swap_stack(newArmor)

[scotthmccoy]

I figured out a solution! I create a temporary inventory, move everything into there, swap armors, and then move all the items back. If there's a more elegant way to accomplish this, though, I'd love to know!

Code: Select all

mainInventory = game.player.get_main_inventory()
tempInventory = game.create_inventory(#mainInventory)

for i=1, #mainInventory do
	if mainInventory[i].valid_for_read then
  		mainInventory[i].swap_stack(tempInventory[i])
  	end
end

wornArmor = game.player.get_inventory(defines.inventory.character_armor)[1]
newArmor = tempInventory.find_item_stack("power-armor-mk2")
newArmor.swap_stack(wornArmor)

for i=1, #tempInventory do
	if tempInventory[i].valid_for_read then
  		tempInventory[i].swap_stack(mainInventory[i])
  	end
end

tempInventory.destroy()

[eradicator]

Code: Select all

player.character_inventory_slots_bonus = player.character_inventory_slots_bonus + 1000
newArmor.swap_stack(wornArmor)
player.character_inventory_slots_bonus = player.character_inventory_slots_bonus - 1000

Be wary of other mods that might change the bonus during on_player_armor_inventory_changed. You should only do relative changes.

[scotthmccoy]

I thought about doing that but Ive read that lua lacks thread safety and I'm concerned that one thread might modify the value of the inventory slot bonus while it's being read. Is that a cause for concern?

[eradicator]

I thought about doing that but Ive read that lua lacks thread safety and I'm concerned that one thread might modify the value of the inventory slot bonus while it's being read. Is that a cause for concern?

No. Factorio lua runs single-threaded. Thus the only access point for another mod is when LuaItemStack.swap_stack() calls the other mods event handler function - at which point your mod is halted until swap_stack() returns.

[eradicator]

Saw your mod on the portal. Nice idea with the color coding. Just wanted to add that "+1000" was supposed to be an example number that "is probably large enough". Ideally you'd have some fancy logic that calculates a better number - ye never know if some mod doesn't add an armor with a 2000 slot bonus and there's also mods that add grid equipment with bonus. Maybe something like the current size of the inventory.

Code: Select all

#player.get_main_inventory()

[scotthmccoy]

That's a really good point, I'll update that in the next version.

[hyspeed]

I think Scooty's Armor Swap does similar:
https://mods.factorio.com/mod/scootys-armor-swap

[Rseding91]

This was fixed in 1.1.97 so it will simply swap without needing any extra logic.