[1.7.x] Crée de nouvelle valeurs (données) joueurs sauvegarder (exemple : mana)

robin4002

Oui.

BlackDrag00n

Classe : ExtendedEntityPropMoney
Package : fr.paramystick.tutoriel.extprop
:::


package fr.paramystick.tutoriel.extprop;

import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import fr.paramystick.tutoriel.core.CommonProxy;
import fr.paramystick.tutoriel.core.TutorielMod;
import fr.paramystick.tutoriel.packets.PacketMoney;
import net.minecraft.entity.Entity;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.EntityPlayerMP;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.world.World;
import net.minecraftforge.common.IExtendedEntityProperties;
import net.minecraftforge.event.entity.EntityEvent.EntityConstructing;
import net.minecraftforge.event.entity.EntityJoinWorldEvent;
import net.minecraftforge.event.entity.living.LivingDeathEvent;

public class ExtendedEntityPropMoney implements IExtendedEntityProperties
{
//Identifiant à la propriété
public final static String EXT_PROP_NAME = "ExtPropMoney";
private final EntityPlayer player;
public long Money, maxMoney;
//–---------------------------------------------------------------------------------------

//Le constructeur. Il prend en paramètre le player auquel nous allons toucher.
public ExtendedEntityPropMoney(EntityPlayer player)
{
this.player = player;
this.Money = 12;
this.maxMoney = 20;
}
//-----------------------------------------------------------------------------------------

//méthodes concernant la sauvegarde et l'obtention de ces données
public static final void register(EntityPlayer player)
{
player.registerExtendedProperties(ExtendedEntityPropMoney.EXT_PROP_NAME, new ExtendedEntityPropMoney(player));
}

public static final ExtendedEntityPropMoney get(EntityPlayer player)
{
return (ExtendedEntityPropMoney) player.getExtendedProperties(EXT_PROP_NAME);
}
//-----------------------------------------------------------------------------------------

// Synchronisation client/serveur
public final void sync() {
PacketMoney packetMoney = new PacketMoney(this.maxMoney, this.Money);
//La ligne suivante dépend de votre manière d'envoyer les packets. Celle-ci vient de mon mod, je ne la changerais pas car je ne peux l'appliquer à votre mod, mais elle reste bonne pour un exemple.
TutorielMod.TutorielModPacketHandler.sendToServer(packetMoney);

if (!player.worldObj.isRemote) {
EntityPlayerMP player1 = (EntityPlayerMP) player;
//Ici, même chose que précédemment, sauf que le packet est envoyé au player.
TutorielMod.TutorielModPacketHandler.sendTo(packetMoney, player1);
}
}
//-----------------------------------------------------------------------------------------

//Encore une méthode de sauvegarde
private static String getSaveKey(EntityPlayer player)
{
return player.getDisplayName() + ":" + EXT_PROP_NAME;
}
//-----------------------------------------------------------------------------------------

public static void saveProxyData(EntityPlayer player) {
ExtendedEntityPropMoney playerData = ExtendedEntityPropMoney.get(player);
NBTTagCompound savedData = new NBTTagCompound();

playerData.saveNBTData(savedData);
CommonProxy.storeEntityData(getSaveKey(player), savedData);
}

public static void loadProxyData(EntityPlayer player) {
ExtendedEntityPropMoney playerData = ExtendedEntityPropMoney.get(player);
NBTTagCompound savedData = CommonProxy.getEntityData(getSaveKey(player));

if (savedData != null)
{
playerData.loadNBTData(savedData);
}

playerData.sync();
}
//-----------------------------------------------------------------------------------------

@Override
public void saveNBTData(NBTTagCompound compound)
{
// TODO Auto-generated method stub
NBTTagCompound properties = new NBTTagCompound();

properties.setLong("Money", this.Money);
properties.setLong("maxMoney", this.maxMoney);

compound.setTag(EXT_PROP_NAME, properties);
}

@Override
public void loadNBTData(NBTTagCompound compound)
{
// TODO Auto-generated method stub
NBTTagCompound properties = (NBTTagCompound) compound.getTag(EXT_PROP_NAME);
this.Money = properties.getLong("Money");
this.maxMoney = properties.getLong("maxMoney");

}

@Override
public void init(Entity entity, World world)
{
// TODO Auto-generated method stub

}
//-----------------------------------------------------------------------------------------

// Pour des commandes
public boolean pay(int amount)
{
boolean sufficient = amount <= this.Money;

if (sufficient)
{
this.Money -= amount;
this.sync();
} else {
return false;
}

return sufficient;
}

public void addMoney(int amount)
{
this.Money += amount;
this.sync();
}

public long getMoney()
{
return this.Money;
}

public void setMoney(int newMoney)
{
this.Money = newMoney;
this.sync();
}

//-----------------------------------------------------------------------------------------
}

:::

Classe : InfoOverlay
Package : fr.paramystick.tutoriel.hud
:::


package fr.paramystick.tutoriel.hud;

import cpw.mods.fml.client.FMLClientHandler;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import net.minecraft.client.Minecraft;
import net.minecraft.client.gui.FontRenderer;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.MathHelper;
import net.minecraft.util.MovingObjectPosition;
import net.minecraft.world.World;
import net.minecraftforge.client.event.RenderGameOverlayEvent;
import net.minecraftforge.client.event.RenderGameOverlayEvent.ElementType;

import java.text.DateFormat;
import java.text.SimpleDateFormat;
import java.util.Date;

import fr.paramystick.tutoriel.extprop.ExtendedEntityPropMoney;

//------------------------------------------------------------------------------------------

public class InfoOverlay
{
private Minecraft mc = FMLClientHandler.instance().getClient();
private final EntityPlayer player = mc.thePlayer;
private String fpsString;
private long MoneyLong,maxMoneyLong;

@SubscribeEvent
public void renderGameOverlay(RenderGameOverlayEvent.Post event)
{
if(event.type != ElementType.EXPERIENCE)
{
return;
}
fpsString = mc.debug.split(",", 2)[0];
//MoneyLong = 5;
//maxMoneyLong = 10;
MoneyLong = ExtendedEntityPropMoney.get(player).Money;
maxMoneyLong = ExtendedEntityPropMoney.get(player).maxMoney;

FMLClientHandler.instance().getClient().fontRenderer.drawStringWithShadow("Fps : "+fpsString, 5, 5, 0xffffff);
FMLClientHandler.instance().getClient().fontRenderer.drawStringWithShadow("Money : "+MoneyLong+" / "+maxMoneyLong, 5, 25, 0xffffff);
}
}

:::

robin4002

Le problème ne vient pas d’ici.
Je peux avoir ta classe avec tout tes événements + ta classe principale ?

BlackDrag00n

Classe : TutorielModEventHandler
Package : fr.paramystick.tutoriel.core
:::


package fr.paramystick.tutoriel.core;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraftforge.event.entity.EntityJoinWorldEvent;
import net.minecraftforge.event.entity.EntityEvent.EntityConstructing;
import net.minecraftforge.event.entity.living.LivingDeathEvent;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import fr.paramystick.tutoriel.extprop.ExtendedEntityPropMoney;

public class TutorielModEventHandler
{
private CommonProxy proxy;

@SubscribeEvent
public void onEntityConstructing(EntityConstructing event)
{
if (event.entity instanceof EntityPlayer && ExtendedEntityPropMoney.get((EntityPlayer) event.entity) == null)
{
ExtendedEntityPropMoney.register((EntityPlayer) event.entity);
}

if (event.entity instanceof EntityPlayer && event.entity.getExtendedProperties(ExtendedEntityPropMoney.EXT_PROP_NAME) == null)
{
event.entity.registerExtendedProperties(ExtendedEntityPropMoney.EXT_PROP_NAME, new ExtendedEntityPropMoney((EntityPlayer) event.entity));
}
}

@SubscribeEvent
public void onEntityJoinWorld(EntityJoinWorldEvent event)
{
if (!event.entity.worldObj.isRemote && event.entity instanceof EntityPlayer)
{
NBTTagCompound playerData = proxy.getEntityData(((EntityPlayer) event.entity).getDisplayName());
if (playerData != null)
{
((ExtendedEntityPropMoney) (event.entity.getExtendedProperties(ExtendedEntityPropMoney.EXT_PROP_NAME))).loadNBTData(playerData);
}

((ExtendedEntityPropMoney) (event.entity.getExtendedProperties(ExtendedEntityPropMoney.EXT_PROP_NAME))).sync();
}
}

@SubscribeEvent
public void onLivingDeathEvent(LivingDeathEvent event)
{
if (!event.entity.worldObj.isRemote && event.entity instanceof EntityPlayer)
{
NBTTagCompound playerData = new NBTTagCompound();
((ExtendedEntityPropMoney) (event.entity.getExtendedProperties(ExtendedEntityPropMoney.EXT_PROP_NAME))).saveNBTData(playerData);
proxy.storeEntityData(((EntityPlayer) event.entity).getDisplayName(), playerData);
ExtendedEntityPropMoney.saveProxyData((EntityPlayer) event.entity);
} else {

}
}
}

:::

Classe : TutorielMod
Package : fr.paramystick.tutoriel.core

:::


package fr.paramystick.tutoriel.core;

import net.minecraftforge.common.MinecraftForge;
import cpw.mods.fml.common.Mod;
import cpw.mods.fml.common.Mod.EventHandler;
import cpw.mods.fml.common.Mod.Instance;
import cpw.mods.fml.common.SidedProxy;
import cpw.mods.fml.common.event.FMLInitializationEvent;
import cpw.mods.fml.common.event.FMLPostInitializationEvent;
import cpw.mods.fml.common.event.FMLPreInitializationEvent;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import fr.minecraftforgefrance.ffmtlibs.network.FFMTPacketHandler;
import fr.minecraftforgefrance.ffmtlibs.network.PacketManager;
import fr.paramystick.tutoriel.hud.InfoOverlay;

@Mod(modid = "TutorielMod", name = "Tutoriel Mod", version = "0.0.1", dependencies = "required-after:ffmtlibs")
public class TutorielMod
{
/**
* TutorielMod's mod Identifier
*/
public static final String MODID = "TutorielMod";

/**
* La suite va se mettre ici
*/
@Instance(MODID)
public static TutorielMod modInstance;

/**
* Initialisation si on est coter SERVEUR ou CLIENT
*/
@SidedProxy(clientSide = "fr.paramystick.tutoriel.core.ClientProxy", serverSide = "fr.paramystick.tutoriel.core.CommonProxy")
public static CommonProxy proxy;

/**
* TutorielMod packethandler
*/
public static final PacketManager TutorielModPacketHandler = new PacketManager("fr.paramystick.tutoriel.packets", MODID, "TutorielMod");

//FMLPreInitializationEvent : c'est la première exécuté, elle va servir pour lire le fichier de configuration, enregistrer les blocs, les items, la plupart des choses qui se trouve dans le GameRegistry
@EventHandler
public void preInit(FMLPreInitializationEvent event)
{

}

//FMLInitializationEvent : toutes les choses secondaires iront ici, enregistrement des events, des rendus, des recettes, etc …
@EventHandler
public void init(FMLInitializationEvent event)
{
proxy.registerRender();
}

//FMLPostInitializationEvent : la dernière chose faite avant le lancement du jeu, permet de principalement d'interagir avec les autres mods.
@EventHandler
public void postInit(FMLPostInitializationEvent event)
{
MinecraftForge.EVENT_BUS.register(new TutorielModEventHandler());
MinecraftForge.EVENT_BUS.register(new InfoOverlay());
}

}

:::

isador

les:
MinecraftForge.EVENT_BUS.register(new TutorielModEventHandler());
MinecraftForge.EVENT_BUS.register(new InfoOverlay());
met les dans le Init

BlackDrag00n

Cela ne change rien, en faite le jeu ce lance mais quand je veux rejoindre un monde j’ai le crash avec les erreurs en question

En gros sa crash car il cherche a récupérer les valeurs de la variable Money pour l’afficher dans mon GUI mais il me dit qu’il est “NULL”, l’auto-completion fonctionne il trouve bien la méthode que je demande.

La derniere fois que j’ai eu cette erreur :

java.lang.NullPointerException: Unexpected error

c’était due a sa :
@robin4002:

java.lang.NullPointerException veut dire que un de tes objets est null.
Et dans ton cas, c’est l’objet mc qui est null.
private Minecraft mc;
Remplace par :
private Minecraft mc = FMLClientHandler.instance().getClient();

Donc je suppose que l’erreur que j’ai est due au fait que dans :

maxMoneyLong = ExtendedEntityPropMoney.get(player).maxMoney;

j’ai l’objet “ExtendedEntityPropMoney.get(player).maxMoney;” qui est “NULL”
Mais je vois pas pourquoi ?

Superloup10

Relance le jeu en remplaçant :


maxMoneyLong = ExtendedEntityPropMoney.get(player).maxMoney;

par :


maxMoneyLong
= ExtendedEntityPropMoney
.get(player)
.maxMoney;

afin de savoir avec précision, ce qui est nul.

BlackDrag00n

Visiblement sa serait le “.get(player);”

robin4002

Heu tu es sur quand sans le code pour le rendu ça ne crash pas non plus ?
Car c’est le gros bordel dans ta classe TutorielModEventHandler.
La variable proxy est null, elle n’a rien à faire la, tu devais normalement utiliser celle de la classe principale

@SubscribeEvent
public void onEntityConstructing(EntityConstructing event)
{
if (event.entity instanceof EntityPlayer && ExtendedEntityPropMoney.get((EntityPlayer) event.entity) == null)
{
ExtendedEntityPropMoney.register((EntityPlayer) event.entity);
}

if (event.entity instanceof EntityPlayer && event.entity.getExtendedProperties(ExtendedEntityPropMoney.EXT_PROP_NAME) == null)
{
event.entity.registerExtendedProperties(ExtendedEntityPropMoney.EXT_PROP_NAME, new ExtendedEntityPropMoney((EntityPlayer) event.entity));
}
}

et ici tu fais exactement deux fois la même chose.

@SubscribeEvent
public void onEntityConstructing(EntityConstructing event)
{
if (event.entity instanceof EntityPlayer && ExtendedEntityPropMoney.get((EntityPlayer) event.entity) == null)
{
ExtendedEntityPropMoney.register((EntityPlayer) event.entity);
}
}

Suffit.

BlackDrag00n

Bah j’ai suivi le tuto qui se trouve > ICI <

robin4002

:va gronder gugu:

BlackDrag00n

^^ lol, mais j’ai essayer comme tu m’a dit j’ai toujours le même problème.

Sinon oui je suis sur si j’inverse les lignes commenté suivant comme ceci :

MoneyLong = 5; // Si je met des valeurs fixe (Sa fonctionne)
maxMoneyLong = 10; // Si je met des valeurs fixe (Sa fonctionne)
//MoneyLong = ExtendedEntityPropMoney.get(player).Money; // Si il va chercher la valeur initialiser sa bug
//maxMoneyLong = ExtendedEntityPropMoney.get(player).maxMoney;// Si il va chercher la valeur initialiser sa bug

Alors, je n’est plus aucun crash !

Et en ayant suivi le conseil de “Superloup10”, sa viendrais visiblement de “.get(player);”

Je voudrais démarrer le jeu en mode “Debug” tous en faisant une surveillance sur une variable je sais que sous Visual Studio c’est faisable, mais depuis éclipse on peut le faire ? si oui pouvez vous me donner la méthode ?

BlackDrag00n

Démarrage en mode “Debug”
Breakpoint sur chaques lignes suivante :


maxMoneyLong
= ExtendedEntityPropMoney
.get(player)
.maxMoney;

Si je regarde les variables a ce moment la j’ai “this.player” qui est “NULL”.

j’ai modifier le code :

MoneyLong = ExtendedEntityPropMoney.get(player).Money;
maxMoneyLong = ExtendedEntityPropMoney.get(player).maxMoney;

par :

MoneyLong = ExtendedEntityPropMoney.get(mc.thePlayer).Money;
maxMoneyLong = ExtendedEntityPropMoney.get(mc.thePlayer).maxMoney;

Le jeu ne crash plus, je ne sais pas pour qu’elle raison …

HUD :

FMLClientHandler.instance().getClient().fontRenderer.drawStringWithShadow("Money : "+MoneyLong+" / "+maxMoneyLong, 5, 25, 0xffffff);

et j’ai un problème c’est qu’il m’affiche les valeur suivante sur mon HUD :

Money : 20 / 0

Alors qu’il devrait m’afficher (pour mes test) :

Money : 12 / 20

Si sa peu aider a débloquer mon problème

robin4002

Tu peux renvoyer la classe de ton paquet ?

BlackDrag00n

Voili voilou, désoler vraiment de t’embêter mais je comprend vraiment pas, j’ai même refait le projets entièrement a l’aide des différents tuto et j’ai toujours la même choses

Classe : PacketMoney
Package : fr.paramystick.tutoriel.packets

:::


package fr.paramystick.tutoriel.packets;

import java.io.IOException;

import net.minecraft.entity.player.EntityPlayer;
import fr.minecraftforgefrance.ffmtlibs.network.FFMTPacket;
import fr.paramystick.tutoriel.money.ExtendedPlayerMoney;
import io.netty.buffer.ByteBuf;
import io.netty.channel.ChannelHandlerContext;

public class PacketMoney extends FFMTPacket
{
private long maxMoney, money;

public PacketMoney()
{

}

public PacketMoney(long maxMoney, long money)
{
this.maxMoney = maxMoney;
this.money = money;
}

@Override
public void writeData(ByteBuf buffer) throws IOException
{
buffer.writeLong(maxMoney);
buffer.writeLong(money);
}

@Override
public void readData(ByteBuf buffer)
{
this.maxMoney = buffer.readInt();
this.money = buffer.readInt();
}

@Override
public void handleClientSide(EntityPlayer player)
{
ExtendedPlayerMoney props = ExtendedPlayerMoney.get(player);
props.maxMoney = this.maxMoney;
props.money = this.money;
}

@Override
public void handleServerSide(EntityPlayer player)
{
ExtendedPlayerMoney props = ExtendedPlayerMoney.get(player);
props.maxMoney = this.maxMoney;
props.money = this.money;
}
}

:::

Ici vers la ligne 30, il y a :

buffer.writeLong(maxMoney);
buffer.writeLong(money);

il ne faudrait pas plutôt mettre ceci :

buffer.writeLong(this.maxMoney);
buffer.writeLong(this.money);

Juste pour info comme je t’est dit j’ai tous refait pour reprendre quelques choses a zéro j’en est donc profitez pour refaire une hierarchie donc ne t’etonne pas si la classe s’appel :

“ExtendedPlayerMoney” à la place de “ExtendedEntityPropMoney”

idem il n’est plus dans le package d’avant il est maintenant dans :

“import fr.paramystick.tutoriel.money;” au lieu de “fr.paramystick.tutoriel.extprop”

Le reste j’ai remis la même chose, sauf que je suis repartit de zero en suivant les tutos et mon problème est toujours présent vis à vis de la récupération de la valeur “Money” :

Soit c’est un problème de récupération de la valeur.
Soit c’est la valeur qui ne s’enregistre pas correctement.

Ma hierarchie de mon projet, sa peut aider :

src/main/java
|>fr.paramystick.tutoriel.core
|>ClientProxy.java
|>CommonProxy.java
|>TutorielModEventHandler.java
|>TutorielMod.java
|>fr.paramystick.tutoriel.hud
|>InfoOverlay.java
|>fr.paramystick.tutoriel.money
|>ExtendedPlayerMoney.java
|>fr.paramystick.tutoriel.packets
|>PacketMoney.java

BlackDrag00n

#J’ai trouvé !!!(J’ai trouvé !!!)

L’erreur se situe ici :

@Override
public void writeData(ByteBuf buffer) throws IOException
{
buffer.writeInt(maxMoney);
buffer.writeInt(money);
}

C’est encore une erreur avec les type de variable “Int” et “Long” …

Il suffit de remplace par ceci :

@Override
public void writeData(ByteBuf buffer) throws IOException
{
buffer.writeLong(maxMoney);
buffer.writeLong(money);
}

Je le rajoute dans les commentaires du tuto pour que le problème soit corriger dans le tuto …

Par contre je pense que c’est normal si je déclare des variables comme ceci :

public ExtendedPlayerMoney(EntityPlayer player) {
this.player = player;
this.money = 5000;
this.maxMoney = 10000;
}

Je crée un nouveau monde et me connect dessus tous va bien dans mon HUD je récupérer bien les valeurs.

Si ensuite je modifie le code précédent par :

public ExtendedPlayerMoney(EntityPlayer player) {
this.player = player;
this.money = 5000;
this.maxMoney = 20000;
}

Je me retrouve toujours dans le HUD avec ceci :

Money : 5000 / 10000

Je suppose qu’il faudrait une sorte d’update si le moneyMax du joueur ne correpond pas au moneyMax définie ?

robin4002

Dans ce cas il ne faudrait pas enregistrer la variable maxMoney dans le tag nbt du joueur, ça devrait être suffisant.
Car le problème actuellement c’est que ta variable maxMoney est enregistré dans le tag du joueur, donc si tu change la valeur par défaut par la suite, ça va garder celle dans le tag nbt du joueur

BlackDrag00n

D’accord oui effectivement vu sous cet angle

Sinon pourrais tu me guider sur le bon chemin ?

Je cherche à faire un système similaire à la faim de minecraft, c’est a dire que cette barre ce vide en permanence comme la faim, elle se vide encore plus dans la cas ou on est en activité physique (Sauté, courir, minée, etc …) on pourrait parler d’une sorte de système de soif par exemple ou encore d’énergie.

robin4002

Il faut passer par les event pour ça. (le tick du joueur surtout)

BlackDrag00n

D’accord merci, je vais chercher de ce côté là me renseigner et encore apprendre je passe ce sujet en résolu.

sa serait sur l’event “onTickClient” ?