Icon not rendering

[GodOfYeti]

heres my code

 

package com.mcpixelplex.blocks;

import com.mcpixelplex.CreativeTabs.BlessedCreativeTab;
import com.mcpixelplex.lib.RefStrings;

import cpw.mods.fml.relauncher.Side;
import cpw.mods.fml.relauncher.SideOnly;
import net.minecraft.block.Block;
import net.minecraft.block.material.Material;
import net.minecraft.client.renderer.texture.IIconRegister;
import net.minecraft.util.IIcon;

public class machineBlock extends Block{

public machineBlock(Material p_i45394_1_) {
	super(p_i45394_1_);
}

@SideOnly(Side.CLIENT)
public static IIcon topIcon;
@SideOnly(Side.CLIENT)
public static IIcon sideIcon;
@SideOnly(Side.CLIENT)
public static IIcon frontIcon;

@SideOnly(Side.CLIENT)
public void registerIIcons(IIconRegister icon) {
	topIcon = icon.registerIcon(RefStrings.MODID + ":machineBlockTop"); //top
	sideIcon = icon.registerIcon(RefStrings.MODID + ":machineBlockSide"); //side
	frontIcon = icon.registerIcon(RefStrings.MODID + ":machineBlockFront"); //front
}

public IIcon getIIcon(int side, int meta) {
if(side == 0 || side == 1) {
		return topIcon;
	} else if(side == 2) {
		return frontIcon;
	} else {
		return sideIcon;
	}
}

}

[Draco18s]

This is why you use @Override

If you had @Override on your getIIcon function you would know what the problem is.

[shadowfacts]

Also, the first letter of your class name should be capitalized. e.g. MachineBlock not machineBlock.

[GodOfYeti]

This is why you use @Override

If you had @Override on your getIIcon function you would know what the problem is.

 

It tells me to remove the override function

[Draco18s]

should

 

Or rather, it's common practice and considered good technique, but the compiler doesn't give a shit.

[shadowfacts]

Yeah, common practice to make it easier to tell wether something is a class, method, or field.

Seriously, use it. This naming convention is used in pretty much every programming language that has classes.

[GodOfYeti]

Not to be rude but ya getting a little bit off topic, the getIIcon tells me to remove @override

[Draco18s]

Yeah, common practice to make it easier to tell wether something is a class, method, or field.

 

Package, class, constant, methods.  Yep.

(packages are all lower case.  Classes are camel case with a capitalized first letter, constants are ALL CAPS, methods, fields, and other variables are camel case with lower case first letter).

 

Not to be rude but ya getting a little bit off topic, the getIIcon tells me to remove @override

 

Because your method signature is wrong.  It's not overriding any function in the super class.  Look closely.

[GodOfYeti]

Yeah, common practice to make it easier to tell wether something is a class, method, or field.

 

Package, class, constant, methods.  Yep.

(packages are all lower case.  Classes are camel case with a capitalized first letter, constants are ALL CAPS, methods, fields, and other variables are camel case with lower case first letter).

 

Not to be rude but ya getting a little bit off topic, the getIIcon tells me to remove @override

 

Because your method signature is wrong.  It's not overriding any function in the super class.  Look closely.

 

 

What can I do to fix it

[shadowfacts]

Look at the method in the Block class. Then look at yours. What is different?

[GodOfYeti]

Look guys im trying here but im getting no where

[shadowfacts]

*facepalms*

getIcon not getIIcon and registerIcons not registerIIcons.

 

[Draco18s]

getIcon != getIIcon

[GodOfYeti]

*facepalms*

getIcon not getIIcon and registerIcons not registerIIcons.

 

Ive tried that

[GodOfYeti]

This is what I got so far

 

package com.mcpixelplex.blocks;

import com.mcpixelplex.CreativeTabs.BlessedCreativeTab;
import com.mcpixelplex.lib.RefStrings;

import cpw.mods.fml.relauncher.Side;
import cpw.mods.fml.relauncher.SideOnly;
import net.minecraft.block.Block;
import net.minecraft.block.material.Material;
import net.minecraft.client.renderer.texture.IIconRegister;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.item.ItemStack;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.IIcon;
import net.minecraft.util.MathHelper;
import net.minecraft.world.World;

public class MachineBlock extends Block{

public MachineBlock(int id) {
	super(Material.iron);
	setBlockName("machineBlock");
	setCreativeTab(BlessedCreativeTab.tabBlocks);
	setHardness(5F);
	setResistance(10F);
}

//public TileEntity createNewTileEntity(World world) {
//	return null;
//}

@SideOnly(Side.CLIENT)
public static IIcon topIcon;
@SideOnly(Side.CLIENT)
public static IIcon sideIcon;
@SideOnly(Side.CLIENT)
public static IIcon frontIcon;

@SideOnly(Side.CLIENT)
public void registerIcons(IIconRegister icon) {
	topIcon = icon.registerIcon(RefStrings.MODID + ":machineBlockTop"); //top
	sideIcon = icon.registerIcon(RefStrings.MODID + ":machineBlockSide"); //side
	frontIcon = icon.registerIcon(RefStrings.MODID + ":machineBlockFront"); //front
}

    @Override
public IIcon getIcon(int side, int meta) {
	if(side == 0 || side == 1) {
		return topIcon;
	} else if(side == 2) {
		return frontIcon;
	} else {
		return sideIcon;
	}
}

}

[GodOfYeti]

Well I ended up rewriting the entire block and it seems to be working but I would like to know how to make it rotate depending on the players location

[shadowfacts]

Make a new topic.