Not breaking with tool

[chasingfire_hpps]

I wanted to make this break with a shovel faster than by hand. I set the material to sand, am I missing something?

 

package Blocks;

 

import net.minecraft.block.material.Material;

import net.minecraft.block.Block;

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

import net.minecraft.creativetab.CreativeTabs;

import net.minecraft.util.AxisAlignedBB;

import net.minecraft.world.World;

 

public class Coal_Pile extends Block {

 

    public Coal_Pile(int id) {

        super(id, Material.sand);

        this.setBlockBounds(0.0F, 0.0F, 0.0F, 1.0F , 1.5F, 1.0F);

setCreativeTab(CreativeTabs.tabBlock);

setHardness(3F);

setUnlocalizedName(Blockinfo.Coal_Pile_UNLOCALIZED_NAME);

    }

       

    public AxisAlignedBB getCollisionBoundingBoxFromPool(World world, int x, int y, int z) {

        return null;

    }

       

    public int getRenderType() {

        return 6;

    }

   

    public boolean isOpaqueCube() {

        return false;

        }

       

    @Override

    public void registerIcons(IconRegister register){

   

    blockIcon = register.registerIcon(Blockinfo.TEXTURE_LOCATION + ":" + Blockinfo.Coal_Pile);

    }

}

 

 

 

package Blocks;

 

import net.minecraft.block.Block;

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

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

 

public class Cfblocks5 {

 

public static Block Coal_Pile;

 

public static void init() {

Coal_Pile = new Coal_Pile(Blockinfo.Coal_Pile_ID);

GameRegistry.registerBlock(Coal_Pile, Blockinfo.Coal_Pile_KEY);

 

 

}

 

 

public static void addNames() {

LanguageRegistry.addName(Coal_Pile, Blockinfo.Coal_Pile_NAME);

 

}

}

 

 

 

 

package Blocks;

 

public class Blockinfo {

 

public static final String TEXTURE_LOCATION = "lotsofcoals";

 

public static int Anthracite_ore_ID = 1620;

public static final String Anthracite_ore_KEY = "Anthracite_ore";

public static final int Anthracite_ore_DEFAULT = 1360;

public static final String Anthracite_ore_UNLOCALIZED_NAME = "Anthracite_ore";

public static final String Anthracite_ore_NAME = "Anthracite Ore";

 

public static final String Anthracite_ore = "anthracite_ore";

 

public static final String Anthracite_ore_TOP="anthracite_ore";

public static final String Anthracite_ore_BOT="anthracite_ore";

public static final String Anthracite_ore_SIDE="anthracite_ore";

 

public static int Bituminous_ore_ID = 1621;

public static final String Bituminous_ore_KEY = "Bituminous_ore";

public static final int Bituminous_ore_DEFAULT = 1361;

public static final String Bituminous_ore_UNLOCALIZED_NAME = "Bituminous_ore";

public static final String Bituminous_ore_NAME = "Bituminous Ore";

 

public static final String Bituminous_ore = "Bituminous_ore";

public static final String Bituminous_ore_TOP="bituminus_ore";

public static final String Bituminous_ore_BOT="bituminus_ore";

public static final String Bituminous_ore_SIDE="bituminus_ore";

 

public static int Lignite_ore_ID = 1622;

public static final String Lignite_ore_KEY = "Lignite_ore";

public static final int Lignite_ore_DEFAULT = 1362;

public static final String Lignite_ore_UNLOCALIZED_NAME = "Lignite_ore";

public static final String Lignite_ore_NAME = "Lignite Ore";

 

public static final String Lignite_ore = "Lignite_ore";

public static final String Lignite_ore_TOP="lignite_ore";

public static final String Lignite_ore_BOT="lignite_ore";

public static final String Lignite_ore_SIDE="lignite_ore";

 

public static int Peat_ID = 1623;

public static final String Peat_KEY = "Peat";

public static final int Peat_DEFAULT = 1363;

public static final String Peat_UNLOCALIZED_NAME = "Peat";

public static final String Peat_NAME = "Peat";

 

public static final String Peat = "Peat";

public static final String Peat_TOP="peat_block";

public static final String Peat_BOT="peat_block";

public static final String Peat_SIDE="peat_block";

 

public static int Blast_furnace_ID = 1624;

public static final String Blast_furnace_KEY = "Blast_furnace";

public static final int Blast_furnace_DEFAULT = 1364;

public static final String Blast_furnace_UNLOCALIZED_NAME = "Blast_furnace";

public static final String Blast_furnace_NAME = "Blast Furnace";

 

public static final String Blast_furnace = "Blast_furnace";

public static final String Blast_furnace_TOP="Coal_Furnace_Side";

public static final String Blast_furnace_BOT="Coal_Furnace_Side";

public static final String Blast_furnace_SIDE="Coal_Furnace_Side";

public static final String Blast_furnace_FRONT="Coal_Furnace_Front";

 

public static int Coal_Pile_ID = 1625;

public static final String Coal_Pile_KEY = "Coal_Pile";

public static final int Coal_Pile_DEFAULT = 1365;

public static final String Coal_Pile_UNLOCALIZED_NAME = "Coal_Pile";

public static final String Coal_Pile_NAME = "Coal Pile";

public static final String Coal_Pile = "Coal_Pile";

 

}

 

 

 

 

 

 

 

 

 

 

 

 

 

 

 

 

 

 

 

[sequituri]

See? Basic java folks.

 

This is the reason there are java naming conventions. People who code in java need to learn the basics: For one thing you have a variable "Coal_Pile" with the exact same name as a class "Coal_Pile". This is always a nono.

 

• You never name variable and fields starting with uppercase!
  
• Start class and enum names with uppercase.
  
• Don't use underscores "_" in anything except ALL_UPPERCASE or all_lowercase names.
  
• Don't use ALL_UPPERCASE for anything except manifest constants i.e. 'static final'
  
• Almost never use all_lowercase, learn camel-case.
  
• All non-constant variable and fields start with a lowercased letter (so they are not confused with Class names.
  

Camel case uses worldAnotherAndAnother or WordHereToo for example.

 

[chasingfire_hpps]

Ok its how I named it sorry. It doesn't affect anything now are you going to help.

[sequituri]

Ok its how I named it sorry. It doesn't affect anything now are you going to help.

 

That's nice of you to not bother learning how to code. I hope someone cares, because I give up on you.