Problème Méthode d'instance compare()

Bonjour,

en fait j'essaye de faire un exo et je bloque voila mon énoncé :

Dans la classe Livre écrivez une méthode d'instance compare pour comparer 2 livres sur leur nombre de pages.
compare prend un livre en paramètre et elle renvoie 0 si le livre a le même nombre de pages que l'instance qui reçoit le message, 1 si l'instance courante ("this") a plus de pages que le parametre et -1 sinon.

Pour tester, vous utiliserez un code semblable au suivant :

Code:

---

1 2 3 4 5 6

System.out.print("L'auteur du plus gros livre est "); String auteurPlusGrosLivre; // Le code pour trouver le livre qui a le plus de pages (entre 2 livres) // Ecrivez 2 versions : une avec if et l'autre avec switch . . . System.out.println(auteurPlusGrosLivre);

---

Et voila le code de ma méthode :

Code:

---

1 2 3 4 5 6 7 8 9

public void compare(Object l) { if (this.nbPages == nbPages) { System.out.println("0"); } else if (this.nbPages > nbPages) { System.out.println("1"); } else { System.out.println("-1"); } }

---

Je n'ai pas d'erreur à la compilation mais quand j'exécute il renvoie toujours 0, alors qu'il ne devrait pas dans certains cas.

Merci de m'aider je bloque depuis un moment je ne sais pas comment faire. :?

Bonjour,

c'est normal car tu compares le même champ, donc tu as toujours une égalité.

Ta méthode compare doit prendre un livre en paramètre et ensuite tu compareras param.nbPages à this.nbPages.

Au passage, ta méthode compare est censée renvoyer -1/0/1 et non pas l'afficher.

Comme Deaf a dit il faut modifier ton code au lieu de

Code:

---

1 2	this.nbPages OP nbPages

---

Faire ceci si tu n'as pas mis de getter

Code:

---

1 2	this.nbPages OP l.nbPages

---

Ou avec getter

Code:

---

1 2	this.nbPages OP l.getNbPages()

---

Ou OP est une des operation de comparaison
==, > ou <

Ta classe devrai être comme suit:

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public class Livre { int nbPages; //....autres champs et constructeurs public int compare(Livre l) { if(nbPages < l.nbPages) return -1; if(nbPages > l.nbPages) return 1; return 0; } }

---

A++

Je rajouterai juste un cas au début de la méthode si l est null.

Merci à tous pour vos réponses j'ai réussi à compiler et à exécuter ma méthode et elle marche parfaitement.