Hi all,

 

I'm trying to make a custom picaxe that will smelt the block you mine. However, when I try to get the smelt result of a block, it always returns null, EXCEPT for the first time for each new block.

The first stone I mined, smelted into stone, however after that it returned null every time. The first iron ore I mined dropped an iron ingot, after that FurnaceRecipes.smelting().getSmeltingResult(new ItemStack(Block.blocksList[par3])) always returned null for iron ore (I have checked to make sure that new ItemStack(Block.blocksList[par3]) is of the correct type). Can anyone help me? Here is my onBlockDestroyed code

 

@Override
public boolean onBlockDestroyed(ItemStack par1ItemStack, World par2World, int par3, int x, int y, int z, EntityLiving par7EntityLiving) {

	        boolean dropModified = false;
		if (!par2World.isRemote)
		{


			ItemStack res = FurnaceRecipes.smelting().getSmeltingResult(new ItemStack(Block.blocksList[par3]));
			if (res != null)
			{
				float f = 0.7F;
				double d0 = (double)(par2World.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
				double d1 = (double)(par2World.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
				double d2 = (double)(par2World.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
				EntityItem entityitem = new EntityItem(par2World, (double)x + d0, (double)y + d1, (double)z + d2, res);
				entityitem.delayBeforeCanPickup = 10;
				par2World.spawnEntityInWorld(entityitem);
				dropModified = true;
			}

	        }
                        if (!dropModified)
		      super.onBlockDestroyed(par1ItemStack, par2World, par3, x, y, z, par7EntityLiving);
                        else
                             par2World.destroyBlock(x, y, z, false);
	        return true;
}

Do you need the

                        if (!dropModified)
		      super.onBlockDestroyed(par1ItemStack, par2World, par3, x, y, z, par7EntityLiving);
                        else
                             par2World.destroyBlock(x, y, z, false);

? If not, try deleting it.

which heu ......

 

which class has the code for onBlockDestroyed?

 

The MyPickaxe extends ItemPickaxe class

Do you need the

                        if (!dropModified)
		      super.onBlockDestroyed(par1ItemStack, par2World, par3, x, y, z, par7EntityLiving);
                        else
                             par2World.destroyBlock(x, y, z, false);

? If not, try deleting it.

 

The whole statement? Then the block will drop the smelted contents and the original contents too won't it?

 

I need the pickaxe should only smelt things SOMETIMES as in there's a chance it will drop regularly. I didnt put that part into the code I posted but it shouldn't affect it. My issue is that FurnaceRecipes returns null even though it shouldn't.

which heu ......

 

which class has the code for onBlockDestroyed?

 

im serious btw, this might be a huge problem

which heu ......

 

which class has the code for onBlockDestroyed?

 

im serious btw, this might be a huge problem

 

Block class has the onBlockDestroyedByPlayer.

which heu ......

 

which class has the code for onBlockDestroyed?

 

im serious btw, this might be a huge problem

 

As I said my class is extending ItemPickaxe.

 

OnBlockDestroyed is part of ItemTool

which heu ......

 

which class has the code for onBlockDestroyed?

 

im serious btw, this might be a huge problem

 

As I said my class is extending ItemPickaxe.

 

OnBlockDestroyed is part of ItemTool

 

ItemPickaxe extends ItemTool

which heu ......

 

which class has the code for onBlockDestroyed?

 

im serious btw, this might be a huge problem

 

As I said my class is extending ItemPickaxe.

 

OnBlockDestroyed is part of ItemTool

 

ItemPickaxe extends ItemTool

 

Yes, that is how I overrode it.

just for fun, try

 

ItemStack iss = (ItemStack)FurnaceRecipes.smelting().getSmeltingList().get(par3);

 

its basicly what vanilla is doing. also safecheck non-null of "iss"

just for fun, try

 

ItemStack iss = (ItemStack)FurnaceRecipes.smelting().getSmeltingList().get(par3);

 

its basicly what vanilla is doing. also safecheck non-null of "iss"

 

It seems like my problem is now in this part:

 

if (!par2World.isRemote && par2World.getGameRules().getGameRuleBooleanValue("doTileDrops"))
			{
				float f = 0.7F;
				double d0 = (double)(par2World.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
				double d1 = (double)(par2World.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
				double d2 = (double)(par2World.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
				EntityItem entityitem = new EntityItem(par2World, (double)x + d0, (double)y + d1, (double)z + d2, iss);
				entityitem.delayBeforeCanPickup = 10;
				par2World.spawnEntityInWorld(entityitem);
				dropModified = true;
			}

 

Its not actually spawning anything even though the code is run just fine. It spawns something once then stops

I'm not that familiar with java, but have you tried a for loop?

I'm not that familiar with java, but have you tried a for loop?

 

How exactly would a for loop help?