[1.11.2] [SOLVED] TileEntity null while it shouldn't

[Bektor]

Hi,

 

I've got the following problem:

 

	protected boolean canCableConnect(IBlockAccess worldIn, BlockPos pos, EnumFacing facing) {
	    // 188 72 265
	    BlockPos posi = pos.offset(facing); // correct
	    TileEntity tile = worldIn.getTileEntity(posi); // correct
	    if(tile != null) // why null????
	        return EnergyUtils.hasCapability(tile, facing.getOpposite());
	    
	    return false;
	}

(debugging got me to the point that I can write the comments behind the line of code)

 

The method is called in 

public IBlockState getActualState(IBlockState state, IBlockAccess worldIn, BlockPos pos) {

.

 

My problem here is that the tile is null while it shouldn't be null. I checked the positions and the positions match perfectly the one of my transfer pipe.

 

public class BlockTransfer extends Block {
    @Override
    public boolean hasTileEntity() {
        return true;
    }
    
    @Override
    public TileEntity createTileEntity(World world, IBlockState state) {
        return new TileEntityPipeTransferEnergy();
    }
}

 

Why is my method's tile null when my BlockTransfer has a tile entity?

 

Note: the canCableConnect method checks to which blocks the cable can connect to in order to display the correct block state (connected to south, not connected, etc.).

 

Thx in advance.

Bektor

 

EDIT:

public class BlockTransfer extends Block {
    @Override
    public boolean hasTileEntity(IBlockState state) {
        return true;
    }
    
    @Override
    public TileEntity createTileEntity(World world, IBlockState state) {
        return new TileEntityPipeTransferEnergy();
    }
}

 

Edited October 16, 2017 by Bektor
update code

[Bektor]

2 minutes ago, diesieben07 said:

Please verify using World::getBlockState that the block actually exists at the position.

It exists. 

	    IBlockState state = worldIn.getBlockState(posi);
	    
	    TileEntity tile = worldIn.getTileEntity(posi);
	    if(tile != null)
	        return EnergyUtils.hasCapability(tile, facing.getOpposite());

Output using debugger:
state	BlockStateContainer$StateImplementation  (id=15672)	   justanotherenergy:transfer

tile    null

 

[Bektor]

1 minute ago, diesieben07 said:

The version of hasTileEntity you are using is deprecated. Please don't just ignore deprecation warnings.

Oh... hm.. my IDE didn't told me that... With adding super.hasTileEntity it tells me that, thought.

Which one should I use then?

[Bektor]

48 minutes ago, diesieben07 said:

The one that is not deprecated...

Changed it. Doesn't solve the problem, thought.

[Bektor]

Just now, diesieben07 said:

Post updated code, ideally the whole block class.

public class BlockTransfer extends Block {
    
    public BlockTransfer() {
        super(Material.ROCK);
        this.setCreativeTab(ModCreativeTabs.mcpowerTab);
    }
    
    @Override
    public void onNeighborChange(IBlockAccess world, BlockPos pos, BlockPos neighbor) {
    	super.onNeighborChange(world, pos, neighbor);
    	
    	// World implements IBlockAccess
    	if(((World)world).isRemote)
    	    return;
    	
    	// add direct connections to the tile entity
    	TileEntity tile = world.getTileEntity(pos);
    	if(tile instanceof TileEntityPipeTransferEnergy) {
    		TileEntityPipeTransferEnergy pipe = (TileEntityPipeTransferEnergy) tile;
    		
    		if(world.getTileEntity(neighbor) != null) {
    			// check for all directions
    			TileEntity neighbor_te = world.getTileEntity(neighbor);
    			if(neighbor_te instanceof TileEntityEnergy)
    			    pipe.shouldRecalculate = true;
    			else {
    			    // check the facing of the block to make sure which side is connected to us
    			    for(EnumFacing face : EnumFacing.VALUES) {
    			        BlockPos offset = pos.offset(face);
    			        if(offset == neighbor) {
    			            if(EnergyUtils.hasCapability(neighbor_te, face))
    			                pipe.shouldRecalculate = true;
    			        }
    			    }
    			}
    		} else {
    			// remove dead objects from the list
    			pipe.shouldRecalculate = true;
    		}
    	}
    }
    
    @Override
    public boolean hasTileEntity(IBlockState state) {
        return true;
    }
    
    @Override
    public TileEntity createTileEntity(World world, IBlockState state) {
        return new TileEntityPipeTransferEnergy();
    }
}

public class BlockCable extends BlockConnectorBase {
    
	[...]
	
	public BlockCable() {
		super(Material.IRON);
		this.setHardness(.25f);
		this.setResistance(3f);
		
		this.setCreativeTab(ModCreativeTabs.mcpowerTab);
	}
	
	@Override
	protected boolean canCableConnect(IBlockAccess worldIn, BlockPos pos, EnumFacing facing) {
	    // 188 72 265
	    BlockPos posi = pos.offset(facing);
	    
	    IBlockState state = worldIn.getBlockState(posi);
	    
	    TileEntity tile = worldIn.getTileEntity(posi);
	    if(tile != null)
	        return EnergyUtils.hasCapability(tile, facing.getOpposite());
	    
	    return false;
	}
	
	@Override
	public void onBlockAdded(World worldIn, BlockPos pos, IBlockState state) {
	    super.onBlockAdded(worldIn, pos, state);
	    
	    TileEntity tile = worldIn.getTileEntity(pos);
	    if(tile != null && tile instanceof TileEntityPipeEnergy) {
	        TileEntityPipeEnergy cable = (TileEntityPipeEnergy) tile;
	        cable.searchNetwork();
	    }
	}
	
	@Override
	public void breakBlock(World worldIn, BlockPos pos, IBlockState state) {
       TileEntity tile = worldIn.getTileEntity(pos);
        if(tile != null && tile instanceof TileEntityPipeEnergy) {
            TileEntityPipeEnergy cable = (TileEntityPipeEnergy) tile;
            cable.searchNetwork();
        }
        
        super.breakBlock(worldIn, pos, state);
	}
	
	@Override
    public IBlockState getActualState(IBlockState state, IBlockAccess worldIn, BlockPos pos) {
        for(final EnumFacing facing : EnumFacing.VALUES)
            state = state.withProperty(CONNECTED.get(facing.getIndex()), this.canCableConnect(worldIn, pos, facing));
        
        return state;
    }
	
	[...] (Collision box related stuff)
	
	    @Override
    protected BlockStateContainer createBlockState() {
        return new BlockStateContainer(this, CONNECTED.toArray(new IProperty[CONNECTED.size()]));
    }
    
    @Override
    public int getMetaFromState(IBlockState state) {
        return 0;
    }
    
    @Override
    public IBlockState getStateFromMeta(int meta) {
        return this.getDefaultState();
    }
	
    @Override
    public boolean hasTileEntity(IBlockState state) {
        return true;
    }
	
	@Override
	public TileEntity createTileEntity(World world, IBlockState state) {
	    return new TileEntityPipeEnergy();
	}
}

 

[Bektor]

1 minute ago, diesieben07 said:

Did you create a new world?

Yep.

[Bektor]

On 10/15/2017 at 2:11 AM, diesieben07 said:

I am not sure then...

Ok, don't ask what I changed as I got totally no clue what exaclty I changed, but it works now.

[Rikka0_0]

I also have this kind of problem. I do null check in my getActualBlock state. I think possible reason can be, sometimes the IBlockState passed into getActualBlock() can be implemented by a ChunkCache, and the TileEntity doesn't exist in it yet.