Faire un compteur de tick (Bar de soif)

robin4002

thirstValue ne devrait pas être static dans la classe PacketThirstCapabilities.
Je ne vois pas d’autres problèmes sinon.

Alexandre1156

Changes


@Override
  public NBTTagCompound serializeNBT()
  {
      NBTTagCompound compound = new NBTTagCompound();
      compound.setInteger("ThirstVal", this.getThirstVal());
      return compound;               
  }

  @Override
  public void deserializeNBT(NBTTagCompound compound)
  {
      this.setThirstVal(compound.getInteger("ThirstVal"));
  }

Par


 @Override
  public NBTTagCompound serializeNBT()
  {
      return Storage.writeNBT(ModSurvivant.T_CAP, this, null);//Le this est peut-être faux           
  }

  @Override
  public void deserializeNBT(NBTTagCompound compound)
  {
      Storage.readNBT(ModSurvivant.T_CAP, this, null, compound);//Pareil
  }

Et


 @Override
      public NBTBase writeNBT(Capability <thirstcapabilities>capability, ThirstCapabilities instance, EnumFacing side)
      {
          return null;
      }

      @Override
      public void readNBT(Capability <thirstcapabilities>capability, ThirstCapabilities instance, EnumFacing side, NBTBase nbt)
      { 

      }

Par


@Override
      public NBTBase writeNBT(Capability <thirstcapabilities>capability, ThirstCapabilities instance, EnumFacing side)
      {
          return new NBTTagInt(instance.getThirstVal());
      }

      @Override
      public void readNBT(Capability <thirstcapabilities>capability, ThirstCapabilities instance, EnumFacing side, NBTBase nbt)
      {
          if(nbt instanceof NBTTagInt)
             instance.setThirstVal(((NBTTagInt) nbt).getInt());
      }

Aussi, le fait que tu mettes tout dans une class n’aide pas.
Je te conseil de tout mettre en 5 classes :
-IThirst → Interface qui comprend les méthodes getThirstVal et setThirstVal,
-**ThirstCap **→ Class qui implémente **IThirst **et qui contient la variable thirstValue en field,
-**ThirstProvider **→ Class qui implémente ICapabilitySerializable<nbtbase>, et qui contient notamment l’instance de Capability<ithirst>,
-**ThirstStorage **→ Class qui implémente IStorage<ithirst>. C’est ici où tu dois écrire et lire les NBT,
-**CapabilityHandler **→ Class qui contient l’event AttachCapabilitiesEvent. <type>(ici Entity) afin d’attacher le capability au joueur.
Ça permettrait d’avoir un code plus lisible.

Aussi, si le joueur change de dimension, il va perdre toutes les valeurs de son capability. Pour cela, avec l’event PlayerEvent.Clone, vérifie que isWasDeath() est *false *et ainsi mettre les valeurs du capability de l’ancien joueur sur le nouveau.

Voici un tutorial en anglais qui explique très bien comment faire tout cela : http://www.planetminecraft.com/blog/forge-tutorial-capability-system/.</type></ithirst></ithirst></nbtbase></thirstcapabilities></thirstcapabilities></thirstcapabilities></thirstcapabilities>

elx9000

J’ai suivi le tuto sans sucés puisque le jeu crash à la génération du monde. Aussi, voici si join le dossier avec les classes concernées.

Les capability et moi décidément ça fait 2, il n’y a pas un moyen plus simple d’enregistrer une valeur pour un joueur ?

Merci par avance.

Alexandre1156

Changes


    @Override
    public <t>T getCapability(Capability <t>capability, EnumFacing facing)
    {
        return capability == THIRST_CAP ? (T)this.instance : null;
    }

Par


    @Override
    public <t>T getCapability(Capability <t>capability, EnumFacing facing)
    {
        return capability == THIRST_CAP ? THIRST_CAP.<t>cast(this.instance) : null;
    }

Si tu crash encore, envoies le crash report.</t></t></t></t></t>

elx9000

J’ai une erreur sur cast.

Alexandre1156

Laquelle ? Envoies le message d’erreur ainsi que la méthode.

elx9000


<t>T getCapability(Capability <t>capability, EnumFacing facing)    {        return capability == THIRST_CAP ? THIRST_CAP.<t>cast(this.instance) : null;       }</t></t></t>

Le cast de cast(this.instance) est souligné en rouge il veut que je cast THIRST_CAP.

Alexandre1156

Je viens d’essayer en 1.8.9 (étant donner que je faisait tout en 1.11.2) et la méthode cast n’existe pas. Donc laisse comme c’était avant, et envoies le crash report s’il y en a.

elx9000

Voici le crash report :

–-- Minecraft Crash Report ----
// Hi. I'm Minecraft, and I'm a crashaholic.

Time: 08/05/17 14:31
Description: Unexpected error

java.lang.NullPointerException: Unexpected error
at com.survivant.mod.SurvivantThirstBar.onRenderGameOverlay(SurvivantThirstBar.java:83)
at net.minecraftforge.fml.common.eventhandler.ASMEventHandler_13_SurvivantThirstBar_onRenderGameOverlay_RenderGameOverlayEvent.invoke(.dynamic)
at net.minecraftforge.fml.common.eventhandler.ASMEventHandler.invoke(ASMEventHandler.java:49)
at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:140)
at net.minecraftforge.client.GuiIngameForge.post(GuiIngameForge.java:854)
at net.minecraftforge.client.GuiIngameForge.renderExperience(GuiIngameForge.java:561)
at net.minecraftforge.client.GuiIngameForge.renderGameOverlay(GuiIngameForge.java:145)
at net.minecraft.client.renderer.EntityRenderer.updateCameraAndRender(EntityRenderer.java:1135)
at net.minecraft.client.Minecraft.runGameLoop(Minecraft.java:1107)
at net.minecraft.client.Minecraft.run(Minecraft.java:380)
at net.minecraft.client.main.Main.main(Main.java:116)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
at net.minecraft.launchwrapper.Launch.main(Launch.java:28)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at net.minecraftforge.gradle.GradleStartCommon.launch(GradleStartCommon.java:97)
at GradleStart.main(GradleStart.java:26)

A detailed walkthrough of the error, its code path and all known details is as follows:
---------------------------------------------------------------------------------------

-- Head --
Stacktrace:
at com.survivant.mod.SurvivantThirstBar.onRenderGameOverlay(SurvivantThirstBar.java:83)
at net.minecraftforge.fml.common.eventhandler.ASMEventHandler_13_SurvivantThirstBar_onRenderGameOverlay_RenderGameOverlayEvent.invoke(.dynamic)
at net.minecraftforge.fml.common.eventhandler.ASMEventHandler.invoke(ASMEventHandler.java:49)
at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:140)
at net.minecraftforge.client.GuiIngameForge.post(GuiIngameForge.java:854)
at net.minecraftforge.client.GuiIngameForge.renderExperience(GuiIngameForge.java:561)
at net.minecraftforge.client.GuiIngameForge.renderGameOverlay(GuiIngameForge.java:145)

-- Affected level --
Details:
Level name: MpServer
All players: 1 total; [EntityPlayerSP['Player338'/168, l='MpServer', x=8,50, y=65,00, z=8,50]]
Chunk stats: MultiplayerChunkCache: 0, 0
Level seed: 0
Level generator: ID 01 - flat, ver 0\. Features enabled: false
Level generator options:
Level spawn location: -52,00,4,00,414,00 - World: (-52,4,414), Chunk: (at 12,0,14 in -4,25; contains blocks -64,0,400 to -49,255,415), Region: (-1,0; contains chunks -32,0 to -1,31, blocks -512,0,0 to -1,255,511)
Level time: 0 game time, 0 day time
Level dimension: 0
Level storage version: 0x00000 - Unknown?
Level weather: Rain time: 0 (now: false), thunder time: 0 (now: false)
Level game mode: Game mode: survival (ID 0). Hardcore: false. Cheats: false
Forced entities: 1 total; [EntityPlayerSP['Player338'/168, l='MpServer', x=8,50, y=65,00, z=8,50]]
Retry entities: 0 total; []
Server brand: fml,forge
Server type: Integrated singleplayer server
Stacktrace:
at net.minecraft.client.multiplayer.WorldClient.addWorldInfoToCrashReport(WorldClient.java:383)
at net.minecraft.client.Minecraft.addGraphicsAndWorldToCrashReport(Minecraft.java:2645)
at net.minecraft.client.Minecraft.run(Minecraft.java:409)
at net.minecraft.client.main.Main.main(Main.java:116)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
at net.minecraft.launchwrapper.Launch.main(Launch.java:28)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at net.minecraftforge.gradle.GradleStartCommon.launch(GradleStartCommon.java:97)
at GradleStart.main(GradleStart.java:26)

– System Details --
Details:
Minecraft Version: 1.8.9
Operating System: Windows 8.1 (amd64) version 6.3
Java Version: 1.8.0_121, Oracle Corporation
Java VM Version: Java HotSpot(TM) 64-Bit Server VM (mixed mode), Oracle Corporation
Memory: 658588368 bytes (628 MB) / 1038876672 bytes (990 MB) up to 1038876672 bytes (990 MB)
JVM Flags: 3 total; -Xincgc -Xmx1024M -Xms1024M
IntCache: cache: 0, tcache: 0, allocated: 0, tallocated: 0
FML: MCP 9.19 Powered by Forge 11.15.1.1747 4 mods loaded, 4 mods active
States: 'U' = Unloaded 'L' = Loaded 'C' = Constructed 'H' = Pre-initialized 'I' = Initialized 'J' = Post-initialized 'A' = Available 'D' = Disabled 'E' = Errored
UCHIJAAAA mcp{9.19} [Minecraft Coder Pack] (minecraft.jar)
UCHIJAAAA FML{8.0.99.99} [Forge Mod Loader] (forgeSrc-1.8.9-11.15.1.1747.jar)
UCHIJAAAA Forge{11.15.1.1747} [Minecraft Forge] (forgeSrc-1.8.9-11.15.1.1747.jar)
UCHIJAAAA survivant{1.0.0} [Mod Survivant] (bin)
Loaded coremods (and transformers):
GL info: ' Vendor: 'Intel' Version: '4.0.0 - Build 10.18.10.3621' Renderer: 'Intel(R) HD Graphics 4000'
Launched Version: 1.8.9
LWJGL: 2.9.4
OpenGL: Intel(R) HD Graphics 4000 GL version 4.0.0 - Build 10.18.10.3621, Intel
GL Caps: Using GL 1.3 multitexturing.
Using GL 1.3 texture combiners.
Using framebuffer objects because OpenGL 3.0 is supported and separate blending is supported.
Shaders are available because OpenGL 2.1 is supported.
VBOs are available because OpenGL 1.5 is supported.

Using VBOs: No
Is Modded: Definitely; Client brand changed to 'fml,forge'
Type: Client (map_client.txt)
Resource Packs:
Current Language: Français (France)
Profiler Position: N/A (disabled)
CPU: 4x Intel(R) Core(TM) i5-3210M CPU @ 2.50GHz

Alexandre1156

Si tu n’as fait aucun changement par rapport à la dernière fois, la ligne 83 correspond à ça :


mc.ingameGUI.drawTexturedModalRect(posX + 1, posY + 1, 0, 9, (int)thirst.getThirst(), 7);

Je pense que la variable *thirst *doit être null, ce qui cause le crash.
Pour voir si j’ai raison, rajoute cette ligne avant :


System.out.println("isThirstNull : "+(thirst == null));

Si elle est sur true, il faudrait regarder du côté de le class **CapabilityHandler **et voir si le capability est bien assigné au joueur.
Conseil : regarde cette vidéo pour apprendre à lire un crash report : https://www.youtube.com/watch?v=Uipvf_DmyQc

elx9000

Effectivement c’est sur true, par contre dans capabilityHandler j’ai l’impression qu’elle est bien assignée au joueur. Merci par avance.

elx9000

Pas d’idée pour corriger la class CapabilityHandler ?Car moi je sais pas quoi faire. Merci par avance.

Alexandre1156

@‘elx9000’:

Pas d’idée pour corriger la class CapabilityHandler ?Car moi je sais pas quoi faire. Merci par avance.

Faire du debugging.
Dans ton cas, vérifier que l’event fonctionne, vérifier que l’event arrive jusqu’à addCapability , vérifier que le capability est bien assigné au joueur, vérifie que THIRST_CAP n’est pas null…

elx9000

Justement, je suis désolé mais ça je ne sais pas faire , comment il faut que je m’y prenne ?

AymericRed

Il faut que tu mettes des “System.out.println(“fonction truc”);” dans les fonctions concernées, si ça apparaît dans la console c’est bon, sinon c’est quelle n’est pas appelée.

elx9000

J’ai mis un System.out.println(“attachCapability OK”); dans la fonction attachCapability de la class CapabilityHandler, mais cette phrase n’est pas apparu. J’ai compris que c’est le fait que thirstValue soit null mais je n’ai aucune idée pour que ça fonctionne.

Merci par avance pour votre aide.

Alexandre1156

Dans la class Common Proxy, fais comme ceci :


public void registerRenders()
    {
       //Tu ne dois rien mettre ici étant donner que le render ne se fait que côté client
    }

    public void init(FMLInitializationEvent e)
    {
    CapabilityManager.INSTANCE.register(IThirst.class, new ThirstStorage(), ThirstCap.class);
    MinecraftForge.EVENT_BUS.register(new CapabilityHandler());
    }

Et dans ta class principale :


 @EventHandler
    public void init (FMLInitializationEvent event)
    {        
        proxy.registerRenders();

        proxy.init(event);

        MinecraftForge.EVENT_BUS.register(new SurvivantBlockEvent());       
        MinecraftForge.EVENT_BUS.register(new SurvivantThirstBar());
        EntityHandler.RegisterFish(EntityFish.class, "Fish");
        EntityHandler.RegisterSalmon(EntitySalmon.class, "Salmon");
        EntityHandler.RegisterRay(EntityRay.class, "Ray");
    }

L’event ne s’enregistré pas étant donné que proxy.registerRenders() ne s’effectue que dans ClientProxy et non dans **CommonProxy *(à moins de mettre super.registerRenders()*).

elx9000

OK, le système en lui même fonctionne bien. J’ai fait ceci, mais il reste quelque problème :


package com.survivant.mod;

import com.survivant.mod.Capabilities.IThirst;
import com.survivant.mod.Capabilities.ThirstProvider;

import net.minecraft.client.Minecraft;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.potion.Potion;
import net.minecraft.potion.PotionEffect;
import net.minecraft.util.ChatComponentText;
import net.minecraft.util.DamageSource;
import net.minecraft.util.ResourceLocation;
import net.minecraftforge.client.event.GuiScreenEvent;
import net.minecraftforge.client.event.RenderGameOverlayEvent;
import net.minecraftforge.client.event.RenderGameOverlayEvent.ElementType;
import net.minecraftforge.event.AttachCapabilitiesEvent;
import net.minecraftforge.event.entity.player.PlayerEvent;
import net.minecraftforge.event.entity.player.PlayerSleepInBedEvent;
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;
import net.minecraftforge.fml.common.gameevent.PlayerEvent.PlayerLoggedInEvent;
import net.minecraftforge.fml.common.gameevent.PlayerEvent.PlayerRespawnEvent;
import net.minecraftforge.fml.common.gameevent.TickEvent;

public class SurvivantThirstBar
{
   public static int i = 0;

   @SubscribeEvent
   public void onPlayerLogsIn(PlayerLoggedInEvent event)
   {
       EntityPlayer player = event.player;
       IThirst thirst = player.getCapability(ThirstProvider.THIRST_CAP, null);
   }

   @SubscribeEvent
   public void onPlayerSleep(PlayerSleepInBedEvent event)
   {
       EntityPlayer player = event.entityPlayer;
       if (player.worldObj.isRemote)
       {
           return;
       }
       IThirst thirst = player.getCapability(ThirstProvider.THIRST_CAP, null);
   }

   @SubscribeEvent
   public void onPlayerClone(PlayerEvent.Clone event)
   {
       EntityPlayer player = event.entityPlayer;
       IThirst thirst = player.getCapability(ThirstProvider.THIRST_CAP, null);
       thirst.set(86);
   }

   @SubscribeEvent
   public void playerTick(TickEvent.PlayerTickEvent event)
   {    
       EntityPlayer player = event.player;
       IThirst thirst = player.getCapability(ThirstProvider.THIRST_CAP, null);
       if (!event.player.capabilities.isCreativeMode)
       {
           i++;
           if (i == 800)
           {
               thirst.consume(1);
               System.out.println("thirst = " + thirst.getThirst());
               i = 0;
           }
           if (thirst.getThirst() == 0)
           {
               event.player.attackEntityFrom(ModSurvivant.damageSourceDehydration, 1.0F);
           }
       }
   }

   @SubscribeEvent
   public void onRenderGameOverlay(RenderGameOverlayEvent event)
   {
       if (!event.isCancelable() && event.type == ElementType.EXPERIENCE)
       {
           Minecraft mc = Minecraft.getMinecraft();

           if (!mc.thePlayer.capabilities.isCreativeMode)
           {
               EntityPlayer player = mc.thePlayer;
               IThirst thirst = player.getCapability(ThirstProvider.THIRST_CAP, null);

               int posX = event.resolution.getScaledWidth() / 2 + 7;
               int posY = event.resolution.getScaledHeight() - 50;

               mc.renderEngine.bindTexture(new ResourceLocation("survivant:textures/gui/thirstBar.png"));
               mc.ingameGUI.drawTexturedModalRect(posX, posY, 0, 0, 88, 9);
               mc.ingameGUI.drawTexturedModalRect(posX + 1, posY + 1, 0, 9, (int)thirst.getThirst(), 7);
           }
       }
   }  
}

Les problèmes sont les suivants :

quand la variable thirst arrive à 0 (vérifié à l’aide d’un System.out.println("thirst = "+thirst.getThrist())), normalement je suis censé prendre des dégâts, or il ne se passe rien même si la ligne est présente.
quand je me déco-reco ou que j’étais sur un monde puis que je ferme le jeu et que je reviens. La variable thrist reste bien la même quelle était avant la deconnection (vérifié grâce au System.out.println(…)), pourtant en jeu, le rendu graphique de la barre de soif est total : graphiquemment la barre est pleine alors que ce n’est pas le cas dans la console.
Merci par avance pour votre aide.

Alexandre1156

@‘elx9000’:

quand la variable thirst arrive à 0 (vérifié à l’aide d’un System.out.println("thirst = "+thirst.getThrist())), normalement je suis censé prendre des dégâts, or il ne se passe rien même si la ligne est présente.

quand je me déco-reco ou que j’étais sur un monde puis que je ferme le jeu et que je reviens. La variable thrist reste bien la même quelle était avant la deconnection (vérifié grâce au System.out.println(…)), pourtant en jeu, le rendu graphique de la barre de soif est total : graphiquemment la barre est pleine alors que ce n’est pas le cas dans la console.
Merci par avance pour votre aide.

Problème 1 :
-La variable *i *est augmenté côté Client et Serveur. Or, non seulement i augmente 2x plus vite (800 / 40ticks = 20s, or ici : 800 / 80ticks = 10s), mais en plus c’est à chaque fois le côté Client qui va atteindre 800 et donc descendre thirst de 1 alors que côté Server, thirst reste à 85. Les dégâts se faisant côté Server, il ne se passe rien car t’a condition est vrai côté Client mais faux côté Server. La solution est de tout faire côté Server (grâce à event.side.server) et lorsque thirst diminue, le faire descendre aussi côté Client via les packets.
Problème 2 :
-Les capabilities ne sont sauvegardé que côté serveur, or RenderGameOverlayEvent est côté client, il faut donc les syncer en utilisant les packets lorsque le joueur rejoint un monde[size=small.]

elx9000

Je suis désolé mais je ne comprends pas ce que c’est que les packets et comment on les utilises. Merci d’avance.