Bounding Box Manipulation

[MajesticMadman]

Hi,

First question: I have a bounding box of 3x1 (blocks) . As far as entities go works fine, but you can only see the bounding box or destroy the block on the middle block. Is there a way like below to change the clickable hitbox:

	    @Override
	    public AxisAlignedBB getBoundingBox(IBlockState state, IBlockAccess source, BlockPos pos) {
	    	return new AxisAlignedBB(westPos,topPos,northPos,eastPos,bottomPos,southPos);
	    }

 

Second Question: What is the best way of rotating the bounding box when placed?
My idea here is that by default the bounding box is setup for the block facing north or south. If I create an IF for the direction of the block being east or west and then if true it swaps the North & South with the East & West Positions.

However I'm having issues, strFacing needs to be an IBlockState, how do I get this into a value that I can compare?

 

 

	public String strFacing;

//////////////////////////////////////////////////////////
//Handles block rotation
//////////////////////////////////////////////////////////	
    public void onBlockAdded(World worldIn, BlockPos pos, IBlockState state)
    {super.onBlockAdded(worldIn, pos, state);}
    
    
	  public IBlockState withRotation(IBlockState state, Rotation rot)
	    {
	        return state.withProperty(BlockHorizontal.FACING, rot.rotate((EnumFacing)state.getValue(BlockHorizontal.FACING)));
	    }
	
	    public IBlockState getStateForPlacement(World worldIn, BlockPos pos, EnumFacing facing, float hitX, float hitY, float hitZ, int meta, EntityLivingBase placer)
	    {
	        return this.getDefaultState().withProperty(BlockHorizontal.FACING, placer.getHorizontalFacing().getOpposite());	
	        strFacing = this.getDefaultState().withProperty(BlockHorizontal.FACING, placer.getHorizontalFacing().getOpposite());	
	    }
	  
	    public IBlockState getStateFromMeta(int meta)
	    {
	        return this.getDefaultState().withProperty(BlockHorizontal.FACING, EnumFacing.getHorizontal(meta));
	    }
	    
	    public int getMetaFromState(IBlockState state)
	    {
	        return ((EnumFacing)state.getValue(BlockHorizontal.FACING)).getHorizontalIndex();
	    }

	    protected BlockStateContainer createBlockState()
	    {
	        return new BlockStateContainer(this, new IProperty[] {BlockHorizontal.FACING});
	    }

//////////////////////////////////////////////////////////
//Sets hitbox
//////////////////////////////////////////////////////////	  
	   
	    @Override
	    public AxisAlignedBB getBoundingBox(IBlockState state, IBlockAccess source, BlockPos pos) {
	    	
	    	if(strFacing == "East" || strFacing ==  "West") {
	    		northPosHolder = northPos;
	    		southPosHolder = southPos;	
	    		northPos = eastPos;
	    		southPos = westPos;
	    		eastPos = northPosHolder;
	    		westPos = southPosHolder;
	    	}
	    	
	    	
	    	return new AxisAlignedBB(westPos,topPos,northPos,eastPos,bottomPos,southPos);
	    }

}

 

 

[V0idWa1k3r]

  On 11/9/2018 at 10:42 AM, MajesticMadman said:

change the clickable hitbox:

Expand  

No. The game will only display/handle your hitbox when you are directly looking at the block in question. While block hitboxes are raytraced it doesn't apply to hitboxes bigger than 1x1.5x1. If you need blocks bigger than that you will have to fill in the space with "phantom" blocks.

 

  On 11/9/2018 at 10:42 AM, MajesticMadman said:

public String strFacing;

Expand  

Don't do stringly-typed code. Minecraft already has a class for directions, it's EnumFacing.

 

Also you can't store variables in blocks like that. Blocks are singletons, there is only ever one instance per registry entry. If you need to store additional data per block then you need to use either blockstates or TileEntities.

 

  On 11/9/2018 at 10:42 AM, MajesticMadman said:

if(strFacing == "East" || strFacing == "West")

Expand  

Don't compare strings like that, use equals. This only works because java has string pools and those can be disabled(I think)

 

  On 11/9/2018 at 10:42 AM, MajesticMadman said:

return this.getDefaultState().withProperty(BlockHorizontal.FACING, placer.getHorizontalFacing().getOpposite());

strFacing = this.getDefaultState().withProperty(BlockHorizontal.FACING, placer.getHorizontalFacing().getOpposite());

Expand  

This doesn't even compile.

 

What are you actually trying to do? have the bounding box be dependent on the block's rotation? You can get the IBlockState passed to you in the Block#getBoundingBox and compare against the value of the rotation property of that blockstate.

[Cadiboo]

STOP! If your still using the block that you got from McCreator just stop and learn java first.

 

Edit: Sorry, you have the same Avatar & a similar name to someone with a different problem that would be solved by learning java

Edited November 10, 2018 by Cadiboo

[MajesticMadman]

  On 11/9/2018 at 11:23 AM, Cadiboo said:

STOP! If your still using the block that you got from McCreator just stop and learn java first.

Expand  

Didn't get the block from McCreator, didn't even know that was a thing till I looked it up after you mentioned it.

Models JSONs were made using blockbench.
Blockstates were taken from minecraft assets (Though was told forges way was better, but went back to vanilla for rotation).

Rotation was taken from minecrafts pumpkin.

The original bounding box was got on a quick search and then modified so I could easily apply to all custom modeled blocks rather than creating a separate one for each model size.

[Animefan8888]

  On 11/9/2018 at 10:42 AM, MajesticMadman said:

Second Question: What is the best way of rotating the bounding box when placed?

Expand  

Store multiple bounding boxes for each rotation. Rotating them is just a pain and not worth it.

 

  On 11/9/2018 at 10:42 AM, MajesticMadman said:

First question: I have a bounding box of 3x1 (blocks) . As far as entities go works fine

Expand  

Are you sure about this if I'm not mistaken Minecraft is really only supposed to be 1x1.5x1 bounding boxes, and the fence is more or less hacked in.