OnItemRightClick sur une entity.

Gabs

Moi marche pas comme ça en gros:

Alors j’ai 1 stack de bois comme toi et 1 stack de terre, je me /kill je respawn lors du respawn j’ai plus rien dans mon inventaire et quand je vais sur le lieux ou je suis mort je retrouve le stack de bois mais pas le stack de terre et moi se que je veux c’est que le stack de bois disparaisse et le stack de terre revienne dans mon inventaire.

Et comme code j’ai ça:

private Map <entityplayer, itemstack="">itemsToKeep = new HashMap<entityplayer, itemstack="">();

@SubscribeEvent
public void onDead(LivingDeathEvent event)
{
    if(event.entityLiving instanceof EntityPlayer)
    {
        EntityPlayer player = (EntityPlayer)event.entityLiving;
        for(int i = 0; i < player.inventory.getSizeInventory(); i++)
        {
            ItemStack stack = player.inventory.getStackInSlot(i);
            if(stack != null && stack.getItem() == Item.getItemFromBlock(Blocks.dirt))
            {
                itemsToKeep.put(player, stack);
                player.inventory.setInventorySlotContents(i, null);
            }
        }
    }
}

@SubscribeEvent
public void onRespawn(PlayerRespawnEvent event)
{
    if(itemsToKeep.containsKey(event.player))
    {
        event.player.inventory.addItemStackToInventory(itemsToKeep.get(event.player));
        itemsToKeep.remove(event.player);
    }
}

}
```</entityplayer,></entityplayer,>

SCAREX

1 - Tu devrais utiliser les UUID plutôt que l’instance du joueur
2 - Ta HashMap devrait avoir une liste d’ItemStack car s’il y a plusieurs slots contenant de la dirt, seulement le dernier sera gardé
3 - Vérifie que l’inventaire du joueur n’est pas clear après l’appelle de l’événement

Gabs

Pour le 2 c’est pas garve car je vais changer par la suite je vais mettre un item de mon mod qui se stack par 1 et que le joueur aura qu’une fois pour la 3 l’inventaire n’est pas clear il me semble.

Snowy_1803

PlayerRespawnEvent est un event FML. As-tu enregistré ton GuiHandler dans le FMLEventBus ?

Gabs

J’ai ça chez moi:

@EventHandler
public void init(FMLInitializationEvent event) {
proxy.init(event);
NetworkRegistry.INSTANCE.registerGuiHandler(INSTANCE, proxy); 
MinecraftForge.EVENT_BUS.register(new EntityEvent());
    FMLCommonHandler.instance().bus().register(new EntityEvent());

}

PS: EntityEvent = la class avec onRespawn et onDead.

Snowy_1803

Ça devrait être bon, alors.

robin4002

Non, ce ne sont que des event Forge.
Je vais regarder de mon côté. Il faut clear pour chaque item si tu ne veux que rien reste.
Et en effet scarex, mon code est perfectible (uuid et tableau comme tu as dit). Ce qu’il faudrait aussi faire c’est sauvegarder la HashMap dans la save au cas le joueur se déconnecte au lieu de respawn tout de suite et que le serveur est coupé entre temps.

Gabs

D’acc ^^ j’attend ton retour, merci à vous

Snowy_1803

Pourtant le package de PlayerRespawnEvent indique FML…

robin4002

Ah oui autant pour moi. Je suis en 1.8.9 donc tout passe par l’event bus de Forge.
EDIT : ahhh c’est pour ça que ça ne fonctionne pas chez floriangabet.
Dans ta classe principale mets :

EntityEvent entityEvent = new EntityEvent();
FMLCommonHandler.instance().bus().register(entityEvent );
MinecraftForge.EVENT_BUS.register(entityEvent);

Sinon tu aura deux instances différentes de EntityEvent, donc deux maps différentes.

Snowy_1803

Je venais d’y penser…
Sinon, tu peux aussi mettre la variable static.

robin4002

Oui, ça fonctionne aussi.
Bon code modifier :

private Map <uuid, itemstack[]="">itemsToKeep = new HashMap<uuid, itemstack[]="">();

@SubscribeEvent
public void onDead(LivingDeathEvent event)
{
if(event.entityLiving instanceof EntityPlayer)
{
EntityPlayer player = (EntityPlayer)event.entityLiving;
ItemStack[] stackToKeep = new ItemStack[player.inventory.getSizeInventory()];
for(int i = 0; i < player.inventory.getSizeInventory(); i++)
{
ItemStack stack = player.inventory.getStackInSlot(i);
if(stack != null)
{
if(stack.getItem() == Items.diamond_sword)
{
stackToKeep* = stack;
}
player.inventory.setInventorySlotContents(i, null);
}
}
itemsToKeep.put(player.getGameProfile().getId(), stackToKeep);
}
}

@SubscribeEvent
public void onRespawn(PlayerRespawnEvent event)
{
if(itemsToKeep.containsKey(event.player.getGameProfile().getId()))
{
ItemStack[] stackToKeep = itemsToKeep.get(event.player.getGameProfile().getId());
for(int i = 0; i < event.player.inventory.getSizeInventory(); i++)
{
event.player.inventory.setInventorySlotContents(i, stackToKeep*);
}
itemsToKeep.remove(event.player.getGameProfile().getId());
}
}

UUID + tableau d’item stack.
Donc prend en charge plusieurs items. Et supprimes les autres items. J’ai mit une épée en diamant, c’était pour m’assurer que les enchantements étaient gardé y comprit le nom custom.</uuid,></uuid,>

Gabs

Wow magnifique merci !

Gabs

Dernier petit soucis! l’item que je met a la place de l’épée est un item GUI et lorsque je met un item dans l’item Gui et que je le referme l’item dedans disparaît ^^ .

Folgansky

Tu fais un backpack qui reste sur le joueur même mort on dirait. C’est ça?

robin4002

C’est un sac tu veux dire ? Car item gui ça ne veux rien dire …
Si oui ça vient de ton container (ou des nbt).

Gabs

C’est un backpack exactement %(#333333)[size=small][[color=#0072bc]☆Toutoune1008☆].

Gabs

Bizarre le problème ne vient pas de ton code j’ai un problème avec mon countainer pourtant j’ai rien toucher .

Gabs

OK problème résolu ! il me manquer une ligne je sais pas pourquoi ^^ merci à vous de votre aide .