Propriétés pour couper des arbres

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 ?

Deleted

Ce qu’il y avait dans onBlockDestroyed en prenant soin de remplacer les variables par celle que propose le paramètre event

robin4002

Dans l’event tu check si le bloc qui vient d’être cassé est du bois.
Si oui, tu check si le joueur à ton item et main et si oui tu check avec cette condition :
worldIn.getBlockState(pos.up(7)).getBlock().isWood(worldIn, pos.up(7)
que l’arbre n’est pas trop grand. Et si l’arbre est trop grand tu cancel l’event.

Le problème c’est que BlockBreak reste appelé après la destruction du bloc, donc ça permettra juste que la première buche ne soit pas cassé, le message ne s’affichera pas avant de tenter de casser le bloc

elx9000

J’avais une idée. Est-ce qu’on ne peut pas faire ceci ? :

le block visé se casse.
le message s’envoie.
le drop du block s’annule.
le block cassé réapparaît à l’endroit où on cassé le premier block.

robin4002

C’est exactement ce qu’on t’as proposé de faire avec BlockBreakEvent

elx9000

D’accord j’avais pas compris. J’ai fait ça le problème c’est que le message ne s’envoie pas et 7 block de buche se détruisent :

@SubscribeEvent
public void onBlockDestroyed(BlockEvent.BreakEvent event)
{
if (event.world.getBlockState(event.pos).getBlock().isWood(event.world, event.pos))
{
if (event.getPlayer().getItemInUse() == new ItemStack(ItemMod.itemMachete))
{
if (event.world.getBlockState(event.pos.up(7)).getBlock().isWood(event.world, event.pos))
{
event.setCanceled(true);
event.getPlayer().addChatMessage(new ChatComponentText("Cet arbre est trop grand pour pouvoir être coupé !"));
}
else
{
event.setCanceled(false);
}
}
}
}

robin4002

if (event.getPlayer().getItemInUse() == new ItemStack(ItemMod.itemMachete))
->
if (event.getPlayer().getItemInUse() != null && event.getPlayer().getItemInUse().getItem() == ItemMod.itemMachete)

elx9000

Ca veut toujours pas, c’est le même problème.

robin4002

Tu as bien enregistré la classe d’événement dans ta classe principale ?

elx9000

Oui, j’ai d’autres évènement dans cette classe qui fonctionne très bien

elx9000

Une idée ?

elx9000

Vu que BlockEvent.BreakEvent ne semble pas fonctionner, je penser utiliser la commande /setblock comme on peut le faire avec les commandes blocks ou le tchat. Cependant est-il possible d’exécuter cette commande à l’intérieur du mod ? Je voudrais en effet rajouter cette commande à ma méthode onBlockDestroyed pour replacer le block.

AymericRed

Euh… pourquoi passer par une commande alors que tu peux faire world.setBlockState(tonBlock, LaPostion); ?

elx9000

Effectivement, en mettant ceci au niveau de l’envoie du texte :“Cet arbre est trop grand”, ça fonctionne:

%(#6a3e3e)[worldIn].setBlockState(%(#6a3e3e)[[color=#6a3e3e]pos], (IBlockState)%(#6a3e3e)[[color=#6a3e3e]worldIn].getBlockState(%(#6a3e3e)[[color=#6a3e3e]pos].up(1)).getBlock());

Le souci c’est que le bloc qui se repositionne devient invisible, je dois faire un click droit dessus (alors qu’il est invisible) pour le faire apparaître. Comment corriger ceci ?

Merci par avance.

robin4002

worldIn.setBlockState(pos, worldIn.getBlockState(pos.up(1)), 3);

elx9000

Je viens d’essayer, si je met pas .getBlock() , le bloc drop et même en mettant .getBlock() puis ,3 derrière cela ne fonctionne pas. La valeur de 3 correspond à un retard ?