TEs, ItemStackHandler, and dropping all items

[Draco18s 2425]

Feels like a bit of an oversight that the ItemStackHandler class doesn't have a "how many slots do you have" method, especially considering that it throws a runtime exception if you try to access a slot outside its size.

 

On top of that, there doesn't appear to be a suggested way for a block to tell its own TE to drop all of its contents without performing a cast from TileEntity to something else (either its own TE type or custom interface) as there's no ability to be certain (using generic code) to determine that "yes, I've accessed every ItemStackHandler capability the tile can provide."

 

I mean, I'm fine creating my own helper interface that I apply to my TEs so I can just do this:

	public void breakBlock(World worldIn, BlockPos pos, IBlockState state) {
        TileEntity tileentity = worldIn.getTileEntity(pos);

        if (tileentity instanceof IInvenCapHelper) {
            ((IInvenCapHelper)tileentity).dropAllItems(worldIn, pos);
        }

        super.breakBlock(worldIn, pos, state);
    }

And then manually looping through all the slots my TE knows it has for all of its ItemStackHandler instances:

	@Override
public void dropAllItems(World worldIn, BlockPos pos) {
	ItemStack stack;
	//magic number `n` because ItemStackHandler does not have a .getSize()
	for(int i=0; i < n; i++) {
		stack = inputSlots.getStackInSlot(i);
		EntityItem entityIn;
		if(stack != null) {
			entityIn = new EntityItem(worldIn, pos.getX(), pos.getY(), pos.getZ(), stack);
			entityIn.setDefaultPickupDelay();
			worldIn.spawnEntityInWorld(entityIn);
		}
	}
	//repeat for outputSlots and any other ItemStackHandler instances
}

 

Or am I missing something?

[diesieben07 7775]

Feels like a bit of an oversight that the ItemStackHandler class doesn't have a "how many slots do you have" method, especially considering that it throws a runtime exception if you try to access a slot outside its size.

It does,

getSlots

.

On top of that, there doesn't appear to be a suggested way for a block to tell its own TE to drop all of its contents without performing a cast from TileEntity to something else (either its own TE type or custom interface) as there's no ability to be certain (using generic code) to determine that "yes, I've accessed every ItemStackHandler capability the tile can provide."

Usually

getCapability(CapabilityItemHandler.ITEM_HANDLER_CAPABILITY, null)

will give you a "full access" handler.

Or am I missing something?

Yes

[Draco18s 2425]

Feels like a bit of an oversight that the ItemStackHandler class doesn't have a "how many slots do you have" method, especially considering that it throws a runtime exception if you try to access a slot outside its size.

It does,

getSlots

.

 

....How in the hell did I overlook that.

 

Usually

getCapability(CapabilityItemHandler.ITEM_HANDLER_CAPABILITY, null)

will give you a "full access" handler.

 

I think that entirely depends on how the block handles its inventory.

 

For example, if I have a block like the furnace which I want to have sided access I need to return a different capability depending on the side (or the possibility of inputting to the output slot exists).

 

Ergo:

	inputSlot = new ItemStackHandler(1);
outputSlot = new ItemStackHandler(1);

 

How would I then go about returning both in the case of a null side?

[diesieben07 7775]

How would I then go about returning both in the case of a null side?

CombinedInvWrapper

can combine multiple

IItemHandler

s into one.

[Draco18s 2425]

Crafty.