Encore problème onUpdate

robin4002

Pourquoi tu as encore PacketArrow ?
Tu es déjà côté serveur, effectue directement le tire.

Et on ne va pas avancer si tu donnes que des petits bouts de code …

Gabs

Ah oui exact.

Marche toujours pas, donc tien:

Arc:


package net.legacymod.items;

import cpw.mods.fml.relauncher.Side;
import cpw.mods.fml.relauncher.SideOnly;
import net.legacymod.LegacyMod;
import net.legacymod.packets.PacketArrow;
import net.legacymod.proxy.ClientProxy;
import net.minecraft.client.renderer.texture.IIconRegister;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.enchantment.Enchantment;
import net.minecraft.enchantment.EnchantmentHelper;
import net.minecraft.entity.Entity;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.projectile.EntityArrow;
import net.minecraft.entity.projectile.EntityLargeFireball;
import net.minecraft.init.Items;
import net.minecraft.item.EnumAction;
import net.minecraft.item.Item;
import net.minecraft.item.ItemBow;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTBase;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.util.DamageSource;
import net.minecraft.util.IIcon;
import net.minecraft.util.Vec3;
import net.minecraft.world.World;
import net.minecraftforge.common.MinecraftForge;
import net.minecraftforge.event.entity.living.LivingEvent.LivingUpdateEvent;
import net.minecraftforge.event.entity.player.ArrowLooseEvent;
import net.minecraftforge.event.entity.player.ArrowNockEvent;

public class FlameBow extends ItemBow {

public static final String[] bowPullIconNameArray = new String[] {"pulling_0", "pulling_1", "pulling_2"};
   @SideOnly(Side.CLIENT)
   private IIcon[] iconArray;

   public boolean isExplosive;

   public FlameBow()
   {
       this.maxStackSize = 1;
       this.setMaxDamage(1000);
       this.setFull3D();
       this.setCreativeTab(CreativeTabs.tabCombat);
   }

   /**
    * called when the player releases the use item button. Args: itemstack, world, entityplayer, itemInUseCount
    */
   public void onPlayerStoppedUsing(ItemStack stack, World world, EntityPlayer player, int p_77615_4_)
   {
       int j = this.getMaxItemUseDuration(stack) - p_77615_4_;

       ArrowLooseEvent event = new ArrowLooseEvent(player, stack, j);
       EntityLargeFireball fire = new EntityLargeFireball(world);
       MinecraftForge.EVENT_BUS.post(event);
       if (event.isCanceled())
       {
           return;
       }
       j = event.charge;

       boolean flag = player.capabilities.isCreativeMode || EnchantmentHelper.getEnchantmentLevel(Enchantment.infinity.effectId, stack) > 0;

       if (flag || player.inventory.hasItem(Items.arrow))
       {
           float f = (float)j / 20.0F;
           f = (f * f + f * 2.0F) / 3.0F;

           if ((double)f < 0.1D)
           {
               return;
           }

           if (f > 1.0F)
           {
               f = 1.0F;
           }

           EntityArrow entityarrow = new EntityArrow(world, player, f * 2.0F);
           EntityArrow entityarrow2 = new EntityArrow(world, player, f * 2.0F);
           EntityArrow entityarrow3 = new EntityArrow(world, player, f * 2.0F);
           EntityArrow entityarrow4 = new EntityArrow(world, player, f * 2.0F);
           EntityArrow entityarrow5 = new EntityArrow(world, player, f * 2.0F);

           if (f == 1.0F)
           {
               entityarrow.setIsCritical(true);
           }

           int k = EnchantmentHelper.getEnchantmentLevel(Enchantment.power.effectId, stack);

           if (k > 0)
           {
               entityarrow.setDamage(entityarrow.getDamage() + (double)k * 0.5D + 0.5D);
           }

           int l = EnchantmentHelper.getEnchantmentLevel(Enchantment.punch.effectId, stack);

           if (l > 0)
           {
               entityarrow.setKnockbackStrength(l);
           }

           if (EnchantmentHelper.getEnchantmentLevel(Enchantment.flame.effectId, stack) > 0)
           {
               entityarrow.setFire(100);
           }

           stack.damageItem(1, player);
           world.playSoundAtEntity(player, "random.bow", 1.0F, 1.0F / (itemRand.nextFloat() * 0.4F + 1.2F) + f * 0.5F);

           if (flag)
           {
               entityarrow.canBePickedUp = 2;
           }
            if(ClientProxy.keyBindSortG2.isPressed() && stack.getTagCompound().getInteger("timer") >= 20 * 20)
           {
            LegacyMod.network.sendToServer(new PacketArrow(f * 2.0F, 1));
            System.out.println("flèche explosive");
            stack.getTagCompound().setInteger("timer", 0);
           }

           else if (!world.isRemote && stack.getTagCompound().getInteger("key") == 3) // c'est ici le reste des else if je m'en charge une fois celui-ci opérationnel.
           {
        world.spawnEntityInWorld(entityarrow);
            entityarrow.setFire(2 * 5);
            System.out.println("flèche en feu");
            //stack.getTagCompound().setInteger("timer2", 0);
           }
           else if (player.isSneaking() && !world.isRemote)
           {
          System.out.println("5 flèches");
        LegacyMod.network.sendToServer(new PacketArrow(2.0F, 2));

            stack.getTagCompound().setInteger("timer3", 0);

           }else if(!world.isRemote)
           {
            world.spawnEntityInWorld(entityarrow);
            System.out.println("flèche normale");
           }
       }
   }

   public ItemStack onEaten(ItemStack p_77654_1_, World p_77654_2_, EntityPlayer p_77654_3_)
   {
       return p_77654_1_;
   }

   /**
    * How long it takes to use or consume an item
    */
   public int getMaxItemUseDuration(ItemStack p_77626_1_)
   {
       return 72000;
   }

   /**
    * returns the action that specifies what animation to play when the items is being used
    */
   public EnumAction getItemUseAction(ItemStack p_77661_1_)
   {
       return EnumAction.bow;
   }

   /**
    * Called whenever this item is equipped and the right mouse button is pressed. Args: itemStack, world, entityPlayer
    */
   public ItemStack onItemRightClick(ItemStack p_77659_1_, World p_77659_2_, EntityPlayer p_77659_3_)
   {
       ArrowNockEvent event = new ArrowNockEvent(p_77659_3_, p_77659_1_);
       MinecraftForge.EVENT_BUS.post(event);
       if (event.isCanceled())
       {
           return event.result;
       }

       if (p_77659_3_.capabilities.isCreativeMode || p_77659_3_.inventory.hasItem(Items.arrow))
       {
           p_77659_3_.setItemInUse(p_77659_1_, this.getMaxItemUseDuration(p_77659_1_));
       }

       return p_77659_1_;
   }

   /**
    * Return the enchantability factor of the item, most of the time is based on material.
    */
   public int getItemEnchantability()
   {
       return 1;
   }

   @SideOnly(Side.CLIENT)
   public void registerIcons(IIconRegister p_94581_1_)
   {
       this.itemIcon = p_94581_1_.registerIcon(this.getIconString() + "_standby");
       this.iconArray = new IIcon[bowPullIconNameArray.length];

       for (int i = 0; i < this.iconArray.length; ++i)
       {
           this.iconArray* = p_94581_1_.registerIcon(this.getIconString() + "_" + bowPullIconNameArray*);
       }
   }

   /**
    * used to cycle through icons based on their used duration, i.e. for the bow
    */
   @SideOnly(Side.CLIENT)
   public IIcon getItemIconForUseDuration(int p_94599_1_)
   {
       return this.iconArray[p_94599_1_];
   }

public void onUpdate(ItemStack item, World world, Entity player, int slotIndex, boolean inHand) {
if(!item.hasTagCompound() && !world.isRemote) {
      item.setTagCompound(new NBTTagCompound());
}
if(world.isRemote){
int timer = !item.getTagCompound().hasKey("timer") ? 0 : item.getTagCompound().getInteger("timer");
timer++;
item.getTagCompound().setInteger("timer", timer);

int timer2 = !item.getTagCompound().hasKey("timer2") ? 0 : item.getTagCompound().getInteger("timer2");
timer2++;
item.getTagCompound().setInteger("timer2", timer2);

int timer3 = !item.getTagCompound().hasKey("timer3") ? 0 : item.getTagCompound().getInteger("timer3");
timer3++;
item.getTagCompound().setInteger("timer3", timer3);

super.onUpdate(item, world, player, slotIndex, inHand);
}
}

@Override//Prob ici
   public IIcon getIcon(ItemStack stack, int renderPass, EntityPlayer player, ItemStack usingItem, int useRemaining)
   {
       if(usingItem != null && usingItem.getItem() == LegacyMod.FlameBow)
       {
           int k = usingItem.getMaxItemUseDuration() - useRemaining;
           if(k >= 18)
               return iconArray[2];
           if(k > 13)
               return iconArray[1];
           if(k > 0)
               return iconArray[0];
       }
       return getIconIndex(stack);
}

}

Proxy:


package net.legacymod.proxy;

import net.legacymod.LegacyMod;
import net.legacymod.items.ItemEpee3D;
import net.legacymod.mobs.EntityAlienMob;
import net.legacymod.mobs.RenderAlienMob;
import net.legacymod.mobs.alien;
import net.legacymod.packets.PacketClassesGUI;
import net.legacymod.packets.PacketKey;
import net.minecraft.client.Minecraft;
import net.minecraft.client.settings.KeyBinding;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.world.World;
import net.minecraftforge.client.IItemRenderer;
import net.minecraftforge.client.MinecraftForgeClient;

import org.lwjgl.input.Keyboard;

import cpw.mods.fml.client.registry.ClientRegistry;
import cpw.mods.fml.client.registry.RenderingRegistry;
import cpw.mods.fml.common.FMLCommonHandler;
import cpw.mods.fml.common.event.FMLInitializationEvent;
import cpw.mods.fml.common.event.FMLPreInitializationEvent;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import cpw.mods.fml.common.gameevent.InputEvent.KeyInputEvent;

public class ClientProxy extends CommonProxy{

    private static KeyBinding keyBindGUI;
    public static KeyBinding keyBindSortG1;
    public static KeyBinding keyBindSortG2;

@Override
   public void preInit(FMLPreInitializationEvent e)
   {

   }

@Override
public void init(FMLInitializationEvent e) {
RenderingRegistry.registerEntityRenderingHandler(EntityAlienMob.class, new RenderAlienMob(new alien(), 0.5F));
MinecraftForgeClient.registerItemRenderer(LegacyMod.epee, (IItemRenderer)new ItemEpee3D());
}

   public ClientProxy(){
    FMLCommonHandler.instance().bus().register(this);
    keyBindGUI = new KeyBinding("Choix de grade!", Keyboard.KEY_NUMPAD5, "key.categories.gameplay");
       ClientRegistry.registerKeyBinding(keyBindGUI);

       keyBindSortG1 = new KeyBinding("Sort Guerrier 1", Keyboard.KEY_NUMPAD6, "key.categories.gameplay");
       ClientRegistry.registerKeyBinding(keyBindSortG1);

       keyBindSortG2 = new KeyBinding("Sort Guerrier 2", Keyboard.KEY_NUMPAD7, "key.categories.gameplay");
       ClientRegistry.registerKeyBinding(keyBindSortG2);
   }

   @SubscribeEvent
   public void onEvent(KeyInputEvent event)
   {
       if(keyBindGUI.isPressed())
       {
           keyTestTyped();
       }
   }

   private void keyTestTyped()
   {
       EntityPlayer player = Minecraft.getMinecraft().thePlayer;
       World world = player.worldObj;
       LegacyMod.network.sendToServer(new PacketClassesGUI());

   }

   @SubscribeEvent
   public void onEvent2(KeyInputEvent event)
   {
       if(keyBindSortG1.isPressed())
       {
           keyTestTyped2();
       }
   }

   private void keyTestTyped2()
   {
    LegacyMod.network.sendToServer(new PacketKey());   
    System.out.println("num pad 6");
   }

}

Packetkey:


package net.legacymod.packets;

import io.netty.buffer.ByteBuf;
import net.legacymod.LegacyMod;
import net.legacymod.gui.ClassesGUI;
import net.minecraft.client.Minecraft;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import cpw.mods.fml.common.network.simpleimpl.IMessage;
import cpw.mods.fml.common.network.simpleimpl.IMessageHandler;
import cpw.mods.fml.common.network.simpleimpl.MessageContext;

public class PacketKey implements IMessage {

public PacketKey(){

}

@Override
public void fromBytes(ByteBuf buf) {

}

@Override
public void toBytes(ByteBuf buf) {

}

public static class Handler implements IMessageHandler <packetkey, imessage="">{

   public IMessage onMessage(PacketKey message, MessageContext ctx) {

    ItemStack stack = ctx.getServerHandler().playerEntity.inventory.getItemStack();
    if(stack != null){
    stack.setTagCompound(new NBTTagCompound());
    NBTTagCompound nbt = (NBTTagCompound) stack.getTagCompound();
    stack.getTagCompound().setInteger("key", 3);
    System.out.println("packet");
    }
    return null;

       }
   }
}

```</packetkey,>

robin4002

fais un print de stack.getTagCompound().getInteger(“key”) pour voir la valeur.

Le reste de ton code est aussi un gros bordel, tu devrais le réorganiser.

Gabs

@‘robin4002’:

fais un print de stack.getTagCompound().getInteger(“key”) pour voir la valeur.

Le reste de ton code est aussi un gros bordel, tu devrais le réorganiser.

Il est à 0 car le packet n’est pas envoyer j’ai mis un print dans mon packet dans les logs j’ai pas le print.

Pour le réorganiser c’est prévu!

edit:

J’ai rien dis j’ai mis un print dans frombyte:

[15:34:51] [Server thread/INFO] [STDOUT]: [net.legacymod.packets.PacketKey:fromBytes:23]: packet

Mais le truc c’est que c’est toujours a 0

packet:


public class PacketKey implements IMessage {

public PacketKey(){

}

@Override
public void fromBytes(ByteBuf buf) {
System.out.println("packet");
}

@Override
public void toBytes(ByteBuf buf) {

}

public static class Handler implements IMessageHandler <packetkey, imessage="">{

   public IMessage onMessage(PacketKey message, MessageContext ctx) {

    ItemStack stack = ctx.getServerHandler().playerEntity.inventory.getItemStack();
    if(stack != null){
    stack.setTagCompound(new NBTTagCompound());
    stack.getTagCompound().setInteger("key", 3);
    System.out.println("packet 2");
    System.out.println(stack.getTagCompound().getInteger("key"));
    }
    return null;

       }
   }
}

log:


[15:34:50] [Server thread/INFO]: floriangabet a rejoint la partie
[15:34:51] [Client thread/INFO] [STDOUT]: [net.legacymod.proxy.ClientProxy:keyTestTyped2:89]: num pad 6
[15:34:51] [Server thread/INFO] [STDOUT]: [net.legacymod.packets.PacketKey:fromBytes:23]: packet
[15:34:53] [Server thread/INFO] [STDOUT]: [net.legacymod.items.FlameBow:onPlayerStoppedUsing:144]: flèche normale
0

```</packetkey,>

robin4002

System.out.println(“packet 2”);
ne s’affiche pas.

Tu peux envoyer la ligne où tu as enregistré le paquet ?

Gabs

@‘robin4002’:

System.out.println(“packet 2”);
ne s’affiche pas.

Tu peux envoyer la ligne où tu as enregistré le paquet ?

Oui,
tien: ```java
network.registerMessage(PacketKey.Handler.class, PacketKey.class, 4, Side.SERVER);

robin4002

étrange, je ne vois pas pourquoi ça ne fonctionne pas.
Sinon envoies-moi un zip de ton dossier src.

Gabs

Mon src:

robin4002

La flèche de feu est ok, appuies sur 6 pour changer de mode.
En passant j’ai corrigé la convention Java et formaté ton code.

Gabs

@‘robin4002’:

La flèche de feu est ok, appuies sur 6 pour changer de mode.
En passant j’ai corrigé la convention Java et formaté ton code.

Super! merci c’est exactement se qui me fallait!

Par contre j’ai un soucis,

Je m’explique:

Comme vous le savez très bien je veux faire un système de recharge pour chaque sort sur l’arc avec la fonction onUpdate.
Le soucis est que cette fonction fait bug mon arc comme ceci: https://youtu.be/wThsb3Fi4jU
Du coup pour évité ce problème je r’ajoute un world.isRemote comme ça:


 @Override
    public void onUpdate(ItemStack item, World world, Entity player, int slotIndex, boolean inHand)
    {
        if(!item.hasTagCompound())
        {
            item.setTagCompound(new NBTTagCompound());
        }
        if(world.isRemote) // Comme ça
        {
            int timer = !item.getTagCompound().hasKey("timer") ? 0 : item.getTagCompound().getInteger("timer");
            timer++;
            item.getTagCompound().setInteger("timer", timer);

            int timer2 = !item.getTagCompound().hasKey("timer2") ? 0 : item.getTagCompound().getInteger("timer2");
            timer2++;
            item.getTagCompound().setInteger("timer2", timer2);

            int timer3 = !item.getTagCompound().hasKey("timer3") ? 0 : item.getTagCompound().getInteger("timer3");
            timer3++;
            item.getTagCompound().setInteger("timer3", timer3);

            super.onUpdate(item, world, player, slotIndex, inHand);
        }
    }

Mais le soucis c’est que le world.isremote cause un soucis du coup les timers ils sont complètement faux j’ai fias un test sans le world.isremote avec le bug et la les timers marche.

Du coup je vois pas du tout comment faire car même avec un !world.isRemote c’est le même soucis!

J’espère que vous m’avez compris sinon faite le moi savoir!

Merci en tout cas robin

robin4002

Faudrait voir si on peut désactiver la synchro du tag nbt.
Car c’est ça qui fait ce bug visuel.

Gabs

D’acc je cherche de mon côté demain la je vais dodo !

Edit:

J’ai pas trouvé perso sur internet.

C’est possible tu penses ?

BrokenSwing


@Override
public boolean getShareTag()
{
return false;
}

Gabs

@‘BrokenSwing’:


@Override
public boolean getShareTag()
{
    return false;
}

Salut,

J’ai ajouté cette fonction dans la classe de mon arc et j’ai ensuite enlever le world.isremote mais j’ai toujours le problème “visuel” comme dis robin.

Je précise juste que ce bug influence aussi sur le “chargement” de l’arc c’est un peu compliqué a expliqué.

Je sais pas si vous voyez se que je veux dire au pire vous avez mon src juste au dessus vous enlevez le if(world.isremote) dans la fonction onUpdate de l’arc et vous verrez par vous même.

En tout cas merci quand même x) .

SCAREX

Je vais tester de mon côté, les sources n’ont pas été modifiées ?

EDIT : j’ai pris les sources données par robin. Mis à part quelques petits trucs qui ne sont pas très correctes, le code est globalement bon. En revanche je n’est pas ton problème lorsque je test de mon côté, est-ce que tu testes sur cauldron ou quelque chose comme ça ?

Eikins

Coucou !
Bon, alors j’avais étudier le système de minecraft car je voulais faire un truc similaire.
En fait c’est simple : tu peux pas.

Quand tu mets à jour un NBT, ça met à jour ton ItemStack et ça fait planter le “Player Using Item”.
On peut très bien récupérer des données via le getter des NBTTagCompound, mais faut surtout pas utiliser les setters.
Pour les sceptiques, faites le test vous verrez.

Du coup t’as plusieurs options : soit tu refait un système de “Use Item”, un peu relou, soit tu utilises les ExtendedProperties et tu stock les cooldowns “sur le joueur”, par contre c’est un peu sale. (Ou une autre méthode à toi de voir, là c’est les deux qui me viennent à l’esprit tout de suite)

SCAREX

Pourtant moi quand je teste j’ai aucun problème

Eikins

C’est bizarre, car en 1.7.10, c’est justement un problème.
Utiliser les NBT dans le onUpdate ou dans le onUsingTick va empêcher la méthode onPlayerStoppedUsing d’être appelée. (Et en plus ça fait le problème visuel)
Si tu veux jeter un oeil : http://www.minecraftforge.net/forum/index.php?topic=35035.0

SCAREX

Peut être que le problème a été réglé dans ma version

Gabs

@‘SCAREX’:

Peut être que le problème a été réglé dans ma version

Euhh tu as enlever la ligne if(world.isRemote) dans la fonction onUpdate

Salut Eikins,
Merci des infos, du coup c’est chiant ouais…
Y’a vraiment aucun moyen autre fonction ou une autre version de forge ?

En tout cas merci de votre aides!