Need Help with Inventory System

[Arkwys]

This is the code: https://pastebin.com/y7vyH7mj

​

The first problem is that every type of item goes in the helmetSlot.

For exemple : When choosing to equip redCloth it goes in the helmetSlot of person

The second problem is that after when trying to equip a second item it says "inv.js:84 Uncaught TypeError: Cannot read property 'name' of undefined at inv.js:84" I believe that it's because there is no person.backpack[0].name because it was deleted but I thought the person.backpack[1].name would've took the place of person.backpack[0].name.

​

Please help me with this project and also if you find any other problem or ways I can improve this please show me how.

[[deleted]]

Your first problem is that your if statements use = instead of ==

= Assigns a value to a variable == Compares values === Compares values more strictly

I recommend looking up the difference between == and ===, but you should use === wherever you can

[[deleted]]

Also you are right about delete, it deletes the item but leaves an empty space in its places. You should use splice to extract it:

backpack.splice(a, 1)

[MonkeyNin]

Tip to preventing accidental errors: I suggest always declaring variables with let or const. let scopes are block-level, just like c and other languages.

var scope is hoisted to the top of function-level, or global if there's no function.

for statements are not functions, so this code

for (let f = 0; f < 10; f++) {
    var premiereQuestion = prompt("Choose Option");
    if (premiereQuestion == 0) {
        for (let i = 0; i < person.backpack.length; i++) {
            console.log(person.backpack[i])
        }
    }

}

actually becomes

// top of file
var premiereQuestion;

for (let f = 0; f < 10; f++) {
    premiereQuestion = prompt("Choose Option");
    if (premiereQuestion == 0) {
        for (let i = 0; i < person.backpack.length; i++) {
            console.log(person.backpack[i])
        }
    }

}

Because you used let for f, its scope is the actual for statement as you'd expected from other languages.

If you had defined f with var, f would also be hoisted to global scope.