What am I doing wrong? (Inventory System)

Hi y'all!

I'm working on an inventory system and for some reason it's not updating the slots. I "pick" a item from other layout, set the key "PickedItem" in a Dictionary as the picked item, then I go back to the inventory and call a while expression to check the next empty slot (the inventory is in an array) and add this picked item there.

But the problem is that it's not adding the item properly to the first empty slot. Actually it's not adding anywhere, it stays as the "PickedItem", the "PickedItem" slot never empties and the slots remains as 0.

What is strange is that for some miliseconds, it works as it should: 0 for picked item, 2 in the slot 1 (uncommon medicine herb code), 0 in the slot 2 and slot 3 as well. But after this miliseconds, it gets back to the wrong structure showed bellow on the output image.

So: what am I doing wrong!?

Thank you so much for your time and for helping me!!! Really appreciate it :)

Here is my code:

Here is my inventory array:

Here is my item list dictionary: (used to call the picked item code into the slot)

Here is the Inventory Layout (dummy texts on the bottom to test it):

Here is the output from the code:

Hey Lionz!! Thank you for your response!! :)

Hm, yes, I did share! It's in the same dictionary as the picked item (3rd image that I shared), and it's "code" is the number 2.

So, I don't understand why the 'while' loop should not work properly as the 'for' loop, since both of them should find the empty slot.

Also, ok, I could add the picked item to a global variable, but since it's a mobile app game, any layout suspension will need it to save this global variable. If I start adding global variable for everything, it will be a mess to save all of them, so that's why I use dictionaries and arrays. And I found many posts also recommending it, to use arrays and dictionaries instead of global variables, so I'm confused now! haha

And sorry, the function thing, I didn't get it, is there any other way I can call the while without using functions? Or should I optimze it somehow?

thank you again! :))

What I mean about the logic is that it only shows you searching for an empty slot and adding one item to the inventory on start of layout. It sounds like you are storing items to later add to an array, why not add to the array at the moment they are picked up?

There would only be one global var to store an item as you pick it up, but once you have verified it is working you wouldn't even need that. With the function, if you are making an add item function, you can send the item name through the function as a parameter. Read up on functions if you don't know about this.

What you've done here is combine looking for the empty row and adding the item into one function which is fine but I don't understand the use of dictionaries and list of items.

I would advise using debugging with the Browser object, you can log things to the console to check how the function is working.

Hm, got it. I understand that there is simpler ways to do what I'm trying to do, but still, I don't understand why it's not working the way I structured it! I mean, what's wrong with my code, it should work, doesn't it?

What are you loading with ajax? Maybe it's not finished. Also you are getting the dictionary item within the function which may take time and it could be blank or 0. I suggested getting the dictionary item before you call the function then call it with the dict item as a parameter.

I don't know why, but the images didn't load properly.

Here is the new code (I excluded the function):

It's no use talking about events that we can't see, you should provide all important events. So it's resolved now then, not everything is loading for you.

I provided all the important events! It's up here, the loading, the text update, the invntory adding item code! And no, everything is loading properly! As I said, the LocalStorageLoading always get the "1" value... :/

I mean initially in the first post you didn't provide the important load events. I can see now you're loading things from ajax on start of layout while at the same time running a function on start of layout before the data has loaded. But also I mentioned this method is unnecessary, you shouldn't be saving and loading to local storage between layouts to add items to an inventory.

Ok, but I said this is a mobile app, let's say that the player suspend the app and return to the it's homescreen. I need to save it, regardless the layout.. What do you suggest?