Extended Entity Properties

[bin4r1s]

Bonjour,
J’ai un problème : Quand je rejoint le monde, je me fais éjecter avec un beau message, je cite : “A fatal error has occured, this connection is terminated” (Littéralement : “Une erreur fatale est survenue, cette connection est terminée”). Je regarde la console d’eclipse, qui m’indique que j’ai un NPE dans la fonction loadNBTData, sur mes "getShort(“gnagnagna”). POur voir si c’est bien l’origine du problème, je les retire, mais là ( la même chose, sauf que mes classes ne figurent plus dans le log) :

 
io.netty.handler.codec.DecoderException: java.lang.NullPointerException
 
at io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:99) ~[MessageToMessageDecoder.class:?]
at io.netty.handler.codec.MessageToMessageCodec.channelRead(MessageToMessageCodec.java:111) ~[MessageToMessageCodec.class:?]
at io.netty.channel.DefaultChannelHandlerContext.invokeChannelRead(DefaultChannelHandlerContext.java:337) [DefaultChannelHandlerContext.class:?]
at io.netty.channel.DefaultChannelHandlerContext.fireChannelRead(DefaultChannelHandlerContext.java:323) [DefaultChannelHandlerContext.class:?]
at io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:785) [DefaultChannelPipeline.class:?]
at io.netty.channel.embedded.EmbeddedChannel.writeInbound(EmbeddedChannel.java:169) [EmbeddedChannel.class:?]
at cpw.mods.fml.common.network.internal.FMLProxyPacket.processPacket(FMLProxyPacket.java:86) [FMLProxyPacket.class:?]
at net.minecraft.network.NetworkManager.processReceivedPackets(NetworkManager.java:241) [NetworkManager.class:?]
at net.minecraft.client.Minecraft.runTick(Minecraft.java:2141) [Minecraft.class:?]
at net.minecraft.client.Minecraft.runGameLoop(Minecraft.java:1028) [Minecraft.class:?]
at net.minecraft.client.Minecraft.run(Minecraft.java:951) [Minecraft.class:?]
at net.minecraft.client.main.Main.main(Main.java:164) [Main.class:?]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_11]
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_11]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_11]
at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_11]
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135) [launchwrapper-1.11.jar:?]
at net.minecraft.launchwrapper.Launch.main(Launch.java:28) [launchwrapper-1.11.jar:?]
at net.minecraftforge.gradle.GradleStartCommon.launch(GradleStartCommon.java:78) [start/:?]
at GradleStart.main(GradleStart.java:45) [start/:?]
Caused by: java.lang.NullPointerException
at io.netty.buffer.AbstractByteBuf.readBytes(AbstractByteBuf.java:676) ~[AbstractByteBuf.class:?]
at fr.MrBlockTNT.Dragons.network.PacketData$01.fromBytes(PacketData$01.java:34) ~[PacketData$01.class:?]
at cpw.mods.fml.common.network.simpleimpl.SimpleIndexedCodec.decodeInto(SimpleIndexedCodec.java:17) ~[SimpleIndexedCodec.class:?]
at cpw.mods.fml.common.network.simpleimpl.SimpleIndexedCodec.decodeInto(SimpleIndexedCodec.java:7) ~[SimpleIndexedCodec.class:?]
at cpw.mods.fml.common.network.FMLIndexedMessageToMessageCodec.decode(FMLIndexedMessageToMessageCodec.java:77) ~[FMLIndexedMessageToMessageCodec.class:?]
at cpw.mods.fml.common.network.FMLIndexedMessageToMessageCodec.decode(FMLIndexedMessageToMessageCodec.java:17) ~[FMLIndexedMessageToMessageCodec.class:?]
at io.netty.handler.codec.MessageToMessageCodec$2.decode(MessageToMessageCodec.java:81) ~[MessageToMessageCodec$2.class:?]
at io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:89) ~[MessageToMessageDecoder.class:?]
… 19 more
[18:17:38] [Client thread/ERROR] [FML]: There was a critical exception handling a packet on channel dragons:CHANNEL_1
io.netty.handler.codec.DecoderException: java.lang.NullPointerException
at io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:99) ~[MessageToMessageDecoder.class:?]
at io.netty.handler.codec.MessageToMessageCodec.channelRead(MessageToMessageCodec.java:111) ~[MessageToMessageCodec.class:?]
at io.netty.channel.DefaultChannelHandlerContext.invokeChannelRead(DefaultChannelHandlerContext.java:337) ~[DefaultChannelHandlerContext.class:?]
at io.netty.channel.DefaultChannelHandlerContext.fireChannelRead(DefaultChannelHandlerContext.java:323) ~[DefaultChannelHandlerContext.class:?]
at io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:785) ~[DefaultChannelPipeline.class:?]
at io.netty.channel.embedded.EmbeddedChannel.writeInbound(EmbeddedChannel.java:169) ~[EmbeddedChannel.class:?]
at cpw.mods.fml.common.network.internal.FMLProxyPacket.processPacket(FMLProxyPacket.java:86) [FMLProxyPacket.class:?]
at net.minecraft.network.NetworkManager.processReceivedPackets(NetworkManager.java:241) [NetworkManager.class:?]
at net.minecraft.client.Minecraft.runTick(Minecraft.java:2141) [Minecraft.class:?]
at net.minecraft.client.Minecraft.runGameLoop(Minecraft.java:1028) [Minecraft.class:?]
at net.minecraft.client.Minecraft.run(Minecraft.java:951) [Minecraft.class:?]
at net.minecraft.client.main.Main.main(Main.java:164) [Main.class:?]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_11]
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_11]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_11]
at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_11]
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135) [launchwrapper-1.11.jar:?]
at net.minecraft.launchwrapper.Launch.main(Launch.java:28) [launchwrapper-1.11.jar:?]
at net.minecraftforge.gradle.GradleStartCommon.launch(GradleStartCommon.java:78) [start/:?]
at GradleStart.main(GradleStart.java:45) [start/:?]
Caused by: java.lang.NullPointerException
at io.netty.buffer.AbstractByteBuf.readBytes(AbstractByteBuf.java:676) ~[AbstractByteBuf.class:?]
at fr.MrBlockTNT.Dragons.network.PacketData$01.fromBytes(PacketData$01.java:34) ~[PacketData$01.class:?]
at cpw.mods.fml.common.network.simpleimpl.SimpleIndexedCodec.decodeInto(SimpleIndexedCodec.java:17) ~[SimpleIndexedCodec.class:?]
at cpw.mods.fml.common.network.simpleimpl.SimpleIndexedCodec.decodeInto(SimpleIndexedCodec.java:7) ~[SimpleIndexedCodec.class:?]
at cpw.mods.fml.common.network.FMLIndexedMessageToMessageCodec.decode(FMLIndexedMessageToMessageCodec.java:77) ~[FMLIndexedMessageToMessageCodec.class:?]
at cpw.mods.fml.common.network.FMLIndexedMessageToMessageCodec.decode(FMLIndexedMessageToMessageCodec.java:17) ~[FMLIndexedMessageToMessageCodec.class:?]
at io.netty.handler.codec.MessageToMessageCodec$2.decode(MessageToMessageCodec.java:81) ~[MessageToMessageCodec$2.class:?]
at io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:89) ~[MessageToMessageDecoder.class:?]
… 19 more
[18:17:38] [Server thread/INFO]: Player730 lost connection: TextComponent{text='Disconnected', siblings=[], style=Style{hasParent=false, color=null, bold=null, italic=null, underlined=null, obfuscated=null, clickEvent=null, hoverEvent=null}}
[18:17:38] [Server thread/INFO]: Player730 left the game
[18:17:38] [Server thread/INFO]: Stopping singleplayer server as player logged out
 

Pouvez-vous m’éclairer sur l’origine possible de cette erreur ? (A mon avis c’est tout bête mais ça fait un petit peu que je cherche)
Je précise aussi que je suis en 1.7.10 mais ça ne doit rien changer de ce côté (enfin je pense).

Code intéressant de PlayerData.java :

 
@Override
 
public void loadNBTData(NBTTagCompound nbt) 
{
NBTTagCompound properties = (NBTTagCompound) nbt.getTag(DATA_NAME);
 
this.mana = properties.getShort("Mana");
this.playerMaxMana = properties.getShort("MaxMana");
this.spell_levels = properties.getByteArray("Spell_Levels");
}

[robin4002]

NBTTagCompound properties = (NBTTagCompound) nbt.getTag(DATA_NAME);
System.out.println(properties == null)
Si il t’indique true dans la console c’est que nbt.getTag(DATA_NAME) est null, et donc que tu as un problème au niveau des tags nbt.

[bin4r1s]

Retourne false, mais j’ai trouvé d’ou vient le bug :
Je voulais envoyer (packet) un tableau de byte, mais je ne trouvais pas (sur internet) comment récupérer alors j’ai tenté des choses hasardeuses :

 
@Override
 
    public void fromBytes(ByteBuf buf) 
    {
        mana = buf.readShort();
        playerMaxMana = buf.readShort();
        buf.readBytes(levels); // ICI
    }

Donc si quelqu’un sait comment faire…

[robin4002]

Pour envoyer un tableau il faut passer par des boucles for.
Dans un premier temps tu envoie la taille du tableau :

buf.writeInt(tonTableau.length);

puis tu écris toutes les valeurs :

for(int i = 0; i > tonTableau.length;i++)
{
      buf.writeByte(tonTableau*);
}

Et pour le lire de l’autre côté tu l’initialises et tu fais une boucle for en fonction de sa taille :

int size = buf.readInt();
tonTableau = new Byte;
for(int i = 0; i > size;i++)
 
{
      tonTableau* = buf.readByte();
}

[RedRelay]

Pourquoi ne pas passer par un NBTTagList ?
Si c’est un tableau d’entier, il y a aussi une méthode pour écrire directement un tableau d’entier.

[bin4r1s]

Merci maintenant ça marche, mais ça fait laguer mon ordi O_O et en plus, à chaque fois que j’utilise mon item, la console me sort :

[10:26:01] [Client thread/ERROR] [FML]: Detected ongoing potential memory leak. 100 packets have leaked. Top offenders

[10:26:01] [Client thread/ERROR] [FML]: dragons:CHANNEL_1 : 100
[10:26:05] [Client thread/ERROR] [FML]: Detected ongoing potential memory leak. 200 packets have leaked. Top offenders
[10:26:05] [Client thread/ERROR] [FML]: dragons:CHANNEL_1 : 200
[10:26:08] [Client thread/ERROR] [FML]: Detected ongoing potential memory leak. 300 packets have leaked. Top offenders
[10:26:08] [Client thread/ERROR] [FML]: dragons:CHANNEL_1 : 300
[10:26:11] [Client thread/ERROR] [FML]: Detected ongoing potential memory leak. 400 packets have leaked. Top offenders
[10:26:11] [Client thread/ERROR] [FML]: dragons:CHANNEL_1 : 400
[10:26:13] [Client thread/ERROR] [FML]: Detected ongoing potential memory leak. 500 packets have leaked. Top offenders
[10:26:13] [Client thread/ERROR] [FML]: dragons:CHANNEL_1 : 500
[10:26:16] [Client thread/ERROR] [FML]: Detected ongoing potential memory leak. 600 packets have leaked. Top offenders
[10:26:16] [Client thread/ERROR] [FML]: dragons:CHANNEL_1 : 600

[robin4002]

Oula, envoie tout ton code, visiblement tu as des paquets qui se perdent.

[bin4r1s]

PlayerData.java (le code intéressant) :

 
public final void sync() 
{
    PacketData$01 packet = new PacketData$01(this.mana, this.playerMaxMana, this.spell_levels, this.mana_regen_ticks);
   Dragons.network.sendToServer(packet);
 
    if (!player.worldObj.isRemote) 
    {
        EntityPlayerMP player1 = (EntityPlayerMP) player;
        Dragons.network.sendTo(packet, player1);
    }
}

PacketData$01.java :

 
package fr.MrBlockTNT.Dragons.network;
 
import fr.MrBlockTNT.Dragons.event.PlayerData;
import io.netty.buffer.ByteBuf;
import net.minecraft.client.Minecraft;
import net.minecraft.client.network.NetHandlerPlayClient;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.network.NetHandlerPlayServer;
import cpw.mods.fml.common.network.simpleimpl.IMessage;
import cpw.mods.fml.common.network.simpleimpl.IMessageHandler;
import cpw.mods.fml.common.network.simpleimpl.MessageContext;
import cpw.mods.fml.relauncher.Side;
import cpw.mods.fml.relauncher.SideOnly;
 
public class PacketData$01 implements IMessage
{
private short mana, playerMaxMana;
private int regen_ticks;
private byte[] levels;
 
    public PacketData$01() { }
 
    public PacketData$01(short mana, short playerMaxMana, byte[] levels, int r_ticks) 
    {
        this.mana = mana;
        this.playerMaxMana = playerMaxMana;
        this.levels = levels;
        this.regen_ticks = r_ticks;
    }
 
    @Override
    public void fromBytes(ByteBuf buf) 
    {
        mana = buf.readShort();
        playerMaxMana = buf.readShort();
        byte size = buf.readByte();
        levels = new byte;
        for(int i = 0; i < size; i++)
        levels* = buf.readByte();
        regen_ticks = buf.readInt();
    }
 
    @Override
    public void toBytes(ByteBuf buf) 
    {
    buf.writeShort(mana);
    buf.writeShort(playerMaxMana);
    buf.writeByte(levels.length);
    for(int i = 0; i < levels.length; i++)
    buf.writeByte(levels*);
    buf.writeInt(regen_ticks);
    }
 
    public static class Handler implements IMessageHandler <packetdata$01, imessage="">{
 
        @Override
        public IMessage onMessage(PacketData$01 message, MessageContext ctx) 
        {
        if(ctx.netHandler instanceof NetHandlerPlayServer)
        {
        PlayerData dat = PlayerData.get(getClientPlayer());
        dat.update(message.mana, message.playerMaxMana, message.levels, message.regen_ticks);
        }
        else if(ctx.netHandler instanceof NetHandlerPlayClient)
        {
        PlayerData dat = PlayerData.get(getClientPlayer());
        dat.update(message.mana, message.playerMaxMana, message.levels, message.regen_ticks);
        }
            return null;
        }
 
        @SideOnly(Side.CLIENT)
        public EntityPlayer getClientPlayer()
        {
        return Minecraft.getMinecraft().thePlayer;
        }
    }
}
```</packetdata$01,>

[robin4002]

if(ctx.netHandler instanceof NetHandlerPlayServer)
{
PlayerData dat = PlayerData.get(getClientPlayer());

??!!??

Et autre problème, le nom de la classe. Mettre un $ dans le nom est une très mauvaise idée.

[bin4r1s]

En fait, je ne voyais pas trop comment récupérer le joueur pour le côté serveur  mais je viens de me rendre compte que c’est complètement c** stupide puisque le serveur ne pourra pas le récupérer (SideOnly). Une idée ? (J’avais aussi pensé à envoyer l’entity player mais il semblerait que je ne puisse pas (et puis ça serai trop lourd…)).

Sinon pour le nom je voulais un séparateur entre le nom du packet et son index x)

[robin4002]

PlayerData dat = PlayerData.get(((NetHandlerPlayServer)netHandler).playerEntity);

Le $ est utilisé pour les noms des classes lorsque tu utilise des sous classe.
Par exemple si tu fais :

public class BlaBla
{
     public static class AutreClasse
     {
     }
}

Lors de la compilation ça va créer deux fichiers .class :
BlaBla.class
BlaBla$AutreClasse.class

Donc c’est très déconseillé de mettre des $ dans le nom de la classe, ça risque de créer des problèmes avec ça.

[bin4r1s]

Ah d’accord, merci  je vais changer ça.

EDIT :
Ah ben en fait je perds toujours des packets  . Nouveau code :

 
@Override
 
public IMessage onMessage(PacketData_01 message, MessageContext ctx) 
{
         if(ctx.netHandler instanceof NetHandlerPlayServer)
        {
                 PlayerData dat = PlayerData.get(((NetHandlerPlayServer)ctx.netHandler).playerEntity);
                 dat.update(message.mana, message.playerMaxMana, message.levels, message.regen_ticks);
        }
        else if(ctx.netHandler instanceof NetHandlerPlayClient)
        {
                 PlayerData dat = PlayerData.get(getClientPlayer());
                 dat.update(message.mana, message.playerMaxMana, message.levels, message.regen_ticks);
        }
        return null;
}
 

[bin4r1s]

Up, help svp 

[robin4002]

Ta fonction dat.update fait quoi ?

[bin4r1s]

Elle met juste à jour les propriétés du joueur.

[robin4002]

Un copier/coller de la fonction aurait été utile. Car je suspecte que tu renvoie un paquet à chaque fois, enfin je ne vois pas d’autres explications.

[bin4r1s]

 
public void update(short mana, short playerMaxMana, byte[] levels, int regen)
 
{
       this.mana = mana;
       this.playerMaxMana = playerMaxMana;
       this.spell_levels = levels;
       this.mana_regen_ticks = regen;
}

En effet ta théorie sur le fait que je renvoie le packet à chaque fois n’est pas impossible. Je check ça.

EDIT : Rien trouvé  (Enfin intelligible pour moi)

Code (peut-être auras-tu plus de chance  ) :
PlayerData.java
PacketData_01.java
CommonProxy.java

Il y a quelques erreurs dans un constructeur et la méthode update notamment car je suis en train de mettre en place quelque chose.

[bin4r1s]

Up

[robin4002]

Je ne vois pas d’erreur non plus
Sinon envoie un zip de ton dossier src, je vais faire des tests en local.

[bin4r1s]

Regarde tes MPs