Extended Entity Props. qui ne se savent pas

[robin4002]

Non, ce n’est pas ça le problème. La sauvegarde est sensé se faire automatiquement. Le problème se trouve dans la fonction saveNBTData

​    @Override

    public void saveNBTData(NBTTagCompound compound)
    {

        NBTTagCompound properties = new NBTTagCompound();
        compound.setTag(EXT_PROP_NAME, properties);
        properties.setDouble("Mana", this.mana);
        properties.setDouble("MaxMana", this.maxMana);
        properties.setString("spellRight", this.spellRight);
        properties.setString("spellLeft", this.spellLeft);
        properties.setInteger("spellsListSize", this.spellsList.size());
        if(this.spellsList != null){
            for(String spell : this.spellsList)
            {
                properties.setString("spellsList", spell);
            }
        }
    }

Regarde bien ce que tu fais. Dans un premier temps tu créé une variable local de type NBTTagCompound, puis tu écris dans le nbt tag d’entity ta variable, et seulement après tu ajoutes des choses dans ta variable local. Donc au final rien est écrit, sauf un tab nbt vide.
Cela devrait être comme ça :

​    @Override

    public void saveNBTData(NBTTagCompound compound)
    {

        NBTTagCompound properties = new NBTTagCompound();
        properties.setDouble("Mana", this.mana);
        properties.setDouble("MaxMana", this.maxMana);
        properties.setString("spellRight", this.spellRight);
        properties.setString("spellLeft", this.spellLeft);
        properties.setInteger("spellsListSize", this.spellsList.size());
        if(this.spellsList != null){
            for(String spell : this.spellsList)
            {
                properties.setString("spellsList", spell);
            }
        }
compound.setTag(EXT_PROP_NAME, properties);
}

[sosoh]

J’ai corrigé mais ca ne marche pas

​package fr.sosoh.hogsmod.common.event;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.util.ChatComponentText;
import net.minecraftforge.event.entity.EntityEvent.EntityConstructing;
import net.minecraftforge.event.entity.EntityJoinWorldEvent;
import net.minecraftforge.event.entity.living.LivingDeathEvent;
import net.minecraftforge.event.entity.living.LivingEvent;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import cpw.mods.fml.common.gameevent.PlayerEvent;
import fr.sosoh.hogsmod.common.entity.props.ExtendedEntityProps;
import fr.sosoh.hogsmod.proxy.CommonProxy;

public class EventHandlerExtendedProps {

private CommonProxy proxy;
public int ticks;

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

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

@SubscribeEvent
public void onEntityJoinWorld(EntityJoinWorldEvent event) {
if (!(event.entity.worldObj.isRemote) && event.entity instanceof EntityPlayer) {

EntityPlayer player = (EntityPlayer)event.entity;
NBTTagCompound playerData = proxy.getEntityData(((EntityPlayer) event.entity).getDisplayName());

if (playerData != null) {
((ExtendedEntityProps) (event.entity.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).loadNBTData(playerData);
}

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

@SubscribeEvent
public void onPlayerLoggedOut(PlayerEvent.PlayerLoggedOutEvent event){
if (!(event.player.worldObj.isRemote)) {

NBTTagCompound playerData = proxy.getEntityData((event.player).getDisplayName());

if (playerData != null) {
((ExtendedEntityProps) (event.player.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).saveNBTData(playerData);
}

((ExtendedEntityProps) (event.player.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).sync();
}
}
}

[SCAREX]

@‘sosoh’:

J’ai corrigé mais ca ne marche pas

​package fr.sosoh.hogsmod.common.event;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.util.ChatComponentText;
import net.minecraftforge.event.entity.EntityEvent.EntityConstructing;
import net.minecraftforge.event.entity.EntityJoinWorldEvent;
import net.minecraftforge.event.entity.living.LivingDeathEvent;
import net.minecraftforge.event.entity.living.LivingEvent;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import cpw.mods.fml.common.gameevent.PlayerEvent;
import fr.sosoh.hogsmod.common.entity.props.ExtendedEntityProps;
import fr.sosoh.hogsmod.proxy.CommonProxy;

public class EventHandlerExtendedProps {

private CommonProxy proxy;
public int ticks;

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

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

EntityPlayer player = (EntityPlayer)event.entity;
NBTTagCompound playerData = proxy.getEntityData(((EntityPlayer) event.entity).getDisplayName());

if (playerData != null) {
((ExtendedEntityProps) (event.entity.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).loadNBTData(playerData);
}

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

@SubscribeEvent
public void onPlayerLoggedOut(PlayerEvent.PlayerLoggedOutEvent event){
if (!(event.player.worldObj.isRemote)) {

NBTTagCompound playerData = proxy.getEntityData((event.player).getDisplayName());

if (playerData != null) {
((ExtendedEntityProps) (event.player.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).saveNBTData(playerData);
}

((ExtendedEntityProps) (event.player.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).sync();
}
}
}

Mets un System.out.print (ou un log.info si tu en as un à disposition) dans ta fonction de sauvegarde et regarde s’il s’affiche quand tu te deco/reco etc.

[sosoh]

J’ai mis ca dans mes events, il affiche que c’est null (je peux t’envoyer mon src ?):

​@SubscribeEvent

public void onLivingDeathEvent(LivingDeathEvent event)
{
if (!event.entity.worldObj.isRemote && event.entity instanceof EntityPlayer){
System.out.println("Death");
NBTTagCompound playerData = new NBTTagCompound();
((ExtendedEntityProps)(event.entity.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).saveNBTData(playerData);
proxy.storeEntityData(((EntityPlayer) event.entity).getDisplayName(), playerData);
ExtendedEntityProps.saveProxyData((EntityPlayer) event.entity);
}
}

@SubscribeEvent
public void onEntityJoinWorld(EntityJoinWorldEvent event) {
if (!(event.entity.worldObj.isRemote) && event.entity instanceof EntityPlayer) {
System.out.println("LoggedIn");
EntityPlayer player = (EntityPlayer)event.entity;
NBTTagCompound playerData = proxy.getEntityData(((EntityPlayer) event.entity).getDisplayName());

if (playerData != null) {
System.out.println("LoggedIn-null");
((ExtendedEntityProps) (event.entity.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).loadNBTData(playerData);
}

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

@SubscribeEvent
public void onPlayerLoggedOut(PlayerEvent.PlayerLoggedOutEvent event){
if (!(event.player.worldObj.isRemote)) {

NBTTagCompound playerData = proxy.getEntityData((event.player).getDisplayName());

if (playerData != null) {
((ExtendedEntityProps) (event.player.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).saveNBTData(playerData);
System.out.println("LoggedOut-Null");
}
System.out.println("LoggedOut");
((ExtendedEntityProps) (event.player.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).sync();
}
}

[22:15:48] [Server thread/INFO] [STDOUT]: [fr.sosoh.hogsmod.common.event.EventHandlerExtendedProps:onLivingDeathEvent:32]: Death
[22:15:48] [Server thread/INFO]: sosoh fell out of the world
[22:15:48] [Client thread/INFO]: [CHAT] sosoh fell out of the world
[22:15:48] [Client thread/INFO]: [CHAT] Ouch! That looked like it hurt
[22:15:50] [Server thread/INFO] [STDOUT]: [fr.sosoh.hogsmod.common.event.EventHandlerExtendedProps:onEntityJoinWorld:43]: LoggedIn
[22:15:50] [Server thread/INFO] [STDOUT]: [fr.sosoh.hogsmod.common.event.EventHandlerExtendedProps:onEntityJoinWorld:48]: LoggedIn-null

[SCAREX]

@‘sosoh’:

J’ai mis ca dans mes events, il affiche que c’est null (je peux t’envoyer mon src ?):

​@SubscribeEvent

public void onLivingDeathEvent(LivingDeathEvent event)
{
if (!event.entity.worldObj.isRemote && event.entity instanceof EntityPlayer){
System.out.println("Death");
NBTTagCompound playerData = new NBTTagCompound();
((ExtendedEntityProps)(event.entity.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).saveNBTData(playerData);
proxy.storeEntityData(((EntityPlayer) event.entity).getDisplayName(), playerData);
ExtendedEntityProps.saveProxyData((EntityPlayer) event.entity);
}
}
 
@SubscribeEvent
public void onEntityJoinWorld(EntityJoinWorldEvent event) {
if (!(event.entity.worldObj.isRemote) && event.entity instanceof EntityPlayer) {
System.out.println("LoggedIn");
EntityPlayer player = (EntityPlayer)event.entity;
NBTTagCompound playerData = proxy.getEntityData(((EntityPlayer) event.entity).getDisplayName());

if (playerData != null) {
System.out.println("LoggedIn-null");
((ExtendedEntityProps) (event.entity.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).loadNBTData(playerData);
}

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

@SubscribeEvent
public void onPlayerLoggedOut(PlayerEvent.PlayerLoggedOutEvent event){
if (!(event.player.worldObj.isRemote)) {

NBTTagCompound playerData = proxy.getEntityData((event.player).getDisplayName());

if (playerData != null) {
((ExtendedEntityProps) (event.player.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).saveNBTData(playerData);
System.out.println("LoggedOut-Null");
}
System.out.println("LoggedOut");
((ExtendedEntityProps) (event.player.getExtendedProperties(ExtendedEntityProps.EXT_PROP_NAME))).sync();
}
}

[22:15:48] [Server thread/INFO] [STDOUT]: [fr.sosoh.hogsmod.common.event.EventHandlerExtendedProps:onLivingDeathEvent:32]: Death
[22:15:48] [Server thread/INFO]: sosoh fell out of the world
[22:15:48] [Client thread/INFO]: [CHAT] sosoh fell out of the world
[22:15:48] [Client thread/INFO]: [CHAT] Ouch! That looked like it hurt
[22:15:50] [Server thread/INFO] [STDOUT]: [fr.sosoh.hogsmod.common.event.EventHandlerExtendedProps:onEntityJoinWorld:43]: LoggedIn
[22:15:50] [Server thread/INFO] [STDOUT]: [fr.sosoh.hogsmod.common.event.EventHandlerExtendedProps:onEntityJoinWorld:48]: LoggedIn-null

Mets les system.out.println dans ta fonction de sauvegarde, pas dans tes fonctions d’event. ça servirai pas à grand chose de m’envoyer tes sources car je m’y connait pas trop en EntityExtendedProperties.

[sosoh]

J’ai mis dans la fonction store de mon common proxy, ça marche. Quelqu’un a une idée ?

[SCAREX]

@‘sosoh’:

J’ai mis dans la fonction store de mon common proxy, ça marche. Quelqu’un a une idée ?

Le problème vient de là:

​
@Override

public void saveNBTData(NBTTagCompound compound)
{

NBTTagCompound properties = new NBTTagCompound();
properties.setDouble("Mana", this.mana);
properties.setDouble("MaxMana", this.maxMana);
properties.setString("spellRight", this.spellRight);
properties.setString("spellLeft", this.spellLeft);
properties.setInteger("spellsListSize", this.spellsList.size());
if(this.spellsList != null){
for(String spell : this.spellsList)
{
properties.setString("spellsList", spell);
}
}
compound.setTag(EXT_PROP_NAME, properties);
}

Ta boucle for supprime l’ancienne valeur : il faut utiliser un NBTTagList.

[sosoh]

Tu peux m’expliquer comment car je ne maîtrise pas tout à fait?

[SCAREX]

@‘sosoh’:

Tu peux m’expliquer comment car je ne maîtrise pas tout à fait?

Regarde dans le code tu TileEntity coffre car j’ai jamais utilisé ce NBTTag mais je sais qu’il existe.

[sosoh]

Mais je mets un NBTTagList pour juste l’array ou pour toutes mes props ?

[SCAREX]

@‘sosoh’:

Mais je mets un NBTTagList pour juste l’array ou pour toutes mes props ?

juste pour ton array spellList.

[sosoh]

J’ai fait ca, mais ca ne marche pas:

​@Override

public void saveNBTData(NBTTagCompound compound) 
    {

        NBTTagCompound properties = new NBTTagCompound();
        NBTTagList nbtlist = new NBTTagList();
        properties.setDouble("Mana", this.mana);
        properties.setDouble("MaxMana", this.maxMana);
        properties.setString("spellRight", this.spellRight);
        properties.setString("spellLeft", this.spellLeft);
        properties.setInteger("spellsListSize", this.spellsList.size());
        nbtlist.appendTag(properties);
        if(this.spellsList != null){
            for(String spell : this.spellsList)
            {
            NBTTagCompound nbttag = new NBTTagCompound();
            nbttag.setString("spellsList", spell);
                nbtlist.appendTag(nbttag);
            }
        }
        compound.setTag(EXT_PROP_NAME, nbtlist);
    }

@Override
public void loadNBTData(NBTTagCompound compound) 
{
NBTTagList nbtlist = (NBTTagList) compound.getTag(EXT_PROP_NAME);
NBTTagCompound properties = nbtlist.getCompoundTagAt(1);
this.mana = properties.getDouble("Mana");
this.maxMana = properties.getDouble("MaxMana");
this.spellRight = properties.getString("spellRight");
this.spellLeft = properties.getString("spellLeft");
int size = properties.getInteger("spellsListSize");
if(size > 0){
for(int i = 0; i < size; i++){
NBTTagCompound nbttag = nbtlist.getCompoundTagAt(i);
this.spellsList.add(i, nbttag.getString("spellsList"));
}
}
}

[SCAREX]

@‘sosoh’:

J’ai fait ca, mais ca ne marche pas:

​@Override

public void saveNBTData(NBTTagCompound compound) 
    {

        NBTTagCompound properties = new NBTTagCompound();
        NBTTagList nbtlist = new NBTTagList();
        properties.setDouble("Mana", this.mana);
        properties.setDouble("MaxMana", this.maxMana);
        properties.setString("spellRight", this.spellRight);
        properties.setString("spellLeft", this.spellLeft);
        properties.setInteger("spellsListSize", this.spellsList.size());
        nbtlist.appendTag(properties);
        if(this.spellsList != null){
            for(String spell : this.spellsList)
            {
            NBTTagCompound nbttag = new NBTTagCompound();
            nbttag.setString("spellsList", spell);
                nbtlist.appendTag(nbttag);
            }
        }
        compound.setTag(EXT_PROP_NAME, nbtlist);
    }

@Override
public void loadNBTData(NBTTagCompound compound) 
{
NBTTagList nbtlist = (NBTTagList) compound.getTag(EXT_PROP_NAME);
NBTTagCompound properties = nbtlist.getCompoundTagAt(1);
this.mana = properties.getDouble("Mana");
this.maxMana = properties.getDouble("MaxMana");
this.spellRight = properties.getString("spellRight");
this.spellLeft = properties.getString("spellLeft");
int size = properties.getInteger("spellsListSize");
if(size > 0){
for(int i = 0; i < size; i++){
NBTTagCompound nbttag = nbtlist.getCompoundTagAt(i);
this.spellsList.add(i, nbttag.getString("spellsList"));
}
}
}

regarde comment les NBTTagList fonctionne dans le TileEntityChest

[sosoh]

C’est ce que j’ai fais^^

[SCAREX]

@‘sosoh’:

C’est ce que j’ai fais^^

Regarde mieux alors car je peux pas beaucoup t’aider là-dessus (PS : le NBTTagList doit être créer EN-DEHORS de la boucle for)

[sosoh]

C’est ce que j’ai fait

[SCAREX]

@‘sosoh’:

C’est ce que j’ai fait

Le problème est que tu mets tes variables dans la liste, voici arborescence que tu devrais avoir :
-NBTTagCompound (joueur) {
  -NBTTagCompound (ton extendedentityproperties){
-NBTTagCeQueTuVeux Variable 1
    -NBTTagCeQueTuVeux Variable 2
    -NBTTagCeQueTuVeux Variable n
    -NBTTagList {
     -NBTTagString spell 1
     -NBTTagString spell 2
     -NBTTagString spell n
}
}
}

[sosoh]

Faut que je fasses un NBTTagCompound pour chaque variable ? Et les NBTTagCompounds de chaque variable dans mon NBTTagList ?

[SCAREX]

@‘sosoh’:

Faut que je fasses un NBTTagCompound pour chaque variable ? Et les NBTTagCompounds de chaque variable dans mon NBTTagList ?

Un NBTTagCompound est l’équivalent d’un dossier, NBTTagString, NBTTagInt, etc sont des équivalents de fichiers et les NBTTagList sont une sorte de dossier dans lesquels tu peux mettre qu’un seul type de fichiers (ou de dossier).

Donc tu créer un NBTTagCompound pour ton entityExtendedProperties dans lequel tu mets tes variables puis tu créer un NBTTagList avec dedans tout tes NBTTagString pour ta liste de String.

(je fais le code complet et je te le donnes sinon on va encore y passer plusieurs heures)

[sosoh]

Merci ! je crois avoir compris mais avec le code que tu vas me donner je pense que ça ce être plus explicit Juste si j’ai bien compris si je veux enregistrer une string je dois la mettre dans une NBTTagString ?