Propriétés pour couper des arbres

elx9000

Bonjour à tous, dans mon mod j’ai un outil, une machette, et je voudrais rajouter sur cet outil la propriété suivante :
Quand on utilise la machette sur un petit arbre (comme un petit chêne par exemple) je voudrais que tous les blocs de buches au - dessus du bloc cassé se détruisent.
Et pour que mon outil serve à quelque chose il faudrait aussi que le joueur ne puisse casser les blocs de buches qu’avec la machette (pas à la main …) . Et si les arbres son trop grand (comme un arbre de la jungle par exemple) je voudrais que la machette ne fonctionne pas sur cet arbre. Je n’ai aucune idée comment coder ceci. Merci par avance pour votre aide.

sventus

il existe un tuto fait par robin pour la 1.7 je sais pas si tout change mais ça peut peut-être te mettre sur une piste : https://www.minecraftforgefrance.fr/showthread.php?tid=702

elx9000

Cela me donne une idée, j’ai tenté quelque chose mais j’ai l’impression qu’en 1.8.9 la position du block est gérée par BlockPos, le souci c’est que je n’ai aucune idée comment s’occuper des coordonnées avec BlockPos, j’ai compris l’idée principale mais avec les BlockPos je n’ai aucune idée, comment faire ?

robin4002

Tu peux utiliser la fonction up pour augmenter le y d’un bloc pos, down pour diminuer le y, et il y a aussi nort, south, east et weast.
Il y a aussi une fonction add où tu peux changer les 3 coordonnées d’un coup :

BlockPos pos = new BlockPos(0, 0, 0) // x = 0, y = 0, z = 0
pos.up(50); // x = 0, y = 50, z = 0
pos.add(-20, 0, 50) // x = -20, y = 50, z =50

elx9000

J’ai fait ça :

public boolean onBlockDestroyed(ItemStack stack, World worldIn, Block blockIn, BlockPos pos, EntityLivingBase playerIn)
{
for (double i = pos.getY() ; i < 256 ; i++)
{
if (Blocks.log.isWood(worldIn, pos))
{
stack.damageItem(1, playerIn);

if (!worldIn.isRemote && worldIn.getGameRules().getBoolean("doTileDrops"))
{
float f = 0.7F;

double d0 = (double)(worldIn.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
double d1 = (double)(worldIn.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
double d2 = (double)(worldIn.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;

EntityItem entityItem = new EntityItem(worldIn, (double)pos.getX() + d0, (double)pos.getY() + d1, (double)pos.getZ() + d2, new ItemStack(Item.*getItemFromBlock*(blockIn), 1));
entityItem.setDefaultPickupDelay();
worldIn.spawnEntityInWorld(entityItem);
}
worldIn.setBlockToAir(pos.add(pos.getX(), pos.getY(), pos.getZ()));
pos.up(1);
}
else
{
return super.onBlockDestroyed(stack, worldIn, blockIn, pos, playerIn);
}
}
return super.onBlockDestroyed(stack, worldIn, blockIn, pos, playerIn);
}

Le problème c’est que quand je casse le block de la base de l’arbre, il n’y a qu’un block qui se casse (celui visé) et j’obtient environ 3 stack et 61 buches de bois.

robin4002

Ça devrait être plutôt comme ça :

public boolean onBlockDestroyed(ItemStack stack, World worldIn, Block blockIn, BlockPos pos, EntityLivingBase playerIn)
{
    for (int i = 0 ; i < 256 - pos.get(Y); i++)
    {
        BlockPos testPos = pos.up(i);
        if (Blocks.log.isWood(worldIn, testPos))
        {
            stack.damageItem(1, playerIn);

            if (!worldIn.isRemote && worldIn.getGameRules().getBoolean("doTileDrops"))
            {
                float f = 0.7F;

                double d0 = (double)(worldIn.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
                double d1 = (double)(worldIn.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
                double d2 = (double)(worldIn.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;

                EntityItem entityItem = new EntityItem(worldIn, (double)testPos.getX() + d0, (double)testPos.getY() + d1, (double)testPos.getZ() + d2, new ItemStack(Item.getItemFromBlock(blockIn), 1));
                entityItem.setDefaultPickupDelay();
                worldIn.spawnEntityInWorld(entityItem);
            }
            worldIn.setBlockToAir(testPos);
        }
        else
        {
            return super.onBlockDestroyed(stack, worldIn, blockIn, pos, playerIn);
        }
    }
    return super.onBlockDestroyed(stack, worldIn, blockIn, pos, playerIn);
}

elx9000

Cela fonctionne bien, le seul souci c’est que lorsque je détruis un arbre de 5 buches de hauts j’obtient en retour plusieurs stack de buches, comme si tous les blocks au dessus de moi était du bois.

robin4002

if (Blocks.log.isWood(worldIn, testPos))
->
if (worldIn.getBlockState(testPos).getBlock().isWood(worldIn, testPos))

elx9000

Super ça fonctionne, mais il me reste une dernière chose pour finaliser mon item. Je voudrais que si un arbre possède plus que 6 buches, l’item ne puisse pas être utilisé et qu’on envoie un message comme “Cette arbre est trop grand pour pouvoir être coupé”. Une idée ?

robin4002

public boolean onBlockDestroyed(ItemStack stack, World worldIn, Block blockIn, BlockPos pos, EntityLivingBase playerIn)
{
   if(worldIn.getBlockState(pos.up(7)).getBlock().isWood(worldIn, pos.up(7))
   {
        if(playerIn instanceof EntityPlayer && !worldIn.isRemote)
        {
             ((EntityPlayer)playerIn).sendChatMessage(new ChatComponantText("Cet arbre est trop grand pour pouvoir être coupé"));
        }
        return super.onBlockDestroyed(stack, worldIn, blockIn, pos, playerIn);
   }
   for (int i = 0 ; i < 7; i++)
   {
       BlockPos testPos = pos.up(i);
       if(worldIn.getBlockState(testPos).getBlock().isWood(worldIn, testPos))
       {
           stack.damageItem(1, playerIn);

           if (!worldIn.isRemote && worldIn.getGameRules().getBoolean("doTileDrops"))
           {
               float f = 0.7F;

               double d0 = (double)(worldIn.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
               double d1 = (double)(worldIn.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;
               double d2 = (double)(worldIn.rand.nextFloat() * f) + (double)(1.0F - f) * 0.5D;

               EntityItem entityItem = new EntityItem(worldIn, (double)testPos.getX() + d0, (double)testPos.getY() + d1, (double)testPos.getZ() + d2, new ItemStack(Item.getItemFromBlock(blockIn), 1));
               entityItem.setDefaultPickupDelay();
               worldIn.spawnEntityInWorld(entityItem);
           }
           worldIn.setBlockToAir(testPos);
       }
       else
       {
           return super.onBlockDestroyed(stack, worldIn, blockIn, pos, playerIn);
       }
   }
   return super.onBlockDestroyed(stack, worldIn, blockIn, pos, playerIn);
}

elx9000

Quand je coupe un arbre, ça ne coupe plus que la buche visée et cela m’envoie 2 fois le message même si l’arbre et de la bonne taille, et je peux toujours utilisé l’outil sur les arbres trop grands.

robin4002

J’ai édité mon message précédent, j’avais oublié d’appliquer la modif de mon avant dernier message.

elx9000

Ca fonctionne, le seul petit souci c’est que lorsque j’essaye de casser un “grand” arbre je ne casse que la buche visée et le message s’envoi après. Est-il possible de faire en sorte que si l’arbre est trop grand le message s’envoie puis l’item ne puisse pas être utilisé : le block ne se détruit pas ? Je pensais utilisé la fonction onItemUse pour vérifier le nombre de block dans l’arbre.

robin4002

Je ne vois aucune fonction dans la classe Item qui permettrai ça

AymericRed

Sinon tu peux utiliser l’event BlockEvent.BreakEvent et le cancel.

LeBossMax2

Ici, l’even HarvestCheck correspond plus car il s’exécute quand le joueur commence à casser l’arbre et que tu peux faire “setCanHarvest(false)” (ou “event.success = false” selon la version où tu es) pour que le joueur ne puisse pas casser l’arbre.

AymericRed

Ah oui tu as raison, car avec BreakEvent ça aurait juste fait comme si ça reposait le block juste après l’avoir cassé.

elx9000

Ok d’accord mais où je met l’event ?

robin4002

https://www.minecraftforgefrance.fr/showthread.php?tid=716

elx9000

Ok mais je met quoi dans l’event ?

Résolu Propriétés pour couper des arbres