Savoir si un item est présent dans l'inventaire

BrokenSwing

Le problème vient du fait que les arguments de la fonctions ne sont pas le bon ordre, le constructeur n’a rien à voir

yveslefou

@‘BrokenSwing’:

Les Item n’ont pas forcement besoin d’un constructeur, ensuite il est possible que l’itemstack de mana qu’il ai sur lui n’est pas le même métadata (vu que c’est un truc de mana j’imagine que ce n’est pas infini et du coup ya une durabilité où un truc comme ça (peut-être));
Et les arguments de la fonctions ne sont pas bons
Donc utilise plutôt :
public class Sort1 extends Item{

public ItemStack onItemRightClick(ItemStack itemStackIn, World world, EntityPlayer player)
{
if (!world.isRemote && player.inventory.hasItem(Main.Mana)) {
player.addPotionEffect(new PotionEffect(3, 200, 2));

}
else{
player.addChatMessage(new ChatComponentText("[Magie] Mana insuffisant"));
}
return stack ;

}
}

Il n’y a pas de méthadata mais il est vrai que j’avais les avais mis dans le mauvais ordre sans m’en rendre compte mais maintenant bien que j’ais mes manas dans l’inventaire cela les détecte pas mais je pense que je peus régler sa en regardant un peu merci beaucoup

BrokenSwing

Non je parle de l’ORDRE, d’abord l’ItemStack, puis le World puis l’EntityPlayer, tu peut laisser stack comme nom de varariable
EDIT : Quand tu override une fonction pense toujours à l’annoté comme ça tu sais si tu as fait une erreur

Dylem

J’ai testé le code en 1.8.9 donc je ne peux pas garantir qu’il marche en 1.7, sinon par convention on ne donne pas de majuscule aux variables
Tu as bien définit ton Item en public static ?

yveslefou

Oui il est bien en public ^^

BrokenSwing

Il faut que ta fonction soit comme ça et c’est tout :


@Override
public ItemStack onItemRightClick(ItemStack stack, World world,EntityPlayer player)
{
if (!world.isRemote && player.inventory.hasItemStack(new ItemStack(Main.Mana))) {
player.addPotionEffect(new PotionEffect(3, 200, 2));

}
else{
player.addChatMessage(new ChatComponentText("[Magie] Mana insuffisant"));
}
return stack ;

}
}

Dylem

Sinon effectivement je n’avais pas vu mais ça ne marchera pas si tu change l’ordre des paramètres…

yveslefou

en effet le problémé venais bien de l’ordre mais maintenant j’ais un tout petit soucie mais je en regardant un peu dessus je suis sur que je peus le regler tout seul. C’est que cela me donne l’effet et m’affiche le message.

Sinon un grand merci à vous deux que dis je un énorme ^^

Dylem

@‘yveslefou’:

en effet le problémé venais bien de l’ordre mais maintenant j’ais un tout petit soucie mais je en regardant un peu dessus je suis sur que je peus le regler tout seul. C’est que cela me donne l’effet et m’affiche le message.

Sinon un grand merci à vous deux que dis je un énorme ^^

C’est tout simplement parce que le world.isRemote sera vrai et donc il affiche le message.

@Override
public ItemStack onItemRightClick(ItemStack stack, World world,EntityPlayer player)
{
if (!world.isRemote) {
if player.inventory.hasItemStack(new ItemStack(Main.Mana)) {
player.addPotionEffect(new PotionEffect(3, 200, 2));
}
else{
player.addChatMessage(new ChatComponentText("[Magie] Mana insuffisant"));
}
}
return stack ;

}

yveslefou

@‘Dylem’:

@‘yveslefou’:

en effet le problémé venais bien de l’ordre mais maintenant j’ais un tout petit soucie mais je en regardant un peu dessus je suis sur que je peus le regler tout seul. C’est que cela me donne l’effet et m’affiche le message.

Sinon un grand merci à vous deux que dis je un énorme ^^

C’est tout simplement parce que le !world.isRemote sera faux et donc il affiche le message.
@Override
public ItemStack onItemRightClick(ItemStack stack, World world,EntityPlayer player)
{
if (!world.isRemote) {
if player.inventory.hasItemStack(new ItemStack(Main.Mana)) {
player.addPotionEffect(new PotionEffect(3, 200, 2));
}
else{
player.addChatMessage(new ChatComponentText("[Magie] Mana insuffisant"));
}
}
return stack ;

}

Je venais juste de trouver la raison quand tu as publié le message. Je te remercie encore car surtout grâce à toi le mod pourra voir le jour.

yveslefou

A en faite la solution trouvé tout les deux ne peus pas convenir car cela fais crash le jeux.

yveslefou

En faite c’est bon il y avais juste une petit erreur dans ce que tu as envoyé et un } qui manqué dans ma version. Encore merci à toi

SCAREX

Minimum 24h entre 2 messages.

PS : si le sujet est résolu, pense à la balise