[SANS SUITE] Crash Serveur

robin4002

Envoies ta classe Events entière.

Gabs

@‘robin4002’:

Envoies ta classe Events entière.

Tien : (J’ai fais quelques modif au niveau de l’event ondeath mais je pense pas que se soit ça:


public class Events {

public static boolean coma;

@SubscribeEvent
public void onDeath(LivingDeathEvent event)
{

booleantrue();
System.out.println(coma);
if(event.entityLiving instanceof EntityPlayer && coma == true)
{

event.entityLiving.setHealth(1);

Main.network.sendToServer(new PacketGui());

System.out.println("Mort");
System.out.println(coma);

}

}

@SubscribeEvent
public void onInventoryOpen(GuiOpenEvent event)
{
EntityPlayer player = Minecraft.getMinecraft().thePlayer;
if(event.gui instanceof GuiInventory && !player.capabilities.isCreativeMode)
{
event.setCanceled(true);
}
}

public static void booleantrue()
{
coma = true;
}
public static void booleanfalse()
{
coma = false;
}

@SubscribeEvent
public void pickupfalse(EntityItemPickupEvent event)
{
if(coma == true && event.entity instanceof EntityPlayer)
{
event.setCanceled(true);
}
}
@SubscribeEvent
public void onRespawn(PlayerEvent.PlayerRespawnEvent event)
{
if(coma == true)
{
booleanfalse();
System.out.println(coma);
}
}

@SubscribeEvent
public void onDecocoma(PlayerEvent.PlayerLoggedOutEvent event)
{
if(coma == true)
{
event.player.setDead();
}
}

}

robin4002

Mets un @SideOnly(Side.CLIENT) au dessus de cette fonction : public void onInventoryOpen(GuiOpenEvent event)

Et dans onDeath Main.network.sendToServer(new PacketGui()); devrait être Main.network.sendTo(new PacketGui(), (EntityPlayer)event.entityLiving);
Comment veux-tu envoyer un paquet au serveur si tu es déjà sur le serveur ?

Gabs

@‘robin4002’:

Mets un @SideOnly(Side.CLIENT) au dessus de cette fonction : public void onInventoryOpen(GuiOpenEvent event)

Et dans onDeath Main.network.sendToServer(new PacketGui()); devrait être Main.network.sendTo(new PacketGui(), (EntityPlayer)event.entityLiving);
Comment veux-tu envoyer un paquet au serveur si tu es déjà sur le serveur ?

Mon gui s’ouvre plus du coup:


@SubscribeEvent
public void onDeath(LivingDeathEvent event)
{

booleantrue();
System.out.println(coma);
if(event.entityLiving instanceof EntityPlayer && coma == true)
{

event.entityLiving.setHealth(1);

Main.network.sendTo(new PacketGui(), (EntityPlayerMP)event.entityLiving);

System.out.println("Mort");
System.out.println(coma);

}

}

AymericRed

1. ```java
EntityPlayer player = Minecraft.getMinecraft().thePlayer;

2\. Pourquoi avoir des fonction booleantrue() et booleanfalse() alors que tu peux directement changer la valeur de coma sans ces fonctions ?
3\. ```java
if(event.entityLiving instanceof EntityPlayer && coma == true)
``` pourquoi tester si come == true alors que tu as appelé booleantrue() juste avant ?
4\. ```java
Main.network.sendToServer(new PacketGui());
```, j'ai déjà dit qu'il faut

> que tu envoies le packet au client qui vient de mourir

, alors pourquoi tu l'envoies depuis le serveur vers le serveur ?

EDIT : Robin a répondu avant, et tu as ensuite répondu, est-ce que "Mort" apparaît dans la console ?

Gabs

@‘AymericRed’:

1. ```java
EntityPlayer player = Minecraft.getMinecraft().thePlayer;

2\. Pourquoi avoir des fonction booleantrue() et booleanfalse() alors que tu peux directement changer la valeur de coma sans ces fonctions ?
3\. ```java
if(event.entityLiving instanceof EntityPlayer && coma == true)
``` pourquoi tester si come == true alors que tu as appelé booleantrue() juste avant ?
4\. ```java
Main.network.sendToServer(new PacketGui());
```, j'ai déjà dit qu'il faut

> que tu envoies le packet au client qui vient de mourir

, alors pourquoi tu l'envoies depuis le serveur vers le serveur ?

EDIT : Robin a répondu avant, et tu as ensuite répondu, est-ce que "Mort" apparaît dans la console ?

2 : Je préfères faire des fonctions je sais que je peux directement avec coma = true / false;
3 : bonne question.
4 : J’avais pas remarqué.
et pour le edit le “Mort” marcher avec un world.isremote maintenant sans le isremote oui il s’affiche dans la console.

AymericRed

Donc le packet devrait être envoyé, tu l’as bien register comme devant arriver sur le client ? Mets un print dans la fonction onMessage.

Gabs

@‘AymericRed’:

Donc le packet devrait être envoyé, tu l’as bien register comme devant arriver sur le client ? Mets un print dans la fonction onMessage.

J’ai mis un print dans la fonction onMessage en solo le print est bien envoyer et le gui s’ouvre bien mais en multi il s’ouvre pas.

robin4002

Pas normal du tout.
Renvoies ton code. (gui, paquet, event)

Gabs

Surement les sides que ont a mis au débuts.

Gui:


public class Mort extends GuiScreen {

@SideOnly(Side.CLIENT)

private ResourceLocation ressource = new ResourceLocation("dd:textures/gui/death.png");

private int medecin;

private int spawn;

public static int timer = 10000;

int guiWidth = 256;

int guiHeight = 256;

public Mort() {

}

@Override
public void initGui() {

int guiX = (this.width - guiWidth) / 2;

int guiY = (this.height - guiHeight) / 2;

medecin = 0;

spawn = 1;

buttonList.clear();

buttonList.add(new GuiButton(medecin, guiX + 170, guiY + 226 + 5, 58, 20, "§aAppeller un médecin"));

buttonList.add(new GuiButton(spawn, guiX + 80, guiY, 120, 20, "§5Respawn possible dans"));

super.initGui();
}

@Override
protected void keyTyped(char typedChar, int keyCode) {

}

@Override
protected void actionPerformed(GuiButton button) {

switch(button.id) {
case 0:
Main.network.sendToServer(new PacketMedecin());
break;
case 1:
if(timer > 0)
{
EntityPlayer player1 = Minecraft.getMinecraft().thePlayer;
ChatComponentText text1 = new ChatComponentText("§7Vous ne pouvez pas respawn ! il vous reste: " + timer / 100 + " §7secondes");
player1.addChatComponentMessage(text1);
timer –;
break;
}
else
{

EntityPlayer player1 = Minecraft.getMinecraft().thePlayer;
player1.setDead();
Minecraft.getMinecraft().thePlayer.closeScreen();
break;
}
default:
break;
}

super.actionPerformed(button);
}

@Override
public void updateScreen() {
super.updateScreen();
}

public void drawScreen(int mouseX, int mouseY, float partialTick) 

{

String tilename = "§5 : " + timer-- / 100;

mc.getTextureManager().bindTexture(ressource);

this.drawTexturedModalRect(this.width / 2 - 135, this.height / 2 - 127, 0, 0, 256, 255);
if(timer > 0)
{
this.fontRendererObj.drawString(tilename, this.width / 2 + 95 - this.fontRendererObj.getStringWidth(tilename) / 2, 1, 0);
}
super.drawScreen(mouseX, mouseY, partialTick);
    }

public static void settimer(int newtimer)
{
timer = newtimer;
}

@Override
public void onGuiClosed() {

}

}

Packet:


public class PacketGui implements IMessage {

public int i;

public PacketGui(){

}
public PacketGui(int i){
this.i = i;
}

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

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

public static class Handler implements IMessageHandler <packetgui, imessage="">{
@SideOnly(Side.CLIENT)
   public IMessage onMessage(PacketGui message, MessageContext ctx) {

        Minecraft.getMinecraft().displayGuiScreen(new Mort());
        Mort.settimer(10000);
        System.out.println("packet");
        return message;
       }

   }

}

Event:


@SubscribeEvent
public void onDeath(LivingDeathEvent event)
{

booleantrue();
System.out.println(coma);
if(event.entityLiving instanceof EntityPlayer)
{

event.entityLiving.setHealth(1);

Main.network.sendTo(new PacketGui(), (EntityPlayerMP)event.entityLiving);

System.out.println("Mort");
System.out.println(coma);

}

}

Register Packet:


network = NetworkRegistry.INSTANCE.newSimpleChannel("dd:");

network.registerMessage(PacketGui.Handler.class, PacketGui.class, 0, Side.CLIENT);
``` // ici quand je met client le gui ne s'ouvre pas solo ( j'ai pas essayer en multi )</packetgui,>

Deleted

Vire le constructeur du packet ayant un integer en paramètre, il ne te sert à rien. Et vire le
@SideOnly(Side.CLIENT) dans ta sous classe Handler et dans ta classe Gui. Tu n’en as pas besoin.

EDIT = Ton onGuiClosed que tu as override dans ta classe gui, ne sert à rien, puisque tu l’as laissé vide, et que de base (n’ai pas les sources sous les yeux), je crois qu’il est déjà vide.
Ton boolean coma dans ta classe EventHandler, sera commun à tous les joueurs, or je pense que tu ne le souhaites pas ? Si oui, go ––> l’enregistrer dans ton ExtProp

AymericRed

Dans ta fonction onMessage, “return message;” devrait être “return null;”, t comme Plaigon l’a dit, retires tout ce qui touches à l’int i.

Deleted

Dernières remarques :

À quoi ton setter setTimer peut te servir si ta variable timer est déjà en static ?
Comme ton onGuiClosed, même remarque pour ta fonction updateScreen
Pourquoi divises-tu ta variabe timer par 100 dans actionPerformed, alors que tu indiques qu’il s’agit d seconde. Je te rappelle que 20 ticks est égale à 1 seconde. Donc faut mieux diviser par 20
Lorsque tu te sers du setDead dans ta classe gui Mort, je ne suis pas trop sûr, mais il y a des chances que cela ne marche pas, car setDead() doit être appelé côté serveur (si quelqu’un pouvait me confirmer :))

Gabs

@‘Plaigon’:

Dernières remarques :

À quoi ton setter setTimer peut te servir si ta variable timer est déjà en static ?

Comme ton onGuiClosed, même remarque pour ta fonction updateScreen

Pourquoi divises-tu ta variabe timer par 100 dans actionPerformed, alors que tu indiques qu’il s’agit d seconde. Je te rappelle que 20 ticks est égale à 1 seconde. Donc faut mieux diviser par 20

Lorsque tu te sers du setDead dans ta classe gui Mort, je ne suis pas trop sûr, mais il y a des chances que cela ne marche pas, car setDead() doit être appelé côté serveur (si quelqu’un pouvait me confirmer :))

Alors,

J’ai fais ça du coup:


public class PacketGui implements IMessage {

public PacketGui(){

}

@Override
public void fromBytes(ByteBuf buf) {
}

@Override
public void toBytes(ByteBuf buf) {
}

public static class Handler implements IMessageHandler <packetgui, imessage="">{
   public IMessage onMessage(PacketGui message, MessageContext ctx) {

        Minecraft.getMinecraft().displayGuiScreen(new Mort());
        Mort.settimer(10000);
        System.out.println("packet");
        return null;
       }

   }

}

Crash au démarrage du serveur au niveau de la main class fonction init java 56 il me semble sa pointe le register du packetGui.</packetgui,>

AymericRed

Mets un @SideOnly(Side.CLIENT) sur ta fontcion onMessage.

SCAREX

Le problème ne vient pas de là, le side only n’est pas obligatoire

Gabs

@‘SCAREX’:

Le problème ne vient pas de là, le side only n’est pas obligatoire

J’ai une question dites moi si je me trompe,
Dans un packet le double constructeur n’est pas obligatoire ??

SCAREX

@‘floriangabet’:

@‘SCAREX’:

Le problème ne vient pas de là, le side only n’est pas obligatoire

J’ai une question dites moi si je me trompe,
Dans un packet le double constructeur n’est pas obligatoire ??

Si car tu as besoin d’un constructeur vide (qui sera appelé au moment où tu enregistres le paquet) et un constructeur que tu utiliseras pour envoyer tes données

Gabs

Du coup vous savez pas? Si il faut je me trompe ça ne vient pas de la, je vous glisse le crash report au cas ou:

http://pastebin.com/SJ8uasK5

normalement c’est la ligne 56 dans la main class je vous la passe aussi:

network.registerMessage(PacketGui.Handler.class, PacketGui.class, 0, Side.CLIENT);

robin4002

@‘Plaigon’:

@SideOnly(Side.CLIENT) dans ta sous classe Handler et dans ta classe Gui. Tu n’en as pas besoin.

@‘SCAREX’:

Le problème ne vient pas de là, le side only n’est pas obligatoire

Si. S’il n’y est pas le serveur crash lors du chargement de la classe à cause du Minecraft.getMinecraft() qui se trouve dans la fonction.