Texture item

[Deleted]

Non, les hit n’ont rien à voir avec les coordonnées du block sur lequel le joueur frappe. On a un BlockPos en paramètre, il est + qu’évident qu’il faut get la TE avec, et non refaire une nouvelle instance.

[Dylem]

@‘Plaigon’:

Non, les hit n’ont rien à voir avec les coordonnées du block sur lequel le joueur frappe. On a un BlockPos en paramètre, il est + qu’évident qu’il faut get la TE avec, et non refaire une nouvelle instance.

ah oui juste, autant pour moi, j’ai regardé en diagonale

[Sebenf0rce]

Merci j’ai modifié le code en suivant ce que vous m’avez dis, du coup le TileEntity n’est plus nul et contient le chest.
Par contre j’ai un soucis avec le openGui ça n’affiche rien.

Voici mon code maintenant :

@Override
    public EnumActionResult onItemUse(EntityPlayer player, World world, BlockPos pos, EnumHand hand, EnumFacing facing, float hitX, float hitY, float hitZ)
    {
if (!world.isRemote) {
System.out.println("test1");
TileEntity tile = world.getTileEntity(pos);
System.out.println(tile); //renvoie toujours null

if (tile == null) {
return EnumActionResult.PASS;
}
if(tile instanceof TileEntityChest) {
System.out.println("test2");
player.openGui(Main.instance, 13, world, pos.getX(), pos.getY(), pos.getZ());
return EnumActionResult.SUCCESS;
}
}
return EnumActionResult.PASS;
    }

(le test2 est bien affiché)

[BrokenSwing]

Tu as quoi dans ton GuiHandler ?

[Deleted]

Sûrement un problème avec ton GuiHandler. File nous sa classe stp

[Sebenf0rce]

J’utilise pas de GuiHandler perso.
J’utilise directement la méthode de la classe player :

/**
    * Opens a GUI with this player, uses FML's IGuiHandler system.
    * Allows for extension by modders.
    *
    * @param mod The mod trying to open a GUI
    * @param modGuiId GUI ID
    * @param world Current World
    * @param x Passed directly to IGuiHandler, data meaningless Typically world X position
    * @param y Passed directly to IGuiHandler, data meaningless Typically world Y position
    * @param z Passed directly to IGuiHandler, data meaningless Typically world Z position
    */
   public void openGui(Object mod, int modGuiId, World world, int x, int y, int z)

C’est au niveau des position où je ne sais pas trop quoi mettre. En 1.7.10 je l’avais utilisé mais il y avait des paramètres supplémentaires (x,y et z)

[Deleted]

En fait t’as pas vraiment le choix :x

[BrokenSwing]

Cette fonction est une fonction de forge qui redirige vers le GuiHandler du mod passé en paramètres. Si tu veux utiliser la fonction de Minecraft, il te faut utiliser : player.displayGUIChest

[Sebenf0rce]

D’accord autant pour moi.
Du coup j’ai mis mon gui cependant ça plante. Voici ma classe GuiHandler :

package fr.sebenforce.mod;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.util.math.BlockPos;
import net.minecraft.world.World;
import net.minecraftforge.fml.common.network.IGuiHandler;

public class GuiHandler implements IGuiHandler {

public Object getServerGuiElement(int ID, EntityPlayer player, World world, int x, int y, int z) {
switch (ID) {
case 0:
return new ContainerChestExplorer(world.getTileEntity(new BlockPos(x, y, z)));
}
return null;
}

public Object getClientGuiElement(int ID, EntityPlayer player, World world, int x, int y, int z) {
switch (ID) {
case 0:
return new GuiChestExplorer(world.getTileEntity(new BlockPos(x, y, z)));
}
return null;
}
}

Et voici le crash report :

https://pastebin.com/9TtG71MD

[AymericRed]

Montres ta classe “ContainerChestExplorer” car à la vue du crash report je pense que tu as spécifié un Inventory null pour un des slots.

[Sebenf0rce]

@‘AymericRed’:

Montres ta classe “ContainerChestExplorer” car à la vue du crash report je pense que tu as spécifié un Inventory null pour un des slots.

Oui la voici :

 package fr.sebenforce.mod;

 import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.inventory.Container;
import net.minecraft.inventory.IInventory;
import net.minecraft.item.ItemStack;
import net.minecraft.tileentity.TileEntity;

 public class ContainerChestExplorer extends Container
 {
   IInventory inventory;

  public ContainerChestExplorer(TileEntity tile)
   {
    this.inventory = ((IInventory)tile);

    int i = 0;
    int j = 0;

    for (i = 0; i < 108; i++) {
      if (i % 12 == 0)
         j++;
      int u = i % 12 + 1;
      if (i < this.inventory.getSizeInventory()) {
         addSlotToContainer(new SlotChestExplorer(this.inventory, i, u * 18 - 6, j * 18 - 10));
      } else {
        addSlotToContainer(new SlotChestExplorer(new InventoryDummy(), 0, u * 18 - 6, j * 18 - 10));
       }
    }
   }

   public boolean canInteractWith(EntityPlayer player) {
    return this.inventory.isUsableByPlayer(player);
  }

  public ItemStack slotClick(int slotIndex, int buttonPressed, int flag, EntityPlayer player)
   {
     return null;
   }

   public ItemStack transferStackInSlot(EntityPlayer player, int quantity)
   {
     return null;
  }
 }

[Sebenf0rce]

Personne a une idée ?

[AymericRed]

Ah non, ce n’est pas un Inventory null mais un ItemStack qui est null, en 1.11 les item stack ne peuvent pas être null, il faut remplacer cela par ItemStack.EMPTY donc passes en revue la classe de ton TileEntity et la classe “InventoryDummy” pour remplacer tout ce qui ferait référence à des ItemStack null.
PS : pour savoir si un stack est vide (ou avant si un stack était null), c’est “lestack.isEmpty()”.

[Sebenf0rce]

@AymericRed:

Ah non, ce n’est pas un Inventory null mais un ItemStack qui est null, en 1.11 les item stack ne peuvent pas être null, il faut remplacer cela par ItemStack.EMPTY donc passes en revue la classe de ton TileEntity et la classe “InventoryDummy” pour remplacer tout ce qui ferait référence à des ItemStack null.
PS : pour savoir si un stack est vide (ou avant si un stack était null), c’est “lestack.isEmpty()”.

Merci, ça ne crash plus désormais cependant mon gui ne s’affiche pas et je ne comprends pas pourquoi.

J’ai bien ma méthode onItemUse qui appel le gui :

@Override
public EnumActionResult onItemUse(EntityPlayer player, World world, BlockPos pos, EnumHand hand, EnumFacing facing, float hitX, float hitY, float hitZ)
{
if (!world.isRemote) {
System.out.println("test1");
TileEntity tile = world.getTileEntity(pos);
System.out.println(tile); //renvoie toujours null

if (tile == null) {
return EnumActionResult.PASS;
}
if(tile instanceof TileEntityChest) {
System.out.println("test2");
player.openGui(Main.instance, 0, world, pos.getX(), pos.getY(), pos.getZ());
return EnumActionResult.SUCCESS;
}
}
return EnumActionResult.PASS;
}

Et mon guiHandler :

public class GuiHandler implements IGuiHandler {

@Override
public Object getServerGuiElement(int ID, EntityPlayer player, World world, int x, int y, int z) {
switch (ID) {
case 0:
return new ContainerChestExplorer(world.getTileEntity(new BlockPos(x, y, z)));
}
return null;
}

@Override
public Object getClientGuiElement(int ID, EntityPlayer player, World world, int x, int y, int z) {
switch (ID) {
case 0:
return new GuiChestExplorer(world.getTileEntity(new BlockPos(x, y, z)));
}
return null;
}
}

[AymericRed]

Vérifies que la fonction isUseableByPlayer de ton TileEntity retourne bien true et ensuite mets des prints/du debug dans les fonctions de draw, d’init et le constructeur du gui pour voir où ça bloque.

[Sebenf0rce]

J’ai dû rater un truc, il faut un tileEntity pour un item ? Je pensais que c’était juste pour les blocs.

[AymericRed]

Euh…. tu sais ce que tu veux faire au moins, tu connais ton code ?
T’as un TileEntity ici :

public ContainerChestExplorer(TileEntity tile)
``` et ici aussi ```java
return this.inventory.isUsableByPlayer(player);

Bon il se trouve que c’est le TileEntity du coffre vanilla donc devrait pas poser de problème mais je te conseillerait de vérifier si sa fonction “isUsableByPlayer” retourne bien true à ton Container.

[Sebenf0rce]

Comment je peux tester la fonction isUsableByPlayer car il faut y passer en paramètre le player mais je n’y ai pas accès dans ma classe ContainerChestExplorer.

J’ai remarqué également en faisant différent test que le programme ne rentre jamais dans ma fonction
public boolean canInteractWith(EntityPlayer player) {
return this.inventory.isUsableByPlayer(player);
}

Est-ce normal ?

[AymericRed]

Tu y as accès dans canInteractWith.

Non mais ça va avec le fait que ton gui s’ouvre pas.
@AymericRed:

[…] et ensuite mets des prints/du debug dans les fonctions de draw, d’init et le constructeur du gui pour voir où ça bloque.

Il te reste à faire ça

[Sebenf0rce]

Salut,

Du coup j’ai mis différents print :

Classe GuiHandler :

public class GuiHandler implements IGuiHandler {

@Override
public Object getServerGuiElement(int ID, EntityPlayer player, World world, int x, int y, int z) {
switch (ID) {
case 0:
System.out.println(" création container chest explorer");
return new ContainerChestExplorer(world.getTileEntity(new BlockPos(x, y, z)));
}
return null;
}

@Override
public Object getClientGuiElement(int ID, EntityPlayer player, World world, int x, int y, int z) {
switch (ID) {
case 0:
System.out.println(" création gui chest explorer");
return new GuiChestExplorer(world.getTileEntity(new BlockPos(x, y, z)));
}
return null;
}
}

Les deux print s’affiche bien.

Classe ContainerChestExplorer :

public class ContainerChestExplorer extends Container
 {
   IInventory inventory;

  public ContainerChestExplorer(TileEntity tile)
   {
 System.out.println("constructeur container");
    this.inventory = ((IInventory)tile);

    int i = 0;
    int j = 0;

    for (i = 0; i < 108; i++) {
      if (i % 12 == 0)
         j++;
      int u = i % 12 + 1;
      if (i < this.inventory.getSizeInventory()) {
         addSlotToContainer(new SlotChestExplorer(this.inventory, i, u * 18 - 6, j * 18 - 10));
      } else {
        addSlotToContainer(new SlotChestExplorer(new InventoryDummy(), 0, u * 18 - 6, j * 18 - 10));
       }
    }
   }

   public boolean canInteractWith(EntityPlayer player) {
  System.out.println("AAAAAAAAAAAAAAA" + this.inventory.isUsableByPlayer(player));
    return this.inventory.isUsableByPlayer(player);
  }

  public ItemStack slotClick(int slotIndex, int buttonPressed, int flag, EntityPlayer player)
   {
     return null;
   }

   public ItemStack transferStackInSlot(EntityPlayer player, int quantity)
   {
     return null;
  }
 }

Le print du constructeur s’affiche bien mais celui de la méthode de canInteractWith n’apparait pas et je ne comprends pas pourquoi.

Classe GuiChestExplorer :

public class GuiChestExplorer extends GuiContainer
 {
  TileEntity tile;
   ResourceLocation bg = new ResourceLocation("sebenforcemod:textures/gui/chestexplorer.png");

   public GuiChestExplorer(TileEntity te) {
    super(new ContainerChestExplorer(te));
    System.out.println("constructeur gui");
     this.xSize = 256;
    this.ySize = 177;
   }

  protected void drawGuiContainerBackgroundLayer(float p_146976_1_, int p_146976_2_, int p_146976_3_)
   {
    this.mc.renderEngine.bindTexture(this.bg);
    System.out.println("draw gui");
     drawDefaultBackground();
     GL11.glColor4f(1.0F, 1.0F, 1.0F, 1.0F);
     int x = (this.width - this.xSize) / 2;
     int y = (this.height - this.ySize) / 2;
     drawTexturedModalRect(x, y, 0, 0, 256, 256);
   }
 }

Les deux print s’affichent bien.

Est ce qu’il est possible que le gui du coffre de base s’affiche par dessus le mien ?