I am making a mod that adds Terraria into Minecraft. To start, I registered all 3991 items into the game. However, when I try and run the game, the build fails with this error: 

C:Myfilesandstuff\ModEventSubscriber.java:13: error: code too large
    public static void onRegisterItems(RegistryEvent.Register<Item> event) {

Any idea for a workaround?

Post logs please, and a link to code, preferably a github repo that can be cloned/build/debugged

6 minutes ago, diesieben07 said:

A method in Java can only be 64KB of bytecode.

Your methods are too big.

WTF are you doing? Did you literally write out 3991 items by hand?! Instead of using some kind of loop or something?

I used Google Sheets to pull the item names from the wiki, replace the spaces with underscores, and convert all letters to lowercase. Then I used Sheets to add the register item thing around each one and pasted it into IntelliJ

Edited April 6, 20205 yr by N/A

2 minutes ago, Ugdhar said:

Post logs please, and a link to code, preferably a github repo that can be cloned/build/debugged

Ok, the github repo is here: https://github.com/SimplyCmd/Terracraft

Also, the only log is right here:

> Task :compileJava
C:\Blablabla\src\main\java\io\github\simplycmd\terracraft\ModEventSubscriber.java:13: error: code too large
    public static void onRegisterItems(RegistryEvent.Register<Item> event) { // Tells Forge that you want this method to be called when it is time for your mod to register its items.
                       ^
1 error

> Task :compileJava FAILED

 

2 minutes ago, diesieben07 said:

yeah... dont do that.

Hmmm... Is there a way to break it up into multiple files or methods?

Edited April 6, 20205 yr by N/A

https://en.wikipedia.org/wiki/Don't_repeat_yourself

Edited April 6, 20205 yr by N/A

Also, your names aren't all lower case.

setup(new Item(new Item.Properties()),"IronPickaxe"),

Also, do you really need to add a new iron pickaxe item to Minecraft? We already have one.

And mushrooms, torches, dirt blocks, various swords, axes, ores, arrows, chests...

 

Edit, I think these items are my favorites:

https://github.com/SimplyCmd/Terracraft/blob/master/main/java/io/github/simplycmd/terracraft/ModEventSubscriber.java#L3989-L4000

Edited April 6, 20205 yr by Draco18s

3 minutes ago, Draco18s said:

Also, your names aren't all lower case.

setup(new Item(new Item.Properties()),"IronPickaxe"),

Also, do you really need to add a new iron pickaxe item to Minecraft? We already have one.

And mushrooms, torches, dirt blocks, various swords, axes, ores, arrows, chests...

 

Edit, I think these items are my favorites:

https://github.com/SimplyCmd/Terracraft/blob/master/main/java/io/github/simplycmd/terracraft/ModEventSubscriber.java#L3989-L4000

Sorry, that was outdated from last night. I pushed the most recent commit. Just refresh the page

20 minutes ago, N/A said:

https://en.wikipedia.org/wiki/Don't_repeat_yourself

Yes. Sadly you are repeating yourself an awful lot.

 

Imagine

for (type in tool_types) // COPPER/TIN, IRON/LEAD, etc

    setup(new Item(new Item.Properties()), type + "_pickaxe");

    setup(new Item(new Item.Properties()), type + "_axe");

    // ETC

2 minutes ago, Animefan8888 said:

Yes. Sadly you are repeating yourself an awful lot.

 

Imagine

for (type in tool_types) // COPPER/TIN, IRON/LEAD, etc

    setup(new Item(new Item.Properties()), type + "_pickaxe");

    setup(new Item(new Item.Properties()), type + "_axe");

    // ETC

Good idea. If I did that, would it fix the error?

36 minutes ago, N/A said:

Good idea. If I did that, would it fix the error?

If you did it for all types of the items. Like the furniture and the wood, etc.

Ok, thanks for the help!