Créer un bloc type four (machine)

robin4002

Le problème est toujours le même.
Un ArrayIndexOutOfBoundsException ce n’est pas compliqué à corriger.

ZephyrinGames

je sais pas comment faire y a t-il un tuto sur le site qui explique comment le corriger ou pouvez-vous m’aider???

robin4002

Google -> ArrayIndexOutOfBoundsException
Un peu d’autonomie …
C’est de la programmation Java. Faut connaitre les bases pour pouvoir coder.

ZephyrinGames

Tkt je connais un peu les bases c’est juste que je débute et mon problème est fix!!!

Minantcraft

Super TUTO!

Folgansky

Plop les gens,
J’ai fais deux machines à 3slots grâce à ce tuto bien réalisé.

Mais maintenant que j’essaie de faire une machine à un slot et une pourvu de deux slots, j’ai un crash identique dans les deux cas:

–-- Minecraft Crash Report ----
// I blame Dinnerbone.

Time: 22/06/16 17:41
Description: Ticking memory connection

java.lang.ArrayIndexOutOfBoundsException: 3
at fr.powergame.modpg2.client.TileEntitySechoir.getStackInSlot(TileEntitySechoir.java:38)
at net.minecraft.inventory.Slot.getStack(Slot.java:88)
at net.minecraft.inventory.Container.getInventory(Container.java:67)
at net.minecraft.inventory.Container.addCraftingToCrafters(Container.java:53)
at cpw.mods.fml.common.network.internal.FMLNetworkHandler.openGui(FMLNetworkHandler.java:88)
at net.minecraft.entity.player.EntityPlayer.openGui(EntityPlayer.java:2501)
at fr.powergame.modpg2.common.BlockSechoir.onBlockActivated(BlockSechoir.java:46)
at net.minecraft.server.management.ItemInWorldManager.activateBlockOrUseItem(ItemInWorldManager.java:409)
at net.minecraft.network.NetHandlerPlayServer.processPlayerBlockPlacement(NetHandlerPlayServer.java:593)
at net.minecraft.network.play.client.C08PacketPlayerBlockPlacement.processPacket(C08PacketPlayerBlockPlacement.java:74)
at net.minecraft.network.play.client.C08PacketPlayerBlockPlacement.processPacket(C08PacketPlayerBlockPlacement.java:122)
at net.minecraft.network.NetworkManager.processReceivedPackets(NetworkManager.java:241)
at net.minecraft.network.NetworkSystem.networkTick(NetworkSystem.java:182)
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:726)
at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:614)
at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:118)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:485)
at net.minecraft.server.MinecraftServer$2.run(MinecraftServer.java:752)

A detailed walkthrough of the error, its code path and all known details is as follows:
---------------------------------------------------------------------------------------

-- Head --
Stacktrace:
at fr.powergame.modpg2.client.TileEntitySechoir.getStackInSlot(TileEntitySechoir.java:38)
at net.minecraft.inventory.Slot.getStack(Slot.java:88)
at net.minecraft.inventory.Container.getInventory(Container.java:67)
at net.minecraft.inventory.Container.addCraftingToCrafters(Container.java:53)
at cpw.mods.fml.common.network.internal.FMLNetworkHandler.openGui(FMLNetworkHandler.java:88)
at net.minecraft.entity.player.EntityPlayer.openGui(EntityPlayer.java:2501)
at fr.powergame.modpg2.common.BlockSechoir.onBlockActivated(BlockSechoir.java:46)
at net.minecraft.server.management.ItemInWorldManager.activateBlockOrUseItem(ItemInWorldManager.java:409)
at net.minecraft.network.NetHandlerPlayServer.processPlayerBlockPlacement(NetHandlerPlayServer.java:593)
at net.minecraft.network.play.client.C08PacketPlayerBlockPlacement.processPacket(C08PacketPlayerBlockPlacement.java:74)
at net.minecraft.network.play.client.C08PacketPlayerBlockPlacement.processPacket(C08PacketPlayerBlockPlacement.java:122)
at net.minecraft.network.NetworkManager.processReceivedPackets(NetworkManager.java:241)

-- Ticking connection --
Details:
Connection: net.minecraft.network.NetworkManager@13f82374
Stacktrace:
at net.minecraft.network.NetworkSystem.networkTick(NetworkSystem.java:182)
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:726)
at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:614)
at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:118)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:485)
at net.minecraft.server.MinecraftServer$2.run(MinecraftServer.java:752)

Du coup voici la classe du tileEntity pour la machine mono-slot, les tableaux etc, ça me passe encore au dessus =/

package fr.powergame.modpg2.client;

import cpw.mods.fml.relauncher.Side;
import cpw.mods.fml.relauncher.SideOnly;
import fr.powergame.modpg2.common.AlambicRecipes;
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.network.NetworkManager;
import net.minecraft.network.Packet;
import net.minecraft.network.play.server.S35PacketUpdateTileEntity;
import net.minecraft.tileentity.TileEntity;

public class TileEntitySechoir extends TileEntity implements IInventory
{
    private byte direction;
    private ItemStack[] contents = new ItemStack[3];
    private int workingTime = 0;
    private int workingTimeNeeded = 200;

    @SideOnly(Side.CLIENT)
    public int getCookProgress()
    {
        return this.workingTime * 41 / this.workingTimeNeeded;
    }

    @Override
    public int getSizeInventory()
    {
        return this.contents.length;
    }

    @Override
    public ItemStack getStackInSlot(int slotIndex)
    {
        return this.contents[slotIndex];
    }

    public ItemStack decrStackSize(int slotIndex, int amount)
    {
         if (this.contents[slotIndex] != null)
            {
                ItemStack itemstack;

                if (this.contents[slotIndex].stackSize <= amount)
                {
                    itemstack = this.contents[slotIndex];
                    this.contents[slotIndex] = null;
                    this.markDirty();
                    return itemstack;
                }
                else
                {
                    itemstack = this.contents[slotIndex].splitStack(amount);

                    if (this.contents[slotIndex].stackSize == 0)
                    {
                        this.contents[slotIndex] = null;
                    }

                    this.markDirty();
                    return itemstack;
                }
            }
            else
            {
                return null;
            }
    }

    @Override
    public ItemStack getStackInSlotOnClosing(int slotIndex)
    {
        if (this.contents[slotIndex] != null)
       {
           ItemStack itemstack = this.contents[slotIndex];
           this.contents[slotIndex] = null;
           return itemstack;
       }
       else
       {
           return null;
       }
    }

    @Override
    public void setInventorySlotContents(int slotIndex, ItemStack stack)
    {
        this.contents[slotIndex] = stack;

       if (stack != null && stack.stackSize > this.getInventoryStackLimit())
       {
           stack.stackSize = this.getInventoryStackLimit();
       }

       this.markDirty();
    }

    @Override
    public String getInventoryName()
    {
        return "tile.Sechoir";
    }

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

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

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

    @Override
    public void openInventory()
    {
    }

    @Override
    public void closeInventory()
    {        
    }

    @Override
    public boolean isItemValidForSlot(int slot, ItemStack stack)
    {
        return slot == 1 ? false : true;
    }

    public boolean isBurning()
    {
        return this.workingTime > 0;
    }

    private boolean canSmelt()
    {
        if (this.contents[0] == null)
        {
            return false;
        }
        else
        {
            ItemStack itemstack = AlambicRecipes.smelting().getSmeltingResult(new ItemStack[]{this.contents[0]});
            if (itemstack == null) return false;
            if (this.contents[1] == null) return true;
            if (!this.contents[1].isItemEqual(itemstack)) return false;
            int result = contents[1].stackSize + itemstack.stackSize;
            return result <= getInventoryStackLimit() && result <= this.contents[1].getMaxStackSize();
        }
    }

    public void updateEntity()
    {
        if(this.isBurning() && this.canSmelt())
        {
            ++this.workingTime;
        }
        if(this.canSmelt() && !this.isBurning())
        {
            this.workingTime = 1;
        }
        if(this.canSmelt() && this.workingTime == this.workingTimeNeeded)
        {
            this.smeltItem();
            this.workingTime = 0;
        }
        if(!this.canSmelt())
        {
            this.workingTime= 0;
        }
    }

    public void smeltItem()
    {
        if (this.canSmelt())
        {
            ItemStack itemstack = AlambicRecipes.smelting().getSmeltingResult(new ItemStack[]{this.contents[0]});
             if (this.contents[1] == null)
             {
                  this.contents[1] = itemstack.copy();
             }
             else if (this.contents[1].getItem() == itemstack.getItem())
             {
                  this.contents[1].stackSize += itemstack.stackSize;
             }

             –this.contents[0].stackSize;

             if (this.contents[0].stackSize <= 0)
             {
                 this.contents[0] = null;
             }
        }
    }

    @Override
    public void writeToNBT(NBTTagCompound compound)
    {
        super.writeToNBT(compound);
        NBTTagList nbttaglist = new NBTTagList();

        for (int i = 0; i < this.contents.length; ++i)
        {
            if (this.contents* != null)
            {
                NBTTagCompound nbttagcompound1 = new NBTTagCompound();
                nbttagcompound1.setByte("Slot", (byte)i);
                this.contents*.writeToNBT(nbttagcompound1);
                nbttaglist.appendTag(nbttagcompound1);
            }
        }

        compound.setTag("Items", nbttaglist);
        compound.setShort("workingTime",(short)this.workingTime);
        compound.setShort("workingTimeNeeded", (short)this.workingTimeNeeded);
        compound.setByte("Direction", this.direction);
    }

    @Override
    public void readFromNBT(NBTTagCompound compound)
    {
        super.readFromNBT(compound);

        NBTTagList nbttaglist = compound.getTagList("Items", 10);
        this.contents = new ItemStack[this.getSizeInventory()];

        for (int i = 0; i < nbttaglist.tagCount(); ++i)
        {
            NBTTagCompound nbttagcompound1 = nbttaglist.getCompoundTagAt(i);
            int j = nbttagcompound1.getByte("Slot") & 255;

            if (j >= 0 && j < this.contents.length)
            {
                this.contents[j] = ItemStack.loadItemStackFromNBT(nbttagcompound1);
            }
        }

        this.workingTime = compound.getShort("workingTime");
        this.workingTimeNeeded = compound.getShort("workingTimeNeeded");
        this.direction = compound.getByte("Direction");
    }

    public byte getDirection()
    {
        return direction;
    }

    public void setDirection(byte direction)
    {
        this.direction = direction;
        this.worldObj.markBlockForUpdate(this.xCoord, this.yCoord, this.zCoord);
    }

    public Packet getDescriptionPacket()
    {
        NBTTagCompound nbttagcompound = new NBTTagCompound();
        this.writeToNBT(nbttagcompound);
        return new S35PacketUpdateTileEntity(this.xCoord, this.yCoord, this.zCoord, 0, nbttagcompound);
    }

    public void onDataPacket(NetworkManager net, S35PacketUpdateTileEntity pkt)
    {
        this.readFromNBT(pkt.func_148857_g());
        this.worldObj.markBlockRangeForRenderUpdate(this.xCoord, this.yCoord, this.zCoord, this.xCoord, this.yCoord, this.zCoord);
    }
}

Qu’est ce que j’ai oublié de modifier pour parvenir à un bloc fonctionnel, svp?

SCAREX

private ItemStack[] contents = new ItemStack[3]; -> à changer en fonction du nombre de slots et il faut aussi modifier le Container

Folgansky

J’ai mis à 2, ça crash aussi. (Le trois est un fail, j’ai fais des tests et j’ai laissé cette valeur pour voir un peu de mon côté mais bon)

Edit: Voici le container

package fr.powergame.modpg2.common;

import fr.powergame.modpg2.client.TileEntitySechoir;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.inventory.Container;
import net.minecraft.inventory.Slot;
import net.minecraft.item.ItemStack;

public class SechoirContainer extends Container
{
    private TileEntitySechoir tileSechoir;

    public SechoirContainer(TileEntitySechoir tile, InventoryPlayer inventory)
    {
        this.tileSechoir = tile;
        this.addSlotToContainer(new Slot(tile, 1, 89, 29));
        this.addSlotToContainer(new SechoirSlotResult(tile, 3, 89, 89));
        this.bindPlayerInventory(inventory);    
    }

    @Override
    public boolean canInteractWith(EntityPlayer player)
    {
        return this.tileSechoir.isUseableByPlayer(player);
    }

    private void bindPlayerInventory(InventoryPlayer inventory)
    {
        int i;
        for(i = 0; i < 3; ++i)
        {
            for(int j = 0; j < 9; ++j)
            {
                this.addSlotToContainer(new Slot(inventory, j + i * 9 + 9, 17 + j * 18, 125 + i * 18));
            }
        }

        for(i = 0; i < 9; ++i)
        {
            this.addSlotToContainer(new Slot(inventory, i, 17 + i * 18, 183));
        }
    }

    public ItemStack transferStackInSlot(EntityPlayer player, int quantity)
    {
        ItemStack itemstack = null;
        Slot slot = (Slot)this.inventorySlots.get(quantity);

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

            if (quantity < this.tileSechoir.getSizeInventory())
            {
                if (!this.mergeItemStack(itemstack1, this.tileSechoir.getSizeInventory(), this.inventorySlots.size(), true))
                {
                    return null;
                }
            }
            else if (!this.mergeItemStack(itemstack1, 0, this.tileSechoir.getSizeInventory(), false))
            {
                return null;
            }

            if (itemstack1.stackSize == 0)
            {
                slot.putStack((ItemStack)null);
            }
            else
            {
                slot.onSlotChanged();
            }
        }

        return itemstack;
    }

    public void onContainerClosed(EntityPlayer player)
    {
        super.onContainerClosed(player);
        this.tileSechoir.closeInventory();
    }
}

BrokenSwing

Essai de mettre ça comme constructeur dans ton container


public SechoirContainer(TileEntitySechoir tile, InventoryPlayer inventory)
   {
       this.tileSechoir = tile;
       this.addSlotToContainer(new Slot(tile, 0, 89, 29));
       this.addSlotToContainer(new SechoirSlotResult(tile, 1, 89, 89));
       this.bindPlayerInventory(inventory);    
   }

Folgansky

Ah oui, j’avais bêtement supprimé les autres lignes sans réaliser ce que ces paramètres représentaient.

Mais ça “crash” toujours, en fait ça fait “terminated” la connexion.
Peut-être mon GUIHandler qui n’est pas bon, j’ai eu des hésitations en ajoutant les nouveaux GUI:

package fr.powergame.modpg2.common;

import cpw.mods.fml.common.network.IGuiHandler;
import fr.powergame.modpg2.client.TileEntityAlambic;
import fr.powergame.modpg2.client.TileEntityEntubeuse;
import fr.powergame.modpg2.client.TileEntitySechoir;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.world.World;

public class GuiHandler implements IGuiHandler
{
    @Override
    public Object getServerGuiElement(int ID, EntityPlayer player, World world, int x, int y, int z)
    {
        TileEntity tile = world.getTileEntity(x, y, z);

        if(tile instanceof TileEntityAlambic)
        {
            return new AlambicContainer((TileEntityAlambic)tile, player.inventory);
        }
        if(tile instanceof TileEntitySechoir)
        {
            return new SechoirContainer((TileEntitySechoir)tile, player.inventory);
        }
        if(tile instanceof TileEntityEntubeuse)
        {
            return new EntubeuseContainer((TileEntityEntubeuse)tile, player.inventory);
        }
        return null;
    }

    @Override
    public Object getClientGuiElement(int ID, EntityPlayer player, World world, int x, int y, int z)
    {
        TileEntity tile = world.getTileEntity(x, y, z);

        if(tile instanceof TileEntityAlambic)
        {
            return new AlambicGUI((TileEntityAlambic)tile, player.inventory);
        }
        if(tile instanceof TileEntitySechoir)
        {
            return new SechoirContainer((TileEntitySechoir)tile, player.inventory);
        }
        if(tile instanceof TileEntityEntubeuse)
        {
            return new EntubeuseContainer((TileEntityEntubeuse)tile, player.inventory);
        }
        return null;
    }
}

edit: sachant que je mets “0” pour les trois blocs:

public boolean onBlockActivated(World world, int x, int y, int z, EntityPlayer player, int side, float hitx, float hity, float hitz)
    {
        if (world.isRemote)
        {
            return true;
        }
        else
        {
            player.openGui(ModPg2.instance, 0, world, x, y, z);
            return true;
        }
    }

Ps: je sais que mes noms de classes devraient être en anglais, mais c’est un mod qui ne sera pas redistribué ou autre, ça vous pique simplement un peu les yeux sur le forum

AymericRed

Dans ta fonction getClientGuiElement :

if(tile instanceof TileEntitySechoir)
{
return new SechoirContainer((TileEntitySechoir)tile, player.inventory);
}
if(tile instanceof TileEntityEntubeuse)
{
return new EntubeuseContainer((TileEntityEntubeuse)tile, player.inventory);
}

Le problème vient de là car il faut toujours retourner un Gui, donc tu dois changer les “return UnContainer” par des “return UnGui”.

PS: “Entubeuse”, quel nom !

Folgansky

Hum, je ne comprend pas, l’Alambic ne pose aucun soucis alors qu’il retourne un Container lui-aussi.

:::
(https://t2.ftcdn.net/jpg/00/03/53/61/500_F_3536130_VwfKdvaF345kuC0foyHZNXq8eo7dI5.jpg)
:::

AymericRed

Non c’est un gui : " return new AlambicGUI((TileEntityAlambic)tile, player.inventory);"

Folgansky

Ouki! Sacrée poutre que tu me retires de l’oeil, je me persuadais moi-même qu’il n’y avait qu’une fonction dans cette classe… Bref merci pour ce point qui m’empêchait d’avancer.

Du coup la machine à deux slots fonctionnent bien, pas de soucis.
Celle à un seul par contre ne semble pas comprendre les recettes suggérées.

Comme les autres fonctionnent par tableau de stacks mais que je suppose que ce n’est pas nécessaire et j’ai donc regardé la classe du four vanilla pour obtenir cette classe:

package fr.powergame.modpg2.common;

import java.util.HashMap;
import java.util.Iterator;
import java.util.Map;
import java.util.Map.Entry;

import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;

public class SechoirRecipes
{
    private static final SechoirRecipes smeltingBase = new SechoirRecipes();

    @SuppressWarnings("rawtypes")
    private Map smeltingList = new HashMap();

//    private Map <itemstack[], itemstack="">smeltingList = new HashMap<itemstack[], itemstack="">();

    public SechoirRecipes()
    {
        this.addRecipe(new ItemStack(ModPg2.itemMetadata, 1, 21), new ItemStack(ModPg2.itemMetadata, 1, 22));
        this.addRecipe(new ItemStack(ModPg2.itemMetadata, 1, 23), new ItemStack(ModPg2.itemMetadata, 1, 24));    
    }

    @SuppressWarnings("unchecked")
    public void addRecipe(ItemStack stack1, ItemStack stack2)
    {
        this.smeltingList.put(stack1, stack2);
    }

    public void addRecipe(Item item1, ItemStack stack)
    {
        this.addRecipe(new ItemStack(item1), stack);
    }

//    @SuppressWarnings("rawtypes")
//    public ItemStack getSmeltingResult(ItemStack[] stack)
//    {
//        Iterator iterator = this.smeltingList.entrySet().iterator();
//        Entry entry;
//        
//        do
//        {
//            if (!iterator.hasNext())
//            {
//                return null;
//            }
//            entry = (Entry)iterator.next();
//        }
//        while (!this.isSameKey(stack, (ItemStack[])entry.getKey()));
//        
//        return (ItemStack)entry.getValue();
//    }
//    
//    private boolean isSameKey(ItemStack[] stackList, ItemStack[] stackList2)
//    {
//        boolean isSame = false;
//        for(int i=0; i<=2; i++)
//        {
//            if(stackList*.getItem() == stackList2*.getItem() && stackList*.getItemDamage() == stackList2*.getItemDamage())
//            {
//                isSame = true;
//            }
//            else
//            {
//                return false;
//            }
//        }
//        return isSame;
//    }

    @SuppressWarnings("rawtypes")
    public ItemStack getSmeltingResult(ItemStack stack)
    {
        Iterator iterator = this.smeltingList.entrySet().iterator();
        Entry entry;

        do
        {
            if (!iterator.hasNext())
            {
                return null;
            }            
            entry = (Entry)iterator.next();
        }
        while (!this.isSameKey(stack, (ItemStack)entry.getKey()));

        return (ItemStack)entry.getValue();
    }

    private boolean isSameKey(ItemStack stack1, ItemStack stack2)
    {
        return stack2.getItem() == stack1.getItem() && (stack2.getItemDamage() == 32767 || stack2.getItemDamage() == stack1.getItemDamage());
    }

    @SuppressWarnings("rawtypes")
    public Map getSmeltingList()
    {
        return this.smeltingList;
    }

    public static SechoirRecipes smelting()
    {
        return smeltingBase;
    }
}

Je ne sais pas ce que devient le smeltingList dans le cas où il n’y a qu’un slot d’input</itemstack[],></itemstack[],>

AymericRed

Tu as juste à remplacer les ItemStack[] par des ItemStacks.
PS : Tu n’as pas de crash, parce que tu essayes de cast un ItemStack[] en un ItemStack à la ligne 84;

Folgansky

Euh non je n’ai pas de crash si c’est une question et j’ai retiré déjà tous les ItemStack[] que j’ai vu, le code ne fonctionne pas cela dit.

Tu as peut-être répondu avant que j’édit mon message pour la dernière fois, mauvais timing de ma part.
J’ai changé la fonction isSameKey et du coup les arguments de la fonction précédente et le mapping, enfin le “Map truc bidule” comme je dirais…

AymericRed

Là ça devrait marcher, montre la classe de ton TE si les fonctions canSmelt, smelt ou updateEntity ont changé.
Et essaye de mettre workingTime >= workingTimeNeeded au lieu de workingTime == workingTimeNeeded dans ta fonction updateEntity().

Folgansky

J’ai essayé de faire le plus attention à changer les bons slots, à voir =d

Edit:

            ItemStack itemstack = SechoirRecipes.smelting().getSmeltingResult(new ItemStack[]{this.contents[0]});

Mon erreur était là à cause d’un oubli, j’avais laissé les recettes de l’alambic comme un idiot (je travaille trop tard visiblement)

Mais du coup, le new ItemStack[]{this.contents[0]} devient quoi? Je teste des choses mais c’pas conventionnel visiblement

SCAREX

ça dépend de ce que tu veux faire mais si tu n’as besoin que d’un seul slot tu n’es pas obligé d’utiliser une array d’ItemStack

Folgansky

Oui clairement je n’ai qu’un slot et donc pas besoin de cet array d’ItemStack

C’est mon “niveau” de java qui me bloque

Je teste " new ItemStack({this.contents[0]}) " ou encore " new ItemStack{this.contents[0]} " mais eclipse semble se demander ce que j’essaie de faire.