[1.12.2] problème avec mon GUI

robin4002

Le problème ne se trouve pas dans ton bloc mais dans ton tile entity.

TheXrayFR

package fr.thexrayfr.familygame.tile;
 
 import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.inventory.ItemStackHelper;
 import net.minecraft.item.ItemStack;
 import net.minecraft.nbt.NBTTagCompound;
 
 public class TileEntityBarrel extends TileEntityStorage
 {
   public int getSizeInventory()
   {
     return 18;
   }

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

     @Override
     public ItemStack getStackInSlot(int index) {
         return null;
     }

     @Override
     public ItemStack decrStackSize(int index, int count) {
         return null;
     }

     @Override
     public ItemStack removeStackFromSlot(int index) {
         return null;
     }

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

     @Override
     public boolean isUsableByPlayer(EntityPlayer player) {
         return false;
     }

     @Override
     public void openInventory(EntityPlayer player) {

     }

     @Override
     public void closeInventory(EntityPlayer player) {

     }

     @Override
     public boolean isItemValidForSlot(int index, ItemStack stack) {
         return false;
     }

     @Override
     public int getField(int id) {
         return 0;
     }

     @Override
     public void setField(int id, int value) {

     }

     @Override
     public int getFieldCount() {
         return 0;
     }

     @Override
     public void clear() {

     }


     public void readFromNBT(NBTTagCompound compound)
   {
     super.readFromNBT(compound);
     ItemStackHelper.loadAllItems(compound, this.stacks);
   }
   
 
   public NBTTagCompound writeToNBT(NBTTagCompound compound)
   {
     ItemStackHelper.saveAllItems(compound, this.stacks);
     return super.writeToNBT(compound);
   }

     @Override
     public String getName() {
         return null;
     }

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

il ma demander quand je lais mise a joue d’importer ca donc apret

robin4002

TileEntityStorage contient quoi ?

Et tu as pleins de fonction non implémenté qui retourne null, ce n’est pas bon. getStackInSlot, decrStackSize, removeStackFromSlot etc.

TheXrayFR

@robin4002

package fr.thexrayfr.familygame.tile;
 
 import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.inventory.IInventory;
 import net.minecraft.inventory.ItemStackHelper;
 import net.minecraft.item.ItemStack;
 import net.minecraft.tileentity.TileEntity;
 import net.minecraft.util.NonNullList;

 
 public abstract class TileEntityStorage extends TileEntity implements IInventory
 {

   public NonNullList<ItemStack> stacks = NonNullList.withSize(getSizeInventory(), ItemStack.EMPTY);

   public String getName()
   {
    return null;
   }

   public boolean hasCustomName()
   {
     return false;
   }
   
   public boolean isEmpty()
   {
     for (ItemStack itemstack : this.stacks)
     {
       if (!itemstack.isEmpty())
       {
         return false;
       }
     }
     
     return true;
   }
 
   public ItemStack getStackInSlot(int index)
   {
     return (ItemStack)this.stacks.get(index);
   }

   public ItemStack decrStackSize(int index, int count)
   {
     return ItemStackHelper.getAndSplit(this.stacks, index, count);
   }

   public ItemStack removeStackFromSlot(int index)
   {
     this.stacks.set(index, ItemStack.EMPTY);
     return getStackInSlot(index);
   }

   public void setInventorySlotContents(int index, ItemStack stack)
   {
     this.stacks.set(index, stack);
   }

   public int getInventoryStackLimit()
   {
     return 64;
   }

   public boolean isUsableByPlayer(EntityPlayer player)
   {
     if (this.world.getTileEntity(this.pos) != this)
     {
       return false;
     }

     return player.getDistanceSq(this.pos.getX() + 0.5D, this.pos.getY() + 0.5D, this.pos.getZ() + 0.5D) <= 64.0D;
   }

   public void openInventory(EntityPlayer player) {}

   public void closeInventory(EntityPlayer player) {}

   public boolean isItemValidForSlot(int index, ItemStack stack)
   {
     return true;
   }

   public int getField(int id)
   {
     return 0;
   }
   public void setField(int id, int value) {}

   public int getFieldCount()
   {
     return 0;
   }

   public void clear()
   {
     this.stacks.clear();
   }
 }

TheXrayFR

jais fait le ménage

package fr.thexrayfr.familygame.tile;
 
 import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.inventory.ItemStackHelper;
 import net.minecraft.item.ItemStack;
 import net.minecraft.nbt.NBTTagCompound;
 
 public class TileEntityBarrel extends TileEntityStorage
 {
     
   public int getSizeInventory()
   {
     return 18;
   }

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

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

     @Override
     public boolean isUsableByPlayer(EntityPlayer player) {
         return false;
     }

     @Override
     public void openInventory(EntityPlayer player) {

     }

     @Override
     public void closeInventory(EntityPlayer player) {

     }

     @Override
     public boolean isItemValidForSlot(int index, ItemStack stack) {
         return false;
     }

     @Override
     public int getField(int id) {
         return 0;
     }

     @Override
     public void setField(int id, int value) {

     }

     @Override
     public int getFieldCount() {
         return 0;
     }

     @Override
     public void clear() {

     }

     public void readFromNBT(NBTTagCompound compound)
   {
     super.readFromNBT(compound);
     ItemStackHelper.loadAllItems(compound, this.stacks);
   }
   
 
   public NBTTagCompound writeToNBT(NBTTagCompound compound)
   {
     ItemStackHelper.saveAllItems(compound, this.stacks);
     return super.writeToNBT(compound);
   }

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

TheXrayFR

déja sa crash plus sa ouvre et referme le gui

TheXrayFR

2019-01-16 20-54-15.zip

robin4002

Ta fonction isUsableByPlayer retourne toujours faux, c’est pour ça que cela ne fonctionne pas.

TheXrayFR

une petite vidéo pour te montrer se qui se passe

TheXrayFR

ses bon le hui s’ouvre mes maintenant ses le snift clique qui pose problème

TheXrayFR

@robin4002

ses pas du au:

public ItemStack transferStackInSlot(EntityPlayer playerIn, int index) {
     ItemStack itemstack = ItemStack.EMPTY;
     Slot slot = (Slot) this.inventorySlots.get(index);

     if ((slot != null) && (slot.getHasStack())) {
       ItemStack itemstack1 = slot.getStack();
       itemstack = itemstack1.copy();

       if (index < 9) {
         if (!mergeItemStack(itemstack1, 18, this.inventorySlots.size(), true)) {
           return ItemStack.EMPTY;
         }
       } else if (!mergeItemStack(itemstack1, 0, 18, false)) {
         return ItemStack.EMPTY;
       }

       if (itemstack1.isEmpty()) {
         slot.putStack(ItemStack.EMPTY);
       } else {
         slot.onSlotChanged();
       }
     }

     return itemstack;
   }

???

robin4002

Si c’est bien cette fonction qui gère le shift clic.

TheXrayFR

@robin4002
je comprent pas pourquoi il bug quand je shift click sa crash

TheXrayFR

@robin4002 tu ses a peut prêt dout sa peut être ?

robin4002

Tu peux envoyer le rapport de crash ?

TheXrayFR

@robin4002 il en na pas

robin4002

Si cela crash il y a forcément quelque chose dans les logs.

TheXrayFR

@robin4002
2019-01-16 21-58-41.zip

de meme je peut même pas mètre d’objet a l’interrieur

 package fr.thexrayfr.familygame.container;
 
 import fr.thexrayfr.familygame.tile.TileEntityBarrel;
 import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.entity.player.InventoryPlayer;
 import net.minecraft.inventory.Container;
 import net.minecraft.inventory.Slot;
 import net.minecraft.item.ItemStack;
 
 public class ContainerBarrel extends Container
 {
   private final TileEntityBarrel tileTuto;
   
   public ContainerBarrel(TileEntityBarrel tile, InventoryPlayer inventory)
   {

     this.tileTuto = tile;
     tile.openInventory(inventory.player);

     for (int i = 0; i < 6; i++)
     {
       for (int j = 0; j < 3; j++)
       {
         addSlotToContainer(new Slot(tile, j + i * 3, 62 + j * 18, 18 + i * 18));
       }
     }
     bindPlayerInventory(inventory);
   }
 
   private void bindPlayerInventory(InventoryPlayer inventory)
   {
     for (int i = 0; i < 3; i++)
     {
       for (int j = 0; j < 9; j++)
       {
         addSlotToContainer(new Slot(inventory, j + i * 9 + 9, 8 + j * 18, 138 + i * 18));
       }
     }

     for (int i = 0; i < 9; i++)
     {
       addSlotToContainer(new Slot(inventory, i, 8 + i * 18, 196));
     }
   }

   public ItemStack transferStackInSlot(EntityPlayer playerIn, int index) {
     ItemStack itemstack = ItemStack.EMPTY;
     Slot slot = (Slot) this.inventorySlots.get(index);

     if ((slot != null) && (slot.getHasStack())) {
       ItemStack itemstack1 = slot.getStack();
       itemstack = itemstack1.copy();

       if (index < 9) {
         if (!mergeItemStack(itemstack1, 18, this.inventorySlots.size(), true)) {
           return ItemStack.EMPTY;
         }
       } else if (!mergeItemStack(itemstack1, 0, 18, false)) {
         return ItemStack.EMPTY;
       }

       if (itemstack1.isEmpty()) {
         slot.putStack(ItemStack.EMPTY);
       } else {
         slot.onSlotChanged();
       }
     }

     return itemstack;
   }
 
   public boolean canInteractWith(EntityPlayer player)
   {
     return this.tileTuto.isUsableByPlayer(player);
   }
   
 
   public void onContainerClosed(EntityPlayer player)
   {
     super.onContainerClosed(player);
     this.tileTuto.closeInventory(player);
   }
 }

TheXrayFR

dans le TileEntity

 package fr.thexrayfr.familygame.tile;

 import net.minecraft.inventory.ItemStackHelper;
 import net.minecraft.nbt.NBTTagCompound;
 
 public class TileEntityBarrel extends TileEntityStorage
 {
     @Override
   public int getSizeInventory()
   {
     return 18;
   }

     @Override
   public void readFromNBT(NBTTagCompound compound)
   {
     super.readFromNBT(compound);
     ItemStackHelper.loadAllItems(compound, this.stacks);
   }

     @Override
   public NBTTagCompound writeToNBT(NBTTagCompound compound)
   {
     ItemStackHelper.saveAllItems(compound, this.stacks);
     return super.writeToNBT(compound);
   }

 }

Dans le contenair

 package fr.thexrayfr.familygame.container;
 
 import fr.thexrayfr.familygame.tile.TileEntityBarrel;
 import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.entity.player.InventoryPlayer;
 import net.minecraft.inventory.Container;
 import net.minecraft.inventory.Slot;
 import net.minecraft.item.ItemStack;
 
 public class ContainerBarrel extends Container
 {
   private final TileEntityBarrel tileTuto;
   
   public ContainerBarrel(TileEntityBarrel tile, InventoryPlayer inventory)
   {

     this.tileTuto = tile;
     tile.openInventory(inventory.player);

     for (int i = 0; i < 6; ++i)
     {
       for (int j = 0; j < 3; ++j)
       {
         this.addSlotToContainer(new Slot(tile, j + i * 3, 62 + j * 18, 18 + i * 18));
       }
     }
     this.bindPlayerInventory(inventory);
   }
 
   private void bindPlayerInventory(InventoryPlayer inventory)
   {
     int i;
     for (i = 0; i < 3; ++i)
     {
       for (int j = 0; j < 9; ++j)
       {
         this.addSlotToContainer(new Slot(inventory, j + i * 9 + 9, 8 + j * 18, 138 + i * 18));
       }
     }

     for (i = 0; i < 9; ++i)
     {
       this.addSlotToContainer(new Slot(inventory, i, 8 + i * 18, 196));
     }
   }

   @Override
   public ItemStack transferStackInSlot(EntityPlayer playerIn, int index) {
     ItemStack itemstack = ItemStack.EMPTY;
     Slot slot = (Slot) this.inventorySlots.get(index);

     if (slot != null && slot.getHasStack()) {
       ItemStack itemstack1 = slot.getStack();
       itemstack = itemstack1.copy();

       if (index < 9) {
         if (!this.mergeItemStack(itemstack1, 18, this.inventorySlots.size(), true)) {
           return ItemStack.EMPTY;
         }
       } else if (!this.mergeItemStack(itemstack1, 0, 18, false)) {
         return ItemStack.EMPTY;
       }

       if (itemstack1.isEmpty())
       {
         slot.putStack(ItemStack.EMPTY);
       }
       else
       {
         slot.onSlotChanged();
       }
     }

     return itemstack;
   }

   @Override
   public boolean canInteractWith(EntityPlayer player)
   {
     return this.tileTuto.isUsableByPlayer(player);
   }

   @Override
   public void onContainerClosed(EntityPlayer player)
   {
     super.onContainerClosed(player);
     this.tileTuto.closeInventory(player);
   }
 }

se que j’ai changer ses les ++i et ++j qui étai i++ et j++ donc je les ais inversser