Créer un bloc type four (machine)

[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.

[BrokenSwing]

new ItemStack[]{this.contents[0]}

[Folgansky]

Euh, mais précédemment j’ai changé des fonctions pour enlever l’array partout ailleurs comme:

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();

    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 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;
    }
}

Sinon je n’aurais rien eu à changer à ce niveau là dans la TileEntity BrockenSwing

[AymericRed]

Pour transformer cet array “new ItemStack[]{this.contents[0]}” en variable simple, suffit de l’écrire comme ça “this.contents[0]”.

[Folgansky]

Ok nickel.
Du coup this.content[0] est un ItemStack apparement, je suis plus lent à apprendre que je ne le pensais.

Tout fonctionne, merci d’avoir pris de votre temps pour m’aider à régler ces “détails” en somme.

[AymericRed]

this.contents[0] est un item stack car this.contents est un tableau de item stack, et à partir du moment ou un met “[un nombre]” derrière le nom d’un tableau, le type de ce qu’on obtient est le même que celui du tableau, mais c’est une variable toute simple :“ItemStack unstack = this.content[0];”.

[Hydroen]

Super tuto

je viens de le lire et j’ai voulu le faire, mais j’ai un petit souci, quand je veux clique droit sur mon block, rien ne ce passe…

J’ai vérifier le code et tenté de résoudre mon problème mais impossible.

Pouvez-vous me guider pour tenter de trouver mon problème ? car la je sèche, mais j’aimerai essayé de le trouver

Merci d’avance