Blocks/Items Not Appearing In-Game

[bauerbach]

I created some blocks and items, but they don't appear in-game. How do I make them appear in-game?

 

My code is here: scpmod.zip

[Draco18s]

No one is going to download your zip file and explore it.

Post your code as a working github repository.

[DaemonUmbra]

And please make sure your git root is in the same directory as the .gitignore that Forge's MDK provides, this will make sure that the repository respects and ignores the proper files

Edited June 15, 2018 by DaemonUmbra

[DoctorLOGiQ]

It depends what Forge version you're using, and how you've done it.

For creating items in 1.12, this is my method (not saying it's the best):

 

1) Create a registration class annotated with:

@Mod.EventBusSubscriber

 

2) Create an object holder annotation and constant for each item:

@GameRegistry.ObjectHolder(MOD_ID + ":your_item_name")
public static final YourItemClass YOUR_ITEM = new YourItemClass();

 

3) Create an array of items and store each item in it. I know this isn't ideal but it works:

private static final Item[] ITEMS =
  {
    YOUR_ITEM,
    ANOTHER_ITEM,
    ETC
  };

 

4) Subscribe to the item registration event and register your items:

@SubscribeEvent
protected static void registerItems(RegistryEvent.Register<Item> event)
{
   final IForgeRegistry<Item> registry = event.getRegistry();
   registry.registerAll(ITEMS);
}

 

3) Subscribe to the model registration event and register your item models:

@SubscribeEvent
protected static void registerItemRenderers(ModelRegistryEvent event)
{
   for (Item item : ITEMS)
   {
      ModelLoader.setCustomModelResourceLocation(item, 0, new ModelResourceLocation(item.getRegistryName(), "inventory"));
   }
}

 

And that's it!

For blocks it's a similar process, except you also need to create and register an ItemBlock for each block. Your blocks class should subscribe to the RegistryEvent.Register<Item> event to do this. Make sure for blocks you change the <Item> to <Block>.

 

I hope you find this helpful.

[Draco18s]

But he didn't use a stupid IHasModel interface! ?

Edited June 16, 2018 by Draco18s

[DoctorLOGiQ]

  On 6/16/2018 at 1:03 PM, diesieben07 said:

@DoctorLOGiQ You have several problems there. Most importantly you are creating item instances in static initializers. This is a very bad idea.

Expand  

Furthermore I have the inability to count to 5, apparently

But in all seriousness, why is that a bad idea? Because it works very well.

[Draco18s]

Because you can't control the order of static initializers.

If you created a new class called ModTabs and created CreativeTabs in a static initializer and those constructors reference an item in your ModItems class (what item it uses for an icon), and an item references ModTabs in its constructor (what tab it appears in)....

 

Which one gets created first and has a null value?

 

i.e. do you have a tab with a null icon?

or do you have an item with a null creative tab?

Edited June 17, 2018 by Draco18s