Créer une arme custom simple

[Folgansky]

:::

public class ItemRailGun extends Item 
{   
    private int timer = 0;

    public void onPlayerStoppedUsing(ItemStack stack, World world, EntityPlayer player, int useTime)
    {
if (this.timer < 60)
{
this.timer = 0;
}
if (this.timer >= 60)
{
       EntityBullet entityBullet = new EntityBullet(world, player);
       world.playSoundAtEntity(player, "note.bd", 0.3F, 1.0F);
       if (!world.isRemote)
       {
           player.worldObj.spawnEntityInWorld(entityBullet);
       this.timer = 0;
       } 
}
     }

    public void onUsingTick(ItemStack stack, EntityPlayer player, int count)
    {
        this.timer++;
    }

    public void onUpdate(ItemStack stack, World world, Entity entity, int i, boolean b) 
    {
    if(!stack.hasTagCompound())
{
stack.setTagCompound(new NBTTagCompound());
}
int timer = stack.getTagCompound().getInteger("timer");
stack.getTagCompound().setInteger("timer", timer);
    }    

    public ItemStack onEaten(ItemStack stack, World world, EntityPlayer player)
    {
        return stack;
    }

    public int getMaxItemUseDuration(ItemStack stack)
    {
        return 300;
    }

    public EnumAction getItemUseAction(ItemStack stack)
    {
        return EnumAction.bow;
    }

    public boolean onLeftClickEntity(ItemStack stack, EntityPlayer player, Entity entity)
    {
        return true;
    }    

    public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player)
    {
        if (player.capabilities.isCreativeMode || player.inventory.hasItem(ModPg2.itemRailGun))
        {
            player.setItemInUse(stack, this.getMaxItemUseDuration(stack));
        }
        return stack;
     } 
}

:::

En faisant comme ça, la valeur du timer est toujours la bonne, celle enregistrée dans les nbt?

[SCAREX]

Je redis la même chose qu’avant, tu ne dois pas avoir de variable dans ta classe, les nbt tu les lis au moment où tu en as besoin

[Folgansky]

J’veux bien mais je ne comprend pas ce que ça veut dire. (en java)

Le “private int timer” vire je suppose et je fais comment pour récupérer la valeur du nbt tag voulu, svp?

[BrokenSwing]

Quand tu veux récupéré le timer présent dans le NBTTagCompound tu fait :

int timer = stack.getTagCompound().getInteger("timer");

[Folgansky]

D’acc, est-ce que j’en ai trop mis ou pas assez?

:::

public class ItemRailGun extends Item 
{      
    public void onPlayerStoppedUsing(ItemStack stack, World world, EntityPlayer player, int useTime)
    {
   int timer = stack.getTagCompound().getInteger("timer");
if (timer < 60)
{
timer = 0;
   stack.getTagCompound().setInteger("timer", timer);
}
if (timer >= 60)
{
       EntityBullet entityBullet = new EntityBullet(world, player);
       world.playSoundAtEntity(player, "note.bd", 0.3F, 1.0F);
       if (!world.isRemote)
       {
           player.worldObj.spawnEntityInWorld(entityBullet);
       timer = 0;
   stack.getTagCompound().setInteger("timer", timer);
       } 
}
     }

    public void onUsingTick(ItemStack stack, EntityPlayer player, int count)
    {
   int timer = stack.getTagCompound().getInteger("timer");
        timer++;
   stack.getTagCompound().setInteger("timer", timer);
    }

    public void onUpdate(ItemStack stack, World world, Entity entity, int i, boolean b) 
    {
    if(!stack.hasTagCompound())
{
stack.setTagCompound(new NBTTagCompound());
}
    int timer = stack.getTagCompound().getInteger("timer");
   stack.getTagCompound().setInteger("timer", timer);
    }    

    public ItemStack onEaten(ItemStack stack, World world, EntityPlayer player)
    {
        return stack;
    }

    public int getMaxItemUseDuration(ItemStack stack)
    {
        return 300;
    }

    public EnumAction getItemUseAction(ItemStack stack)
    {
        return EnumAction.bow;
    }

    public boolean onLeftClickEntity(ItemStack stack, EntityPlayer player, Entity entity)
    {
        return true;
    }    

    public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player)
    {
        if (player.capabilities.isCreativeMode || player.inventory.hasItem(ModPg2.itemRailGun))
        {
            player.setItemInUse(stack, this.getMaxItemUseDuration(stack));
        }
        return stack;
     } 
}

:::

J’ai supposé qu’il fallait set à chaque fois que ma valeur change.
De ce que j’ai compris "   stack.getTagCompound().setInteger(“timer”, timer);" permet d’enregistrer la valeur.

[SCAREX]

Tout est bon sauf dans le onUpdate où tout ce que tu fais c’est récupérer la variable puis “set” la variable sans aucune action entre

[Folgansky]

Ouki, j’ai viré cette méthode du coup.

Par contre j’ai un problème avec l’arme à cause du onUsingTick. Si je set la valeur après le timer++; l’arme “pète un plomb” (fais l’action de l’arc comme voulu mais fonctionne comme un clignotant) et la valeur monte toute seule même sans utiliser l’arme.

Si je retire le set, évidemment la valeur reste à 1 mais l’action “arc” se déroule normalement.

:::

public class ItemRailGun extends Item 
{      
public void onPlayerStoppedUsing(ItemStack stack, World world, EntityPlayer player, int useTime)
    {
int timer = stack.getTagCompound().getInteger("timer");
if (timer < 60)
{
timer = 0;
stack.getTagCompound().setInteger("timer", timer);
}
if (timer >= 60)
{
EntityBullet entityBullet = new EntityBullet(world, player);
world.playSoundAtEntity(player, "note.bd", 0.3F, 1.0F);
if (!world.isRemote)
{
player.worldObj.spawnEntityInWorld(entityBullet);
timer = 0;
stack.getTagCompound().setInteger("timer", timer);
} 
}
     }

    public void onUsingTick(ItemStack stack, EntityPlayer player, int count)
    {
    if(!stack.hasTagCompound())
        {
        stack.setTagCompound(new NBTTagCompound());
        }
    int timer = stack.getTagCompound().getInteger("timer");
        timer++;
        stack.getTagCompound().setInteger("timer", timer);
        System.out.println(timer);
    }   

    public ItemStack onEaten(ItemStack stack, World world, EntityPlayer player)
    {
        return stack;
    }

    public int getMaxItemUseDuration(ItemStack stack)
    {
        return 300;
    }

    public EnumAction getItemUseAction(ItemStack stack)
    {
        return EnumAction.bow;
    }

    public boolean onLeftClickEntity(ItemStack stack, EntityPlayer player, Entity entity)
    {
        return true;
    }    

    public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player)
    {
        if (player.capabilities.isCreativeMode || player.inventory.hasItem(ModPg2.itemRailGun))
        {
            player.setItemInUse(stack, this.getMaxItemUseDuration(stack));
        }
        return stack;
     } 
}

:::

[SCAREX]

Regarde si la méthode est appelée en permanence, si oui rajoute une condition pour vérifier que le joueur est en train de l’utiliser.

Au passage : tu as déjà un timer incrémenté tous les ticks, regarde le dernier paramètre de la fonction onPlayerStoppedUsing.

[Folgansky]

Oui j’avais vu ce paramètre (après) mais le fonctionnement de ma deuxième arme nécessite tout de même d’enregistrer une valeur dans les nbt (en gros pour faire diminuer le count quand le joueur n’utilise pas l’arme).

Par contre je me sers d’une valeur booléenne pour savoir si l’item est utilisé, y a-t-il plus simple?
En gros l’arme réagit comme si on était en train de l’utiliser alors que non.

Voici la classe de ma deuxième arme (modifiée selon les derniers bons conseils et possédant déjà une valeur booléenne pour tenter de résoudre le problème), mais le problème persiste

:::

public class ItemPlasmaGun extends Item 
{   
    public void onPlayerStoppedUsing(ItemStack stack, World world, EntityPlayer player, int useTime)
    {
        boolean onUse = stack.getTagCompound().getBoolean("onUse");
    onUse = false;
        stack.getTagCompound().setBoolean("onUse", onUse);
    }

    public void onUsingTick(ItemStack stack, EntityPlayer player, int count)
    {
    int timerTir = stack.getTagCompound().getInteger("timerTir");
    int timerCold = stack.getTagCompound().getInteger("timerCold");
    boolean onUse = stack.getTagCompound().getBoolean("onUse");
    if(onUse == false)
    {
    onUse = true;
    stack.getTagCompound().setBoolean("onUse", onUse);
    }
    if (timerTir <= 150 && onUse == true)
    {
    timerTir++;
    stack.getTagCompound().setInteger("timerTir", timerTir);
    }
    if (timerTir % 5 == 0 && timerCold != 0 && timerTir <= 150 && onUse == true)
    {
    EntityBullet entityBullet = new EntityBullet(player.worldObj, player);
    player.worldObj.playSoundAtEntity(player, "note.bd", 0.3F, 1.0F);
    if (!player.worldObj.isRemote)
    {
    player.worldObj.spawnEntityInWorld(entityBullet);
    } 
    }    
    }

    public void onUpdate(ItemStack stack, World world, Entity entity, int i, boolean b) 
    {
      if(!stack.hasTagCompound())
         {
        stack.setTagCompound(new NBTTagCompound());
         }
      int timerTir = stack.getTagCompound().getInteger("timerTir");
      int timerCold = stack.getTagCompound().getInteger("timerCold");
    boolean onUse = stack.getTagCompound().getBoolean("onUse");
    if(timerTir > 0 && timerTir != 151 && onUse == false)
    {      
    timerTir–; 
    stack.getTagCompound().setInteger("timerTir", timerTir);
    }
    if (timerTir > 150 && timerCold != 0)
         {    
    timerCold--;
    stack.getTagCompound().setInteger("timerCold", timerCold);
         }
         if (timerCold == 0)
         {        
        timerTir = 0;
        timerCold = 100;
        stack.getTagCompound().setInteger("timerTir", timerTir);
        stack.getTagCompound().setInteger("timerCold", timerCold);
         }
        System.out.println(timerTir + "tir");
        System.out.println(timerCold + "cold");
    }    

    public ItemStack onEaten(ItemStack stack, World world, EntityPlayer player)
    {
        return stack;
    }

public int getMaxItemUseDuration(ItemStack stack)
    {
        return 3000;
    }

public EnumAction getItemUseAction(ItemStack stack)
    {
        return EnumAction.bow;
    }

public boolean onLeftClickEntity(ItemStack stack, EntityPlayer player, Entity entity)
{
return true;
}

public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player)
{
if (player.capabilities.isCreativeMode || player.inventory.hasItem(ModPg2.itemPlasmaGun))
{
player.setItemInUse(stack, this.getMaxItemUseDuration(stack));
}
return stack;
}

public void writeToNBT(ItemStack stack) 
{
if (!stack.hasTagCompound()) stack.setTagCompound(new NBTTagCompound());
stack.writeToNBT(stack.getTagCompound());  
}
}

:::

[SCAREX]

Dans ton code il y a beaucoup de choses inutiles et il y a des choses que je ne comprends pas :

• Pourquoi faire ceci :

boolean onUse = stack.getTagCompound().getBoolean("onUse");
onUse = false;
stack.getTagCompound().setBoolean("onUse", onUse);

au lieu de ceci :

stack.getTagCompound().setBoolean("onUse", false);

• A quoi ça sert de faire ça :

public void writeToNBT(ItemStack stack)
{
if (!stack.hasTagCompound()) stack.setTagCompound(new NBTTagCompound());
stack.writeToNBT(stack.getTagCompound());
}

[Folgansky]

Parce que je ne sais absolument pas me servir des nbt et que je tâtonne, du coup merci pour ces précisions, je vais changer ça.

Peut-être que ça réglera mon soucis d’arme qui ne s’arrête plus de tirer (j’éditerai dans tous les cas)

Edit: Non actuellement le code est plus propre mais toujours le même soucis à l’activation

Edit: Le soucis c’est que ma méthode onStoppedUsing a du mal à être appelée, il faut avoir de la chance quand on clique droit, donc le boolean qui doit faire yo-yo , je cherche pourquoi.

:::

public class ItemRailGun extends Item 
{      
public void onPlayerStoppedUsing(ItemStack stack, World world, EntityPlayer player, int useTime)
    {
    stack.getTagCompound().setBoolean("onUse", false);    
int timer = stack.getTagCompound().getInteger("timer");
if (timer < 60)
{
stack.getTagCompound().setInteger("timer", 0);
}
if (timer >= 60)
{
EntityBullet entityBullet = new EntityBullet(world, player);
world.playSoundAtEntity(player, "note.bd", 0.3F, 1.0F);
if (!world.isRemote)
{
player.worldObj.spawnEntityInWorld(entityBullet);
stack.getTagCompound().setInteger("timer", 0);
} 
}
     }

    public void onUsingTick(ItemStack stack, EntityPlayer player, int count)
    {
    if(!stack.hasTagCompound())
        {
        stack.setTagCompound(new NBTTagCompound());
        }
    int timer = stack.getTagCompound().getInteger("timer");
    stack.getTagCompound().setBoolean("onUse", true); 
    if(stack.getTagCompound().getBoolean("onUse") == true)
    {
    timer++;
    stack.getTagCompound().setInteger("timer", timer);
    }
        System.out.println(timer);
    }   

    public ItemStack onEaten(ItemStack stack, World world, EntityPlayer player)
    {
        return stack;
    }

    public int getMaxItemUseDuration(ItemStack stack)
    {
        return 300;
    }

    public EnumAction getItemUseAction(ItemStack stack)
    {
        return EnumAction.bow;
    }

    public boolean onLeftClickEntity(ItemStack stack, EntityPlayer player, Entity entity)
    {
        return true;
    }    

    public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player)
    {
        if (player.capabilities.isCreativeMode || player.inventory.hasItem(ModPg2.itemRailGun))
        {
            player.setItemInUse(stack, this.getMaxItemUseDuration(stack));
        }
        return stack;
     } 
}

:::

[Folgansky]

Petit Up car les armes ont toujours le même soucis.

[SCAREX]

Personnellement dans mon mod d’armes, j’ai créé une proxy client qui envoi un paquet à chaque tentative de tir, ensuite le serveur décide ou non de faire tirer le joueur, je ne sais plus pourquoi j’avais choisi cette méthode mais je crois que c’était du à un problème avec l’appel de ces fonctions

[Folgansky]

Han, les paquets… Justement j’ai voulu créer des armes qui pourraient s’en passer. Visiblement ce n’est pas si facile.
Merci pour cette info Scarex, elle me permet de jeter l’éponge, du moins de mettre ce projet de côté un bon moment.

[Deleted]

Tu sais, il ne faut pas baisser les bras si vite ! Les packets avec de la motivation et de l’attention, c’est quasi un jeu d enfant
Essaie et poste ton code, on sera la pour te dire ce qui va et ce qui ne va pas

[Folgansky]

Re les gens, je suis parti sur un tout autre fonctionnement, plus simple et fonctionnel pour le coup.

Du coup je bosse un peu sur les “balles”.
Mon soucis actuel c’est réglé la vitesse de l’entité (qui est extends EntityThrowable)

public void setArrowHeading(double d, double d1, double d2, float spread, float speed)
{
spread /= 5F;
float f2 = MathHelper.sqrt_double(d * d + d1 * d1 + d2 * d2);
d /= f2;
d1 /= f2;
d2 /= f2;
d *= speed;
d1 *= speed;
d2 *= speed;
d += rand.nextGaussian() * spread * speed * 0.005D ;
d1 += rand.nextGaussian() * spread * speed * 0.005D ;
d2 += rand.nextGaussian() * spread * speed * 0.005D ;
motionX = d;
motionY = d1;
motionZ = d2;
float f3 = MathHelper.sqrt_double(d * d + d2 * d2);
prevRotationYaw = rotationYaw = (float) ((Math.atan2(d, d2) * 180D) / 3.1415927410125732D);
prevRotationPitch = rotationPitch = (float) ((Math.atan2(d1, f3) * 180D) / 3.1415927410125732D);
}

C’est la bonne méthode pour modifier le paramètre j’ai supposé mais je ne suis arrivé à rien de concluant en tentant des trucs comme:

motionX = d * 5;
``` (et le y et Z avec)
Tout bêtement quoi.

[Deleted]

Essaie de te servir de cela sinon
this.getEntityAttribute(SharedMonsterAttributes.movementSpeed).setBaseValue(20D);

[Folgansky]

Y’a le soucis que je sois extends EntityThrowable avec cette solution nan?

[Deleted]

Ah peut-être, je ne sais pas. Les EntityAttribute héritent de la classe EntityLivingBase, je crois….Et EntityThrowable hérite de Entity
Soit tu ré adaptes ta classe pour qu’elle étende d’EntityLiving, soit tu gardes l’idée des motions qui n’est pas si mal