Jump to content

[1.12.2] The GUI container take the second slot of the hotbar


MisterGamer _

Recommended Posts

Hello everyone!

In my GUI i have two slots, one for input and one for output like this:

compactor_gui.png.7eb748b5f252efda06129ef737ea8fa0.png

But when i open the GUI, the input slot is filled with the Item i have in the second hotbar slot.

I think that the problem is in the container code, here it is:

public class CompactorContainer extends Container{

	private CompactorTileEntity te;

    public CompactorContainer(IInventory playerInventory, CompactorTileEntity te) {
        this.te = te;

        addOwnSlots(playerInventory);
        addPlayerSlots(playerInventory);
    }

    private void addPlayerSlots(IInventory playerInventory) {
        for (int row = 0; row < 3; ++row) {
            for (int col = 0; col < 9; ++col) {
                int x = 8 + col * 18;
                int y = row * 18 + 84;
                this.addSlotToContainer(new Slot(playerInventory, col + row * 9 + 10, x, y));
            }
        }

      //add the slots for the hotbar
        for (int col = 0; col < 9; ++col) {
            int x = 8 + col * 18;
            int y = 72 + 70;
            this.addSlotToContainer(new Slot(playerInventory, col, x, y));
        }
    }

    private void addOwnSlots(IInventory pInv) {
        addSlotToContainer(new Slot(pInv, 1, 48, 34));
        addSlotToContainer(new Slot(pInv, 1, 110, 35));
    }

    @Override
    public ItemStack transferStackInSlot(EntityPlayer playerIn, int index) {
        ItemStack itemstack = ItemStack.EMPTY;
        Slot slot = this.inventorySlots.get(index);

        if (slot != null && slot.getHasStack()) {
            ItemStack itemstack1 = slot.getStack();
            itemstack = itemstack1.copy();

            if (index < CompactorTileEntity.SIZE) {
                if (!this.mergeItemStack(itemstack1, CompactorTileEntity.SIZE, this.inventorySlots.size(), true)) {
                    return ItemStack.EMPTY;
                }
            } else if (!this.mergeItemStack(itemstack1, 0, CompactorTileEntity.SIZE, false)) {
                return ItemStack.EMPTY;
            }

            if (itemstack1.isEmpty()) {
                slot.putStack(ItemStack.EMPTY);
            } else {
                slot.onSlotChanged();
            }
        }
        return itemstack;
    }
	
    @Override
    public boolean mergeItemStack(ItemStack stack, int start, int end, boolean reverse){
    	return super.mergeItemStack(stack,start,end,reverse);
    }
    
	@Override
	public boolean canInteractWith(EntityPlayer playerIn) {
		return te.canInteractWith(playerIn);
	}

}

Thanks you all!

Link to comment
Share on other sites

56 minutes ago, MisterGamer _ said:

private void addOwnSlots(IInventory pInv) {

    addSlotToContainer(new Slot(pInv, 1, 48, 34));

    addSlotToContainer(new Slot(pInv, 1, 110, 35));

}

Why are the indices of both of your slots the same? This should never be the case.

As for the issue - look at the object you are passing to this method:

57 minutes ago, MisterGamer _ said:

addOwnSlots(playerInventory);

Obviously the added slots will correspond to the player's inventory since it is the inventory you are passing.

Link to comment
Share on other sites

11 minutes ago, V0idWa1k3r said:

Why are the indices of both of your slots the same? This should never be the case.

As for the issue - look at the object you are passing to this method:

Obviously the added slots will correspond to the player's inventory since it is the inventory you are passing.

So i must create a personal slot, right?

Edited by MisterGamer _
Grammar correction
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.