Monture Custom

robin4002

La fonction onItemUse n’est que appelé quand tu fais un clic droit sur un bloc.
Donc il va falloir passer par onRightClick + un player.rayTrace (qui n’est que côté client donc paquet). La fonction rayTrace te renvoies un MovingObjectPosition et avec lui tu peux savoir s’il y a une entité ou pas et obtenir son instance. Un petit if(mop.hitEntity instanceof TonEntity) et c’est bon.

EDIT : d’après ce post : http://www.minecraftforum.net/forums/mapping-and-modding/minecraft-mods/modification-development/2426102-alternatives-to-player-raytrace?comment=6 il existe autre chose que player.rayTrace pour avoir un MovingObjectPosition.

Folgansky

Ok, alors chui loin du compte.

De ce que j’ai pu comprendre et voir j’ai ça:

:::

    public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer player)
    {
    int metadata = stack.getItemDamage();
    Vec3 look = player.getLookVec();
    float f = 200.0F;
    Vec3 start = new Vec3(player.posX, player.posY + player.getEyeHeight(), player.posZ);
    Vec3 end = new Vec3(player.posX + look.xCoord * f, player.posY + player.getEyeHeight() + look.yCoord * f, player.posZ + look.zCoord * f);
    MovingObjectPosition mop = world.rayTraceBlocks(start, end);

    if(metadata == 14)
    {
    if(mop.entityHit instanceof EntityBolide && mop.entityHit.riddenByEntity == null)
{
    {
//        mop.entityHit.setDead();
//        give l'itemMonture de metadata 1;
       stack.stackSize–;
    }
}
        return stack;
    }
    }

:::

En fait j’ai juste trouvé que c’était entityHit et pas hitEntity…

J’ai une erreur sur les vecteurs (truc auquel je ne comprends pas tant que ça), ça me marque comme quoi le constructeur Vec3(double, double, double) n’est pas visible.

Edit: Après ne serait-il pas plus simple de checker si il n’y a pas une entité à 2/3 blocs du joueur qui serait une entité récupérable?

L’histoire de côté client ou serveur avec les paquets ou non me découragent un peu ^^’

Folgansky

Bon comme je ne sais pas trop quoi faire pour récupérer un “mop” j’ai un peu trifouillé ce qui y touchait et je suis tombé sur une archive parlant des potions dont la méthode
onImpact récupère un mop.

Celui là, c’est par-ici, Il est frais mon poisson.

Bref, passons l’heure,

L’idée serait d’utiliser une potion à impact qui, si elle touche mon entité précise, fait ce que je lui demande (c’est à dire buter l’entité et récupérer un certain item qui en fait sert à la refaire spawn, même si techniquement c’est une toute autre entité).

Est-ce concevable selon vous ce type de potion dans mon cas?

BrokenSwing

Pour ce qui est du fait que le constructeur Vec3 qui n’existe pas, il suffit de faire un CTRL+CLIC sur Vec3 pour entrer dans la classe et voir qu’il faut utiliser

Vec3.createVectorHelper(double, double, double);

Ce qui retourne une instance de Vec3.

Ce que te propose c’est ça :


public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer player, EntityLivingBase entity)
{
if (entity instanceof EntityCreeper)
{
entity.setDead();
–stack.stackSize;
player.inventory.addItemStackToInventory(new ItemStack(Items.baked_potato));
return true;
}
else
{
return false;
}
}

robin4002

Je ne savais même pas que la méthode itemInteractionForEntity existait o_O

BrokenSwing

Moi non plus, j’ai regardé la classe de la selle

Folgansky

Edit: Cette méthode tombe vraiment à pic. Merci Bodri pour ton intuition sur la selle.

Alors j’arrive à récupérer l’item voulu en fonction de l’entité sur laquelle je clique. Le problème est que ça consomme deux fois l’item et me donne deux fois l’autre item ^^’

Par contre si je clique sur une autre entité avec cette item je crash, où dois-je mettre le nullcheck svp?

:::

public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer player, EntityLivingBase entity)
    {
int metadata = stack.getItemDamage();
if (metadata == 14)
{
DataWatcher dw = entity.getDataWatcher();
       if (dw.getWatchableObjectString(30) == "Moto" && entity instanceof EntityMoto)
       {
           entity.setDead();
           –stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 0));
           return true;
       }
       else if (dw.getWatchableObjectString(30) == "Kart2" && entity instanceof EntityKart)
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 12));
           return true;
       }
       else if (entity instanceof EntityBolide)
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 13));
           return true;
       }

}
   else    
   return false;    
}
return false;
    }

:::

robin4002

Envoies le rapport de crash.
Et pour l’action fait en double il faut ajouter if(!world.isRemote)

Folgansky

En faisant:

:::

public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer player, EntityLivingBase entity, World world)
    {
int metadata = stack.getItemDamage();
if (metadata == 14 && !world.isRemote)
{[…]}

:::

Les items ne remplissent plus leur fonction mais font toujours crash quand je clique sur un animal (autre entité que mes véhicules).

Et le rapport de crash (quand je ne met pas le isRemote): Pastebin

robin4002

La condition “entity instanceof EntityMoto” doit être avant “dw.getWatchableObjectString(30) == “Moto””

Folgansky

C’était bien ça.

Par contre toujours le même problème de “double action” exécutée côté client +serveur je suppose mais le !world.isRemote que je rajoute rend obselète la méthode: l’item ne fonctionne plus quoi.

robin4002

Pourtant il n’y a pas de raison pour que ça bloque. Étrange.

Folgansky

La méthode en entière même si ça va pas vraiment aider: (beaucoup de répétitions)

:::

public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer player, EntityLivingBase entity, World world)
    {
int metadata = stack.getItemDamage();
if (!world.isRemote && metadata == 14)
{
DataWatcher dw = entity.getDataWatcher();
       if (entity instanceof EntityMoto && dw.getWatchableObjectString(30) == "Moto")
       {
           entity.setDead();
           –stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 0));
           return true;
       }
       else if (entity instanceof EntityMoto && dw.getWatchableObjectString(30) == "Moto1")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 1));
           return true;
       }
       else if (entity instanceof EntityMoto && dw.getWatchableObjectString(30) == "Moto2")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 2));
           return true;
       }
       else if (entity instanceof EntityBMX && dw.getWatchableObjectString(30) == "BMX")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 3));
           return true;
       }
       else if (entity instanceof EntityBMX && dw.getWatchableObjectString(30) == "BMX1")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 4));
           return true;
       }
       else if (entity instanceof EntityBMX && dw.getWatchableObjectString(30) == "BMX2")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 5));
           return true;
       }
       else if (entity instanceof EntityBuggy && dw.getWatchableObjectString(30) == "Buggy")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 6));
           return true;
       }
       else if (entity instanceof EntityBuggy && dw.getWatchableObjectString(30) == "Buggy1")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 7));
           return true;
       }
       else if (entity instanceof EntityBuggy && dw.getWatchableObjectString(30) == "Buggy2")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 8));
           return true;
       }
       else if (entity instanceof EntityBaignoire)
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 9));
           return true;
       }
       else if (entity instanceof EntityKart && dw.getWatchableObjectString(30) == "Kart")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 10));
           return true;
       }
       else if (entity instanceof EntityKart && dw.getWatchableObjectString(30) == "Kart1")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 11));
           return true;
       }
       else if (entity instanceof EntityKart && dw.getWatchableObjectString(30) == "Kart2")
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 12));
           return true;
       }
       else if (entity instanceof EntityBolide)
       {
           entity.setDead();
           --stack.stackSize;
           player.inventory.addItemStackToInventory(new ItemStack(Modpg.itemMonture, 1, 13));
           return true;
       }    
   else    
   return false;    
}
return false;
    }

:::

Est-ce que je te mets la classe en entière? La même beaucoup de répétitions c’est pas un truc trop fournis, c’pour ça que je vois pas où ça bloque.

robin4002

Ajoutes au début de la fonction:
System.out.println(world.isRemote)
et dit moi ce qu’il y a dans la console.

Folgansky

ça ne marque rien par contre j’ai compris à quel moment ça ne faisait plus rien, c’est quand j’ajoute l’argument World:

public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer player, EntityLivingBase entity, World world)

Alors qu’on m’a conseillé de l’utiliser comme ça

public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer player, EntityLivingBase entity)

La différence est simple, l’un marche, l’autre non Xd.

Erreur en java de ma part?

robin4002

Oui en effet, si tu ajoutes un argument ce n’est plus la même fonction. Comme tu n’as pas d’argument world tu peux utiliser player.worldObj.isRemote

Folgansky

Je sens que ça va finir par rentrer mais là c’est pas encore ça:

:::

public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer player, EntityLivingBase entity)
    {
int metadata = stack.getItemDamage();
if (player.worldObj.isRemote && metadata == 14)
{

:::

Alors en ayant fait ça ça me donne toujours deux items mais en plus en déco/reco les entités reviennent.
J’en déduis juste qu’il n’y a que le côté client qui a été pris en compte… Bref que j’ai pas mis ça comme et où il fallait Xd

edit: Encore plus n’importe quoi avec “!” devant “player” xD

robin4002

!player.worldObj.isRemote
et non player.worldObj.isRemote

Folgansky

Ah bah c’est ce que j’ai de suite tenté et c’est encore plus bizarre:

L’entité disparait deco/reco elle a bien disparu, par contre les items:
je clique, y’a rien, déco/reco: bim y’en a deux qui sont apparus.
J’ai supprimé la carte et j’ai en créé une autre au cas où, ça ne change rien.

Folgansky

Bon visiblement si je mets comme condition que le joueur doit être en train de sneak c’est bon.
Reste le test serveur mais ça devrait être ça