So, I just fixed a problem related to texture path and override errors. I added a second item using the same fixed methods as before, but now they show up with the same name, texture, and in the same tab. I'm 99.99% sure that they aren't somehow linked in the code. Any help?

 

Base Mod Code

 

 

package PrimalCraft;

 

import net.minecraft.block.Block;

import net.minecraft.item.Item;

import net.minecraft.item.ItemStack;

import cpw.mods.fml.common.Mod;

import cpw.mods.fml.common.Mod.Init;

import cpw.mods.fml.common.Mod.Instance;

import cpw.mods.fml.common.Mod.PostInit;

import cpw.mods.fml.common.Mod.PreInit;

import cpw.mods.fml.common.SidedProxy;

import cpw.mods.fml.common.event.FMLInitializationEvent;

import cpw.mods.fml.common.event.FMLPostInitializationEvent;

import cpw.mods.fml.common.event.FMLPreInitializationEvent;

import cpw.mods.fml.common.network.NetworkMod;

import cpw.mods.fml.common.registry.LanguageRegistry;

 

@Mod(modid="PrimalCraft", name="PrimalCraft", version="0.0.1")

@NetworkMod(clientSideRequired=true, serverSideRequired=false)

public class PrimalCraftBase {

 

        // The instance of your mod that Forge uses.

        @Instance("Generic")

        public static PrimalCraftBase instance;

        private final static Item ItemKnappingStone = new ItemKnappingStone(7500);

        private final static Item ItemPickaxeheadStone = new ItemKnappingStone(7501);

       

        // Says where the client and server 'proxy' code is loaded.

        @SidedProxy(clientSide="PrimalCraftClient.ClientProxy", serverSide="PrimalCraft.CommonProxy")

        public static CommonProxy proxy;

       

        @PreInit

        public void preInit(FMLPreInitializationEvent event) {

                // Stub Method

        }

       

        @Init

        public void load(FMLInitializationEvent event) {

                LanguageRegistry.addName(ItemKnappingStone, "Knapping Stone");

                LanguageRegistry.addName(ItemPickaxeheadStone, "Stone Pickaxe Head");

                proxy.registerRenderers();

        }

       

       

       

        @PostInit

        public void postInit(FMLPostInitializationEvent event) {

                // Stub Method

        }

       

}

 

 

 

Item Code 1

 

 

package PrimalCraft;

 

import cpw.mods.fml.relauncher.Side;

import cpw.mods.fml.relauncher.SideOnly;

import net.minecraft.client.renderer.texture.IconRegister;

import net.minecraft.creativetab.CreativeTabs;

import net.minecraft.item.Item;

import net.minecraft.util.Icon;

 

public class ItemKnappingStone extends Item

{

 

public ItemKnappingStone(int id)

{

 

super(id);

 

setMaxStackSize(1);

setCreativeTab(CreativeTabs.tabTools);

setUnlocalizedName("knappingStone");

 

}

 

@SideOnly(Side.CLIENT)

@Override

public void registerIcons(IconRegister iconRegister)

{

    itemIcon = iconRegister.registerIcon("PrimalCraft:knappingstone");

}

 

}

 

 

 

 

Item Code 2

 

 

package PrimalCraft;

 

import cpw.mods.fml.relauncher.Side;

import cpw.mods.fml.relauncher.SideOnly;

import net.minecraft.client.renderer.texture.IconRegister;

import net.minecraft.creativetab.CreativeTabs;

import net.minecraft.item.Item;

import net.minecraft.util.Icon;

 

public class ItemPickaxeheadStone extends Item

{

 

public ItemPickaxeheadStone(int id)

{

 

super(id);

 

setMaxStackSize(1);

setCreativeTab(CreativeTabs.tabMaterials);

setUnlocalizedName("pickaxeheadStone");

 

}

 

@SideOnly(Side.CLIENT)

@Override

public void registerIcons(IconRegister iconRegister)

{

    itemIcon = iconRegister.registerIcon("PrimalCraft:pickaxeheadstone");

}

 

}

 

 

You might also want to register the items with the GameRegistry;

 

GameRegistry.registerItem(ItemKnappingStone, "Knapping Stone");

Would that be in the @Init block? And thanks for resolving both of my questions!

Yes, it goes under @Init.

So I put that under the LanguageRegistry. It still won't work. 

You made 2 different knapping stones:

        private final static Item ItemKnappingStone = new ItemKnappingStone(7500);
        private final static Item ItemPickaxeheadStone = new ItemKnappingStone(7501);

In case you aren't kicking yourself right now because you can't believe you missed it, the proper code would be:

        private final static Item ItemKnappingStone = new ItemKnappingStone(7500);
        private final static Item ItemPickaxeheadStone = new ItemPickaxeheadStone(7501);

Enjoy and good luck. 

 

-edit

Also, you may want to (from a coding perspective) start a habit of giving your variables names that start with a lowercase letter:

private final static Item itemKnappingStone = new ItemKnappingStone(7500);

instead of

private final static Item ItemKnappingStone = new ItemKnappingStone(7500);

It is small, but there may come a time where you are confused if you are referring to a variable or a class and the distinction will help. Variables start with lowercase and classes start with uppercase. Just being ocd. Take it or leave it.

Thank you! I knew it was a simple thing like that! I guess I just passed over it a lot...

You made 2 different knapping stones:

 

Missed that when I glanced at his code.

 

I've done that before.

You made 2 different knapping stones:

 

Missed that when I glanced at his code.

 

I've done that before.

 

Everyone has

I didn't see this when I glanced at the code just now via my phone.. I blame the tiny screen <3

Getting code-blind on your own code is quite common, that's the reason why I haven't solved my problems myself like 80% of the times I have asked for help here ^^

 

 

Getting code-blind on your own code is quite common, that's the reason why I haven't solved my problems myself like 80% of the times I have asked for help here ^^

 

Ever solved a problem through the process of writing out the post requesting help?

 

I once solved an issue (not a technical one, but what a recipe should be) in a dream.  Well, less dream and more "how would I explain this to my mother?" exploration as I fell asleep.