Crash

DailyCraft

Pourquoi je crash quand j’utilise la class onItemRightClick ?

public class StrenghtStick extends Item
{
    public StrenghtStick()
    {
        super(new Properties().group(ModGroups.MYGROUP));
        setRegistryName("strengt_stick");
    }

    @Override
    public ActionResult<ItemStack> onItemRightClick(World world, PlayerEntity player, Hand hand)
    {
        player.addPotionEffect(new EffectInstance(Effect.get(5), 40));

        return null;
    }

    @Override
    public int getUseDuration(ItemStack stack) {
        return 20;
    }
}

robin4002

return super.onItemRightClick(world, player, hand)
ou alors simplement aller voir dans la fonction de la classe mère comment c’est géré …

JustAnDev

alerte aux LOGS!!!

DailyCraft

Les voila :
crash-2019-06-28_17.09.49-client.txt

JustAnDev

@DailyCraft a dit dans Crash :

    return null;

La magie des NPE aident a comprendre d’où vient l’erreur…

public ItemStack stack;
return stack;

DailyCraft

Pourquoi en public ?

JustAnDev

@DailyCraft a dit dans Crash :

Pourquoi en public ?

a oui ca sert a rien ca entraine meme des erreurs.Retire le.

DailyCraft

Ca me mets que la variable n’est pas initialiser

JustAnDev

= new ItemStack()

DailyCraft

Quand je mets rien dans les parenthese il me dit Cannot resolve constructor et au return, il me dit qu’il require net.minecraft.util.ActionResult<net.minecraft.item.ItemStack>

JustAnDev

ah oui il faut ajouter apres le stack<ActionResult> et il y a quoi au niveau de stack comme soluce?

DailyCraft

Au ActionResult il me dit Expression Expected et a la variable je sais pas si je suis un boulet mais quand je mets un final et que je vire le = new ItemStack(), il n’y a plus d’erreur

robin4002

return super.onItemRightClick(world, player, hand)
ou alors simplement aller voir dans la fonction de la classe mère comment c’est géré …

DailyCraft

Merci, ça as marcher mais maintenant comment je peux faire pour que je puisse enlever un de dura à chaque fois ?

robin4002

player.damageItem

DailyCraft

ça existe pas

robin4002

Ah oui, cela a changé avec la 1.14 :

stack.damageItem(1, player, (living) -> {
         living.sendBreakAnimation(EquipmentSlotType.MAINHAND);
      });

DailyCraft

Il n’y a pas de stack dans onItemRightClick

robin4002

il faut utiliser ItemStack stack = player.getHeldItem(hand); ou quelque chose comme ça.

DailyCraft

Merci ça a marché