Accommodating for Multiple Stacks of Items in an Inventory System

Help and Feedback Scripting Support
#41

Like all the function calls, but not the functions themselves

#42

No that was just my example, essentially you would only call them when needed.

#43

I have this code format which was used to add items before. Could this be left intact or does anything have to change?

local Info = {
	Item1 = "Bucket",
	Num1 = 1,
}
script.Parent.Triggered:Connect(function(player)
	script.Parent.Enabled = false
	local PlayerID = player.UserId
	local Location = script
	
	game.ReplicatedStorage.Inventory.AddItemBindable:Fire(player, {
		Location = Location,
		PlayerID = PlayerID,
		Num1 = Info.Num1,
		Item1 = Info.Item1
	})

end)
#44

Well, how you do it seems fine but you could definitely improve the naming structure of the data: “Item” instead of “Item1” and “Amount” instead of “Num1”. Also seems like “Location” is just the slot so rename it to smth like “SlotId”

#45

I used Location so that the Inventory handler knows where the remote event was fired from, so then it can fire events inside that script which ill do things like delete the part, or return a certain amount if the inventory is overflowing.

#46

Oh I c, I don’t suppose you have to change any of that since its tied to your game logic

#47

Okay thanks. I need to figure out what to do now for finding the correct player inventory. Should I check for the UserId still or the name?

#48

I actually set the index as the player instance since instances are pretty much unique but ye you can opt for UserId but don’t do name

#49

Should this conversation be moved to DMs so this thread doesn’t continue to expand?

#50

Absolutely, tho it’d be better if it was in discord rather than here in the forum

#51

Okay that would work out. Send me your username

#52

mystxryxd

#53

This topic was automatically closed 14 days after the last reply. New replies are no longer allowed.