Jump to content
Search In
  • More options...
Find results that contain...
Find results in...

GameRegistry.register problem(beginner)


Dr_Blockenstein
 Share

Recommended Posts

I have an error at an area of my code that I can't figure out at all. I've been looking at this guide (http://mcforge.readthedocs.io/en/latest/concepts/registries/) and read it a few times over but can't wrap my head over how to fix this code problem in ModItems.java

Code:https://pastebin.com/YRDDxP4v

When hovering over text: The method register(K) from the type GameRegistry is not visible

 

 

MC version modding to:1.12

Edited by Dr_Blockenstein
Need to show I'm new to modding.
Link to comment
Share on other sites

Don't use the ModelMesher, use the ModelLoader in PreInit instead. 

Also stop with the substring nonsense. Unlocalized names have nothing to do with resources. You should be using getRegistryName instead. 

Edited by Draco18s

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

1 hour ago, loordgek said:

@Draco18s it is best to use ModelRegistryEvent for it because in 1.12 the RegistryEvent is now fired after preinit

Yes, but using the ModelMesher is still the worst option.

  • Like 1

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

30 minutes ago, Kitsune_Ultima said:

Have the same problem how do I go about fixing this?

In code preferably

 

Did you read the thread?

 

You need to use registry events.

Please don't PM me to ask for help. Asking your question in a public thread preserves it for people who are having the same problem in the future.

Link to comment
Share on other sites

Create a new class.

Annotate this new class with @EventBusSubscriber

Create two public static void methods with Register<Block> and Register<Item> parameters respectively.

Either manually call event.registry::registerAll(), adding all blocks inside of the method call, or if you have a collection of blocks/items, for that collection call forEach(event.registry::register)

 

Also previously known as eAndPi.

"Pi, is there a station coming up where we can board your train of thought?" -Kronnn

Published Mods: Underworld

Handy links: Vic_'s Forge events Own WIP Tutorials.

Link to comment
Share on other sites

Package is irrelevant and only for your own convenience.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

On 7/1/2017 at 9:37 AM, Matryoshika said:

Create a new class.

Annotate this new class with @EventBusSubscriber

Create two public static void methods with Register<Block> and Register<Item> parameters respectively.

Either manually call event.registry::registerAll(), adding all blocks inside of the method call, or if you have a collection of blocks/items, for that collection call forEach(event.registry::register)

 

Like This? vvvvvvvvvvvvvvvv

Screenshot_15.thumb.png.4f45f04574637edf59c2edcab5fc1ae9.png

Link to comment
Share on other sites

  1. You need to put @EventBusSubscriber above the public class Register
  2. The methods (registerBlocks & registerItems) need to be static
  3. You still need to have reference points for your blocks and items
    1. Create a static field in the Register class called OBSIDIAN_INGOT or similar.
    2. Inside the registerAll method, make OBSIDIAN_INGOT = new ItemObsidianIngot(). This way you can use your items in code by calling Register.OBSIDIAN_INGOT.

The @EventBusSubscriber annotation makes sure that Forge knows that this class contains methods that rely on events. You have to use the annotation because these events are fired before preInit.

 

Edited by Matryoshika

Also previously known as eAndPi.

"Pi, is there a station coming up where we can board your train of thought?" -Kronnn

Published Mods: Underworld

Handy links: Vic_'s Forge events Own WIP Tutorials.

Link to comment
Share on other sites

1 hour ago, Matryoshika said:
  1. Create a static field in the Register class called OBSIDIAN_INGOT or similar.

Or somewhere, at least.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

1 hour ago, Matryoshika said:
  1. You need to put @EventBusSubscriber above the public class Register
  2. The methods (registerBlocks & registerItems) need to be static
  3. You still need to have reference points for your blocks and items
    1. Create a static field in the Register class called OBSIDIAN_INGOT or similar.
    2. Inside the registerAll method, make OBSIDIAN_INGOT = new ItemObsidianIngot(). This way you can use your items in code by calling Register.OBSIDIAN_INGOT.

The @EventBusSubscriber annotation makes sure that Forge knows that this class contains methods that rely on events. You have to use the annotation because these events are fired before preInit.

 

 

33 minutes ago, Draco18s said:

Or somewhere, at least.

Thank you both, this makes so much more sense then what the docs say.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share



×
×
  • Create New...

Important Information

By using this site, you agree to our Privacy Policy.