Furnace Broken When Extracting Items

[OBCLetter]

When I try to extract an item from a furnace, the furnace is broken. Not like glitched broken, but like survival player mines the block broken. Here is my code:

	@Override
	public void updateTask() {
		if (soldier.wantsToGoTo != BlockPos.NULL_VECTOR) {
			soldier.getNavigator().tryMoveToXYZ(soldier.wantsToGoTo.getX(), soldier.wantsToGoTo.getY(),
					soldier.wantsToGoTo.getZ(), speedIn);
			double distance = soldier.getPosition().getDistance(soldier.wantsToGoTo.getX(), soldier.wantsToGoTo.getY(),
					soldier.wantsToGoTo.getZ());
			if (distance < 5) {
				TileEntityFurnace furnace = soldier.getFurnace();
				if (soldier.getHeldItem(EnumHand.MAIN_HAND).isEmpty()) {
					soldier.setHeldItem(EnumHand.MAIN_HAND, furnace.getStackInSlot(2));
					furnace.getStackInSlot(2).splitStack(furnace.getStackInSlot(2).getCount());
				} else if (soldier.getHeldItem(EnumHand.MAIN_HAND) == furnace.getStackInSlot(2)) {
					soldier.setHeldItem(EnumHand.MAIN_HAND, new ItemStack(furnace.getStackInSlot(2).getItem(),
							furnace.getStackInSlot(2).getCount() + soldier.getHeldItem(EnumHand.MAIN_HAND).getCount()));
					furnace.getStackInSlot(2).splitStack(furnace.getStackInSlot(2).getCount());
				}

			}
		}
	}

 

[Draco18s]

Do not use setHeldItem

You want to insert or extract items from an Inventory?

 

Step 1: Use GetCapability to get the inventory

Step 2: Use the insert/extract methods to insert and extract

 

59 minutes ago, OBCLetter said:

soldier.getFurnace();

Also, what the bollocks? Why is this method in your entity class at all?

Edited April 13, 2018 by Draco18s

[OBCLetter]

8 minutes ago, Draco18s said:

Also, what the bollocks? Why is this method in your entity class at all?

It remembers which furnace it originally built when building a house. When the original furnace is destroyed, it seeks a new one and sets that one as it's furnace.

[Draco18s]

Okay then.

In any case, the code you've posted cannot destroy blocks.