"Accroché" une entité à une autre entité

Deleted

Non, tu dois te servir de la méthode itemInteractionForEntity, de ta classe ItemRope. Non de ta classe EntityRope.

Axaurus

C’est déjà plus simple. Mais j’ai un problème (sans doute simple) : j’ai fait ça

   @Override
   public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer playerIn, EntityLivingBase target, EnumHand hand)
   {
    super.itemInteractionForEntity(stack, playerIn, target, hand);
    if(target instanceof EntityReindeer)
    {
    System.out.println("est une instance de reindeer");
    //EntityReindeer reindeer = (EntityReindeer)target;

    return true;
    }
       else
       {
        System.out.println("n'est pas une instance de reindeer");
        return true;
       }
   }
```Quand je click droit sur un mob autre qu'un renne, ça m'affiche le 2ème message (comme quoi c'est pas un renne) (normal). Mais quand je click droit sur un renne, rien du tout. C'est censé m'affiché le 1er message mais rien du tout.

Deleted

Print target pour voir ce que te dis la console.

Axaurus

Quand je faisais clic droit sur un autre mob, ça me mettait toutes ses infos (nom de la classe, client ou server et sa position. Quand je faisait click droit sur le renne, toujours rien jusqu’à que je décide de mettre en commentaire la fonction processInteract de EntityReindeer. Maintenant, ça affiche bien mais pourquoi la fonction processInteract est en conflit avec ? Et pis faut pas que je l’enlève, j’en ai besoin

EDIT : j’ai trouvé. Ce n’est pas vraiement la fonction qui est en conflit, c’est quand je disais que quand je click droit avec la rope, ça l’accrochait. J’ai enlevé ça et ça marche bien. Du coup, autant mettre ça dans l’ItemRope

EDIT n°2 : Désolé mais j’ai encore des problèmes : j’ai fait ```java
@Override
public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer playerIn, EntityLivingBase target, EnumHand hand)
{
super.itemInteractionForEntity(stack, playerIn, target, hand);
System.out.println(target);
if(target instanceof EntityReindeer)
{
System.out.println(“est une instance de reindeer”);
EntityReindeer reindeer = (EntityReindeer)target;
if (reindeer.canBeLeashedTo(playerIn))
{
reindeer.setLeashedToEntity(playerIn, true);
–stack.stackSize;
}

int ReindeerId = reindeer.getEntityId();

System.out.println(ReindeerId);
return true;
}
else
{
System.out.println(“n’est pas une instance de reindeer”);
return true;
}
}

Deleted

Sinon oublie les id’s, tu n’enregistre qu’une seule valeur dans les NBT, le nombre de rennes attachés. Et lors de la liaison avec le traîneau, tu fais une boucle for, qui spawnera de nouveaux rennes et tueras ceux attachés à la corde. Les id’s ne sont peut-être pas indispensables.
Essaie avec l’event EntityInteractEvent, voir si ton traîneau est détecté. Sinon tu seras forcé de l’extends EntityLivingBase, et de nous montrer de + près la stacktrace.

Axaurus

Salut
Désolé du temps de la réponse.
Je pense avoir trouvé une alternative aux id : les uuids. Ils restent tout le temps les mêmes.

Dans mon ItemRope, j’ai fait : ```java
public void reindeerToNBT(ItemStack stack, EntityLivingBase target)
{
NBTTagList nbttaglist = this.getReindeer(stack);
boolean flag = true;

for (int i = 0; i < nbttaglist.tagCount(); i++)
{
NBTTagCompound nbttagcompound = nbttaglist.getCompoundTagAt(i);

if (nbttagcompound.getUniqueId(“uuid”) != target.getUniqueID())
{
nbttagcompound.setUniqueId(“uuid”, target.getUniqueID());
}

flag = false;
break;
}

if (flag)
{
NBTTagCompound nbttagcompound1 = new NBTTagCompound();
nbttagcompound1.setUniqueId(“uuid”, target.getUniqueID());
nbttaglist.appendTag(nbttagcompound1);
}

if (!stack.hasTagCompound())
{
stack.setTagCompound(new NBTTagCompound());
}

stack.getTagCompound().setTag(“Reindeers”, nbttaglist);
}

public NBTTagList getReindeer(ItemStack stack)
{
NBTTagCompound nbttagcompound = stack.getTagCompound();
return nbttagcompound != null && nbttagcompound.hasKey(“Reindeers”, 9) ? (NBTTagList)nbttagcompound.getTag(“Reindeers”) : new NBTTagList();
}
et j'appelle la première fonction grâce à java
@Override
public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer playerIn, EntityLivingBase target, EnumHand hand)
{
super.itemInteractionForEntity(stack, playerIn, target, hand);
System.out.println(target);
if(target instanceof EntityReindeer)
{
System.out.println(“est une instance de reindeer”);
EntityReindeer reindeer = (EntityReindeer)target;
if (reindeer.canBeLeashedTo(playerIn))
{
reindeer.setLeashedToEntity(playerIn, true);
–stack.stackSize;
this.reindeerToNBT(stack, target);
}

UUID ReindeerId = reindeer.getUniqueID();
System.out.println(ReindeerId);
return true;
}
else
{
System.out.println(“n’est pas une instance de reindeer”);
return true;
}
}


Dans mon CommonEventHandler, j'ai fait :```java
@SubscribeEvent
public void test(EntityInteract event)
{
Entity target = event.getTarget();
EntityPlayer player = event.getEntityPlayer();

if(player instanceof EntityPlayer)
{
if(target instanceof EntitySleigh)
{
EntitySleigh sleigh = (EntitySleigh)event.getTarget();
Item item = event.getItemStack().getItem();

if(item instanceof ItemRope)
{
ItemRope rope = (ItemRope)event.getItemStack().getItem();
/*appelle d'une fonction qui return des reindeers (ou leurs UUIDs)*/

for(int i=/*nombre de reindeer dans le nbt de l'itemrope*/; /*je sais pas quoi mettre*/; i--)
{
/*reindeer récupéré*/.setDead;
EntityReindeer reindeer = new EntityReindeer(event.getWorld());
event.getWorld().spawnEntityInWorld(reindeer);
reindeer.setLeashedToEntity(sleigh, true);
}

}
}
}
}
``` J'ai préparé la fonction, mais il manque certaines choses.

Par contre, j'ai un problème et un petit warning dans le code : le problème c'est que quand j'attache des rennes à ma corde, ça accroche bien mais ne me suivent plus, ils ne bougent pas (j'ai testé avant de modifié le code). Le warning c'est dans le for de la fonction reindeerToNbt, à l'incrémentation de i. Ca me dit que c'est un code mort (dead code). Qu'est ce que ça veut dire ?

Merci d'avance

BrokenSwing

Normalement du code mort c’est qu’il ne pourra jamais être atteint, de quelque façon qu’il soit

LeBossMax2

Ici, le problème, c’est que ta boucle est exécutée qu’une seul fois à cause du “break”. Je pence que le break et le “flag = false” devraient aller dans la condition, après “nbttagcompound.setUniqueId(…)”

Axaurus

Merci, ça marche mieux (enfin …, ce que je veux dire par là, c’est que y’a plus le warning). Mais je sais toujours pas comment appelé ces fonctions et surtout si c’est juste.

Merci d’avance

EDIT : Je pense avoir trouvé. J’ai modifié l’ItemRope pour ajouté une liste de type UUID qui s’enregistre dans le nbt. Du coup, dans le nbt, j’aurais 2 tags : un de type UUID et un de type int (qui récupère la taille de la liste UUID). J’ai également fait deux nouvelles fonctions qui return l’uuid et le nombre de renne (la taille de la liste) (tout les deux regardant dans les nbt). Et dans l’event EntityInteract, j’ai aussi un peu modifié : j’ai ajouté un petit while qui tant que le nombre de renne n’est pas à 0 récupère l’uuid, tue le renne et en fait spawn un autre qui est directement attaché au traîneau. Mais il me manque un bout de code.

Voilà ce que j’ai fait (comme je l’ai dit je n’explique pas très bien, aussi vous comprendrez mieux) :
dans l’itemRope


ArrayList <uuid>nmbReindeer = new ArrayList<uuid>();
@Override
public boolean itemInteractionForEntity(ItemStack stack, EntityPlayer playerIn, EntityLivingBase target, EnumHand hand)
{
super.itemInteractionForEntity(stack, playerIn, target, hand);
System.out.println(target);
if(target instanceof EntityReindeer)
{
System.out.println("est une instance de reindeer");
EntityReindeer reindeer = (EntityReindeer)target;
if (reindeer.canBeLeashedTo(playerIn))
{
reindeer.setLeashedToEntity(playerIn, true);
--stack.stackSize;
this.reindeerToNBT(stack, target);
nmbReindeer.add(reindeer.getUniqueID());
}
return true;
}
else
{
System.out.println("n'est pas une instance de reindeer");
return true;
}
}

public void reindeerToNBT(ItemStack stack, EntityLivingBase target)
{
NBTTagList nbttaglist = this.getReindeer(stack);
boolean flag = true;

for (int i = 0; i < nbttaglist.tagCount(); i++)
{
NBTTagCompound nbttagcompound = nbttaglist.getCompoundTagAt(i);

if (nbttagcompound.getUniqueId("uuidReindeer") != target.getUniqueID())
{
nbttagcompound.setUniqueId("uuidReindeer", target.getUniqueID());
flag = false;
break;
}
if(nbttagcompound.getInteger("nmbReindeer") != nmbReindeer.size())
{
nbttagcompound.setInteger("nmbReindeer", nmbReindeer.size());
}
}

if (flag)
{
NBTTagCompound nbttagcompound1 = new NBTTagCompound();
nbttagcompound1.setUniqueId("uuidReindeer", target.getUniqueID());
nbttaglist.appendTag(nbttagcompound1);

NBTTagCompound nbttagcompound2 = new NBTTagCompound();
nbttagcompound1.setInteger("nmbReindeer", nmbReindeer.size());
nbttaglist.appendTag(nbttagcompound2);
}

if (!stack.hasTagCompound())
{
stack.setTagCompound(new NBTTagCompound());
}

stack.getTagCompound().setTag("Reindeers", nbttaglist);
}

public NBTTagList getReindeer(ItemStack stack)
{
NBTTagCompound nbttagcompound = stack.getTagCompound();
return nbttagcompound != null && nbttagcompound.hasKey("Reindeers", 9) ? (NBTTagList)nbttagcompound.getTag("Reindeers") : new NBTTagList();
}

public int getNmbReindeer(ItemStack stack)
{
NBTTagCompound nbttagcompound = stack.getTagCompound();
if(nbttagcompound != null && nbttagcompound.hasKey("nmbReindeer"))
{
int nmbReindeer = nbttagcompound.getInteger("nmbReindeer");
return nmbReindeer;
}
else
{
return 0;
}
}
public UUID getUUIDReindeer(ItemStack stack)
{
NBTTagCompound nbttagcompound = stack.getTagCompound();
if(nbttagcompound != null && nbttagcompound.hasKey("uuidReindeer"))
{
UUID uuidReindeer = nbttagcompound.getUniqueId("uuidReindeer");
return uuidReindeer;
}
else
{
return (UUID)null;
}
}

Et dans l’event EntityInteract

@SubscribeEvent
public void test(EntityInteract event)
{
Entity target = event.getTarget();
EntityPlayer player = event.getEntityPlayer();

if(player instanceof EntityPlayer)
{
if(target instanceof EntitySleigh)
{
EntitySleigh sleigh = (EntitySleigh)event.getTarget();
Item item = event.getItemStack().getItem();

if(item instanceof ItemRope)
{
ItemRope rope = (ItemRope)event.getItemStack().getItem();
int nmbReindeer = rope.getNmbReindeer(event.getItemStack());

while (nmbReindeer > 0)
{

UUID uuid = rope.getUUIDReindeer(event.getItemStack());

/*reindeer récupéré*/.setDead;
EntityReindeer reindeer = new EntityReindeer(event.getWorld());
event.getWorld().spawnEntityInWorld(reindeer);
reindeer.setLeashedToEntity(sleigh, true);
nmbReindeer--;
}

}
}
}
}

Voilà, je sais pas du tout si ça va marcher (en tout cas je pense que ça marchera déjà mieux que ce que j’avais proposé avant), et même si ça marche je sais pas du tout ce que ça va donner.

Si vous pouvez (encore) m’aidez, merci d’avance

Merci d’avance</uuid></uuid>

LeBossMax2

Je pence que tu pourrais faire comme quand on attache un mob à une barrière :


double d0 = 7.0D;
for (EntityReindeer entityliving : worldIn.getEntitiesWithinAABB(EntityReindeer.class, new AxisAlignedBB((double)player.posX - d0, (double)player.posY - d0, (double)player.posZ - d0, (double)player.posX + d0, (double)player.posY + d0, (double)player.posZ + d0)))

{
    if (entityliving.getLeashed() && entityliving.getLeashedToEntity() == player)
    {
        entityliving.setLeashedToEntity(sleigh, true);
    }
}

Axaurus

Justement. Je voulais faire ça mais il fallait détecté le click droit du joueur, si il clickait droit sur le traîneau. Et à partir de ça, on est arrivé à l’event EntityInteract

EDIT : Salut. Au pire, si nous n’avons pas de solution, je peux faire un autre truc qui sera surement plus simple : je créer un item qui, quand on click droit avec sur un renne, ça le supprime du monde et j’obtient un nouveau item. Et quand je fais click droit sur le traîneau avec cet item, ça fait spawn un nouveau renne (si possible avec les mêmes tags que l’autre) directement accroché au traîneau. C’est surement plus simple. Après, bien sûr je continuerai quand même de chercher comment faire, mais c’est une alternative.