Help with Inventory system!

[H.I]

Hi. In that code the loops are redundant as you're not using the "x" and "y" variables.
The function can be reduced to just this line, "self.arr[iy][ix] = 0"

Thank you for noticing, i guess i didn't notice that.

[RNavega]

Thank you for noticing, i guess i didn't notice that.

Reading it again, I should've said "the way that the loops are used", without mentioning the index variables, because you can have a totally useful loop where the index variable happens to not be used.
For example:

Code: Select all

-- Saving time by both emptying a table and calling a method of its items.
for unusedVariable = 1, #myTable do
    local myItem = table.remove(myTable)
    myItem:method()
    (...)
end

Edit: that is more performant than "while #myTable > 0" because it only calculates the table length once, when the FOR loop starts, instead of on each iteration of that WHILE.