Affichage de capabilities

robin4002

La fonction est bien appelé ?
Le paquet est bien enregistré ?

Jerem_Tech

Salut ,

Oui c’est une des première chose que j’ai vérifié ….
Toutefois , en creusant un peu plus , j’ai trouver cette erreur qui surviens uniquement dans la console du serveur :

[Server thread/ERROR] [FML]: SimpleChannelHandlerWrapper exception
java.lang.RuntimeException: Missing
at net.minecraftforge.fml.server.FMLServerHandler.getClientToServerNetworkManager(FMLServerHandler.java:288) ~[FMLServerHandler.class:?]
at net.minecraftforge.fml.common.FMLCommonHandler.getClientToServerNetworkManager(FMLCommonHandler.java:545) ~[FMLCommonHandler.class:?]
at net.minecraftforge.fml.common.network.FMLOutboundHandler$OutboundTarget$8.selectNetworks(FMLOutboundHandler.java:245) ~[FMLOutboundHandler$OutboundTarget$8.class:?]
at net.minecraftforge.fml.common.network.FMLOutboundHandler.write(FMLOutboundHandler.java:293) ~[FMLOutboundHandler.class:?]
at io.netty.channel.AbstractChannelHandlerContext.invokeWrite(AbstractChannelHandlerContext.java:658) ~[AbstractChannelHandlerContext.class:4.0.23.Final]
at io.netty.channel.AbstractChannelHandlerContext.write(AbstractChannelHandlerContext.java:716) ~[AbstractChannelHandlerContext.class:4.0.23.Final]
at io.netty.channel.AbstractChannelHandlerContext.write(AbstractChannelHandlerContext.java:651) ~[AbstractChannelHandlerContext.class:4.0.23.Final]
at io.netty.handler.codec.MessageToMessageEncoder.write(MessageToMessageEncoder.java:112) ~[MessageToMessageEncoder.class:4.0.23.Final]
at io.netty.handler.codec.MessageToMessageCodec.write(MessageToMessageCodec.java:116) ~[MessageToMessageCodec.class:4.0.23.Final]
at io.netty.channel.AbstractChannelHandlerContext.invokeWrite(AbstractChannelHandlerContext.java:658) ~[AbstractChannelHandlerContext.class:4.0.23.Final]
at io.netty.channel.AbstractChannelHandlerContext.write(AbstractChannelHandlerContext.java:716) ~[AbstractChannelHandlerContext.class:4.0.23.Final]
at io.netty.channel.AbstractChannelHandlerContext.writeAndFlush(AbstractChannelHandlerContext.java:706) ~[AbstractChannelHandlerContext.class:4.0.23.Final]
at io.netty.channel.AbstractChannelHandlerContext.writeAndFlush(AbstractChannelHandlerContext.java:741) ~[AbstractChannelHandlerContext.class:4.0.23.Final]
at io.netty.channel.DefaultChannelPipeline.writeAndFlush(DefaultChannelPipeline.java:895) ~[DefaultChannelPipeline.class:4.0.23.Final]
at io.netty.channel.AbstractChannel.writeAndFlush(AbstractChannel.java:240) ~[AbstractChannel.class:4.0.23.Final]
at net.minecraftforge.fml.common.network.simpleimpl.SimpleNetworkWrapper.sendToServer(SimpleNetworkWrapper.java:294) [SimpleNetworkWrapper.class:?]
at fr.jeremy60800.dofuscraftcore.capabilities.DofusCraftLevels.sync(DofusCraftLevels.java:25) [DofusCraftLevels.class:?]
at fr.jeremy60800.dofuscraftcore.handlers.PlayerHandler.onPlayerLogging(PlayerHandler.java:29) [PlayerHandler.class:?]
at net.minecraftforge.fml.common.eventhandler.ASMEventHandler_8_PlayerHandler_onPlayerLogging_PlayerLoggedInEvent.invoke(.dynamic) [?:?]
at net.minecraftforge.fml.common.eventhandler.ASMEventHandler.invoke(ASMEventHandler.java:90) [ASMEventHandler.class:?]
at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:185) [EventBus.class:?]
at net.minecraftforge.fml.common.FMLCommonHandler.firePlayerLoggedIn(FMLCommonHandler.java:565) [FMLCommonHandler.class:?]
at net.minecraft.server.management.PlayerList.initializeConnectionToPlayer(PlayerList.java:238) [PlayerList.class:?]
at net.minecraftforge.fml.common.network.handshake.NetworkDispatcher.completeServerSideConnection(NetworkDispatcher.java:263) [NetworkDispatcher.class:?]
at net.minecraftforge.fml.common.network.handshake.NetworkDispatcher.access$100(NetworkDispatcher.java:73) [NetworkDispatcher.class:?]
at net.minecraftforge.fml.common.network.handshake.NetworkDispatcher$1.update(NetworkDispatcher.java:212) [NetworkDispatcher$1.class:?]
at net.minecraft.network.NetworkManager.processReceivedPackets(NetworkManager.java:309) [NetworkManager.class:?]
at net.minecraft.network.NetworkSystem.networkTick(NetworkSystem.java:197) [NetworkSystem.class:?]
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:807) [MinecraftServer.class:?]
at net.minecraft.server.dedicated.DedicatedServer.updateTimeLightAndEntities(DedicatedServer.java:408) [DedicatedServer.class:?]
at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:688) [MinecraftServer.class:?]
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:537) [MinecraftServer.class:?]
at java.lang.Thread.run(Thread.java:748) [?:1.8.0_131]

Et voilà les lignes qui sont désignées :

DofusCraftLevels:25

[font=Courier Newpublic void ]sync(EntityPlayerMP player)
{
  DofusCraftCore.*network*.sendTo(new PacketLevel(player.getCapability(DofusCraftLevelsProvider.*LEVEL_CAP*, null).getLevel()), player)%(#cc7832)[;
]}

PlayerHandler:29

[font=Courier New@SubscribeEvent
]public void onPlayerLogging(net.minecraftforge.fml.common.gameevent.PlayerEvent.PlayerLoggedInEvent event)
{
  new DofusCraftLevels().sync((EntityPlayerMP) event.player)%(#cc7832)[;
]}

Visiblement c’est la fonction sync qui fonctionne pas …
Mais je ne voit pas pourquoi ça ne fonctionne pas …

Cordialement ,

robin4002

L’erreur ne correspond pas au code.
Dans le code tu fais un sendTo alors que dans le stacktrace on voit un sendToServer (d’où l’erreur, il y a envoie de paquet du client vers le serveur alors que tu es côté serveur).

Par ailleurs, il serait mieux d’avoir une fonction sync comme ceci :

public void sync(EntityPlayerMP player)
{
DofusCraftCore.network.sendTo(new PacketLevel(this.level), player);
}

Et pour l’appeler :

@SubscribeEvent
public void onPlayerLogging(net.minecraftforge.fml.common.gameevent.PlayerEvent.PlayerLoggedInEvent event)
{
event.player.getCapability(DofusCraftLevelsProvider.LEVEL_CAP, null)sync((EntityPlayerMP) event.player);

}

Jerem_Tech

Salut ,

J’ai donc fait les changements que tu m’a dit mais l’affichage ne fonctionne toujours pas ….
On a 1 , la valeur par défaut …

Cordialement ,

robin4002

Ton paquet est bien enregistré ?
Tu peux envoyer le code du paquet ?

Jerem_Tech

Oui il est bien enregistré et il retourne la bonne valeur .
C’est uniquement le fait de devoir l’afficher qui pose problème car l’entityplayer de la fonction addinformation() c’est un entotyplayerSP et on a besoin d’un entityplayerMP ….

Sinon voici le packet :

[font=Courier Newpublic class ]PacketLevel implements IMessage
{

  IDofusCraftLevels level%(#cc7832)[;
]
   public PacketLevel()
  {
  }

  public PacketLevel(IDofusCraftLevels level)
  {
     this.level = level%(#cc7832)[;
]   }

  %(#bbb529)[@Override
]   public void fromBytes(ByteBuf buf)
  {
     if(this.getLevel() != null)
     {

        this.getLevel().setLevel(buf.readInt())%(#cc7832)[;
]
     }
  }

  %(#bbb529)[@Override
]   public void toBytes(ByteBuf buf)
  {
     if(this.getLevel() != null)
     {

        buf.writeInt(this.getLevel().getLevel())%(#cc7832)[;
]
     }
  }

  public void setLevel(IDofusCraftLevels level)
  {
     this.level = level%(#cc7832)[;
]   }

  public  IDofusCraftLevels getLevel()
  {
     return this.level%(#cc7832)[;
]   }

  public static class PacketLevelHandlerClient implements IMessageHandler <packetlevel, imessage="">{

     %(#bbb529)[@Override
]      public IMessage onMessage(PacketLevel message, MessageContext ctx)
     {

        IThreadListener thread = DofusCraftCore.*proxy*.getListener(ctx)%(#cc7832)[;
]         final EntityPlayer player = DofusCraftCore.*proxy*.getPlayer(ctx)%(#cc7832)[;
]         final IDofusCraftLevels value = message.getLevel()%(#cc7832)[;
]         thread.addScheduledTask(() -> {
           if(player != null) {
              IDofusCraftLevels capabilities = player.getCapability(DofusCraftLevelsProvider.*LEVEL_CAP*, null)%(#cc7832)[;
]               if(capabilities != null && value != null) {
                 capabilities.setLevel(value.getLevel())%(#cc7832)[;
]               }
           }
        })%(#cc7832)[;
]
%(#cc7832)[         return null;
]      }
  }

  public static class PacketLevelHandlerServer implements IMessageHandler <packetlevel, imessage="">{

     %(#bbb529)[@Override
]      public IMessage onMessage(PacketLevel message, MessageContext ctx)
     {
        IThreadListener thread = DofusCraftCore.*proxy*.getListener(ctx)%(#cc7832)[;
]         final EntityPlayer player = DofusCraftCore.*proxy*.getPlayer(ctx)%(#cc7832)[;
]         final IDofusCraftLevels value = message.getLevel()%(#cc7832)[;
]         thread.addScheduledTask(() -> {
           if(player != null) {
              IDofusCraftLevels capabilities = player.getCapability(DofusCraftLevelsProvider.*LEVEL_CAP*, null)%(#cc7832)[;
]               if(capabilities != null && value != null) {
                 capabilities.setLevel(value.getLevel())%(#cc7832)[;
]               }
           }
        })%(#cc7832)[;
]%(#cc7832)[         return null;
]      }
  }

}

Cordialement ,</packetlevel%(#cc7832)[,></packetlevel%(#cc7832)[,>

robin4002

Non tu n’as pas besoin d’un entityplayerMP pour afficher la valeur d’une capability.
Je le fais dans plusieurs de mes mods sans problème.

La synchro coince quelque part, mais je ne vois pas où car ton code est correcte (du moins pour les classes que j’ai vu).
Vérifies que toutes les fonctions sont bien un appelé (avec le débogguer + point d’arrêt, il faut vérifier toute la chaine, envoie du paquet, réception par le client, modification de la valeur de la capa sur le client)

Jerem_Tech

C’est la modification de la capa côté client qui ne fonctionne pas ……
Car le packet j’ai bien le bon niveau en réponse et l’envoi fonctionne nickel …

Le truc c’est que je sais pas comment fixer le problème …

Cordialement ,

robin4002

Je vois pourquoi.
à l’arrivé du paquet IDofusCraftLevels est null.
Il faudrait lui donner une valeur ou alors le remplacer directement par un int et changer la valeur de capabilities.getLevel().level au lieu de changer l’objet level qui se trouve dans capabilities.

(d’ailleurs je ne comprends pas pourquoi tu mets ta variable level à l’intérieur d’un objet level. Si tu veux une classe à part pour la question des niveaux, il serait mieux de nommer la classe LevelMananger ou quelque chose du genre, afin d’avoir nom différent pour éviter les possibles confusions).

Jerem_Tech

Je suis censé faire ça comment ?
Pour l’histoire de l’objet , en fait c’est parce que après il y aura 13 variables correspondantes à un métier chacune ….

Cordialement ,

robin4002

D’accord.

Mais dans ce cas je ne vois pas la pertinence d’avoir un setter dans la classe de capability l’avoir dans dans la classe IDofusCraftLevels aurait plus de sens.

public class PacketLevel implements IMessage
{

private int level;

public PacketLevel()
{
}

public PacketLevel(IDofusCraftLevels level)
{
this.level = level.getLevel();
}

@Override
public void fromBytes(ByteBuf buf)
{
level = buf.readInt();
}

@Override
public void toBytes(ByteBuf buf)
{
buf.writeInt(this.level);
}

public static class PacketLevelHandlerClient implements IMessageHandler <packetlevel, imessage="">{

@Override
public IMessage onMessage(final PacketLevel message, MessageContext ctx)
{

IThreadListener thread = DofusCraftCore.proxy.getListener(ctx);
final EntityPlayer player = DofusCraftCore.proxy.getPlayer(ctx);
thread.addScheduledTask(() -> {
if(player != null) {
IDofusCraftLevels capabilities = player.getCapability(DofusCraftLevelsProvider.LEVEL_CAP, null);
if(capabilities != null) {
capabilities.getLevel().setLevel(message.level);
}
}
});

return null;
}
}

public static class PacketLevelHandlerServer implements IMessageHandler <packetlevel, imessage="">{

@Override
public IMessage onMessage(final PacketLevel message, MessageContext ctx)
{
IThreadListener thread = DofusCraftCore.proxy.getListener(ctx);
final EntityPlayer player = DofusCraftCore.proxy.getPlayer(ctx);
thread.addScheduledTask(() -> {
if(player != null) {
IDofusCraftLevels capabilities = player.getCapability(DofusCraftLevelsProvider.LEVEL_CAP, null);
if(capabilities != null) {
capabilities.getLevel().setLevel(message.level);
}
}
});
return null;
}
}
}
```</packetlevel,></packetlevel,>

Jerem_Tech

Changements effectuées , simplement ça change rein xD
Et est-ce une erreur ceci ?

%(#a9b7c6)[[font=Courier New]capabilities.getLevel.setLevel(message].%(#9876aa)[level])%(#cc7832)[;]

Ce n’est pas plutôt ça :

%(#a9b7c6)[[font=Courier New]capabilities.setLevel(message].%(#9876aa)[level])%(#cc7832)[;]

Après dans le second cas ça ne fonctionne quand même , et dans le premier bah moi j’ai pas de méthodes que je peut appeler ainsi ….

Cordialement ,

robin4002

Oula …
Je viens de capter que j’étais à côté de la plaque pour l’IDofusCraftLevels.
Je croyais pendant un moment que c’était une classe que tu avais en plus alors que c’est ta capability.

Autant pour moi, donc en effet c’est juste capabilities.setLevel(message.level)

Par contre je ne comprends pas pourquoi la valeur ne synchro toujours pas.
Regardes ce qu’il se passe avec le déboguer.

Jerem_Tech

J’ai rien dit en redémarrant le jeu ça fonctionne
Donc je peut enfin passer ce sujet en résolu !

Mille merci !