Auto-stocking chests with random items from a list

What the title said. Here is my code, but it doesn't seem to work:

Code:java

@EventHandler
    public void fillChests(PlayerJoinEvent event) {
        World world = Bukkit.getWorld("world");
        for(Chunk c : world.getLoadedChunks()){
        for(BlockState b : c.getTileEntities()){
        if(b instanceof Chest){
        Inventory inv = ((Chest) b).getBlockInventory();
        inv.clear();
        for (int i = 0; i <= 36; i++) {
            Random r = new Random();
            if (i == r.nextInt(36)) {
                inv.addItem(new ItemStack(Material.WOOL, 1), new ItemStack(Material.BOWL, 1), new ItemStack(Material.BED, 1), new ItemStack(Material.ARROW, 1), new ItemStack(Material.CAKE, 1), new ItemStack(Material.CARROT, 1));
            }
        }
        }
        }
        }
    }

 

ok do you really want to clear all the loaded chests in a world? is that what you really want to happen?
secondly you're random thingy will be weird it will randomly give the chest up to 36 of everything.
cheers, Q

 

Add the items you want into a ItemStack array,

Code:

ItemStack[] array_instance = new ItemStack[]{new ItemStack(...), ...}

Code:

addItem(array_instance[random.nextInt(array_instance.length)])
// Warning, Pseudocode

Put that inside the for loop for how many items to add to the inventory.
Of course you should be able to add an empty itemstack (Material.AIR) to the array for the chance of no item for that specific loop cycle.

Also in your title you imply you're selecting random items from a list, however you didn't use a list in your code. If you want to have the ability to dynamically add and remove items from the possible items, use a list.