Slot index out of range [0,1]

[naturaGodhead]

When opening my gui, it pauses for about 2 seconds, then crashes. 

 

Container class: 

https://github.com/linkisheroic/rosetteMod/blob/master/main/java/com/natura/rosette/tileentity/PedestalContainer.java

 

Tile entity class:

https://github.com/linkisheroic/rosetteMod/blob/master/main/java/com/natura/rosette/tileentity/PedestalEntity.java

 

Crash: 

java.lang.RuntimeException: Slot 1 not in valid range - [0,1)
    at net.minecraftforge.items.ItemStackHandler.validateSlotIndex(ItemStackHandler.java:210)
    at net.minecraftforge.items.ItemStackHandler.getStackInSlot(ItemStackHandler.java:75)
    at net.minecraftforge.items.SlotItemHandler.getStack(SlotItemHandler.java:79)

[Dustpuppy]

private ItemStackHandler inventory = new ItemStackHandler(4);

Is wrong.

private ItemStackHandler inventory = new ItemStackHandler[4];

Would be better, i think.

 

[naturaGodhead]

 

15 minutes ago, Dustpuppy said:

private ItemStackHandler inventory = new ItemStackHandler(4);

Is wrong.

private ItemStackHandler inventory = new ItemStackHandler[4];

Would be better, i think.

 

That doesn't work. ItemStackHandler accepts an int value as a size parameter. It's not an array.

[naturaGodhead]

When I label all of the slots as index 0 it works just fine.

[Draco18s]

18 minutes ago, Dustpuppy said:

private ItemStackHandler inventory = new ItemStackHandler(4);

Is wrong.

private ItemStackHandler inventory = new ItemStackHandler[4];

Would be better, i think.

 

No, you're wrong.

 

53 minutes ago, naturaGodhead said:

When opening my gui, it pauses for about 2 seconds, then crashes. 

Did your inventory previously only have 1 slot? Are you reloading a world with a TE that had been previously saved?

 

[naturaGodhead]

3 minutes ago, Draco18s said:

No, you're wrong.

 

Did your inventory previously only have 1 slot? Are you reloading a world with a TE that had been previously saved?

 

It did at one point. I made a new world when I started trying to figure it out though. Passing 0 as the index for each one makes the slots work fine, but I can't reference them with onSlotChanged that way.

[naturaGodhead]

Nevermind, it evidently was just the old data. I loaded up a new world and the block functions correctly.