texture issues

majesticmadman98 · April 23, 2014

Using minecrafts Log file i've sorted out all the errors just need help with textures

@SideOnly(Side.CLIENT)
public IIcon getSideIcon(int p_150163_1_)

{

return this.field_150167_a[p_150163_1_ % this.field_150167_a.length];

}

@SideOnly(Side.CLIENT)

public IIcon getTopIcon(int p_150161_1_)

{

return this.field_150166_b[p_150161_1_ % this.field_150166_b.length];

}

so how do i alter this code to add my textures into this

Godis_apan · April 23, 2014

@SideOnly(Side.CLIENT)

@Override

protected IIcon getSideIcon(int metadata)

{

return mySideIcon;

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getTopIcon(int metadata)

{

return myTopIcon;

}

If you have more metadata wood just return: myIcon[metadata]

majesticmadman98 · April 23, 2014

so im gonna need to create strings for the side and top and make those link to the texture?

Godis_apan · April 24, 2014

Ok like this:

public static final String[] TOP = {"topIcon1Name", "topIcon2Name", "topIcon3Name", and so on..};

public static final String[] SIDE = {"sideIcon1Name", "sideIcon2Name", "sideIcon3Name", and so on..};

private IIcon topIcon[];

private IIcon sideIcon[];

public void registerBlockIcons(IIconRegister iIconRegister)

{

topIcon = new IIcon[TOP.length];

sideIcon = new IIcon[sIDE.length];

for (int i = 0; i < TOP.lenngth; i++)

{

topIcon = iIconRegister.registerIcon("modid" + ":" + TOP);

sideIcon = iIconRegister.registerIcon("modid" + ":" + SIDE)

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getSideIcon(int metadata)

{

return sideIcon[metadata];

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getTopIcon(int metadata)

{

return topIcon[metadata];

}

That should do it! Fairy straight forward code.

majesticmadman98 · April 24, 2014

Ok there is an error it comes up with following you code, which isn't just a hover over and fix,

Heres the full class

package hat.blocks.palmblocks;

import hat.HAT;

import java.util.Random;

import net.minecraft.block.Block;

import net.minecraft.block.BlockRotatedPillar;

import net.minecraft.block.material.Material;

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

import net.minecraft.creativetab.CreativeTabs;

import net.minecraft.item.Item;

import net.minecraft.util.IIcon;

import net.minecraft.world.IBlockAccess;

import net.minecraft.world.World;

import cpw.mods.fml.relauncher.Side;

import cpw.mods.fml.relauncher.SideOnly;

public abstract class palmlog extends BlockRotatedPillar

{

@SideOnly(Side.CLIENT)

protected IIcon[] field_150167_a;

@SideOnly(Side.CLIENT)

protected IIcon[] field_150166_b;

private static final String __OBFID = "CL_00000266";

public static final String[] TOP ={"topIcon1Name"};

public static final String[] SIDE ={"SIDEIcon1Name"};

private IIcon[] topIcon;

private IIcon[] sideIcon;

public palmlog()

{

super(Material.wood);

this.setHardness(2.0F);

this.setStepSound(soundTypeWood);

}

public static int func_150165_c(int p_150165_0_)

{

return p_150165_0_ & 3;

}

/**

* Returns the quantity of items to drop on block destruction.

*/

public int quantityDropped(Random p_149745_1_)

{

return 1;

}

public Item getItemDropped(int p_149650_1_, Random p_149650_2_, int p_149650_3_)

{

return Item.getItemFromBlock(this);

}

public void breakBlock(World p_149749_1_, int p_149749_2_, int p_149749_3_, int p_149749_4_, Block p_149749_5_, int p_149749_6_)

{

byte b0 = 4;

int i1 = b0 + 1;

if (p_149749_1_.checkChunksExist(p_149749_2_ - i1, p_149749_3_ - i1, p_149749_4_ - i1, p_149749_2_ + i1, p_149749_3_ + i1, p_149749_4_ + i1))

{

for (int j1 = -b0; j1 <= b0; ++j1)

{

for (int k1 = -b0; k1 <= b0; ++k1)

{

for (int l1 = -b0; l1 <= b0; ++l1)

{

Block block = p_149749_1_.getBlock(p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1);

if (block.isLeaves(p_149749_1_, p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1))

{

block.beginLeavesDecay(p_149749_1_, p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1);

}

public void registerBlockIcons(IIconRegister iIconRegister){

topIcon = new IIcon[TOP.length];

sideIcon = new IIcon[sIDE.length];

for (int i =0; i < TOP.length;i++){

topIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Top");

sideIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Side");

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getSideIcon(int metadata)

{

return sideIcon[1];

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getTopIcon(int metadata)

{

return topIcon[1];

}

@Override

public boolean canSustainLeaves(IBlockAccess world, int x, int y, int z)

{

return true;

}

@Override

public boolean isWood(IBlockAccess world, int x, int y, int z)

{

return true;

}

any text in red highlight is the part eclipse registers as errors.

public void registerBlockIcons(IIconRegister iIconRegister){
topIcon = new IIcon[TOP.length];

sideIcon = new IIcon[sIDE.length];

for (int i =0; i < TOP.length;i++){

topIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Top");

sideIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Side");

}

}

The errors here want to change this

private IIcon topIcon[];
private IIcon sideIcon[];

into

private IIcon topIcon;
private IIcon sideIcon;

which of course causes array type errors down here

@SideOnly(Side.CLIENT)
@Override

protected IIcon getSideIcon(int metadata)

{

return sideIcon[1];

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getTopIcon(int metadata)

{

return topIcon[1];

}

without resolving these errors

public void registerBlockIcons(IIconRegister iIconRegister){
topIcon = new IIcon[TOP.length];

sideIcon = new IIcon[sIDE.length];

for (int i =0; i < TOP.length;i++){

topIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Top");

sideIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Side");

}

}

Then it will want to change

private IIcon topIcon;
private IIcon sideIcon;

into

private IIcon[] topIcon;
private IIcon[] sideIcon;

and again get these errors

public void registerBlockIcons(IIconRegister iIconRegister){
topIcon = new IIcon[TOP.length];

sideIcon = new IIcon[sIDE.length];

for (int i =0; i < TOP.length;i++){

topIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Top");

sideIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Side");

}

}

and to fix will try change

private IIcon[] topIcon;
private IIcon[] sideIcon;

back to

private IIcon topIcon;
private IIcon sideIcon;

which then keeps looping back through "solutions" which dont work.

larsgerrits · April 24, 2014

In the registerIcons method, you need to change the second topIcon array to an actual array using topIcon = ***;

Godis_apan · April 24, 2014

Just do topIcon = iIconRegister and sideIcon = iIconRegister

coolboy4531 · April 25, 2014

It's IIconRegister.

Godis_apan · April 25, 2014

Yes that the class, but if you look at the parameter name you will see that it's called iIconRegister, as when you write code in java the common way to name variables is to uncap the first word.

majesticmadman98 · April 25, 2014

ok im confused with you all saying different things so Godis_apan change this to what you think it should be

package hat.blocks.palmblocks;

import hat.HAT;

import java.util.Random;

import net.minecraft.block.Block;

import net.minecraft.block.BlockRotatedPillar;

import net.minecraft.block.material.Material;

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

import net.minecraft.creativetab.CreativeTabs;

import net.minecraft.item.Item;

import net.minecraft.util.IIcon;

import net.minecraft.world.IBlockAccess;

import net.minecraft.world.World;

import cpw.mods.fml.relauncher.Side;

import cpw.mods.fml.relauncher.SideOnly;

public abstract class palmlog extends BlockRotatedPillar

{

@SideOnly(Side.CLIENT)

protected IIcon[] field_150167_a;

@SideOnly(Side.CLIENT)

protected IIcon[] field_150166_b;

private static final String __OBFID = "CL_00000266";

public static final String[] TOP ={"topIcon1Name"};

public static final String[] SIDE ={"SIDEIcon1Name"};

private IIcon topIcon[];

private IIcon sideIcon[];

public palmlog()

{

super(Material.wood);

this.setHardness(2.0F);

this.setStepSound(soundTypeWood);

}

public static int func_150165_c(int p_150165_0_)

{

return p_150165_0_ & 3;

}

/**

* Returns the quantity of items to drop on block destruction.

*/

public int quantityDropped(Random p_149745_1_)

{

return 1;

}

public Item getItemDropped(int p_149650_1_, Random p_149650_2_, int p_149650_3_)

{

return Item.getItemFromBlock(this);

}

public void breakBlock(World p_149749_1_, int p_149749_2_, int p_149749_3_, int p_149749_4_, Block p_149749_5_, int p_149749_6_)

{

byte b0 = 4;

int i1 = b0 + 1;

if (p_149749_1_.checkChunksExist(p_149749_2_ - i1, p_149749_3_ - i1, p_149749_4_ - i1, p_149749_2_ + i1, p_149749_3_ + i1, p_149749_4_ + i1))

{

for (int j1 = -b0; j1 <= b0; ++j1)

{

for (int k1 = -b0; k1 <= b0; ++k1)

{

for (int l1 = -b0; l1 <= b0; ++l1)

{

Block block = p_149749_1_.getBlock(p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1);

if (block.isLeaves(p_149749_1_, p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1))

{

block.beginLeavesDecay(p_149749_1_, p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1);

}

}

}

}

}

}

public void registerBlockIcons(IIconRegister iIconRegister){

topIcon = new IIcon[TOP.length];

sideIcon = new IIcon[sIDE.length];

for (int i =0; i < TOP.length;i++){

topIcon = IIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Top");

sideIcon = IIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Side");

}

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getSideIcon(int metadata)

{

return sideIcon[1];

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getTopIcon(int metadata)

{

return topIcon[1];

}

@Override

public boolean canSustainLeaves(IBlockAccess world, int x, int y, int z)

{

return true;

}

@Override

public boolean isWood(IBlockAccess world, int x, int y, int z)

{

return true;

}

}

Godis_apan · April 25, 2014

Yes but you can remove all this:

 @SideOnly(Side.CLIENT)
    protected IIcon[] field_150167_a;
    @SideOnly(Side.CLIENT)
    protected IIcon[] field_150166_b;
    private static final String __OBFID = "CL_00000266";

And they should return 0, like this:

    @SideOnly(Side.CLIENT)
    @Override
    protected IIcon getSideIcon(int metadata)
    {
        return sideIcon[0];
    }

    @SideOnly(Side.CLIENT)
    @Override
    protected IIcon getTopIcon(int metadata)
    {
        return topIcon[0];
    }

majesticmadman98 · April 25, 2014

ok done that but still shows errors

for (int i =0; i < TOP.length;i++){
topIcon = IIconRegister(HAT.MODID + ":Palm_Wood_Log" + "_Top");

sideIcon = IIconRegisterr(HAT.MODID + ":Palm_Wood_Log" + "_Side");

in which case eclipse needs a method "IIconRegister(string)" or as was before

for (int i =0; i < TOP.length;i++){
topIcon = IIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Top");

sideIcon = IIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Side");

which goes into that loop

Godis_apan · April 25, 2014

should be


topIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Top")

Exactly like that.

TheGreyGhost · April 26, 2014

Hi guys

I have sent you a PM which hopefully is helpful.

-TGG

Godis_apan · April 26, 2014

TGG, i know that, it's just that majesticmadman98, no offence, hasn't really got my point here. I my first or second message I explaind it all, just missed to add the little after the icons. But here's a finished code exaktly like you should have it:

public static final String[] TOP = {"Palm_Wood_Log_Top"};
public static final String[] SIDE = {"Palm_Wood_Log_Side"};

private IIcon[] topIcon;
private IIcon[] sideIcon;

public void registerBlockIcons(IIconRegister iIconRegister)
{
      topIcon = new IIcon[this.TOP.length];
      sideIcon = new IIcon[this.SIDE.length];

      for (int i = 0; i < TOP.lenngth; i++)
      {
            topIcon[i] = iIconRegister.registerIcon(HAT.MODID + ":" + this.TOP);
            sideIcon[i] = iIconRegister.registerIcon(HAT.MODID + ":" + this.SIDE)
      }
   
}

@SideOnly(Side.CLIENT)
    @Override
    protected IIcon getSideIcon(int metadata)
    {
        return sideIcon[metadata];
    }

    @SideOnly(Side.CLIENT)
    @Override
    protected IIcon getTopIcon(int metadata)
    {
        return topIcon[metadata];
    }

You shouldm't have to change anything

majesticmadman98 · April 26, 2014

no offense taken, ok the class file is done with no errors but i presume is missing something as in the main file here

palmlog = new palmlog().setBlockName("palmlog").setCreativeTab(hatblocksTab);

it says "Cannot Intitate the type palmlog" which is the part that usually links to the class which is called "palm log". Sorry for being such a pain on this tree building stuff the help is really appreciated guys, sorry if it was hard work to get this far.

Main class(things to do with the log will be in green)

package hat;

import hat.blocks.dyeblocks.dyeblock1;

import hat.blocks.dyeblocks.dyeblock10;

import hat.blocks.dyeblocks.dyeblock11;

import hat.blocks.dyeblocks.dyeblock12;

import hat.blocks.dyeblocks.dyeblock13;

import hat.blocks.dyeblocks.dyeblock14;

import hat.blocks.dyeblocks.dyeblock15;

import hat.blocks.dyeblocks.dyeblock16;

import hat.blocks.dyeblocks.dyeblock2;

import hat.blocks.dyeblocks.dyeblock3;

import hat.blocks.dyeblocks.dyeblock4;

import hat.blocks.dyeblocks.dyeblock5;

import hat.blocks.dyeblocks.dyeblock6;

import hat.blocks.dyeblocks.dyeblock7;

import hat.blocks.dyeblocks.dyeblock8;

import hat.blocks.dyeblocks.dyeblock9;

import hat.blocks.palmblocks.palmleaves;

import hat.blocks.palmblocks.palmlog;

import hat.blocks.palmblocks.palmlogcomplex;

import hat.blocks.palmblocks.palmplank;

import hat.blocks.palmblocks.palmsapling;

import hat.items.combat.mace;

import hat.items.materials.cingot;

import hat.items.materials.cmixture;

import hat.items.materials.singot;

import hat.items.materials.smixture;

import hat.items.materials.wax;

import hat.tabs.icons.blocktabicon;

import hat.tabs.icons.combattabicon;

import hat.tabs.icons.materialtabicon;

import net.minecraft.block.Block;

import net.minecraft.creativetab.CreativeTabs;

import net.minecraft.init.Blocks;

import net.minecraft.init.Items;

import net.minecraft.item.Item;

import net.minecraft.item.Item.ToolMaterial;

import net.minecraft.item.ItemStack;

import net.minecraftforge.common.util.EnumHelper;

import net.minecraftforge.oredict.OreDictionary;

import cpw.mods.fml.common.Mod;

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

import cpw.mods.fml.common.SidedProxy;

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

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

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

@Mod(modid = HAT.MODID, version = HAT.VERSION)

public class HAT

{

public static final String MODID = "hat";

public static final String VERSION = "1.0";

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

public static CommonProxy proxy;

//Tabs

//tab method = new tab();

public static CreativeTabs hatblocksTab = new CreativeTabs("hatblocksTab"){

@Override

public Item getTabIconItem() {

return Item.getItemFromBlock(palmplank);

}

};

public static CreativeTabs hatmaterialsTab = new CreativeTabs("hatmaterialsTab"){

@Override

public Item getTabIconItem() {

return materialtabicon;

}

};

public static CreativeTabs hatcombatTab = new CreativeTabs("hatcombatTab"){

@Override

public Item getTabIconItem() {

return combattabicon;

}

};

public static CreativeTabs hatmusicTab = new CreativeTabs("hatmusicTab"){

@Override

public Item getTabIconItem() {

return Items.baked_potato;

}

};

public static CreativeTabs hatlightningTab = new CreativeTabs("hatlightningTab"){

@Override

public Item getTabIconItem() {

return Items.baked_potato;

}

};

public static CreativeTabs hatmachinesTab = new CreativeTabs("hatmachinesTab"){

@Override

public Item getTabIconItem() {

return Items.baked_potato;

}

};

public static CreativeTabs hattrapsTab = new CreativeTabs("hattrapsTab"){

@Override

public Item getTabIconItem() {

return Items.baked_potato;

}

};

public static CreativeTabs hattransportTab = new CreativeTabs("hattransportTab"){

@Override

public Item getTabIconItem() {

return Items.baked_potato;

}

};

public static CreativeTabs hatmiscTab = new CreativeTabs("hatmiscTab"){

@Override

public Item getTabIconItem() {

return Items.baked_potato;

}

};

//Dye Block Defining

public static Block dyeblock1;

public static Block dyeblock2;

public static Block dyeblock3;

public static Block dyeblock4;

public static Block dyeblock5;

public static Block dyeblock6;

public static Block dyeblock7;

public static Block dyeblock8;

public static Block dyeblock9;

public static Block dyeblock10;

public static Block dyeblock11;

public static Block dyeblock12;

public static Block dyeblock13;

public static Block dyeblock14;

public static Block dyeblock15;

public static Block dyeblock16;

//Palm Defining

public static Block palmlog;

public static Block palmplank;

public static Block palmleaves;

public static Block palmsapling;

//material

public static ToolMaterial macematerial = EnumHelper.addToolMaterial("macematerial", 3, 1000, 5.0F, 6.0F, 10);

//Item Defining

public static Item wax = new wax().setUnlocalizedName("wax").setCreativeTab(hatmaterialsTab);

public static Item cingot = new cingot().setUnlocalizedName("cingot").setCreativeTab(hatmaterialsTab);

public static Item cmixture = new cmixture().setUnlocalizedName("cmixture").setCreativeTab(hatmaterialsTab);

public static Item singot = new singot().setUnlocalizedName("singot").setCreativeTab(hatmaterialsTab);

public static Item smixture = new smixture().setUnlocalizedName("smixture").setCreativeTab(hatmaterialsTab);

//weapon Defining

public static Item mace;

//icon item defining

public static Item blocktabicon = new blocktabicon().setUnlocalizedName("blocktabicon");

public static Item materialtabicon = new materialtabicon().setUnlocalizedName("materialtabicon");

public static Item combattabicon = new combattabicon().setUnlocalizedName("combattabicon");

//Dye block settings

@EventHandler

public void preInit(FMLPreInitializationEvent event){

dyeblock1 = new dyeblock1().setBlockName("dyeblock1").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock1, "dyeblock1");

dyeblock2 = new dyeblock2().setBlockName("dyeblock2").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock2, "dyeblock2");

dyeblock3 = new dyeblock3().setBlockName("dyeblock3").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock3, "dyeblock3");

dyeblock4 = new dyeblock4().setBlockName("dyeblock4").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock4, "dyeblock4");

dyeblock5 = new dyeblock5().setBlockName("dyeblock5").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock5, "dyeblock5");

dyeblock6 = new dyeblock6().setBlockName("dyeblock6").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock6, "dyeblock6");

dyeblock7 = new dyeblock7().setBlockName("dyeblock7").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock7, "dyeblock7");

dyeblock8 = new dyeblock8().setBlockName("dyeblock8").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock8, "dyeblock8");

dyeblock9 = new dyeblock9().setBlockName("dyeblock9").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock9, "dyeblock9");

dyeblock10 = new dyeblock10().setBlockName("dyeblock10").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock10, "dyeblock10");

dyeblock11 = new dyeblock11().setBlockName("dyeblock11").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock11, "dyeblock11");

dyeblock12 = new dyeblock12().setBlockName("dyeblock12").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock12, "dyeblock12");

dyeblock13 = new dyeblock13().setBlockName("dyeblock13").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock13, "dyeblock13");

dyeblock14 = new dyeblock14().setBlockName("dyeblock14").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock14, "dyeblock14");

dyeblock15 = new dyeblock15().setBlockName("dyeblock15").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock15, "dyeblock15");

dyeblock16 = new dyeblock16().setBlockName("dyeblock16").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(dyeblock16, "dyeblock16");

//Palm Settings

palmlog = new palmlog().setBlockName("palmlog").setCreativeTab(hatblocksTab);

palmplank = new palmplank().setBlockName("palmplank").setCreativeTab(hatblocksTab);

// palmleaves = new palmleaves().setBlockName("palmleaves").setCreativeTab(hatblocksTab);

//palmsapling = new palmsapling().setBlockName("palmsapling").setCreativeTab(hatblocksTab);

//weapon defining

Item mace = new mace(macematerial).setUnlocalizedName("mace").setCreativeTab(hatcombatTab);

//Food

//Dye Block Registry

//Palm Registry

GameRegistry.registerBlock(palmplank, "palmplank");

OreDictionary.registerOre("plankWood", palmplank);

GameRegistry.registerBlock(palmlog, "palmlog");

// GameRegistry.registerBlock(palmleaves, "palmleaves");

//Item Registry

GameRegistry.registerItem(wax, "wax");

GameRegistry.registerItem(cingot, "cingot");

GameRegistry.registerItem(cmixture, "cmixture");

GameRegistry.registerItem(singot, "singot");

GameRegistry.registerItem(smixture, "smixture");

//weapon registry

GameRegistry.registerItem(mace, "mace");

//Tab icon registry

GameRegistry.registerItem(blocktabicon, "blocktabicon");

GameRegistry.registerItem(materialtabicon, "materialtabicon");

GameRegistry.registerItem(combattabicon, "combattabicon");

//Dye Block Recipes

GameRegistry.addRecipe(new ItemStack(dyeblock4, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1) });

GameRegistry.addRecipe(new ItemStack(dyeblock3, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,1) });

GameRegistry.addRecipe(new ItemStack(dyeblock5, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,2) });

GameRegistry.addRecipe(new ItemStack(dyeblock6, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,3) });

GameRegistry.addRecipe(new ItemStack(dyeblock7, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,4) });

GameRegistry.addRecipe(new ItemStack(dyeblock8, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,5) });

GameRegistry.addRecipe(new ItemStack(dyeblock9, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,6) });

GameRegistry.addRecipe(new ItemStack(dyeblock2, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,7) });

GameRegistry.addRecipe(new ItemStack(dyeblock10, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1, });

GameRegistry.addRecipe(new ItemStack(dyeblock11, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,9) });

GameRegistry.addRecipe(new ItemStack(dyeblock12, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,10) });

GameRegistry.addRecipe(new ItemStack(dyeblock13, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,11) });

GameRegistry.addRecipe(new ItemStack(dyeblock14, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,12) });

GameRegistry.addRecipe(new ItemStack(dyeblock15, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,13) });

GameRegistry.addRecipe(new ItemStack(dyeblock16, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,14) });

GameRegistry.addRecipe(new ItemStack(dyeblock1, 1), new Object[] {" ","BB ","BB ", 'B', new ItemStack(Items.dye,1,15)});

//Vanilla Changed Recipes

GameRegistry.addShapelessRecipe(new ItemStack(Items.string, 4), new Object [] {Blocks.wool});

GameRegistry.addShapelessRecipe(new ItemStack(Blocks.mossy_cobblestone, 4), new Object [] {Blocks.cobblestone, Blocks.vine});

GameRegistry.addShapelessRecipe(new ItemStack(Items.name_tag, 2), new Object [] {Items.string, Items.paper});

GameRegistry.addRecipe(new ItemStack(Items.iron_horse_armor, 1), new Object[] {" I","II "," W ", 'I', new ItemStack(Items.iron_ingot,1), 'W', new ItemStack(Blocks.wool, 1)});

GameRegistry.addRecipe(new ItemStack(Items.golden_horse_armor, 1), new Object[] {" I","II "," W ", 'I', new ItemStack(Items.gold_ingot,1), 'W', new ItemStack(Blocks.wool, 1)});

GameRegistry.addRecipe(new ItemStack(Items.diamond_horse_armor, 1), new Object[] {" I","II "," W ", 'I', new ItemStack(Items.diamond,1), 'W', new ItemStack(Blocks.wool, 1)});

//material

GameRegistry.addRecipe(new ItemStack(cmixture, 1), new Object[] {" ","RGR"," ", 'R', new ItemStack(Items.redstone,1), 'G', new ItemStack(Items.gold_ingot, 1)});

GameRegistry.addRecipe(new ItemStack(smixture, 1), new Object[] {" ","RGR"," ", 'R', new ItemStack(Items.coal,1), 'G', new ItemStack(Items.iron_ingot, 1)});

GameRegistry.addRecipe(new ItemStack(smixture, 1), new Object[] {" ","RGR"," ", 'R', new ItemStack(Items.coal,1,1), 'G', new ItemStack(Items.iron_ingot, 1)});

//smelting

GameRegistry.addSmelting(Items.reeds, new ItemStack(wax), 0F);

GameRegistry.addSmelting(cmixture, new ItemStack(cingot), 0F);

GameRegistry.addSmelting(smixture, new ItemStack(singot), 0F);

}

@EventHandler

public void init(FMLInitializationEvent event)

{

}

and the class

package hat.blocks.palmblocks;

import hat.HAT;

import java.util.Random;

import net.minecraft.block.Block;

import net.minecraft.block.BlockRotatedPillar;

import net.minecraft.block.material.Material;

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

import net.minecraft.creativetab.CreativeTabs;

import net.minecraft.item.Item;

import net.minecraft.util.IIcon;

import net.minecraft.world.IBlockAccess;

import net.minecraft.world.World;

import cpw.mods.fml.relauncher.Side;

import cpw.mods.fml.relauncher.SideOnly;

public abstract class palmlog extends BlockRotatedPillar

{

public static final String[] TOP ={"Palm_Wood_Log_Top"};

public static final String[] SIDE ={"Palm_Wood_Log_Side"};

public IIcon topIcon[];

public IIcon sideIcon[];

public palmlog()

{

super(Material.wood);

this.setHardness(2.0F);

this.setStepSound(soundTypeWood);

}

public static int func_150165_c(int p_150165_0_)

{

return p_150165_0_ & 3;

}

/**

* Returns the quantity of items to drop on block destruction.

*/

public int quantityDropped(Random p_149745_1_)

{

return 1;

}

public Item getItemDropped(int p_149650_1_, Random p_149650_2_, int p_149650_3_)

{

return Item.getItemFromBlock(this);

}

public void breakBlock(World p_149749_1_, int p_149749_2_, int p_149749_3_, int p_149749_4_, Block p_149749_5_, int p_149749_6_)

{

byte b0 = 4;

int i1 = b0 + 1;

if (p_149749_1_.checkChunksExist(p_149749_2_ - i1, p_149749_3_ - i1, p_149749_4_ - i1, p_149749_2_ + i1, p_149749_3_ + i1, p_149749_4_ + i1))

{

for (int j1 = -b0; j1 <= b0; ++j1)

{

for (int k1 = -b0; k1 <= b0; ++k1)

{

for (int l1 = -b0; l1 <= b0; ++l1)

{

Block block = p_149749_1_.getBlock(p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1);

if (block.isLeaves(p_149749_1_, p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1))

{

block.beginLeavesDecay(p_149749_1_, p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1);

}

public void registerBlockIcons(IIconRegister iIconRegister){

topIcon = new IIcon[this.TOP.length];

sideIcon = new IIcon[this.SIDE.length];

for (int i =0; i < TOP.length;i++){

topIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Top");

sideIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Side");

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getSideIcon(int metadata)

{

return sideIcon[metadata];

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getTopIcon(int metadata)

{

return topIcon[metadata];

}

@Override

public boolean canSustainLeaves(IBlockAccess world, int x, int y, int z)

{

return true;

}

@Override

public boolean isWood(IBlockAccess world, int x, int y, int z)

{

return true;

}

sequituri · April 26, 2014

Class names should always start with in initial Capital letter (including the constructors). Nothing else that is a symbol should, except manifest constants (ALL CAPS). Otherwise, you get completely hard to debug code that has errors as you can see, with no idea why.

Hint:

Your import statement does not import any classes.

2nd hint:

import hat.blocks.palmblocks.palmlog;

^ This could be a file named "palmblocks" containing a class "palmlog" or it may be a file "palmlog" which imports nothing! There is no class name, but you cannot tell because your class names are lowercased!

majesticmadman98 · April 26, 2014

it is set right for the class as if i did the class code as a normal block with no extra stuff it works fine its with the rotated pillar block stuff it doesnt

MrArcane111 · April 26, 2014

A brief suggestion:

Use this icon array declaration at the beginning of your class. This sets the amount of icons we will be working with.

private IIcon[] icon = new IIcon[however_many_icons_you_need_as_an_integer];

Now that you've declared the number of icons to use, we need to determine on which side (and/or on which metadata) to load them:

@Override
@SideOnly(Side.CLIENT)
public IIcon getIcon(int side, int metadata)
{
     // 0 = bottom, 1 = top, for individual sides, please refer to BlockFurnace
}

Now that you've got all that, we need to register the icons so that they'll load in game:

@Override
@SideOnly(Side.CLIENT)
public void registerBlockIcons(IIconRegister ir)
{
     this.icon[0] = ir.registerIcon(""); // For loading textures, please refer to the hundred tutorials. I don't want to write out directory related stuff.
     // Let's say you declared 4 icons at the top. Using the number 0 as I did above uses one of the icons from our array. Still confused? Read more about arrays.
     // Now continue what I did above for each of your textures.
}

Hope this helps!

majesticmadman98 · April 26, 2014

hey thanks for the suggestion but I'm currently using what Godis_apan said, but i cant test it as something about the block class stops the main class using the class

TheGreyGhost · April 26, 2014

hey thanks for the suggestion but I'm currently using what Godis_apan said, but i cant test it as something about the block class stops the main class using the class

What you are effectively doing is

palmlog palmlog;
palmlog = new palmlog();

That is very obviously going to cause trouble because your compiler can't tell whether palmlog is the class definition, or the variable.

That is why everyone names classes in CamelCase

i.e.

Palmlog palmlog = new Palmlog();

If you don't stick to these conventions you're going to make yourself very unpopular with anyone who has to read or debug your code.

-TGG

majesticmadman98 · April 26, 2014

ok heres the parts refering to the palm log now still with the erro and the palm log class.

package hat;

import hat.blocks.dyeblocks.dyeblock1;

import hat.blocks.dyeblocks.dyeblock10;

import hat.blocks.dyeblocks.dyeblock11;

import hat.blocks.dyeblocks.dyeblock12;

import hat.blocks.dyeblocks.dyeblock13;

import hat.blocks.dyeblocks.dyeblock14;

import hat.blocks.dyeblocks.dyeblock15;

import hat.blocks.dyeblocks.dyeblock16;

import hat.blocks.dyeblocks.dyeblock2;

import hat.blocks.dyeblocks.dyeblock3;

import hat.blocks.dyeblocks.dyeblock4;

import hat.blocks.dyeblocks.dyeblock5;

import hat.blocks.dyeblocks.dyeblock6;

import hat.blocks.dyeblocks.dyeblock7;

import hat.blocks.dyeblocks.dyeblock8;

import hat.blocks.dyeblocks.dyeblock9;

import hat.blocks.palmblocks.palmleaves;

import hat.blocks.palmblocks.PalmLog;

import hat.blocks.palmblocks.palmlogcomplex;

import hat.blocks.palmblocks.palmplank;

import hat.blocks.palmblocks.palmsapling;

import hat.items.combat.mace;

import hat.items.materials.cingot;

import hat.items.materials.cmixture;

import hat.items.materials.singot;

import hat.items.materials.smixture;

import hat.items.materials.wax;

import hat.tabs.icons.blocktabicon;

import hat.tabs.icons.combattabicon;

import hat.tabs.icons.materialtabicon;

import net.minecraft.block.Block;

import net.minecraft.creativetab.CreativeTabs;

import net.minecraft.init.Blocks;

import net.minecraft.init.Items;

import net.minecraft.item.Item;

import net.minecraft.item.Item.ToolMaterial;

import net.minecraft.item.ItemStack;

import net.minecraftforge.common.util.EnumHelper;

import net.minecraftforge.oredict.OreDictionary;

import cpw.mods.fml.common.Mod;

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

import cpw.mods.fml.common.SidedProxy;

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

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

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

@Mod(modid = HAT.MODID, version = HAT.VERSION)

public class HAT

{

public static final String MODID = "hat";

public static final String VERSION = "1.0";

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

public static CommonProxy proxy;

//Palm Defining

public static Block palmlog;

palmlog = new PalmLog().setBlockName("palmlog").setCreativeTab(hatblocksTab);

GameRegistry.registerBlock(palmlog, "palmlog");

package hat.blocks.palmblocks;

import hat.HAT;

import java.util.Random;

import net.minecraft.block.Block;

import net.minecraft.block.BlockRotatedPillar;

import net.minecraft.block.material.Material;

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

import net.minecraft.creativetab.CreativeTabs;

import net.minecraft.item.Item;

import net.minecraft.util.IIcon;

import net.minecraft.world.IBlockAccess;

import net.minecraft.world.World;

import cpw.mods.fml.relauncher.Side;

import cpw.mods.fml.relauncher.SideOnly;

public abstract class PalmLog extends BlockRotatedPillar

{

public static final String[] TOP ={"Palm_Wood_Log_Top"};

public static final String[] SIDE ={"Palm_Wood_Log_Side"};

public IIcon topIcon[];

public IIcon sideIcon[];

public PalmLog()

{

super(Material.wood);

this.setHardness(2.0F);

this.setStepSound(soundTypeWood);

}

public static int func_150165_c(int p_150165_0_)

{

return p_150165_0_ & 3;

}

/**

* Returns the quantity of items to drop on block destruction.

*/

public int quantityDropped(Random p_149745_1_)

{

return 1;

}

public Item getItemDropped(int p_149650_1_, Random p_149650_2_, int p_149650_3_)

{

return Item.getItemFromBlock(this);

}

public void breakBlock(World p_149749_1_, int p_149749_2_, int p_149749_3_, int p_149749_4_, Block p_149749_5_, int p_149749_6_)

{

byte b0 = 4;

int i1 = b0 + 1;

if (p_149749_1_.checkChunksExist(p_149749_2_ - i1, p_149749_3_ - i1, p_149749_4_ - i1, p_149749_2_ + i1, p_149749_3_ + i1, p_149749_4_ + i1))

{

for (int j1 = -b0; j1 <= b0; ++j1)

{

for (int k1 = -b0; k1 <= b0; ++k1)

{

for (int l1 = -b0; l1 <= b0; ++l1)

{

Block block = p_149749_1_.getBlock(p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1);

if (block.isLeaves(p_149749_1_, p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1))

{

block.beginLeavesDecay(p_149749_1_, p_149749_2_ + j1, p_149749_3_ + k1, p_149749_4_ + l1);

}

public void registerBlockIcons(IIconRegister iIconRegister){

topIcon = new IIcon[this.TOP.length];

sideIcon = new IIcon[this.SIDE.length];

for (int i =0; i < TOP.length;i++){

topIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Top");

sideIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Side");

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getSideIcon(int metadata)

{

return sideIcon[metadata];

}

@SideOnly(Side.CLIENT)

@Override

protected IIcon getTopIcon(int metadata)

{

return topIcon[metadata];

}

@Override

public boolean canSustainLeaves(IBlockAccess world, int x, int y, int z)

{

return true;

}

@Override

public boolean isWood(IBlockAccess world, int x, int y, int z)

{

return true;

}

sequituri · April 26, 2014

Unless you are purposely snipping parts of your code, there are two problems.

You have not created any event handlers for FML to initialize your mod. So, none of your main mod code is getting executed.

Put

@EventHandler
public static void load(FMLPreinitializationEvent loadEvent) {
//  put your block and item creation code in here
}

Do the same for FMLInitializationEvent with another method.

The other thing is:

for (int i =0; i < TOP.length;i++){
          topIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Top");
          sideIcon = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log" + "_Side");
       }

needs to be:

int i;
for (i =0; i < TOP.length;i++){
      topIcon[i] = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log_Top" + i);
      }
for (i =0; i < SIDE.length;i++){
      sideIcon[i] = iIconRegister.registerIcon(HAT.MODID + ":Palm_Wood_Log_Side" + i);
      }

If TOP.length == SIDE.length, you can merge the loops, like you had. You need to use the [ i ] notation.

majesticmadman98 · April 27, 2014

yeah i snipped bits to but on here because i didnt think people would appreciate having to look through this