[1.6.4] Error with macerator when inserting ore

[ZippyBling]

Hello all.

 

I am having an error when inserting an ore into a slot on my macerator. The crash log can be seen in that pastebin link below.

 

http://pastebin.com/4skAD0B9

 

It works fine when it is in eclipse but when I have recompiled and reobfuscated it, and add it to a mod pack in this instance FTB Monster, it crashes.

I would really like to get this fixed so I can add it to my server. I looked through the error log and went over the code but I cannot see anything that would do it as it runs fine in eclipse!

Macerator TileEntity Class

 

/*

 

package MinersTech.tileentity;

 

import MinersTech.Machines.BlockMacerator;

import MinersTech.MinersTech;

import cpw.mods.fml.relauncher.Side;

import cpw.mods.fml.relauncher.SideOnly;

import net.minecraft.entity.player.EntityPlayer;

import net.minecraft.inventory.ISidedInventory;

import net.minecraft.item.Item;

import net.minecraft.item.ItemStack;

import net.minecraft.item.crafting.FurnaceRecipes;

import net.minecraft.nbt.NBTTagCompound;

import net.minecraft.nbt.NBTTagList;

import net.minecraft.tileentity.TileEntity;

import net.minecraftforge.oredict.OreDictionary;

 

/**

*

* @author ZippyBling

*/

public class TileEntityMacerator extends TileEntity implements ISidedInventory{

private static final int[] slots_top = new int[] {0};

private static final int[] slots_bottom = new int[] {2, 1};

private static final int[] slots_sides = new int[] {1};

 

/**

* The ItemStacks that hold the items currently being used in the furnace

*/

private ItemStack[] slots = new ItemStack[3];

 

/** the speed of this furnace, 200 is normal / how many ticks it takes : 30 ticks = 1 second */

public int maceratingSpeed = 100;

 

/** The number of ticks that the furnace will keep burning */

public int power;

public int maxPower = 15000;

 

/** The number of ticks that the current item has been cooking for */

public int cookTime;

 

private String field_94130_e;

 

/**

* Returns the number of slots in the inventory.

*/

public int getSizeInventory()

{

    return this.slots.length;

}

 

/**

* Returns the stack in slot i

*/

public ItemStack getStackInSlot(int par1)

{

    return this.slots[par1];

}

 

/**

* Removes from an inventory slot (first arg) up to a specified number (second arg) of items and returns them in a

* new stack.

*/

public ItemStack decrStackSize(int par1, int par2)

{

    if (this.slots[par1] != null)

    {

        ItemStack itemstack;

 

        if (this.slots[par1].stackSize <= par2)

      {

            itemstack = this.slots[par1];

            this.slots[par1] = null;

          return itemstack;

        }

        else

        {

            itemstack = this.slots[par1].splitStack(par2);

 

            if (this.slots[par1].stackSize == 0)

            {

                this.slots[par1] = null;

          }

 

            return itemstack;

        }

    }

    else

    {

        return null;

    }

}

 

/**

* When some containers are closed they call this on each slot, then drop whatever it returns as an EntityItem -

* like when you close a workbench GUI.

*/

public ItemStack getStackInSlotOnClosing(int par1)

{

    if (this.slots[par1] != null)

    {

        ItemStack itemstack = this.slots[par1];

        this.slots[par1] = null;

        return itemstack;

    }else{

        return null;

    }

}

 

/**

* Sets the given item stack to the specified slot in the inventory (can be crafting or armor sections).

*/

public void setInventorySlotContents(int par1, ItemStack par2ItemStack)

{

    this.slots[par1] = par2ItemStack;

 

    if (par2ItemStack != null && par2ItemStack.stackSize > this.getInventoryStackLimit())

    {

    par2ItemStack.stackSize = this.getInventoryStackLimit();

    }

}

 

/**

* Returns the name of the inventory.

*/

public String getInvName()

{

    return this.isInvNameLocalized() ? this.field_94130_e : "container.macerator";

}

 

/**

* If this returns false, the inventory name will be used as an unlocalized name, and translated into the player's

* language. Otherwise it will be used directly.

*/

public boolean isInvNameLocalized()

{

    return this.field_94130_e != null && this.field_94130_e.length() > 0;

}

 

/**

* Sets the custom display name to use when opening a GUI linked to this tile entity.

*/

public void setGuiDisplayName(String par1Str)

{

    this.field_94130_e = par1Str;

}

 

/**

* Reads a tile entity from NBT.

*/

public void readFromNBT(NBTTagCompound par1NBTTagCompound)

{

    super.readFromNBT(par1NBTTagCompound);

    NBTTagList nbttaglist = par1NBTTagCompound.getTagList("Items");

    this.slots = new ItemStack[this.getSizeInventory()];

 

    for (int i = 0; i < nbttaglist.tagCount(); ++i)

    {

        NBTTagCompound nbttagcompound1 = (NBTTagCompound)nbttaglist.tagAt(i);

        byte b0 = nbttagcompound1.getByte("Slot");

 

        if (b0 >= 0 && b0 < this.slots.length)

        {

            this.slots[b0] = ItemStack.loadItemStackFromNBT(nbttagcompound1);

        }

    }

 

    this.power = par1NBTTagCompound.getShort("power");

    this.cookTime = par1NBTTagCompound.getShort("CookTime");

 

    if (par1NBTTagCompound.hasKey("CustomName"))

    {

        this.field_94130_e = par1NBTTagCompound.getString("CustomName");

    }

}

 

/**

* Writes a tile entity to NBT.

*/

public void writeToNBT(NBTTagCompound par1NBTTagCompound)

{

    super.writeToNBT(par1NBTTagCompound);

    par1NBTTagCompound.setShort("power", (short)this.power);

    par1NBTTagCompound.setShort("CookTime", (short)this.cookTime);

    NBTTagList nbttaglist = new NBTTagList();

 

    for (int i = 0; i < this.slots.length; ++i)

    {

        if (this.slots != null)

        {

            NBTTagCompound nbttagcompound1 = new NBTTagCompound();

            nbttagcompound1.setByte("Slot", (byte)i);

            this.slots.writeToNBT(nbttagcompound1);

            nbttaglist.appendTag(nbttagcompound1);

        }

    }

 

    par1NBTTagCompound.setTag("Items", nbttaglist);

 

    if (this.isInvNameLocalized())

    {

        par1NBTTagCompound.setString("CustomName", this.field_94130_e);

    }

}

 

/**

* Returns the maximum stack size for a inventory slot. Seems to always be 64, possibly will be extended. *Isn't

* this more of a set than a get?*

*/

public int getInventoryStackLimit()

{

    return 64;

}

 

@SideOnly(Side.CLIENT)

 

/**

* Returns an integer between 0 and the passed value representing how close the current item is to being completely

* cooked

*/

public int getCookProgressScaled(int par1)

{

  return this.cookTime * par1 / this.maceratingSpeed;

}

 

public int getPowerRemainingScaled(int par1){

    return this.power * par1 / this.maxPower;

}

 

/**

* Returns true if the furnace is currently burning

*/

public boolean hasPower()

{

    return this.power > 0;

}

 

public boolean isMacerating(){

return this.cookTime > 0;

}

 

/**

* Allows the entity to update its state. Overridden in most subclasses, e.g. the mob spawner uses this to count

* ticks and creates a new spawn inside its implementation.

*/

public void updateEntity(){

boolean flag = this.power > 0;

    boolean flag1 = false;

 

    if (hasPower() && isMacerating()){

        this.power--;

    }

 

    if (!this.worldObj.isRemote){

        if (this.power < this.maxPower && this.getItemPower(this.slots[1]) > 0){

        this.power += getItemPower(this.slots[1]);

 

        flag1 = true;

       

        if (this.slots[1] != null){

                this.slots[1].stackSize--;

 

                if (this.slots[1].stackSize == 0){

                    this.slots[1] = this.slots[1].getItem().getContainerItemStack(slots[1]);

                }

            }               

        }

 

        if (this.hasPower() && this.canSmelt())

        {

            ++this.cookTime;

 

            if (this.cookTime == this.maceratingSpeed)

            {

                this.cookTime = 0;

                this.smeltItem();

              flag1 = true;

            }

        }

        else

        {

            this.cookTime = 0;

        }

 

        if (flag != this.power > 0)

        {

            flag1 = true;

            BlockMacerator.updateFurnaceBlockState(this.power > 0, this.worldObj, this.xCoord, this.yCoord, this.zCoord);

        }

    }

 

    if (flag1){

        this.onInventoryChanged();

    }

}

 

public boolean isOre(ItemStack itemstack){

String[] oreNames = OreDictionary.getOreNames();

 

for(int i = 0; i < oreNames.length; i++){

if(oreNames.contains("ore")){

    if(OreDictionary.getOres(oreNames) != null){

    if(OreDictionary.getOres(oreNames).get(0).itemID == itemstack.itemID){

    return true;       

    }

    }

}

}

 

return false;

}

 

/**

* Returns true if the furnace can smelt an item, i.e. has a source item, destination stack isn't full, etc.

*/

private boolean canSmelt(){

    if (this.slots[0] == null){

        return false;

    }else{

        ItemStack itemstack = FurnaceRecipes.smelting().getSmeltingResult(this.slots[0]);

        if(itemstack == null) return false;

        if(!isOre(this.slots[0])) return false;

        if(this.slots[2] == null) return true;

        if(!this.slots[2].isItemEqual(itemstack)) return false;

        int result = slots[2].stackSize + (itemstack.stackSize*2);

        return (result <= getInventoryStackLimit() && result <= itemstack.getMaxStackSize());

    }

}

 

/**

* Turn one item from the furnace source stack into the appropriate smelted item in the furnace result stack

*/

public void smeltItem(){

    if(this.canSmelt()){

        ItemStack itemstack = FurnaceRecipes.smelting().getSmeltingResult(this.slots[0]);

       

        if(this.slots[2] == null){

            this.slots[2] = itemstack.copy();

            this.slots[2].stackSize*=2;

        }else if (this.slots[2].isItemEqual(itemstack)){

            slots[2].stackSize += (itemstack.stackSize*2);

        }

 

        --this.slots[0].stackSize;

 

        if(this.slots[0].stackSize <= 0){

            this.slots[0] = null;

        }

    }

}

 

 

/**

* Returns the number of ticks that the supplied fuel item will keep the furnace burning, or 0 if the item isn't

* fuel

*/

public static int getItemPower(ItemStack par0ItemStack){

    if (par0ItemStack == null){

        return 0;

    }else{

    int i = par0ItemStack.getItem().itemID;

   

    if (i == Item.redstone.itemID) return 10;

        if (i == MinersTech.ZianiteCoal.itemID) return 500;

        return 0;

    }

   

 

 

}

 

/**

* Return true if item is a fuel source (getItempower() > 0).

*/

public static boolean isItemFuel(ItemStack par0ItemStack)

{

    return getItemPower(par0ItemStack) > 0;

}

 

/**

* Do not make give this method the name canInteractWith because it clashes with Container

*/

public boolean isUseableByPlayer(EntityPlayer par1EntityPlayer)

{

    return this.worldObj.getBlockTileEntity(this.xCoord, this.yCoord, this.zCoord) != this ? false : par1EntityPlayer.getDistanceSq((double)this.xCoord + 0.5D, (double)this.yCoord + 0.5D, (double)this.zCoord + 0.5D) <= 64.0D;

}

 

public void openChest() {}

 

public void closeChest() {}

 

/**

* Returns true if automation is allowed to insert the given stack (ignoring stack size) into the given slot.

*/

public boolean isItemValidForSlot(int par1, ItemStack par2ItemStack)

{

return par1 == 2 ? false : (par1 == 1 ? isItemFuel(par2ItemStack) : true);

}

 

/**

* Returns an array containing the indices of the slots that can be accessed by automation on the given side of this

* block.

*/

public int[] getAccessibleSlotsFromSide(int par1)

{

    return par1 == 0 ? slots_bottom : (par1 == 1 ? slots_top : slots_sides);

}

 

/**

* Returns true if automation can insert the given item in the given slot from the given side. Args: Slot, item,

* side

*/

public boolean canInsertItem(int par1, ItemStack par2ItemStack, int par3)

{

      return this.isItemValidForSlot(par1, par2ItemStack);

}

 

/**

* Returns true if automation can extract the given item in the given slot from the given side. Args: Slot, item,

* side

*/

public boolean canExtractItem(int par1, ItemStack par2ItemStack, int par3)

{

  return par3 != 0 || par1 != 1 || par2ItemStack.itemID == Item.bucketEmpty.itemID;

}

}

 

 

 

The code was from ScratchForFun's tutorial and I have made a few edits.

 

Thanks!

 

- ZippyBling

Nick

[Mitchellbrine]

Hey, um, I wouldn't make the arrays 0. Just saying. In the crash report, it says that there is an IndexOutOfBoundsException with zero involved. In all the instances, i is equal to 0 or slots[0] is involved.

 

That's most likely your error.

[ZippyBling]

Thanks.

 

I'll check it out.

[SanAndreaP]

Hey, um, I wouldn't make the arrays 0. Just saying. In the crash report, it says that there is an IndexOutOfBoundsException with zero involved. In all the instances, i is equal to 0 or slots[0] is involved.

 

That's most likely your error.

 

If you would read the error, you would see it appears to be in this method:

public boolean isOre(ItemStack itemstack)

Also he doesn't make any array with the size of 0.

 

To ZippyBling:

What I can see is that this is your problem:

             if(OreDictionary.getOres(oreNames) != null){
                if(OreDictionary.getOres(oreNames).get(0).itemID == itemstack.itemID){
                   return true;                 
                }
             }

You check if the value returned by getOres is not null but don't check if it has a size > 0.

[sequituri]

In simple terms, there may be a list (it's not null) but it also may be empty (it has no 0th element). Unless you know for a fact that the list is non-empy, check for it before using any element.