[1.6.2]Problem with Custom Crop

[Neo2k]

Hello together,

 

i have a problem with my custom crop.

 

Here is the code:

 

package xooth.core.blocks;

import java.util.Random;

import cpw.mods.fml.relauncher.Side;
import cpw.mods.fml.relauncher.SideOnly;
import net.minecraft.block.BlockCrops;
import net.minecraft.client.renderer.texture.IconRegister;
import net.minecraft.network.packet.Packet3Chat;
import net.minecraft.server.MinecraftServer;
import net.minecraft.util.Icon;
import net.minecraft.util.MathHelper;
import net.minecraft.world.World;
import xooth.core.CoreItems;
import xooth.core.blocks.*;

public class PlantTomato extends BlockCrops {

@SideOnly(Side.CLIENT)
private Icon[] iconArray;

public PlantTomato(int par1) {
	super(par1);
}

    @SideOnly(Side.CLIENT)
    public Icon getIcon(int par1, int par2)
    {
        if (par2 < 7)
        {
            if (par2 == 6)
            {
                par2 = 5;
            }

            return this.iconArray[par2 >> 1];
        }
        else
        {
            return this.iconArray[4];
        }
    }

protected int getSeedItem() {
	return CoreItems.tomatoSeed.itemID;
}

protected int getCropItem() {
	return CoreItems.tomatoFood.itemID;
}

public int idDropped(int par1, Random par2Random, int par2)
{
	return par1 == 7 ? this.getCropItem() : this.getSeedItem();
}

public int quantityDropped(Random par1Random)
{
	return 1;
}

@SideOnly(Side.CLIENT)
public void registerIcons(IconRegister reg)
{
	this.iconArray = new Icon[5];

	for (int i = 0; i < this.iconArray.length; i++)
	{
		this.iconArray[i] = reg.registerIcon("Core:tomatoCrop_" + i);
	}
}

}

 

The Problem is: my tomato plant has 5 images but the fourth image is skipped. If i use Bonemeal on the plant I must use it one time longer that means if i see the correct last image from the tomato plant i must use one more bonemeal to complete the plant.

 

Can someone help me with my problem?

Sorry for not so good english

 

[GotoLink]

Why ?

@SideOnly(Side.CLIENT)
    public Icon getIcon(int par1, int par2)
    {
        if (par2 < 7)
        {
             if (par2 == 6)
            {
                par2 = 5;
            }

[Neo2k]

I hope you can explain me why I do this with a tutorial and the maker don't explain for what the numbers are...

[GotoLink]

This piece of code does exactly what you are saying, it skips one of the images.

Now, learn some programming. Copying someone else code will take you nowhere.

[Neo2k]

But can you please explain me what i must do in this case?!

[Neo2k]

Can someone tell me what is wrong?

[robustus]

Can someone tell me what is wrong?

 

     if (par2 < 7)
        {
            if (par2 == 6)
            {
                par2 = 5;
            }

 

Do you see what is wrong with this code here?  par2 is the metadata which means everytime the crop gets to the stage that represents metadata 6, you've set it so anytime the crop gets to that stage is is automatically set to metadata 5 the previous crop state.