Help with TileEntity/Container communication?

[Big_Bad_E]

I am trying to make a Tile Entity that stores one item and its amount.

I have a Container which has one slot, that is where you should put the item. The item goes up to the 32 bit integer limit.

My problems are:

Shift clicking is weird, slots 9-10 just go to each other 2-8 goes to 19-26, slot 27-35 and 1 crashes game with this error:

Caused by: java.lang.NullPointerException
	at net.minecraft.inventory.Container.slotClick(Container.java:271) ~[Container.class:?]
	at net.minecraft.client.multiplayer.PlayerControllerMP.windowClick(PlayerControllerMP.java:610) ~[PlayerControllerMP.class:?]
	at net.minecraft.client.gui.inventory.GuiContainer.handleMouseClick(GuiContainer.java:693) ~[GuiContainer.class:?]
	at net.minecraft.client.gui.inventory.GuiContainer.mouseClicked(GuiContainer.java:430) ~[GuiContainer.class:?]
	at net.minecraft.client.gui.GuiScreen.handleMouseInput(GuiScreen.java:611) ~[GuiScreen.class:?]
	at net.minecraft.client.gui.GuiScreen.handleInput(GuiScreen.java:576) ~[GuiScreen.class:?]
	at net.minecraft.client.Minecraft.runTick(Minecraft.java:1884) ~[Minecraft.class:?]
	... 15 more

I'm just gonna post a video cause it's easier to explain.

(Ima add textures/models later ofc)

Container:

public class QuantumContainer extends Container {

    QuantumContainer(InventoryPlayer inventory, TEQuantumStorage tile) {
        SlotItemHandler slot = new SlotQuantum(new QuantumHandler(tile), 36, 80, 40);
        addSlotToContainer(slot);
        slot.putStack(new ItemStack(tile.getItem(), tile.getAmount()));
        int i;
        for (i = 0; i < 3; ++i) {
            for (int j = 0; j < 9; ++j) {
                addSlotToContainer(new Slot(inventory, j + i * 9 + 9,
                        8 + j * 18, 84 + i * 18));
            }
        }
        // add hotbar slots
        for (i = 0; i < 9; ++i) {
            addSlotToContainer(new Slot(inventory, i, 8 + i * 18,
                    142));
        }
    }

    @Override
    @ParametersAreNonnullByDefault
    public boolean canInteractWith(EntityPlayer playerIn) {
        return true;
    }

    @Override
    @MethodsReturnNonnullByDefault
    public ItemStack transferStackInSlot(EntityPlayer playerIn, int slotIndex) {
        ItemStack itemStack1 = null;
        Slot slot = inventorySlots.get(slotIndex);

        if (slot != null && slot.getHasStack()) {
            ItemStack itemStack2 = slot.getStack();
            itemStack1 = itemStack2.copy();
            // player inventory slots
            if (slotIndex >= 9 && slotIndex < 36) {
                if (!mergeItemStack(itemStack2, 9, 35, false))
                    return null;
                // hotbar slots
            } else if (slotIndex < 9 && !mergeItemStack(itemStack2, 2, 28, false)) {
                if (!mergeItemStack(itemStack2, 0, 8, false))
                    return null;
            } else if (!mergeItemStack(itemStack2, 0, 37, false)) {
                return null;
            }
            if (itemStack2.getCount() == 0) {
                slot.putStack(ItemStack.EMPTY);
            } else {
                slot.onSlotChanged();
            }
            if (itemStack2.getCount() == itemStack1.getCount()) {
                return null;
            }
        }
        return itemStack1;
    }
}

QuantumHandler:

private ItemStack stack;
    private TEQuantumStorage tile;

    public QuantumHandler(TEQuantumStorage tile) {
        this.tile = tile;
    }

    @Override
    public int getSlots() {
        return 1;
    }

    @Nonnull
    @Override
    public ItemStack getStackInSlot(int slot) {
        return stack;
    }

    @Nonnull
    @Override
    public ItemStack insertItem(int slot, @Nonnull ItemStack stack, boolean simulate) {
        tile.setItem(stack.getItem());
        tile.addAmount(stack.getCount());
        this.stack = stack;
        this.stack.setCount(tile.getAmount());
        tile.markDirty();
        ItemStack stack1 = stack.copy();
        if(stack.getCount() == 1) {
            stack1.setCount(stack1.getCount() - 1);
            return stack1;
        } else
            return ItemStack.EMPTY;
    }

    @Nonnull
    @Override
    public ItemStack extractItem(int slot, int amount, boolean simulate) {
        ItemStack stack = this.stack.copy();
        if (stack.getCount() > 64) {
            stack.setCount(64);
        }
        if(amount == 1 && tile.getAmount() > 0)
            stack.setCount(1);
        else
            if(tile.getAmount() < 64)
                stack.setCount(tile.getAmount());
        tile.addAmount(-amount);
        this.stack.setCount(this.stack.getCount() - amount);
        tile.markDirty();
        return stack;
    }

    @Override
    public int getSlotLimit(int slot) {
        return 2147483647;
    }

    @Override
    public void setStackInSlot(int slot, @Nonnull ItemStack stack) {
        this.stack = stack;
        tile.markDirty();
    }

SlotQantum:

public class SlotQuantum extends SlotItemHandler {
    public SlotQuantum(IItemHandler handler, int index, int xPosition, int yPosition) {
        super(handler, index, xPosition, yPosition);
    }

    @Override
    public int getSlotStackLimit() {
        return 2147483647;
    }
}

Tile Entity:

public class TEQuantumStorage extends TileEntity {

    private Item item;
    private int amount;

    @Override
    public void readFromNBT(NBTTagCompound nbt) {
        super.readFromNBT(nbt);
        this.item = Item.getItemById(nbt.getInteger("item"));
        this.amount = nbt.getInteger("amount");
    }


    @Override
    public NBTTagCompound writeToNBT(NBTTagCompound nbt) {
        super.writeToNBT(nbt);
        nbt.setInteger("item", Item.getIdFromItem(item));
        nbt.setInteger("amount", amount);
        return nbt;
    }

    public int getAmount() {
        return amount;
    }

    public void setAmount(int amount) {
        this.amount = amount;
    }

    public Item getItem() {
        return item;
    }

    public void setItem(Item item) {
        this.item = item;
    }
}

Sorry for code dump, but I've been at this for hours and have no idea what I'm doing wrong.

Edited August 14, 2018 by Big_Bad_E

[Cadiboo]

Override getMaxStack size or something?

have a look at how AppliedEnergistics or another open source storage mod does it

[Big_Bad_E]

  On 8/10/2018 at 1:59 AM, Cadiboo said:

Override getMaxStack size or something?

have a look at how AppliedEnergistics or another open source storage mod does it

Expand  

No, I am not trying to have a super large item stack, I am trying to store it as a number in the tile entity's NBT data along with the item. Then I can use that to let the player get the item. It's literally the Deep Storage Unit.

[Big_Bad_E]

I'm just gonna bump this because Draco is online

[Animefan8888]

  On 8/10/2018 at 5:23 PM, Big_Bad_E said:

I'm just gonna bump this because Draco is online

Expand  

Lol

  On 8/9/2018 at 11:08 PM, Big_Bad_E said:

GenericSlotHolder implements IInventory

Expand  

Dont do this use the IItenHandler capability instead.

 

[Big_Bad_E]

  On 8/10/2018 at 7:32 PM, Animefan8888 said:

Lol

Dont do this use the IItenHandler capability instead.

 

Expand  

How would I use IItemHandler? I can't set slot's inventory to it because it doesn't implement IInventory.

It looks nice but I can't use it.

 

Also back to the original question what's wrong with my code?

[Draco18s]

  On 8/10/2018 at 5:23 PM, Big_Bad_E said:

I'm just gonna bump this because Draco is online

Expand  

I will then proceed to ignore the fuck out of it because I was on an airplane, and got home at 1am after having almost missed my connecting flight in Phoenix because my first leg was 45 60 minutes late leaving and the gate attendant told me "You'll have 34 minutes between flights, you'll be fine." I had 22 minutes, minus the "doors close 10 minutes before the departure time" so 12 minutes to get from gate B4 to A25.

 

Storing large stacks? You're going to need two inventory slots, an input and an output. The output always contains a stack of size Max(available, 64). The input slot is always empty. When a stack is placed into the input and it matches the contents, destroy the stack and add its size to the TE's integer value of contents.

[Big_Bad_E]

  On 8/11/2018 at 4:24 PM, Draco18s said:

I will then proceed to ignore the fuck out of it because I was on an airplane, and got home at 1am after having almost missed my connecting flight in Phoenix because my first leg was 45 60 minutes late leaving and the gate attendant told me "You'll have 34 minutes between flights, you'll be fine." I had 22 minutes, minus the "doors close 10 minutes before the departure time" so 12 minutes to get from gate B4 to A25.

 

Storing large stacks? You're going to need two inventory slots, an input and an output. The output always contains a stack of size Max(available, 64). The input slot is always empty. When a stack is placed into the input and it matches the contents, destroy the stack and add its size to the TE's integer value of contents.

Expand  

Oh, didn't know, sorry about ur airplane problems. How come we can mass produce nuclear weapons but we cant move from point A to point B in an airplane easily.

 

Input and output is a good idea i'm going to try it.

 

Still being weird, see updated code/errors above.

Edited August 11, 2018 by Big_Bad_E

[Animefan8888]

  On 8/11/2018 at 3:43 PM, Big_Bad_E said:

How would I use IItemHandler? I can't set slot's inventory to it because it doesn't implement IInventory.

It looks nice but I can't use it.

Expand  

Use SlotItemHandler instead of Slot.

[Big_Bad_E]

  On 8/11/2018 at 8:20 PM, Animefan8888 said:

Use SlotItemHandler instead of Slot.

Expand  

Will look into it.

 

I am now using SlotItemHandler, good idea with that one.

Edited August 12, 2018 by Big_Bad_E

[Big_Bad_E]

I just gave up, I've spent way too long working on this.

[Cadiboo]

  On 8/12/2018 at 5:25 PM, Big_Bad_E said:

I just gave up, I've spent way too long working on this.

Expand  

Move on to something else for a couple days, then come back to this. This should be very easy to achieve, and when you come back to it in a couple days with a new perspective, it shouldn’t take you very much time at all to do it! Good luck!

[Big_Bad_E]

  On 8/12/2018 at 11:49 PM, Cadiboo said:

Move on to something else for a couple days, then come back to this. This should be very easy to achieve, and when you come back to it in a couple days with a new perspective, it shouldn’t take you very much time at all to do it! Good luck!

Expand  

My main problem is that I can't figure out what is wrong with my code, and when I use one solution, tons more pop up. Like implements IItemHandler now the input slot only takes one item which won't disappear, and I can't test output, and for some reason when I close the GUI with an item the amount gets set to -22 and it crashes. I've updated my code but I've just run into a lack of solutions. For some reason slot events aren't being called and I'm just frustrated.

 

Now that I'm thinking about it i'm just going to make a slot with a max item limit of the 32 bit integer limit.

[Cadiboo]

  On 8/13/2018 at 12:18 AM, Big_Bad_E said:

My main problem is that I can't figure out what is wrong with my code, and when I use one solution, tons more pop up. Like implements IItemHandler now the input slot only takes one item which won't disappear, and I can't test output, and for some reason when I close the GUI with an item the amount gets set to -22 and it crashes. I've updated my code but I've just run into a lack of solutions. For some reason slot events aren't being called and I'm just frustrated.

 

Now that I'm thinking about it i'm just going to make a slot with a max item limit of the 32 bit integer limit.

Expand  

Use forges IItemHandler system with 2 slots input & output, when a stack goes into the input delete it and add it’s value to your tiles data. Write the code for output later

[Big_Bad_E]

  On 8/13/2018 at 12:27 AM, Cadiboo said:

Use forges IItemHandler system with 2 slots input & output, when a stack goes into the input delete it and add it’s value to your tiles data. Write the code for output later

Expand  

I'd rather just make a single stack. I think it would generally be easier.

[Animefan8888]

  On 8/13/2018 at 2:19 AM, Big_Bad_E said:

I'd rather just make a single stack. I think it would generally be easier.

Expand  

Use an IItemHandler that has one slot and access to the TEs amount field and override insertItem and extractItem as well as getStack or getStacks(whatever that method is called in IItemHandler) and return the appropriate values and update the stack inside the IItemHandler based on the amount filed. No need for a fancy slot.

[Big_Bad_E]

  On 8/13/2018 at 2:23 AM, Animefan8888 said:

Use an IItemHandler that has one slot and access to the TEs amount field and override insertItem and extractItem as well as getStack or getStacks(whatever that method is called in IItemHandler) and return the appropriate values and update the stack inside the IItemHandler based on the amount filed. No need for a fancy slot.

Expand  

That's what i'm doing.

[Animefan8888]

  On 8/13/2018 at 2:31 AM, Big_Bad_E said:

That's what i'm doing.

Expand  

Well considering you have not posted your updated code how was I supposed to know that or help you anymore than that.

[Big_Bad_E]

  On 8/13/2018 at 3:48 AM, Animefan8888 said:

Well considering you have not posted your updated code how was I supposed to know that or help you anymore than that.

Expand  

I haven't updated my code cause I haven't coded it yet. And:

  On 8/13/2018 at 2:19 AM, Big_Bad_E said:

I'd rather just make a single stack. I think it would generally be easier.

Expand  

 

  On 8/12/2018 at 1:10 AM, Big_Bad_E said:

Will look into it.

 

I am now using SlotItemHandler, good idea with that one.

Expand  

I did say what I was going to do.

[Big_Bad_E]

Couple questions:

How do I update the Container/GUI to display the updated item?

How do I stop the block from being placed when I open the GUI?

 

I can fix the shift click problems myself.

[Animefan8888]

  On 8/13/2018 at 6:34 PM, Big_Bad_E said:

How do I update the Container/GUI to display the updated item?

Expand  

Send a packet to the player there are multiple ways to do this, but show your code and I can tell you which on to use. 

  On 8/13/2018 at 6:34 PM, Big_Bad_E said:

How do I stop the block from being placed when I open the GUI?

Expand  

Return true in onBlockActivated

[Big_Bad_E]

  On 8/13/2018 at 9:52 PM, Animefan8888 said:

Send a packet to the player there are multiple ways to do this, but show your code and I can tell you which on to use. 

Return true in onBlockActivated

Expand  

My code is updated, I've never used packets with Forge so it will be fun to try

[Big_Bad_E]

Are you sure packets are the way to go?

I made some packets, and it updated the server, but not the client's slot.

 

How do I get the slot itself to update? I think there is another ItemStack list somewhere that is called to draw the itemstacks. I am gonna look through the itemstack draw method and the container close/update method cause that also updates it.

[Animefan8888]

  On 8/13/2018 at 11:39 PM, Big_Bad_E said:

Are you sure packets are the way to go?

I made some packets, and it updated the server, but not the client's slot.

Expand  

Are you sending the data to the client or to the server?

  On 8/13/2018 at 11:39 PM, Big_Bad_E said:

How do I get the slot itself to update?

Expand  

If changing the IItemHandlers stack on the client doesnt update the container/gui have your packet also update the container.

[Big_Bad_E]

  On 8/14/2018 at 12:10 AM, Animefan8888 said:

Are you sending the data to the client or to the server?

If changing the IItemHandlers stack on the client doesnt update the container/gui have your packet also update the container.

Expand  

The data was sent client to server.

 

My problem was the container had updated data but for some reason the GUI's instance of the data wasn't updated, so I manually just got the itemstack from the TileEntity and used it to update the GUI.