Hello everybody,

im working on a harvester and wrote a tileentity which checks in a 3x3 area for full grown crops.

But it does not work,

i need to clean up my code, i know but ill do that if the harvester works

 

Code : 

public class TileHarvesterTicker extends TileEntity implements ITickable{
	BlockPos checkstart = null;
	IBlockState statebehind = null;
	BlockPos behind = null;
	private int x = 0;
	private int y = 0;
	private int z = 0;
	@Override
	public void update() {
		if (checkstart == null)
			switch (world.getBlockState(pos).getValue(Machine.FACING)) {
			case DOWN:
			{
				break;
			}
			case EAST:
			{
				checkstart = new BlockPos(pos.getX()+1, pos.getY(), pos.getZ()-1);
				break;
			}
			case NORTH:
			{
				checkstart = new BlockPos(pos.getX()-1, pos.getY(), pos.getZ()-3);
				break;
			}
			case SOUTH:
			{
				checkstart = new BlockPos(pos.getX()-1, pos.getY(), pos.getZ()+1);
				break;
			}
			case UP:
			{
				
			}
			case WEST:
			{
				checkstart = new BlockPos(pos.getX()-3, pos.getY(), pos.getZ()-1);
				break;
			}		
			}
		if (x == 3) {
			x=0;z++;
		}else if (z == 4)
			z=0;x=0;
		IBlockState state = world.getBlockState(new BlockPos(checkstart.getX() + x, checkstart.getY(), checkstart.getZ()+z));
		if (state == Blocks.WHEAT.getBlockState().getBaseState().withProperty(net.minecraft.block.BlockCrops.AGE, 7)) {
			if(statebehind == null) {
				switch (world.getBlockState(pos).getValue(Machine.FACING)) {
				case DOWN:
				{
					break;
				}
				case EAST:
				{
					statebehind = world.getBlockState(new BlockPos(pos.getX()-1, pos.getY(), pos.getZ()));
					 behind = new BlockPos(pos.getX()-1, pos.getY(), pos.getZ());
					break;
				}
				case NORTH:
				{
					statebehind = world.getBlockState(new BlockPos(pos.getX(), pos.getY(), pos.getZ()+1));
					 behind = new BlockPos(pos.getX(), pos.getY(), pos.getZ()+1);
					break;
				}
				case SOUTH:
				{
					statebehind = world.getBlockState(new BlockPos(pos.getX(), pos.getY(), pos.getZ()-1));
					 behind = new BlockPos(pos.getX(), pos.getY(), pos.getZ()-1);
					break;
				}
				case UP:
				{
					
				}
				case WEST:
				{
					statebehind = world.getBlockState(new BlockPos(pos.getX()+1, pos.getY(), pos.getZ()));
					 behind = new BlockPos(pos.getX()+1, pos.getY(), pos.getZ());
					break;
				}		
				}
				}
			
			world.setBlockState(new BlockPos(checkstart.getX() + x, checkstart.getY(), checkstart.getZ()+z), state.withProperty(BlockCrops.AGE, 0));
			new EntityItem(world, behind.getX(), behind.getY(), behind.getZ(), new ItemStack(Items.WHEAT));

		
		} else if (state == Blocks.CARROTS.getBlockState().getBaseState().withProperty(net.minecraft.block.BlockCrops.AGE, 7)) {
			if(statebehind == null) {
				switch (world.getBlockState(pos).getValue(Machine.FACING)) {
				case DOWN:
				{
					break;
				}
				case EAST:
				{
					statebehind = world.getBlockState(new BlockPos(pos.getX()-1, pos.getY(), pos.getZ()));
					 behind = new BlockPos(pos.getX()-1, pos.getY(), pos.getZ());
					break;
				}
				case NORTH:
				{
					statebehind = world.getBlockState(new BlockPos(pos.getX(), pos.getY(), pos.getZ()+1));
					 behind = new BlockPos(pos.getX(), pos.getY(), pos.getZ()+1);
					break;
				}
				case SOUTH:
				{
					statebehind = world.getBlockState(new BlockPos(pos.getX(), pos.getY(), pos.getZ()-1));
					 behind = new BlockPos(pos.getX(), pos.getY(), pos.getZ()-1);
					break;
				}
				case UP:
				{
					
				}
				case WEST:
				{
					statebehind = world.getBlockState(new BlockPos(pos.getX()+1, pos.getY(), pos.getZ()));
					 behind = new BlockPos(pos.getX()+1, pos.getY(), pos.getZ());
					break;
				}		
				}
				}
			
			world.setBlockState(new BlockPos(checkstart.getX() + x, checkstart.getY(), checkstart.getZ()+z), state.withProperty(BlockCrops.AGE, 0));
			new EntityItem(world, behind.getX(), behind.getY(), behind.getZ(), new ItemStack(Items.CARROT, (int) (Math.random() * Math.random() * 2 +1)));

		
		}
		x++;
	}

}

source: https://github.com/Eaglesmoddingteam/Eagles-Essentials/tree/master/src/main

Set a breakpoint and run it in the debugger. Figure out what it's not doing that it should be.

21 minutes ago, Draco18s said:

Set a breakpoint and run it in the debugger. Figure out what it's not doing that it should be.

thx

ill try that