[SOLVED][1.6.4] Problem with tree height from sapling block

[Delilah]

Hi hello, I'm doing a mod and I have a small problem but I don't know how to fix it.

 

I did a new world generation with my own world generator and my owns worldgentree and treemanager.

The new generation is working great but the only problem that I have is when I cut a tree and I used the

sapling from the tree, the new tree is my own custom tree which is great but the problem is that the height

of the new tree is not the height that I want. I'm looking in the code of the sapling to see which parameter

is the parameter that control the height and I did a lot of tests but I can't find. I was looking for information

of the vanilla sapling block to understand all the parameters but I didn't see a lot of information so I decided

to open a post to see if someone can tell me which parameter control in my code the height of the tree.

 

I'm going to show here the code of my sapling that grow the new tree and the code of my worldgentree.

 

Sapling code to grow the new tree

 

 

/**

    * Attempts to grow a sapling into a tree

    */

    public void growTree(World par1World, int par2, int par3, int par4, Random par5Random)

    {

        if (!TerrainGen.saplingGrowTree(par1World, par5Random, par2, par3, par4)) return;

 

        int l = par1World.getBlockMetadata(par2, par3, par4) & 3;

        Object object = null;

        int i1 = 0;

        int j1 = 0;

        boolean flag = false;

 

        /*if (l == 1)

        {

            object = new WorldGenTaiga2(true);

        }

        else if (l == 2)

        {

            object = new WorldGenForest(true);

        }

        */

        //else if (l == 3)

        if (l == 3)

        {

            for (i1 = 0; i1 >= -1; --i1)

            {

                for (j1 = 0; j1 >= -1; --j1)

                {

                    if (this.isSameSapling(par1World, par2 + i1, par3, par4 + j1, 3) && this.isSameSapling(par1World, par2 + i1 + 1, par3, par4 + j1, 3) && this.isSameSapling(par1World, par2 + i1, par3, par4 + j1 + 1, 3) && this.isSameSapling(par1World, par2 + i1 + 1, par3, par4 + j1 + 1, 3))

                    {

                        //object = new WorldGenHugeTrees(true, 4 + par5Random.nextInt(20), 3, 3);

                    object = new WorldGenTree(true, 4 + par5Random.nextInt(20), 3, 3, true);

                        flag = true;

                        break;

                    }

                }

 

                if (object != null)

                {

                    break;

                }

            }

 

            if (object == null)

            {

                j1 = 0;

                i1 = 0;

                //object = new WorldGenTree(true, 4 + par5Random.nextInt(7), 3, 3, false);

                object = new WorldGenTree(true, 4  + par5Random.nextInt(7), 3, 3, true);

            }

        }

        else

        {

            object = new WorldGenTree(true);

 

            if (par5Random.nextInt(10) == 0)

            {

                object = new WorldGenTree(true);

            }

        }

 

        if (flag)

        {

        par1World.setBlock(par2 + i1, par3, par4 + j1, 0, 0, 4);

            par1World.setBlock(par2 + i1 + 1, par3, par4 + j1, 0, 0, 4);

            par1World.setBlock(par2 + i1, par3, par4 + j1 + 1, 0, 0, 4);

            par1World.setBlock(par2 + i1 + 1, par3, par4 + j1 + 1, 0, 0, 4);

        }

        else

        {

            par1World.setBlock(par2, par3, par4, 0, 0, 10);

        }

 

        if (!((WorldGenerator)object).generate(par1World, par5Random, par2 + i1, par3, par4 + j1))

        {

            if (flag)

            {

            par1World.setBlock(par2 + i1, par3, par4 + j1, this.blockID, l, 4);

                par1World.setBlock(par2 + i1 + 1, par3, par4 + j1, this.blockID, l, 4);

                par1World.setBlock(par2 + i1, par3, par4 + j1 + 1, this.blockID, l, 4);

                par1World.setBlock(par2 + i1 + 1, par3, par4 + j1 + 1, this.blockID, l, 4);

               

            }

            else

            {

                par1World.setBlock(par2, par3, par4, this.blockID, l, 4);

            }

        }

    }

 

 

 

 

My WorldGenTree

 

 

package JavaBuckets.Modium.Main;

 

import java.util.Random;

 

import net.minecraft.block.Block;

import net.minecraft.block.BlockSapling;

import net.minecraft.world.World;

import net.minecraft.world.gen.feature.WorldGenerator;

import net.minecraftforge.common.ForgeDirection;

 

public class WorldGenTree extends WorldGenerator

{

/** The minimum height of a generated tree. */

private final int minTreeHeight;

/** True if this tree should grow Vines. */

private final boolean vinesGrow;

/** The metadata value of the wood to use in tree generation. */

private final int metaWood;

/** The metadata value of the leaves to use in tree generation. */

private final int metaLeaves;

public WorldGenTree(boolean par1)

{

        this(par1, 4, 0, 0, false);

}

public WorldGenTree(boolean par1, int par2, int par3, int par4, boolean par5)

{

        super(par1);

        this.minTreeHeight = par2;

        this.metaWood = par3;

        this.metaLeaves = par4;

        this.vinesGrow = par5;

}

public boolean generate(World par1World, Random par2Random, int par3, int par4, int par5)

{

        int l = par2Random.nextInt(3) + this.minTreeHeight;

        boolean flag = true;

        if (par4 >= 1 && par4 + l + 1 <= 256)

        {

                int i1;

                byte b0;

                int j1;

                int k1;

                for (i1 = par4; i1 <= par4 + 1 + l; ++i1)

                {

                        b0 = 1;

                        if (i1 == par4)

                        {

                                b0 = 0;

                        }

                        if (i1 >= par4 + 1 + l - 2)

                        {

                                b0 = 2;

                        }

                        for (int l1 = par3 - b0; l1 <= par3 + b0 && flag; ++l1)

                        {

                                for (j1 = par5 - b0; j1 <= par5 + b0 && flag; ++j1)

                                {

                                        if (i1 >= 0 && i1 < 256)

                                        {

                                                k1 = par1World.getBlockId(l1, i1, j1);

                                                Block block = Block.blocksList[k1];

                                                boolean isAir = par1World.isAirBlock(l1, i1, j1);

                                                if (!isAir &&

                                                        !block.isLeaves(par1World, l1, i1, j1) &&

                                                        k1 != MainClass.TeRGrass.blockID && //What blocks the tree will generate on

                                                        k1 != MainClass.ModiumBlock.blockID &&

                                                        !block.isWood(par1World, l1, i1, j1))

                                                     

                                                {

                                                        flag = false;

                                                }

                                        }

                                        else

                                        {

                                                flag = false;

                                        }

                                }

                        }

                }

                if (!flag)

                {

                        return false;

                }

                else

                {

                        i1 = par1World.getBlockId(par3, par4 - 1, par5);

                        Block soil = Block.blocksList[i1];

                        boolean isSoil = (soil != null && soil.canSustainPlant(par1World, par3, par4 - 1, par5, ForgeDirection.UP, (BlockSapling)Block.sapling));

                        if (isSoil && par4 < 256 - l - 1)

                        {

                                soil.onPlantGrow(par1World, par3, par4 - 1, par5, par3, par4, par5);

                                b0 = 3;

                                byte b1 = 0;

                                int i2;

                                int j2;

                                int k2;

                                for (j1 = par4 - b0 + l; j1 <= par4 + l; ++j1)

                                {

                                        k1 = j1 - (par4 + l);

                                        i2 = b1 + 1 - k1 / 2;

                                        for (j2 = par3 - i2; j2 <= par3 + i2; ++j2)

                                        {

                                                k2 = j2 - par3;

                                                for (int l2 = par5 - i2; l2 <= par5 + i2; ++l2)

                                                {

                                                        int i3 = l2 - par5;

                                                        if (Math.abs(k2) != i2 || Math.abs(i3) != i2 || par2Random.nextInt(2) != 0 && k1 != 0)

                                                        {

                                                                int j3 = par1World.getBlockId(j2, j1, l2);

                                                                Block block = Block.blocksList[j3];

                                                                if (block == null || block.canBeReplacedByLeaves(par1World, j2, j1, l2))

                                                                {

                                                                        this.setBlockAndMetadata(par1World, j2, j1, l2, MainClass.TeRTreeLeavesForest.blockID, this.metaLeaves);

                                                                }

                                                        }

                                                }

                                        }

                                }

                                for (j1 = 0; j1 < l; ++j1)

                                {

                                        k1 = par1World.getBlockId(par3, par4 + j1, par5);

                                        Block block = Block.blocksList[k1];

                                        if (k1 == 0 || block == null || block.isLeaves(par1World, par3, par4 + j1, par5))

                                        {

                                                this.setBlockAndMetadata(par1World, par3, par4 + j1, par5, MainClass.TeRTreeWoodForest.blockID, this.metaWood);

                                        }

                                }

                                return true;

                        }

                        else

                        {

                                return false;

                        }

                }

        }

        else

        {

                return false;

        }

}

}

 

 

 

Thanks.

[GotoLink]

Learn to read.

int l = par2Random.nextInt(3) + this.minTreeHeight;

Duh.

[Delilah]

I saw this line, but I was looking in the sapling block. I'll try now, with this parameter. Thanks.

[Delilah]

Thanks I worked with this line and I fixed the problem. Thanks.