Solved Iterating through a player's inventory slots

Hey, I am working on a private kit plugin and to create a kit, I want it to copy the player's inventory and save it to a config. To do this, I am attempting to iterate through a player's inventory slots (0-36) and set them in a config. My question is if I'm iterating through it properly, because I've not really worked with this sort of stuff. Here is the code:

Code:java

        String path = kitName + ".";
        kits.createSection(kitName);
 
        // Iterate through all item slots in the player's inventory.
        for (int i = 36; --i >= 0;) {
            ItemStack is  = p.getInventory().getItem(i);
 
            // If null, loop through the next item and return.
            if (is == null) {
                i++;
                return;
            }
            String slot = path + "items." + i;
            kits.set(slot +".type", is.getType());
 
            if (!is.hasItemMeta()) {
                i++;
                return;
            }
 
            if (is.getItemMeta().hasDisplayName())
                kits.set(slot + ".name", is.getItemMeta().getDisplayName());
            i++;
        }
        plugin.saveConfig();

p is the player, plugin is the main class, kits is a configuration section ("kits")

This is how I want the config to look:

Code:

kits:
  kitName:
    items:
      '0':
        type: DIAMOND
        name: 'blah'

Something like that.
You may be asking: Why aren't I testing it?? This is because the plugin is not even closed to finished yet, and because a lot of methods are right now not-complete the plugin probably won't even work. I want to get this sorted out of the way before I get too far into the kits and have to change a lot of code.

Thank you!

 

AoH_Ruthless
Why don't you just loop through p.getInventory().getContents() ?

 

Gater12
How do I get the slot of the item with that?

 

AoH_Ruthless
Got a point there....

I'm a bit confused on your for loop... i is 36, if one less i >= 0, increment i ?

 

Gater12
I think I meant to do:

Code:java

for (int i = 0; i < 36; i++) {}

Gater12
Is that code right then (with the modified for loop)? I would have edited the post but I don't think you would have saw it so sorry.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

AoH_Ruthless Should seem to work