As only destroy crops?

 

This code:

        int i = MathHelper.floor_double(this.posX);
        int j = MathHelper.floor_double(this.posY);
        int k = MathHelper.floor_double(this.posZ);
        
        for (int l = 0; l < 4; ++l)
        {
            i = MathHelper.floor_double(this.posX + (double)((float)(l % 2 * 2 - 1) * 0.25F));
            j = MathHelper.floor_double(this.posY);
            k = MathHelper.floor_double(this.posZ + (double)((float)(l / 2 % 2 * 2 - 1) * 0.25F));
            BlockPos blockpos = new BlockPos(i, j, k);
               
            Class block = BlockCrops.class;
            IBlockState iblockstate = this.worldObj.getBlockState(blockpos);

            if (iblockstate.getClass() == block);
            {
                this.worldObj.destroyBlock(blockpos, true);
            }
        }

 

Destroys farmland and water.

Why do you check for the class???

just check for the Block instance Blocks.Crops itself...

Anyway, it still destroys farmland when there is water.

            int i = MathHelper.floor_double(this.posX);

            int j = MathHelper.floor_double(this.posY);

            int k = MathHelper.floor_double(this.posZ);

           

            for (int l = 0; l < 4; ++l)

            {

                i = MathHelper.floor_double(this.posX + (double)((float)(l % 2 * 2 - 1) * 0.25F));

                j = MathHelper.floor_double(this.posY);

                k = MathHelper.floor_double(this.posZ + (double)((float)(l / 2 % 2 * 2 - 1) * 0.25F));

                BlockPos blockpos = new BlockPos(i, j, k);

                 

                IBlockState iblockstate = this.worldObj.getBlockState(blockpos);

 

                if (iblockstate.getBlock() == Blocks.CARROTS);

                {

                    this.worldObj.destroyBlock(blockpos, true);

                }

            }

What are you doing with all of these

Math.floor

's for? As far as I can see, that's the X, Y & Z coordinate, which for, when looking for blocks, integers will do just fine.

Just use

BlockPos#getAllInBox

to get a list of BlockPos' that you can loop through.

 

I have my own CropGrower (which also harvests them, resetting them to their youngest stage) here.

Lines of interest are: #57, and #104->115

I use this code?

I use this code?

No. As far as I can read from your posts, what my code does, and what you want, are different.

Read it through.

Learn what happens.

Adapt what suits your needs.

Create your own code.

This code works perfectly.

 

            int i = MathHelper.floor_double(this.posX);
            int j = MathHelper.floor_double(this.posY);
            int k = MathHelper.floor_double(this.posZ);
            
            for (int l = 0; l < 4; ++l)
            {
                i = MathHelper.floor_double(this.posX + (double)((float)(l % 2 * 2 - 1) * 0.25F));
                j = MathHelper.floor_double(this.posY);
                k = MathHelper.floor_double(this.posZ + (double)((float)(l / 2 % 2 * 2 - 1) * 0.25F));
                BlockPos blockpos = new BlockPos(i, j, k);
                   
                IBlockState iblockstate = this.worldObj.getBlockState(blockpos);

    			Block block = worldObj.getBlockState(blockpos).getBlock();
    			if(block instanceof BlockCrops){
                    this.worldObj.destroyBlock(blockpos, true);
    			}
            }

 

Thanks.

We still dont know what your  MathHelper.floor_double Method deos.

nbody will understand this thread in the future

Blocks are organized in x y z . There's no need to round anything as all the coordinates for blocks are whole numbers. There's no need for rounding and flooring. Simply check the block the character is looking at and if is a crop remove it.

We still dont know what your  MathHelper.floor_double Method deos.

nbody will understand this thread in the future

 

It's a vanilla method that returns the greatest

int

less than or equal to the

double

argument. It's equivalent to calling

Math.floor

and casting the result to

int

.