SOLVED Crash Null Pointer Exception

1.7.x
Log in to reply
  • sosoh

    Salut, j’ai mis ce code dans mon container, j’ai fait un Gui, mais je voudrais que lorsqu’on met un sceau d’eau dans un slot (le n°1), ca le vide: ca change le contenu par un sceau vide, mais sans passer par un bouton ou autre.:

    ​public void detectAndSendChanges()

    {
if(tileEntity.getStackInSlot(1).getItem() == Items.water_bucket){
tileEntity.setContenu(1);
tileEntity.setInventorySlotContents(1, new ItemStack(Items.bucket));
}
    }

    mais je crash:

    [22:44:02] [Server thread/ERROR]: Encountered an unexpected exception

    net.minecraft.util.ReportedException: Ticking memory connection
    at net.minecraft.network.NetworkSystem.networkTick(NetworkSystem.java:198) ~[NetworkSystem.class:?]
    at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:726) ~[MinecraftServer.class:?]
    at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:614) ~[MinecraftServer.class:?]
    at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:118) ~[IntegratedServer.class:?]
    at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:485) [MinecraftServer.class:?]
    at net.minecraft.server.MinecraftServer$2.run(MinecraftServer.java:752) [MinecraftServer$2.class:?]
    Caused by: java.lang.NullPointerException
    at com.harrypotter.sosoh.common.container.ContainerChaudron.detectAndSendChanges(ContainerChaudron.java:107) ~[ContainerChaudron.class:?]
    at net.minecraft.inventory.Container.addCraftingToCrafters(Container.java:54) ~[Container.class:?]
    at cpw.mods.fml.common.network.internal.FMLNetworkHandler.openGui(FMLNetworkHandler.java:88) ~[FMLNetworkHandler.class:?]
    at com.harrypotter.sosoh.common.blocks.BlockChaudron.onBlockActivated(BlockChaudron.java:69) ~[BlockChaudron.class:?]
    at net.minecraft.server.management.ItemInWorldManager.activateBlockOrUseItem(ItemInWorldManager.java:409) ~[ItemInWorldManager.class:?]
    at net.minecraft.network.NetHandlerPlayServer.processPlayerBlockPlacement(NetHandlerPlayServer.java:593) ~[NetHandlerPlayServer.class:?]
    at net.minecraft.network.play.client.C08PacketPlayerBlockPlacement.processPacket(C08PacketPlayerBlockPlacement.java:74) ~[C08PacketPlayerBlockPlacement.class:?]
    at net.minecraft.network.play.client.C08PacketPlayerBlockPlacement.processPacket(C08PacketPlayerBlockPlacement.java:122) ~[C08PacketPlayerBlockPlacement.class:?]
    at net.minecraft.network.NetworkManager.processReceivedPackets(NetworkManager.java:241) ~[NetworkManager.class:?]
    at net.minecraft.network.NetworkSystem.networkTick(NetworkSystem.java:182) ~[NetworkSystem.class:?]
    … 5 more

    0
  • Kujaroth

    Essais :

    public void detectAndSendChanges()

   {
if(tileEntity.getStackInSlot(1).getItem() == Items.water_bucket && tileEntity.getStackInSlot(1).getItem() != null){
tileEntity.setContenu(1);
tileEntity.setInventorySlotContents(1, new ItemStack(Items.bucket));
}
   }

    EDIT : Celui la fonctionnera mieux ^^

    public void detectAndSendChanges()
  {
if(tileEntity.getStackInSlot(1).getItem() != null && tileEntity.getStackInSlot(1).getItem() == Items.water_bucket){
tileEntity.setContenu(1);
tileEntity.setInventorySlotContents(1, new ItemStack(Items.bucket));
}
  }
    0
  • jglrxavpok
    Modérateurs

    Unknown, faudrait juste inverser les conditions de ton if pour que ce soit bon 😉

    Sent from my GT-I9000 using Tapatalk 2

    0
  • Kujaroth

    Effectivement…

    Dans le bon sens c’est mieux 😑  Ca semble plus logique aussi x)

    0
  • sosoh

    Je comprends pas pourquoi il faut Check si il n’est pas null on peut m’expliquer ?

    0
  • robin4002
    Administrateurs Rédacteurs Moddeurs confirmés

    Non, le null check n’ai pas bon. Ça ne devrait pas être :

    tileEntity.getStackInSlot(1).getItem() != null

    mais :

    tileEntity.getStackInSlot(1) != null

    Et il faut faire un null check pour la semble et bonne raison que si il n’y a pas d’item dans un slot, le contenu est null. Or si tu appelle une fonction depuis un objet null ->  NPE

    0