Проблема с TileEntity

[Alexandr10]

Суть в том, что предмет лежащий в инвентаре блока можно заменить или забрать положив в хотбар при помощи клавиш 1-9, но нельзя забрать через ЛКМ, а также при нажатие Shift + ПКМ/ЛКМ по предмету или при попытке собрать все предметы в стак двойным нажатием происходит краш. Как это исправить?

Спойлер: КОД

package ru.Alexandr100.arm.blocks;

import cpw.mods.fml.relauncher.Side;
import cpw.mods.fml.relauncher.SideOnly;
import net.minecraft.block.BlockContainer;
import net.minecraft.block.ITileEntityProvider;
import net.minecraft.block.material.Material;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.Item;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.EnumFacing;
import net.minecraft.util.ResourceLocation;
import net.minecraft.world.World;
import ru.Alexandr100.arm.AvaritiaRecipeMaker;
import ru.Alexandr100.arm.tileentities.AvaritiaRecipeMakerTileEntity;

public class AvaritiaRecipeMakerBlock extends BlockContainer {
    public static final int GUI_ID = 1;

    public AvaritiaRecipeMakerBlock() {
        super(Material.iron);
        this.setBlockTextureName("avaritiarecipemakerblock");
        this.setBlockName("avaritiarecipemakerblock");
        this.setCreativeTab(CreativeTabs.tabTransport);
    }

    public TileEntity createNewTileEntity(World worldIn, int meta) {
        return new AvaritiaRecipeMakerTileEntity();
    }

    public boolean onBlockActivated(World w, int x, int y, int z, EntityPlayer p, int side, float hitX, float hitY, float hitZ) {
        if (w.isRemote) {
            return true;
        }
        TileEntity te = w.getTileEntity(x,y,z);

        p.openGui(AvaritiaRecipeMaker.instance, GUI_ID, w, x, y, z);
        return true;
    }
}

package ru.Alexandr100.arm.tileentities;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.ItemStack;
import ru.Alexandr100.arm.misc.RecipeGenerator;

public class AvaritiaRecipeMakerTileEntity extends TileIInventoryBase{
    public AvaritiaRecipeMakerTileEntity() {super(82, 64, "Avaritia recipe maker");}
 
    public ItemStack[] GetArray() {return super.GetArray();}
 
    public void outputToFile(int flag) {try{ RecipeGenerator rg = new RecipeGenerator(this.GetArray(), flag); } catch (Exception e) {e.printStackTrace();}}
 
    public void setItemStackInSlot(ItemStack is, int slot) {this.setSlotContent(slot,is);}
    public ItemStack getItemStackInSlot(int slot) {return this.getSlotContent(slot);}
 
    @Override
    public boolean isItemValidForSlot(int slot, ItemStack i) {return true;}
 
    public boolean canInteractWith(EntityPlayer playerIn) {
        return true;
    }
    @Override
    public boolean isUseableByPlayer(EntityPlayer p) {return true;}
}

package ru.Alexandr100.arm.tileentities;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.inventory.IInventory;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.nbt.NBTTagList;
import net.minecraft.tileentity.TileEntity;
import net.minecraftforge.common.util.Constants;

public abstract class TileIInventoryBase extends TileEntity implements IInventory{
 
    protected TileIInventoryBase(int SizeInventory, int inventoryLimit, String name) {
        this.SizeInventory = SizeInventory;
        this.stackLimit = inventoryLimit;
        this.inventoryName = name;
     
        if(this.Items == null) for(int i = 0; i < this.stackLimit; i++) setSlotContent(i, (ItemStack)null);
     
        if(this.logOther && this.Items != null) System.out.println("this.Items != null");
        else if(this.logOther) System.out.println("this.Items == null");
     
    }
 
    private TileIInventoryBase t = null;
    int SizeInventory = 82;
    int stackLimit = 64;
    String inventoryName = "Unnamed";
    public ItemStack Items[] = new ItemStack[SizeInventory];

    protected ItemStack[] GetArray() {return Items;}

    protected void setSlotContent(int slot, ItemStack is) {
        try {
            this.Items[slot] = is;
        }
        catch (ArrayIndexOutOfBoundsException e) {
            try {
                int i = slot - this.Items.length * ((int)(slot / Items.length));

                this.Items[i] = is;
            }
            catch(ArrayIndexOutOfBoundsException e1) {
            }
        }
    }
    protected ItemStack getSlotContent(int slot) {
        try {
            if(this.Items.length < slot && this.Items.length > 0) return this.Items[slot];
            else return this.Items[(slot - Items.length * (slot % Items.length))];
        }
        catch (ArrayIndexOutOfBoundsException e) {
            try {
                int i = slot - this.Items.length * ((int)(slot / Items.length));
                return this.Items[i];
            }
            catch(ArrayIndexOutOfBoundsException e1) {
            }
        }
        return null;
    }
 
 
 
    @Override
    public int getSizeInventory() {
        return this.SizeInventory;
    }

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

    @Override
    public ItemStack decrStackSize(int slot, int a) {
     
        if(this.getSlotContent(slot) != null) {
            if(this.getSlotContent(slot).stackSize > a)
                return new ItemStack(this.getSlotContent(slot).getItem(), (this.getSlotContent(slot).stackSize - a));
            else return null;
        }
        else return null;
    }

    @Override
    public ItemStack getStackInSlotOnClosing(int slot) {
        if(this.Items.length < slot) {
            if (this.Items[slot] != null)
            {
                ItemStack itemstack = this.Items[slot];
                this.Items[slot] = null;
             
                return itemstack;
            }
            else
            {
                return null;
            }
        }
        else {
            return null;
        }
    }
 
    @Override
    public void setInventorySlotContents(int slot, ItemStack is) {

        ItemStack toReturn = is;
        if(toReturn != null ) if(toReturn.stackSize <= getInventoryStackLimit()) setSlotContent(slot, toReturn);
        else {toReturn.stackSize = getInventoryStackLimit(); setSlotContent(slot, toReturn);}
        else setSlotContent(slot, null);;

        markDirty();
    }

    @Override
    public String getInventoryName() {return inventoryName;}

    @Override
    public boolean hasCustomInventoryName() {return false;}

    @Override
    public int getInventoryStackLimit() {return stackLimit;}

    @Override
    public boolean isUseableByPlayer(EntityPlayer p) {
        return worldObj.getTileEntity(xCoord, yCoord, zCoord) != this ? false :
            p.getDistanceSq((double)xCoord + 0.5D, (double)yCoord + 0.5D, (double)zCoord + 0.5D) <= 64.0D;
    }
    @Override
    public void openInventory() {}

    @Override
    public void closeInventory() {}

    @Override
    public boolean isItemValidForSlot(int slot, ItemStack i) {
        return true;
    }
 
    @Override
    public void readFromNBT(NBTTagCompound nbt)
    {
     
        super.readFromNBT(nbt);
        NBTTagList list = nbt.getTagList("Items", Constants.NBT.TAG_COMPOUND);
        this.Items = new ItemStack[getSizeInventory()];

        for (int i = 0; i < list.tagCount(); ++i) { NBTTagCompound comp = list.getCompoundTagAt(i); int j = comp.getByte("Slot") & 255;
            if (j >= 0 && j < this.Items.length){
                setSlotContent(j, ItemStack.loadItemStackFromNBT(comp));
            }
        }

    }
    @Override
    public void writeToNBT(NBTTagCompound nbt)
    {
        super.writeToNBT(nbt);
        NBTTagList list = new NBTTagList();

        for (int i = 0; i < this.Items.length; ++i)
        {
            if (this.getSlotContent(i) != null)
            {
                NBTTagCompound comp = new NBTTagCompound();
                comp.setByte("Slot", (byte)i);
                this.getSlotContent(i).writeToNBT(comp);
                list.appendTag(comp);

            }
        }
     
        nbt.setTag("Items", list);
    }
}

package ru.Alexandr100.arm.gui;

import java.util.List;
import net.minecraft.client.Minecraft;
import net.minecraft.client.gui.GuiButton;
import net.minecraft.client.gui.inventory.GuiContainer;
import net.minecraft.client.renderer.texture.TextureManager;
import net.minecraft.inventory.Container;
import net.minecraft.item.ItemStack;
import net.minecraft.util.ResourceLocation;
import ru.Alexandr100.arm.containers.AvaritiaRecipeMakerContainer;
import ru.Alexandr100.arm.gui.ConfirmButton;
import ru.Alexandr100.arm.misc.RecipeGenerator;
import ru.Alexandr100.arm.tileentities.AvaritiaRecipeMakerTileEntity;

public class AvaritiaRecipeMakerGui extends GuiContainer {
    public static final int WIDTH = 238;
    public static final int HEIGHT = 256;
    private static final ResourceLocation background = new ResourceLocation("avaritiarecipemaker", "textures/gui/avaritiarecipemaker.png");
    private AvaritiaRecipeMakerTileEntity te;

    public AvaritiaRecipeMakerGui(AvaritiaRecipeMakerTileEntity tileEntity, AvaritiaRecipeMakerContainer container) {
        super((Container)container);
        this.te = tileEntity;
        this.xSize = 238;
        this.ySize = 256;
     
    }

    protected void drawGuiContainerBackgroundLayer(float partialTicks, int mouseX, int mouseY) {
        this.mc.getTextureManager().bindTexture(background);
        this.drawTexturedModalRect(this.guiLeft, this.guiTop, 0, 0, this.xSize, this.ySize);
    }

    public void initGui() {
        super.initGui();
        this.buttonList.add(new ConfirmButton(1, this.guiLeft + 188, this.guiTop + 108));
    }

    public void actionPerformed(GuiButton guiButton) {
        if(guiButton.id == 1) {
            te.outputToFile(0);
        }
        else return;
    }
}

package ru.Alexandr100.arm.containers;

import javax.annotation.Nullable;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.inventory.Container;
import net.minecraft.inventory.IInventory;
import net.minecraft.inventory.Slot;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.util.EnumFacing;
import ru.Alexandr100.arm.tileentities.AvaritiaRecipeMakerTileEntity;


public class AvaritiaRecipeMakerContainer extends Container {
    private AvaritiaRecipeMakerTileEntity te;
 
    private IInventory inv = (IInventory)te;
 
    public AvaritiaRecipeMakerContainer(IInventory playerInventory, AvaritiaRecipeMakerTileEntity te) {    
        this.te = te;
        inv = (IInventory)te;
        int playerSlotIndex = 0;
        int matrixSlotIndex = 0;

        //Player Inventory - Hotbar slots
        for (int col = 0; col < 9; ++col) {
            int x = 39 + col * 18;
            int y = 232;
            this.addSlotToContainer(new Slot(playerInventory, playerSlotIndex, x, y));
            playerSlotIndex++;
        }
     
        //Player Inventory - Inventory(9x3) slots
        for(int i = 0; i < 3; i++) {
            for(int j = 0; j < 9; j++) {
                int y = 174 + i * 18;
                int x = 39 + j * 18;
                this.addSlotToContainer(new Slot(playerInventory, playerSlotIndex, x, y));
                playerSlotIndex++;
            }
        }
     
        //Matrix
        for(int i = 0; i < 9; i++){
            for(int j = 0; j < 9; j++){
                this.addSlotToContainer(new Slot(inv, matrixSlotIndex, 12 + 18 * i, 8 + 18 * j));
                matrixSlotIndex++;
            }
        }
     
        //Output slot
        this.addSlotToContainer(new Slot(inv, matrixSlotIndex, 210, 80));
        matrixSlotIndex++;
    }
 

    public boolean canInteractWith(EntityPlayer player) {
        return this.te.canInteractWith(player);
    }
}

 

[timaxa007]

В контейнере, нету метода transferStackInSlot для "Shift + ПКМ/ЛКМ".

    at net.minecraft.inventory.Container.retrySlotClick(SourceFile:365)
    at net.minecraft.inventory.Container.slotClick(SourceFile:199)

Один метод обращается к другому, другой к первому, без всяких проверок и ограничений, и получается зацикленное обращение, у которого нету прекращения данного действия.
А на остальное уже смотреть не хочется, точнее вникать в код, слишком много его.

 

[Alexandr10]

Я добавил transferStackInSlot и краши при Shift + ПКМ/ЛКМ прекратились, но забрать предмет простым нажатием ЛКМ по прежнему не получается

    @Override
    public ItemStack transferStackInSlot(EntityPlayer player, int slotRaw)
    {
        ItemStack stack = null;
        Slot slot = (Slot)inventorySlots.get(slotRaw);
 
        if (slot != null && slot.getHasStack())
        {
            ItemStack stackInSlot = slot.getStack();
            stack = stackInSlot.copy();
 
            if (slotRaw < 3 * 9)
            {
                if (!mergeItemStack(stackInSlot, 3 * 9, inventorySlots.size(), true))
                {
                    return null;
                }
            }
            else if (!mergeItemStack(stackInSlot, 0, 3 * 9, false))
            {
                return null;
            }
 
            if (stackInSlot.stackSize == 0)
            {
                slot.putStack((ItemStack)null);
            }
            else
            {
                slot.onSlotChanged();
            }
        }
        return stack;
    }

 

[timaxa007]

забрать предмет простым нажатием ЛКМ по прежнему не получается

Причин много может быть.
Что-то в GuiHandler не правильно написано.

AvaritiaRecipeMakerGui(AvaritiaRecipeMakerTileEntity tileEntity, AvaritiaRecipeMakerContainer container)

Убедись, что ты даешь постоянно новый AvaritiaRecipeMakerContainer.
Возможно не правильно сделанный инвентарь по интерфейсу IInventory.
Возможно инстанция твоего мода не правильно написана.
Убедись, что не какой ASM не ломает работу контейнере и его пакетов.
И таких предположений можно много писать.

 

[Alexandr10]

Переписал decrStackSize и остальные проблемы решились