Récupérer et afficher le nom d'un joueur

Minantcraft

@‘Laserflip33’:

@‘LeBossMax2’:

Je pence que l’on peut faire très simple : dans “addInformation”, tu regarde si l’item à le tag “description” ( stack.getTagCompound().hasKey(“description”), n’oublis pas de faire un null check de stack.getTagCompound()). Si il l’a alors tu ajoute son contenu à la list des descriptions sinon, tu le créé de manière random et le rajoute aux tags.

Comme ceci?
public class ItemTest extends Item
{
public void addInformation(ItemStack stack, EntityPlayer player, List list, boolean par4)
    {
if(stack.getTagCompound() != null)
        {
stack.getTagCompound().hasKey("description");
                
list.add(EnumChatFormatting.ITALIC + "description");
        }
        
        else
        {
        String[] randMessages = new String[]{"Alpha", "Bravo", "Charlie"};
            
            String randMessage = randMessages[player.getRNG().nextInt(randMessages.length)];
            
            list.add(EnumChatFormatting.RED + randMessage);
        }
    }
}
Si c’est comme ça le problème est toujours le même ça oscille entre tous les messages

Je crois qu’il vouait dire ça (pas sur):

public class ItemTest extends Item
{
public void addInformation(ItemStack stack, EntityPlayer player, List list, boolean par4)
{
if(stack.getTagCompound() != null && !(stack.getTagCompound().hasKey("description")))
{
String[] randMessages = new String[]{"Alpha", "Bravo", "Charlie"};

String randMessage = randMessages[player.getRNG().nextInt(randMessages.length)];

list.add(EnumChatFormatting.RED + randMessage);

stack.getTagCompound().setString("description", randMessage);
}
}

et grâce à ça, si tu veux récupérer le message random d’un des items tu peux faire ça:

if(stack.hasTagCompound())
{
stack.getTagCompound().getString("description");
}

robin4002

    public void addInformation(ItemStack stack, EntityPlayer player, List list, boolean par4)
    {
        if(stack.hasTagCompound() && stack.getTagCompound().hasKey("description", Constants.NBT.TAG_STRING))
        {
            list.add(EnumChatFormatting.ITALIC + stack.getTagCompound().getString("description"));
        }
        else
        {
            String[] randMessages = new String[] {"Alpha", "Bravo", "Charlie"};

            String randMessage = randMessages[player.getRNG().nextInt(randMessages.length)];
            if(!stack.hasTagCompound())
            {
                stack.setTagCompound(new NBTTagCompound());
            }
            stack.getTagCompound().setString("description", randMessage);
        }
    }

Minantcraft

@‘robin4002’:

Constants.NBT.TAG_STRING

Pourquoi?

@‘robin4002’:

list.add(EnumChatFormatting.ITALIC + stack.getTagCompound().getString("description"));

Ca ne va pas en mettre à l’infini?

LeBossMax2

@‘Minantcraft’:

@‘robin4002’:
Constants.NBT.TAG_STRING
Pourquoi?

@‘robin4002’:
list.add(EnumChatFormatting.ITALIC + stack.getTagCompound().getString("description"));
Ca ne va pas en mettre à l’infini?

“Constants.NBT.TAG_STRING” permet d’être certain que le tag qui a pour nom “description” est de type String donc pour qur “getString(“description”)” fonctionne bien (exemple : si un autre mod ajoute un tag qui porte le même nom mais qui est d’un autre type)
Et le list est réinitialisé à chaque tick (il me semble) dans il faut ajouter la description à chaque tick.

Minantcraft

@‘LeBossMax2’:

“Constants.NBT.TAG_STRING” permet d’être certain que le tag qui a pour nom “description” est de type String donc pour qur “getString(“description”)” fonctionne bien (exemple : si un autre mod ajoute un tag qui porte le même nom mais qui est d’un autre type)
Et le list est réinitialisé à chaque tick (il me semble) dans il faut ajouter la description à chaque tick.

Ok merci

Laserflip33

Ça marche parfaitement! Un seul petit problème persiste : dès que j’actualise l’item (right click avec ou le dropper puis le reprendre par exemple), la valeur affichée change à chaque fois, et ce pendant 2 minutes. C’est à dire que passé ces 2 minutes la valeur se fixe et ne bouge plus, même si on right click, même si on déco/reco, même si on le drop, même si on le met dans un coffre etc…

LeBossMax2

@‘Laserflip33’:

Ça marche parfaitement! Un seul petit problème persiste : dès que j’actualise l’item (right click avec ou le dropper puis le reprendre par exemple), la valeur affichée change à chaque fois, et ce pendant 2 minutes. C’est à dire que passé ces 2 minutes la valeur se fixe et ne bouge plus, même si on right click, même si on déco/reco, même si on le drop, même si on le met dans un coffre etc…

Je n’avais pas pensé à ça, c’est surement un problème de synchro client-serveur, il faut passé par quelque chose qui s’exécute coté serveur, comme onUpdate, onCreated ou des events comme
ItemTooltipEvent (lui, je ne sait pas de quel coté il est appelé)

robin4002

Le problème avec cette méthode c’est qu’en effet tout ce passe côté client.
Donc à la déco reco c’est perdu.
Ou si le serveur renvoie la valeur du tag nbt du serveur (null par défaut) au client, c’est aussi perdu.

C’est pour ça que j’avais proposé de le faire via la méthode onCreated.

Après avec onUpdate ça peut aussi être une bonne idée en utilisant le même principe.

Laserflip33

Dans ce cas il faut obligatoirement passer par des NBT Tag donc?

robin4002

C’est déjà le cas avec le code que je t’ai donné.

Laserflip33

Alors ça fait maintenant plus d’une semaine que je gamberge avec la méthode qu’à donné robin :

@‘robin4002’:

Il faut faire le random exécuté seulement une fois lors de la création (il me semble qu’il existe une fonction onCreate) puis enregistrer les valeurs dans un tag nbt et l’afficher ensuite dans addInformation.

J’ai essayé vraiment énormément de choses mais rien de tout ça ne c’est avéré concluant, alors je m’en remet à vous parce que je suis clairement au point mort.

SCAREX

Où est-ce que tu bloques ?

Laserflip33

Bon du coup désolé pour la loongue absence de plus d’un mois … J’ai enfin décidé de terminer cet item une fois pour toute!

@‘SCAREX’:

Où est-ce que tu bloques ?

En fait je n’arrive pas à enregistrer les valeurs dans un NBT Tag après avoir exécuté le random lors de la création. J’avais un code mais là il faut tout refaire de 0 parce qu’apparemment ce n’est pas la bonne technique

LeBossMax2

@‘Laserflip33’:

Bon du coup désolé pour la loongue absence de plus d’un mois … J’ai enfin décidé de terminer cet item une fois pour toute!

@‘SCAREX’:

Où est-ce que tu bloques ?

En fait je n’arrive pas à enregistrer les valeurs dans un NBT Tag après avoir exécuté le random lors de la création. J’avais un code mais là il faut tout refaire de 0 parce qu’apparemment ce n’est pas la bonne technique

Le code de robin4002 est bon, il faut juste le séparer en deux :
tu garde le “if …. { … }” dans la fonction ‘addInformation’ mais tu met le reste dans l’un des events que l’on t’a proposés. (Il me semble qu’en 1.7, la fonction ‘onCreated’ fonctionne bien mais en 1.8, c’est sûr qu’elle est buggé. Si c’est le cas, je pence que tu est obligé de passer par ‘onUpdate’)

Laserflip33

@‘LeBossMax2’:

@‘Laserflip33’:

Bon du coup désolé pour la loongue absence de plus d’un mois … J’ai enfin décidé de terminer cet item une fois pour toute!

@‘SCAREX’:

Où est-ce que tu bloques ?

En fait je n’arrive pas à enregistrer les valeurs dans un NBT Tag après avoir exécuté le random lors de la création. J’avais un code mais là il faut tout refaire de 0 parce qu’apparemment ce n’est pas la bonne technique

Le code de robin4002 est bon, il faut juste le séparer en deux :
tu garde le “if …. { … }” dans la fonction ‘addInformation’ mais tu met le reste dans l’un des events que l’on t’a proposés. (Il me semble qu’en 1.7, la fonction ‘onCreated’ fonctionne bien mais en 1.8, c’est sûr qu’elle est buggé. Si c’est le cas, je pence que tu est obligé de passer par ‘onUpdate’)

Oui, la logique voudrait que ce soit comme ça :

public class ItemTest extends Item
{
public ItemTest()
{

}

public void onCreated(ItemStack stack, World world, EntityPlayer player, NBTTagCompound comp)
{
String[] randMessages = new String[] {"Alpha", "Bravo", "Charlie"};

        String randMessage = randMessages[player.getRNG().nextInt(randMessages.length)];
        if(!stack.hasTagCompound())
        {
            stack.setTagCompound(new NBTTagCompound());
        }
        stack.getTagCompound().setString("description", randMessage);
}

public void addInformation(ItemStack stack, EntityPlayer player, List list, boolean par4)
    {
        if(stack.hasTagCompound() && stack.getTagCompound().hasKey("description", Constants.NBT.TAG_STRING))
        {
            list.add(EnumChatFormatting.ITALIC + stack.getTagCompound().getString("description"));
        }
    }
}

Mais aucune description ne s’affiche

LeBossMax2

La fonction “onCreated” est exécuté lorsque tu craft l’objet donc si on peut obtenir ton item d’une autre manière qu’en le craftant, il faut utilise “onUpdate”. (Ps : pour tester si ça fonctionne comme ça, il faut donc le crafter et pas juste le prendre dans le menu créatif)

Laserflip33

@‘LeBossMax2’:

La fonction “onCreated” est exécuté lorsque tu craft l’objet donc si on peut obtenir ton item d’une autre manière qu’en le craftant, il faut utilise “onUpdate”. (Ps : pour tester si ça fonctionne comme ça, il faut donc le crafter et pas juste le prendre dans le menu créatif)

Oui c’est ce que j’ai essayé, quand on le récupère dans la table à craft aucune description (avec onCreated), j’ai essayé avec onUpdate ça ne fonctionne pas non plus malheureusement

LeBossMax2

@‘Laserflip33’:

@‘LeBossMax2’:

La fonction “onCreated” est exécuté lorsque tu craft l’objet donc si on peut obtenir ton item d’une autre manière qu’en le craftant, il faut utilise “onUpdate”. (Ps : pour tester si ça fonctionne comme ça, il faut donc le crafter et pas juste le prendre dans le menu créatif)

Oui c’est ce que j’ai essayé, quand on le récupère dans la table à craft aucune description (avec onCreated), j’ai essayé avec onUpdate ça ne fonctionne pas non plus malheureusement

Ajoute un @Override devant tes fonction, il y a surement un problème : pour mois, c’est “onCreated(ItemStack stack, World world, EntityPlayer player)” (en 1.8), tu as mis “NBTTagCompound comp” en trop.
Il y a un autre problème : il faut mettre “if (!world.isRemote)” pour ajouter le tag coté serveur puis “player.inventoryContainer.detectAndSendChanges()” pour synchroniser avec le client.
Au final on a donc :

@Override
public void onCreated(ItemStack stack, World world, EntityPlayer player)
{
    if (!world.isRemote)
    {
        String[] randMessages = new String[] {"Alpha", "Bravo", "Charlie"};

        String randMessage = randMessages[player.getRNG().nextInt(randMessages.length)];
        if(!stack.hasTagCompound())
        {
            stack.setTagCompound(new NBTTagCompound());
        }
        stack.getTagCompound().setString("description", randMessage);
        player.inventoryContainer.detectAndSendChanges();

    }
}

EDIT : Pour moi, “player.inventoryContainer.detectAndSendChanges()” ne fonctionne pas, il faut peut-être utiliser d’autre chose.

EDIT 2 : Remplace “player.inventoryContainer.detectAndSendChanges()” par “((EntityPlayerMP)player).sendContainerToPlayer(player.inventoryContainer);”, ça fonctionne très bien pour moi !

EDIT 3 : Je tiens à te prévenir que la fonction “onCreated” est pas toujours appelé en 1.8 (ex : si je fait shift click ou que je clique sur une touche de 1 à 9) donc tu devrais tester pour voir si il y a les mêmes problèmes en 1.7.

Laserflip33

Ça marche parfaitement avec onCreated merci! Seulement j’aimerai que la description soit affichée quand l’item est give, et non craft. Donc il faudrait passer par l’event onUpdate du coup. J’ai essayé ça :

@Override
public void onUpdate(ItemStack stack, World world, EntityPlayer player, int par4, int par5)
{
   if (!world.isRemote)
   {
       String[] randMessages = new String[] {"Alpha", "Bravo", "Charlie"};

       String randMessage = randMessages[player.getRNG().nextInt(randMessages.length)];
       if(!stack.hasTagCompound())
       {
           stack.setTagCompound(new NBTTagCompound());
       }
       stack.getTagCompound().setString("description", randMessage);
       ((EntityPlayerMP)player).sendContainerToPlayer(player.inventoryContainer);
   }
}

Qui ne marche pas du coup, j’ai une erreur sur le onUpdate. Ça vient du fait que j’ai ajouté @Override, seulement si je l’enlève il n’y a plus d’erreur mais la description ne s’affiche plus en jeu. J’ai aussi essayé d’enlever les int, ça ne marche pas non plus

LeBossMax2

Si ça te dis d’enlever le “@Override”, c’est que tu t’est trompé dans les arguments de la fonction : en 1.8, c’est “onUpdate(ItemStack stack, World worldIn, Entity entityIn, int itemSlot, boolean isSelected)”