[1.7.2] Need help with item code [SOLVED]

[Budboy33]

I've started working on my mod, and am a bit new to forge, I want to know how I would reference an item class that creates the item and registers it, so that it's orderly and I can register all the items at once during preInit, my current code doesn't seem to be working.

 

PreInit code:

 

 

{

 

@EventHandler

public void preInit(FMLPreInitializationEvent e)

{

 

        ModItems.init();

}

 

@EventHandler

public void init(FMLInitializationEvent e)

{

}

}

 

 

ModItems code:

 

 

public class ModItems {

 

public static Item portableWorkbench;

 

{

portableWorkbench = new Item().setUnlocalizedName("portableWorkbench").setCreativeTab(CreativeTabs.tabMisc);

GameRegistry.registerItem(portableWorkbench, "portableWorkbench");

}

public static void init()

{

}

}

 

 

I've just posted the important code, not much else to see besides imports and basic class stuff.

[larsgerrits]

public class ModItems {

   public static Item portableWorkbench;
   
   {
    portableWorkbench = new Item().setUnlocalizedName("portableWorkbench").setCreativeTab(CreativeTabs.tabMisc);
   GameRegistry.registerItem(portableWorkbench, "portableWorkbench"); 
   }
   public static void init() 
   {
   }
}

You have a empty init method?

[Budboy33]

Yeah, is there anything that's supposed to go there that helps initialize the item code?

[TheGreyGhost]

Hi

 

I'd suggest you should use preInit, Init etc to explicitly control when your variables are initialised and items registered etc.  Static initialisers can bite you in the butt if they don't run when you expect and your registrations etc occur at the wrong time.

 

A bit more info here

http://greyminecraftcoder.blogspot.com/2013/11/how-forge-starts-up-your-code.html

It's for 1.6.4 but the concepts are the same for 1.7.2

 

-TGG

[Budboy33]

Thank you! I put all the item code in ModItems and extended the CommonProxy class to include ModItems, and it seems to be working now.

 

ModItems:

public static Item portableWorkbench;

 

and put the item registry code that occurs during preInit in the CommonProxy class

 

public void preInit() {

 

portableWorkbench = new Item().setUnlocalizedName("portableWorkbench").setCreativeTab(CreativeTabs.tabMisc);

GameRegistry.registerItem(portableWorkbench, "portableWorkbench");

 

Not as neat as I had hoped but it should work until I get more of my code done.