How do I get a NullPointerException here?

[superminerJG]

I have attempted to add an alloy furnace, and this has happened. I have no idea why the NullPointerException is thrown, and how I can fix it.

Crash Log and GitHub repo

[V0idWa1k3r]

https://github.com/JGgithub2018/Simple-Batteries/blob/master/src/main/java/jgcodes/sb/SimpleBatteries.java#L24

A common proxy makes no sense. Proxies exist to separate sided-only code. If your code is common then it goes anywhere else but the proxy.

 

https://github.com/JGgithub2018/Simple-Batteries/blob/master/src/main/java/jgcodes/sb/content/ModBlocks.java#L15

1. Don't ever use static initializers for registry entries. They must be instantinated in the appropriate registry event.
2. This is not how ObjectHolders are used. ObjectHolders inject values into fields.

https://github.com/JGgithub2018/Simple-Batteries/blob/master/src/main/java/jgcodes/sb/proxy/CommonProxy.java#L21

Your proxy can't be an event handler since it will load the class and crash the game.

 

https://github.com/JGgithub2018/Simple-Batteries/blob/master/src/main/java/jgcodes/sb/content/blocks/BlockBase.java

Don't do XBase. There is already a BlockBase, it's called Block. Do not abuse inheritance.

 

Your issue is caused by you not setting the registry name for your blocks/items. You need to set the registry name before registering them.

[DavidM]

You probably followed some terrible youtube tutorials...

1. Do not create Base classes (ItemBase, BlockBase, etc). Read the Common issues and recommendation for more explanation.

2. Don't do this. Use normal getters and setters instead.

3. The TileEntityFurnace already has a public static function of getting the burn time of items. No need to copy paste all that here.

4. There's really no much point in overriding methods that relate to customNames (unless you really know what you are doing).

[superminerJG]

On 5/30/2019 at 9:55 PM, DavidM said:

You probably followed some terrible youtube tutorials...

1. Do not create Base classes (ItemBase, BlockBase, etc). Read the Common issues and recommendation for more explanation.

2. Don't do this. Use normal getters and setters instead.

3. The TileEntityFurnace already has a public static function of getting the burn time of items. No need to copy paste all that here.

4. There's really no much point in overriding methods that relate to customNames (unless you really know what you are doing).

Yeah. I used tutorial by Harry Talks, for the Machine stuff because I had no idea how that stuff works.

[DavidM]

HarryTalks is one of the worst modding tutorials.

I would suggest to read vanilla’s code for blocks like furnace as well as to read the code of machines from other mods on GitHub.