Problème GuiScreen Creative/Survival

robin4002

Salut,
De même pour la modification du tag nbt de la carte.
D’ailleurs c’est étonnant qu’en étant en mode créatif ça fonctionne avec du code effectué en client o_O

LeBossMax2

C’est normal que ça fonctionne en créatif car quand tu est en créatif, c’est le client gère l’inventaire, (notamment pour se donner les items quand il les prends dans une crativeTab). Et c’est le client qui informe le serveur des modifications apportées.

Flow

Salut, j’ai regarder le tuto de la 1.7.10 pour m’aider et ca semble fonctionner pour les 5dollars donc en survival j’ai bien mes 5dollars décompté et ils sont bien récuperer dans l’inventaire voici mon code pouvez vous me dire si la façon dont je fais est correcte ? Merci

package fr.fifou.economy.packets;

import fr.fifou.economy.items.ItemsRegistery;
import io.netty.buffer.ByteBuf;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.ItemStack;
import net.minecraftforge.fml.common.network.simpleimpl.IMessage;
import net.minecraftforge.fml.common.network.simpleimpl.IMessageHandler;
import net.minecraftforge.fml.common.network.simpleimpl.MessageContext;

public class PacketCardChange implements IMessage {

private int funds;
public PacketCardChange()
{

}

public PacketCardChange(int funds)
{
this.funds = funds;
}

@Override
public void fromBytes(ByteBuf buf)
{
this.funds = buf.readInt();
}

@Override
public void toBytes(ByteBuf buf)
{
buf.writeInt(this.funds);
}

public static class Handler implements IMessageHandler <packetcardchange, imessage="">{

@Override
public IMessage onMessage(PacketCardChange message, MessageContext ctx)
{
EntityPlayer player = ctx.getServerHandler().playerEntity;
if(player.getHeldItemMainhand().getTagCompound().getInteger("Funds") >= 5)
{
player.inventory.addItemStackToInventory(new ItemStack(ItemsRegistery.itemFiveb));
int funds_new = player.getHeldItemMainhand().getTagCompound().getInteger("Funds");
player.getHeldItemMainhand().getTagCompound().setInteger("Funds", funds_new - 5);

}
else
{
System.out.println("not enough");
System.out.println(player.getHeldItemMainhand().getTagCompound().getInteger("Funds"));
}
return null;
}

}

}

if (button == this.fiveB)
{
ModEconomy.network.sendToServer(new PacketCardChange());

}
```</packetcardchange,>

LeBossMax2

Oui, c’est pas mal, mais dans la méthode onMessage de ton IMessageHandler, il faut d’abord vérifier que le joueur a bien le bon item en main sinon, ça peut peut être bugué dans certains cas.

Et il faut adapter le code pour qu’il fonctionne avec tous les boutons.

Flow

Adapter le code c’est à dire, je pensais juste récupérer le bouton cliquer et ensuite effectuer l’action mais je ne sais pas trop si c’est possible et comment faire

LeBossMax2

Dans la méthode onMessage, tu n’utilise pas la variable funds de PacketCardChange, donc il faudrait la modifier pour que ça donne le bon item.

Flow

@LeBossMax2:

Dans la méthode onMessage, tu n’utilise pas la variable funds de PacketCardChange, donc il faudrait la modifier pour que ça donne le bon item.

Je n’utilise jamais funds et en soit je n’ai besoin d’aucune variable, je ne comprend à quoi sert cet variable en fait

AymericRed

Bah elle sert à indiquer le montant que tu veux ajouter/retirer (donc elle change en fonction du bouton cliqué).

Flow

Ah ok, bah du coup si je fais comme ça :

if (button == this.fiveB)
{
ModEconomy.network.sendToServer(new PacketCardChange(5));
}

et

EntityPlayer player = ctx.getServerHandler().playerEntity;
ItemStack helditem = player.getHeldItemMainhand();
if(funds == 5)
{
if(player.getHeldItemMainhand().getTagCompound().getInteger("Funds") >= 5)
{
player.inventory.addItemStackToInventory(new ItemStack(ItemsRegistery.itemFiveb));
int funds_new = player.getHeldItemMainhand().getTagCompound().getInteger("Funds");
player.getHeldItemMainhand().getTagCompound().setInteger("Funds", funds_new - 5);

}

}

C’est correcte comme code ?

AymericRed

Oui ça marcherai mais là tu risques de faire une suite de if dans le packet alors que tu peux juste faire ça :


if(player.getHeldItemMainhand().getTagCompound().getInteger("Funds") >= funds)
{
ItemStack item = funds == 5 ? ItemsRegistery.itemFiveb : ItemStack.EMPTY;
player.inventory.addItemStackToInventory(new ItemStack(item));
int funds_new = player.getHeldItemMainhand().getTagCompound().getInteger("Funds");
player.getHeldItemMainhand().getTagCompound().setInteger("Funds", funds_new - funds);
}
``` au lieu de ça : ```java
if(funds == 5)
{
if(player.getHeldItemMainhand().getTagCompound().getInteger("Funds") >= 5)
{
player.inventory.addItemStackToInventory(new ItemStack(ItemsRegistery.itemFiveb));
int funds_new = player.getHeldItemMainhand().getTagCompound().getInteger("Funds");
player.getHeldItemMainhand().getTagCompound().setInteger("Funds", funds_new - 5);

}

Ah et par contre n’oublie pas qu’on peut ajouter ET retirer de l’argent (donc il faut faire un if si funds > 0 et un si funds < 0).

BrokenSwing

J’image bien un cheater envoyer un paquet PacketCardChange(-999999999)

AymericRed

Bah pour ajouter de l’argent il suffit que le packet vérifie que le joueur ait bien les billets dans son inventaire.

Flow

@‘BrokenSwing’:

J’image bien un cheater envoyer un paquet PacketCardChange(-999999999)

Oui c’est un problème, je n’ai mis que le retrait dans mon message du dessus mais pour l’ajout d’argent j’avais déjà fait ce que proposais Aymeric c’est à dire :
@‘AymericRed’:

Bah pour ajouter de l’argent il suffit que le packet vérifie que le joueur ait bien les billets dans son inventaire.

ItemStack stack6 = new ItemStack((ItemsRegistery.itemTwohundreedb));
if(player.inventory.hasItemStack(stack6))
{
player.inventory.clearMatchingItems(ItemsRegistery.itemTwohundreedb, 0, 1, null);
int funds_new_12 = player.getHeldItemMainhand().getTagCompound().getInteger("Funds");
player.getHeldItemMainhand().getTagCompound().setInteger("Funds", funds_new_12 + 200);
}

C’est correct ?

Parcontre je n’ai pas compris ce que tu me dis la :

@‘AymericRed’:

Ah et par contre n’oublie pas qu’on peut ajouter ET retirer de l’argent (donc il faut faire un if si funds > 0 et un si funds < 0).

En soit si funds est plus grand ou égal à 5 il donne 5 et diminue les funds ^^ Mais si il n’a pas =<5 alors il ne donne rien ^^

AymericRed

Il manque un truc : tu ne vérifies pas la quantitée des billets.

Flow

@‘AymericRed’:

Il manque un truc : tu ne vérifies pas la quantités des billets.

Si je vérifie qu’il a le billet dans son inventaire ca ne dois pas poser de problème si ? Si il a un item de 5 dollars alors il effectue l’action je ne vois pas le but de vérifier la quantitée …

AymericRed

Imagine le joueur a un item de 5 dollars, et il envoie un packet pour en ajouter 10 000, et bien tant qu’il aura même un seul billet il pourra.

Flow

Ah oui, j’avais pas penser à sa effectivement je vais essayer de chercher et j’édit mon message Tu sauras me dire si le code est juste avec les modifs ? Merci en tout cas ^^

EDIT :

Comme ceci ? Je ne suis pas sur du tout

else if(funds == -500)
{
ItemStack stack6 = new ItemStack((ItemsRegistery.itemFivehundreedb));
int count6 = stack6.getCount();
if(player.inventory.hasItemStack(stack6))
{
if(count6 >= 1)
{
player.inventory.clearMatchingItems(ItemsRegistery.itemFivehundreedb, 0, 1, null);
int funds_new_13 = player.getHeldItemMainhand().getTagCompound().getInteger("Funds");
player.getHeldItemMainhand().getTagCompound().setInteger("Funds", funds_new_13 + 500);
}

}
}

Désolé du double post, mais je sépare ma bêtise –" Hahaha

Du coup c’est comme ça que je dois faire non ?

else if(funds == -500)
{
ItemStack stack6 = new ItemStack((ItemsRegistery.itemFivehundreedb));
int count6 = stack6.getCount();
if(player.inventory.hasItemStack(stack6))
{
if(count6 * 500 >= funds)
{
player.inventory.clearMatchingItems(ItemsRegistery.itemFivehundreedb, 0, 1, null);
int funds_new_13 = player.getHeldItemMainhand().getTagCompound().getInteger("Funds");
player.getHeldItemMainhand().getTagCompound().setInteger("Funds", funds_new_13 + 500);
}

}
}
``` En gros (siNombreBilletDansInventaire * sommeDemandee >= sommeDemandee) alors on autorise ?

AymericRed

Le principe est le bon, sauf que count6 ne correspond pas au nombre de billets dans l’inventaite, mais juste à un ItemStack que tu viens de créer, de taille toujours égale à 1.
Il doit surement exister une méthode d’InventoryPlayer permettant de compter un item, sinon tu dois parcourir l’inventaire et incrémenter une variable pour chaque billet trouvé.

Flow

@AymericRed:

Le principe est le bon, sauf que count6 ne correspond pas au nombre de billets dans l’inventaite, mais juste à un ItemStack que tu viens de créer, de taille toujours égale à 1.
Il doit surement exister une méthode d’InventoryPlayer permettant de compter un item, sinon tu dois parcourir l’inventaire et incrémenter une variable pour chaque billet trouvé.

Yep je cherche mais je ne trouve pas

LeBossMax2

Dans le cas où funds est négatif, cela ne sert à rien de compter le nombre de Billets. Un simple player.inventory.hasItemStack(stack6) suffit. Par contre, c’est quand funds est positif qu’il faut regarder si le joueur à bien assez d’argent sur sa carte pour pouvoir lui give l’item.