Probléme machine

Un Ancien Utilisateur

et la une Class surgit

package com.adamitemod.mod;

import cpw.mods.fml.relauncher.Side;
import cpw.mods.fml.relauncher.SideOnly;
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;

public class TileEntityMachineCraft extends TileEntity implements IInventory {

private ItemStack[] contents = new ItemStack[2]; // #SCAREX : tu n'as que 2 slots
private int workingTime = 0;
private int workingTimeNeeded = 60;

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

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

@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");
    }

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

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

@Override
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.machineTuto";
}

@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 == 3 ? false : true;
}

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

private boolean canSmelt()
    {
        if (this.contents[0] == null) // #SCAREX : tu n'as qu'un seul slot ici
        {
            return false;
        }
        else
        {
            ItemStack itemstack = MachineRecipesCraft.smelting().getSmeltingResult(this.contents[0]); // #SCAREX : un seul slot
            if (itemstack == null) return false;
            if (this.contents[3] == null) return true;
            if (!this.contents[3].isItemEqual(itemstack)) return false;
            int result = contents[3].stackSize + itemstack.stackSize;
            return result <= getInventoryStackLimit() && result <= this.contents[3].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 = MachineRecipesCraft.smelting().getSmeltingResult(this.contents[0]); // #SCAREX : tu n'as qu'un seul input
             if (this.contents[1] == null) // #SCAREX : ton slot d'output devient 1
             {
                  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;
             }
        }
    }
}

Voilla

SCAREX

Pour éviter les doubles posts : tu peux éditer le message.

Du coup si tu veux faire un UncraftingTable, tu as besoin de 1 input, 9 outputs et pas besoin de classe pour les recettes car il y a déjà celle de minecraft.

PS : fait gaffe il y a plusieurs moyens de crafter quelque chose

Snowy_1803

Ligne 13, tu mets 2 slots alors que tu as besoin de 10 slots.

Un Ancien Utilisateur

[font=Ubuntu, sans-serifDu coup si tu veux faire un UncraftingTable, tu as besoin de 1 input, 9 outputs et pas besoin de classe pour les recettes car il y a déjà celle de minecraft.]

[font=Ubuntu, sans-serifPS : fait gaffe il y a plusieurs moyens de crafter quelque chose]

oui mais si je met le class de minecraft je porrais pas decraft les truc de mon mod si ?

sinon sa ne crash plus !

EDIT je veux faire sa :

this.addRecipe(ItemsMod.adamiteShovel, new ItemStack(ItemsMod.adamite, ItemsMod.fission));
sa me demande de retirer sa ItemsMod.fission

Superloup10

this.addRecipe(ItemsMod.adamiteShovel, new ItemStack(ItemsMod.adamite), new ItemStack(ItemsMod.fission));

Un Ancien Utilisateur

j’avais essayer et sa me metais un erreur sur addRecipe et sa me met toujour l’erreur

Superloup10

Tu peux envoyer la class où se trouve cette erreur ?

Un Ancien Utilisateur

package com.adamitemod.mod;

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

import com.adamitemod.mod.init.BlocksMod;
import com.adamitemod.mod.init.ItemsMod;

import net.minecraft.block.Block;
import net.minecraft.init.Blocks;
import net.minecraft.init.Items;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;

public class MachineRecipesCraft {

private static final MachineRecipesCraft smeltingBase = new MachineRecipesCraft();
private HashMap <itemstack, itemstack="">smeltingList = new HashMap(); // #SCAREX : c'est mieux d'indiquer ce que l'on met dans la HashMap

public MachineRecipesCraft()
{

this.addRecipe(ItemsMod.adamiteShovel, new ItemStack(ItemsMod.adamite), new ItemStack(ItemsMod.fission)); // #SCAREX : tu changes ta recette
}

public void addRecipe(ItemStack input, ItemStack output)
{
this.smeltingList.put(input, output);
}

        public void addRecipe(Item input, ItemStack output)
{
this.addRecipe(new ItemStack(input), output);
}

public void addRecipe(Block input, ItemStack output)
{
this.addRecipe(Item.getItemFromBlock(input), output);
}

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 machinesStack, ItemStack key) // #SCAREX : vu que tu n'as qu'un seul item, ça ne sert à rien de faire une boucle
{
return machinesStack.getItem() == key.getItem();
}

public HashMap <itemstack, itemstack="">getSmeltingList()
{
       return this.smeltingList;
    }

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

ses à la ligne 25</itemstack,></itemstack,>

Deleted

Quelle est l’erreur indiquée par eclipse ?

Un Ancien Utilisateur

sur addRecipe: remove argument to match ‘add recipe(item, ItemStack)’

Deleted

Tu as un paramètre en trop

Un Ancien Utilisateur

Bas oui je sait ses sa new ItemStack(ItemsMod.fission)___dez double post mais alors je fait comment pour ajouter un item a mon craft car j’ai neuf ouput et un input je voudrai que quand je mais un item sa me donne un item différent dans le 9ouput donc moi j’ai déja essayer mais j’y suis pas arriver qui sait ?

robin4002

Ton code actuelle n’est pas adapté pour avoir plusieurs output.
Il faut utiliser un tableau d’item stack au lieu d’un simple item stack pour ça :
private HashMap <itemstack, itemstack**[]**=“”>smeltingList = new HashMap();

Il faut adapter en conséquence les autres fonctions :
public void addRecipe(ItemStack input, ItemStack**…** output)
public ItemStack**[]** getSmeltingResult(ItemStack stack)

etc …</itemstack,>

Un Ancien Utilisateur

Bien Re tous le mondes j’ai un pb dans mon code le Voici

package com.adamitemod.mod;

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

import com.adamitemod.mod.init.BlocksMod;
import com.adamitemod.mod.init.ItemsMod;

import net.minecraft.block.Block;
import net.minecraft.init.Blocks;
import net.minecraft.init.Items;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;

public class MachineRecipesCraft {

private static final MachineRecipesCraft smeltingBase = new MachineRecipesCraft();
private HashMap <itemstack, itemstack[]="">smeltingList = new HashMap(); // #SCAREX : c'est mieux d'indiquer ce que l'on met dans la HashMap

public MachineRecipesCraft()
{

public void addRecipe(ItemStack input, ItemStack… output) // #SCAREX : tu changes ta recette
}

public void addRecipe(ItemStack input, ItemStack... output)
{
this.smeltingList.put(input, output);
}

        public void addRecipe(Item input, ItemStack output)
{
this.addRecipe(new ItemStack(input), output);
}

public void addRecipe(Block input, ItemStack output)
{
this.addRecipe(Item.getItemFromBlock(input), output);
}

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 machinesStack, ItemStack key) // #SCAREX : vu que tu n'as qu'un seul item, ça ne sert à rien de faire une boucle
{
return machinesStack.getItem() == key.getItem();
}

public HashMap <itemstack, itemstack[]="">getSmeltingList()
{
       return this.smeltingList;
    }

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

plus précisement à la ligne 25</itemstack,></itemstack,>

AymericRed

Lis ça (au moins le début, les bases, parce que c’est long).
Et regarde tes “{” et “}”, tu ne remarques pas que tu ne fermes pas ton bloc (avec un “}”) après le contructeur (“public MachineRecipesCraft(){”). Et ensuite que tu n’ouvres pas le bloc de ta fonction avec “{” après “public void addRecipe(ItemStack input, ItemStack… output)[font=monospace”.]
Et j’ajoute aussi, là l’erreur était simple à voir, et comme t’a mis la ligne ça allait, mais des fois ça peut être plus compliqué donc ajoute le(s) message(s) que eclipse met sur l’erreur dans le futur.

Un Ancien Utilisateur

l’open classe room je l’ai déja lu . . . mais sinon je dois faire quoi ? car j’ai pas comrpis le tebleau

AymericRed

Lignes 23 à 31 :


public MachineRecipesCraft()
{
public void addRecipe(ItemStack input, ItemStack… output) //Retire cette ligne (je viens de me rendre compte que c'est pas les "{" le problème, mais c'est que t'a dut copier deux fois cette ligne et au mauvais endroit (il y a déjà cette fonction plus bas))

}

public void addRecipe(ItemStack input, ItemStack... output)
{
this.smeltingList.put(input, output);
}

Un Ancien Utilisateur

Je met ou mon craft ,___Dézoler double post mais je mais ou mon craft ducoups ?

SCAREX

1- Pas de double post avant 24h ! Nous ne sommes pas à ton service !
2- Cherche un peu et relis le tutoriel ! c’est sous tes yeux !

Un Ancien Utilisateur

[font=Ubuntu, sans-serif- Pas de double post avant 24h ! Nous ne sommes pas à ton service !]

j’ai jamais dis sa puis j’ai précisé que je m’excuser !

et non dans le tutoriel il me dis pas comment faire un tableau de craft alors lis le sujet avant de poster merci !

du coup qui peut m’aider ?