Just wondering, is there a less intensive way to generate BlockItems from Blocks than manually declaring the whole block list in item form?

 

In earlier versions you declared your Block, and when you registered it automatically created the associated ItemBlock. Now you have to declare all the Blocks and BlockItems separately, maintaining two lists of the same thing. It seems like the sort of thing that could be automated away with a convenience method.

Personally I don't see what's so agonizing about writing an extra line of code per block

Ah. I wouldn't claim it's agonising, but it becomes a bit tedious if you have quite a long list of blocks. And we programmers like to avoid code reuse wherever possible, don't we?

 

My first thought was a method like this:

	private static RegistryObject<Item> registerBlockItem(RegistryObject<Block> block, ItemGroup group)
	{
		return ITEMS.register(block.get().getRegistryName().getPath(), () -> new BlockItem(block.get(), new Item.Properties().group(group)));
	}

but of course it failed because the RegistryObject's value doesn't exist at that point.

If you declare your blocks in a DefferedRegister than you can do something like this:

Quote

 

public class ModBlocks {
    public static final DeferredRegister<Block> BLOCKS = new DeferredRegister<>(ForgeRegistries.BLOCKS, Main.MODID);
	
    (...)
    public static final RegistryObject<Block> EXAMPLE_BLOCK = BLOCKS.register("example_block", () -> new Block(<properties, etc>));
    (...)
}

 

 

Quote

    @SubscribeEvent
    public static void onItemsRegistry(final RegistryEvent.Register<Item> itemRegistryEvent) {
        final IForgeRegistry<Item> registry = itemRegistryEvent.getRegistry();

        ModBlocks.BLOCKS.getEntries().stream()
                .map(RegistryObject::get)
                .forEach(block -> {
                    final Item.Properties properties = new Item.Properties().group(ModItemGroups.MOD_ITEM_GROUP);
                    final BlockItem blockItem;
                    blockItem = new BlockItem(block, properties);
                    blockItem.setRegistryName(Objects.requireNonNull(block.getRegistryName()));
                    registry.register(blockItem);
                });
    }