Une sorte de "familié"

SCAREX

this.isTamed() au lieu de this.Tamed == 2, et je pense que c’est bon

PS : pense à respecter la convention java

leo01418

@‘SCAREX’:

this.isTamed() au lieu de this.Tamed == 2, et je pense que c’est bon

PS : pense à respecter la convention java

sa ne marche pas je pense que ses la solution mais je crois que ma classe est assez mal foutue car quand j’active la méthode vérification il affiche sa
[20:57:14] [Client thread/INFO] [STDOUT]: [ed.enderdeath.mod.entity.Familiar:VERIFICATION:323]: Ligne lvl1
[20:57:14] [Client thread/INFO] [STDOUT]: [ed.enderdeath.mod.entity.Familiar:VERIFICATION:324]: Tamed0
[20:57:14] [Server thread/INFO] [STDOUT]: [ed.enderdeath.mod.entity.Familiar:VERIFICATION:323]: Ligne lvl2
[20:57:14] [Server thread/INFO] [STDOUT]: [ed.enderdeath.mod.entity.Familiar:VERIFICATION:324]: Tamed1

SCAREX

Envoi la classe entière, sinon sans ça je peux rien faire

leo01418

Merci
Je l’envoyerais quand je serais sur le pc

leo01418

http://pastebin.com/3LWfxq7W
voici ma classe alors j’ai touché à des chose

SCAREX

Le problème c’est que tu as fait plein de copier-coller et que tu ne respectes pas la convention java, donc ton code ne ressemble à rien, essaie de faire du ménage dans ton code et essaie de comprendre ce que tu fais, sans ça tu n’arriveras à rien

leo01418

ok merci je vais mi mettre je suite

leo01418

j’avance sur le familier maintenant la classe de mon familier resemble à sa
http://pastebin.com/BHP8S0CJ

SCAREX

Il reste beaucoup d’erreurs, voilà une partie de celles que j’ai trouvé :

ligne 321 : VERIFICATION -> verification sinon non-respect de la convention java
ligne 376 : isWolfsFavoriteMeat() -> un autre nom de fonction serait plus adapté car ce n’est pas un loup
ligne 70 : Tamed -> tamed non-respect de la convention java
ligne 414 à 418 : pourquoi 40 mille fonctions pour mettre le niveau ?!
ligne 484 : fonction inutile
ligne 349 : y’a des problèmes de condition dans cette fonction, je sais pas vraiment ce que tu veux faire

Il y a sûrement d’autres erreurs mais voici les erreurs majeurs

leo01418

Merci Scarex pour ta réponse
ligne 321 [size=small-> je l’ai enlevé et juste pour voir le lvl et le Tamed du familiar j’ai fait un system.out.println x)]
[size=smallligne 376 -> j’ai modifié sa ]
[size=smallligne 70 -> je l’ai pas encore fait ]
[size=smallligne 414 à 418 -> je l’ai corrigé]
[font=Ubuntu, sans-serifligne 484 -> supprimé ]
[font=Ubuntu, sans-serifligne 349 -> pas encore réglé ]

SCAREX

Pour changer les noms : sélectionne le nom voulut fait Alt + shift + r, tape le nouveau puis entrée et ta variable a le même partout pareil

leo01418

Merci pour l’astuce je vais le faire de suite

leo01418

Voici ma classe actuel j’ai modifié quelque truc
http://pastebin.com/fr3PbM8T

SCAREX

Je vois pas de changement avec l’ancienne classe…

leo01418

ah oui Excuse je viens de modifié sa

SCAREX

Il faut que tu revois toutes tes conditions : à un moment tu regardes si la fonction est exécutée côté serveur, puis tu mets un “else if” dans lequel tu accèdes à l’inventaire du joueur côté client (à cause de la condition précédente), ça n’a aucun sens.

leo01418

re
Excuse sa fesait longtemps que je suis pas allé sur le codage ^^
bon en gros c’est ce genre de conditions qui géne ?


if (this.func_152114_e(p_70085_1_) && !this.worldObj.isRemote && !this.isBreedingItem(itemstack)) {
                this.aiSit.setSitting(!this.isSitting());
                this.isJumping = false;
                this.setPathToEntity((PathEntity) null);
                this.setTarget((Entity) null);
                this.setAttackTarget((EntityLivingBase) null);
            }

???

SCAREX

Celle-ci : (ligne 425)

if (!this.worldObj.isRemote) {
if (this.rand.nextInt(3) == 0) {

System.out.println("TEST" + this.isTamed());

this.lvl = 2;
this.setTamed();
this.isTamed();
System.out.println(lvl);

this.setPathToEntity((PathEntity) null);
this.setAttackTarget((EntityLivingBase) null);
this.aiSit.setSitting(true);
this.func_152115_b(p_70085_1_.getUniqueID().toString());
this.playTameEffect(true);
this.worldObj.setEntityState(this, (byte) 7);

} else {
this.playTameEffect(false);
this.worldObj.setEntityState(this, (byte) 6);
}
} else if (itemstack != null && itemstack.getItem() == enderdeath.RubisCroquette && this.isTamed()
&& this.lvl == 2) {
System.out.println("GAGNE");

if (!p_70085_1_.capabilities.isCreativeMode) {
–itemstack.stackSize;
}

if (itemstack.stackSize <= 0) {
p_70085_1_.inventory.setInventorySlotContents(p_70085_1_.inventory.currentItem, (ItemStack) null);
}

if (!this.worldObj.isRemote) {
if (this.rand.nextInt(3) == 0) {

this.setPathToEntity((PathEntity) null);
this.setAttackTarget((EntityLivingBase) null);
this.aiSit.setSitting(true);
System.out.println("GAGNE");
this.lvl = 3;

this.func_152115_b(p_70085_1_.getUniqueID().toString());
this.playTameEffect(true);
this.worldObj.setEntityState(this, (byte) 7);
this.setTamed();
} else {
this.playTameEffect(false);
this.worldObj.setEntityState(this, (byte) 6);
}
if (itemstack != null && itemstack.getItem() == enderdeath.SkinFire && this.isTamed()) {
this.Chanchelog();
this.onParticuleFire();
}
}
}

leo01418

Merci je m’y met de suite

leo01418

je reprend le codage j’avais arrête le pc depuis une semaine
sa fait du bien d’étre de retour x)