Problème IGUIHandler

[AymericRed]

Heu….
T’a bien relu ton code ?
“if(player.getHeldItem() == null) {”
T’es en train d’écrire dans la nbt uniquement si l’item que tu tiens null, faut que tu mettes != null

[Gabs]

Je crash pas mais ça save pas mon item quand je quitte le container, après ça vient peut-etre pas d’ici

[AymericRed]

Oui ça vient d’ici parce que si tu tiens un item, ta condition retournera toujours faux

[Gabs]

Bah je peux le remplacer par quoi?

[Gabs]

Ouais c’est bon^^ par contre petit soucis quand je le met un item dans mon inv et que je referme il drop fin en gros il reste pas dans le slot .
merci d’ailleurs

[Gabs]

Help?

[AymericRed]

Ah dsl, voulais revenir et ai oublié, ton inv, c’est birn un backpack ?
Et met tt ton code en lien avec ça

[Gabs]

Pas grave ^^, Bah en fait j’ai repris les code du backpack de SCAREX mais moi je veux en faire un inventaire, peut-être pour ça que y’a des soucis^^ non?
Birn = ?

[AymericRed]

Il faut que tu saves bien et load bien dans les nbt de l’items et normalement, y a pas de raison que ça loot les items.
Montre ta classe InventoryArma, car il y a une fonction qui pourrait causer ça.

[Gabs]

D’acc je fais ça ce soir tien pour la class inv:

package armacraft.net.GUI;

import armacraft.net.Items.Water;
import net.minecraft.client.Minecraft;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.inventory.IInventory;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.nbt.NBTTagList;
import net.minecraftforge.common.util.Constants;

public class InventaireArma implements IInventory
{
public ItemStack[] content;
public int size;

public InventaireArma(ItemStack container, int size) {
this.size = size;
this.content = new ItemStack;
}

/**
* This methods reads the content of the NBTTagCompound inside the container
* 
* @param comp
*            the container NBTTagCompound
*/
public void readFromNBT(NBTTagCompound comp) {
NBTTagList nbtlist = comp.getTagList("a", Constants.NBT.TAG_COMPOUND);
for (int i = 0; i < nbtlist.tagCount(); i++) {
NBTTagCompound comp1 = nbtlist.getCompoundTagAt(i);
int slot = comp1.getInteger("z");
this.content[slot] = ItemStack.loadItemStackFromNBT(comp1);
}
}

/**
* This methods saves the content inside the container
* 
* @param comp
*            the NBTTagCompound to write in
*/
public void writeToNBT(NBTTagCompound comp) {
NBTTagList nbtlist = new NBTTagList();

for (int i = 0; i < this.size; i++) {
if (this.content* != null) {
NBTTagCompound comp1 = new NBTTagCompound();
comp1.setInteger("Slot", i);
this.content*.writeToNBT(comp1);
nbtlist.appendTag(comp1);
}
}
comp.setTag("e", nbtlist);
}

@Override
public int getSizeInventory() {
return this.size;
}

@Override
public ItemStack getStackInSlot(int index) {
return this.content[index];
}

@Override
public ItemStack decrStackSize(int index, int amount) {
ItemStack stack = getStackInSlot(index);
if (stack != null) {
if (stack.stackSize > amount) {
stack = stack.splitStack(amount);
if (stack.stackSize == 0) this.content[index] = null;
} else {
this.content[index] = null;
}
}
return stack;
}

@Override
public ItemStack getStackInSlotOnClosing(int index) {
ItemStack stack = getStackInSlot(index);
if (stack != null) this.content[index] = null;
return stack;
}

@Override
public void setInventorySlotContents(int index, ItemStack stack) {
this.content[index] = stack;
}

@Override
public String getInventoryName() {
return "amm" + ".inventaire.arma";
}

@Override
public boolean hasCustomInventoryName() {
return false;
}

@Override
public int getInventoryStackLimit() {
return 64;
}

@Override
public void markDirty() {}

@Override
public boolean isUseableByPlayer(EntityPlayer player) {
return true;
}

@Override
public void openInventory() {}

@Override
public void closeInventory() {
}

/**
* Prevents backpack-ception
*/
@Override
public boolean isItemValidForSlot(int index, ItemStack stack) {
return !(stack.getItem() instanceof Water);
}

}

[AymericRed]

ça m’a l’air d’être bon, je vérifierais en détail tout à l’heure en regardant comment la fermeture des container est gérée. Vérifies juste que tes méthodes read et write soient bien appelée (clic droit sur la fonction –> Call hierarchy).
Et je précise que tu tu compliques la vie dans des fonctions de nbt, tu pourrais directement utiliser le tag pasé en paramètre pour set l’id du slot et sont contenu au lieu de passé pas un tag que tu mets dans une liste que tu mets dans le tag d’origine.

[Gabs]

D’accord merci,
[font=Ubuntu, sans-serifJe préfère pas toucher car je connais pas les container et tout si je commence a touché ça va crash et tout donc je préfère laisser comme ça pour le moment du moment que ça marche ça me dérange pas x) .]

[font=Ubuntu, sans-serifEDIT:]

[font=Ubuntu, sans-serifAlors,]

[font=Ubuntu, sans-serifdans le read c’est vide il est pas appeler et dans le write il est juste appeler dans la class container quand on close dans la fonction “onContainerClosed” .]

[AymericRed]

Ton container, il est ouvert avec ton item ?
Donc il faut que tu récupères le held item du joueur, que tu vérifies si il a des tags nbt et si oui, t’appelle ta fonction read avec ces tags.

[Gabs]

Nan il est ouvert avec un keybinding ^^, Je dois faire comment quand c’est comme ça ?

Edit:
J’ai essayer ça:

private void keyTestTyped()
{
EntityPlayer player = Minecraft.getMinecraft().thePlayer;
World world = player.worldObj;

if (stack.hasTagCompound()) stack.setTagCompound(new NBTTagCompound()); // ici la ligne qui cause prob.
inv.readFromNBT(stack.getTagCompound());
player.openGui(AltisMineMod.INSTANCE, 0, world, (int) player.posX, (int) player.posY, (int) player.posZ);
}

J’ai un crash ^^

[AymericRed]

Non mais en fait ce qu’il faut dans ce cas où l’inventaire n’est pas dans un item, c’est faire un extended entity Properties (je n’en ai jamais fait mais y a un tuto je crois)

[Gabs]

Waa le bordel ^^ ehh t’aurais le lien ou des mots clé pour que je trouve ? se serrai super

[AymericRed]

Tiens

[Gabs]

Ah ouais merci, j’avais déjà utilisé ça, par contre a la place je dois prendre mon inv c’est ça ? alors je dois prendre le joueur et dans le packet j’ouvre l’inv au joueur .

Sinon y’a pas un moyen plus simplement normalement on a pas besoin de ça j’crois ^^

[AymericRed]

Si tu n’as pas d’item ou de bloc pour le sauvegarder, t’es obligé de passer pas la

[Gabs]

D’acc bon bah je vais essayer merci je te dis si j’ai besoin d’aide juste tu peux répondre a la question juste au desus ?