[1.7.10] Register Items with 'for' loop

[jackmano]

I need to register a group of item with a for loop. I just need it for registering of some items that are multicolored like spawn eggs, but there's a bunch in an array. I need a way to register the entire array of items.

Item array

 

public static Item[] seedrods = {

//new ItemSeedRod(hex, "name", isItPlural)

new ItemSeedRod(0xFFF32D, "Blaze Rod", false),

new ItemSeedRod(0x2B3DA, "Diamond", false),

new ItemSeedRod(0x496BC9, "Lapis Lazuli", true),

new ItemSeedRod(0x40DE58, "Emerald", false),

new ItemSeedRod(0xFA0714, "Redstone", true),

new ItemSeedRod(0xEEEEEE, "Blaze Rod", false),

};

 

[jeffryfisher]

If you hold off the array assignment until inside your preInit method, then you can have the seedRod constructor do its own registration for each instance. It's the style I use for my items and blocks, but it's a matter of taste (other people love initializers). YMMV.

 

[jackmano]

And how might i do that?

[shadowfacts]

...

You add the GameRegistry.register(Block|Item) call to the item/block's constructor.

If you don't know what a constructor is or how to invoke a method, you need to learn Java.

[EverythingGames]

For my mod Armed, I use a Map to cache a string key and an item type value - string being both the key and unlocalized name for the item. I add new elements to the map in a static block (the Map is a constant) and have methods that iterate through the Map, using a lambda iterator ( Java 8 ), registering the item, setting the unlocalized name with the Map's key, and finally initializing its render. (Registering called on Common, rendering called on Client). That is just how I like to do it - it's very fast when adding new instances / items by having it automatically setup. There are many other ways you can do this, this is a more efficient way.

 

EDIT:

Your looking to do something like this with your code, considering your using an array. You should not use exactly this code, I wrote this quick just to give you an example - reason is because it gets an item from the array based on a given integer index - you're better off grabbing your item instances from an actual static instance.

private static final Item[] ITEMINSTANCES = 
	{
			new Item().setUnlocalizedName("test"),
			new Item().setUnlocalizedName("test2")
	};

public static final Item getItem(int index)
{
	return ITEMINSTANCES[index];
}

public static final void registerItemInstances()
{
	for(Item item : ITEMINSTANCES)
	{
		GameRegistry.registerItem(item, item.getUnlocalizedName().substring(5));
	}
}

[jabelar]

You can only use a loop if there is something "regular" (i.e. predictable or possible to calculate) about the sequence, or if all the information is already stored in a collection/array. But if you just had the items in the array but not the egg color then a loop won't help because it wouldn't be able to figure out the egg color.

 

In other words, you can't save much typing with a loop for spawn eggs because for every item you'll still need to type somewhere the unlocalized name and the egg color and associate that with each actual item. Whether you type that in the preinit handler, or within the item constructor won't avoid the fact you have to type it.

 

Now if the colors could be calculated -- like you could randomize them from a fixed seed so they always turned out the same -- then a loop makes sense.

[jeffryfisher]

...

You add the GameRegistry.register(Block|Item) call to the item/block's constructor.

If you don't know what a constructor is or how to invoke a method, you need to learn Java.

Indeed... Inside the ItemSeedRod constructor, after setting unlocalized name. Where the register function needs an item, pass in "this", the constructor's way of referring to it's own new instance.