edodo7
Messages postés14Date d'inscriptionvendredi 15 janvier 2016StatutMembreDernière intervention28 février 2016
-
14 févr. 2016 à 18:45
edodo7
Messages postés14Date d'inscriptionvendredi 15 janvier 2016StatutMembreDernière intervention28 février 2016
-
14 févr. 2016 à 22:09
Bonjour à tous,je dois reéaliser une méthode qui renvoie true si un tableau est classé par ordre croissant ou décroissant et false sinon. Le problème est que j'ai une erreur me disant qu'il me manque un return.
Voici mon code:
public static boolean ordered( int tableau[]) { int i = 0; if (tableau[0] < tableau[1]) { while (i< tableau.length) { if (tableau[i] > tableau[i+1] && i < tableau.length -1) { return false; } if (i == tableau.length -1 && tableau[i-1] > tableau[i]) { return false; } i++; } return true; } if (tableau[0] > tableau[1]) { while (i < tableau.length) { if (tableau[i] < tableau[i+1] && i < tableau.length -1) { return false; } if (i == tableau.length -1 && tableau[i-1] < tableau[i]) { return false; } i++; } return true; } }
NHenry
Messages postés15150Date d'inscriptionvendredi 14 mars 2003StatutModérateurDernière intervention22 août 2024159 14 févr. 2016 à 18:51
Il ne s'agit pas d'une erreur, mais d'un avertissement.
Que se passe-t-il si :
tableau[0] == tableau[1]
KX
Messages postés16751Date d'inscriptionsamedi 31 mai 2008StatutModérateurDernière intervention27 août 2024127 14 févr. 2016 à 19:07
Il s'agit bien d'une erreur, parce que si le cas que tu indiques se produit il n'y aura aucun return... or il en faut toujours un.
edodo7
Messages postés14Date d'inscriptionvendredi 15 janvier 2016StatutMembreDernière intervention28 février 2016
>
KX
Messages postés16751Date d'inscriptionsamedi 31 mai 2008StatutModérateurDernière intervention27 août 2024 Modifié par edodo7 le 14/02/2016 à 20:22
Je confirme qu'il s'agit bien d'une erreur et non d'un "warning". Cependant je vais traiter le cas où tableau[0] == tableau[1] pour voir si ça fonctionne.
edodo7
Messages postés14Date d'inscriptionvendredi 15 janvier 2016StatutMembreDernière intervention28 février 2016
>
edodo7
Messages postés14Date d'inscriptionvendredi 15 janvier 2016StatutMembreDernière intervention28 février 2016 14 févr. 2016 à 20:34
J'ai donc fait la condition tableau[0] == tableau[1].
Fait étrange puisque quand j'ai essayé simplement en mettant un else, je n'ai eu aucun problème à la compilation mais à l'éxécution j'ai un message me disant qu'une exception a été lancée.
J'essaye donc avec un if(tableau[0] == tableau[1]) et je me retrouve avec l'erreur que j'avais au départ. Dans les deux cas donc, ça ne fonctionne pas
KX
Messages postés16751Date d'inscriptionsamedi 31 mai 2008StatutModérateurDernière intervention27 août 2024127
>
edodo7
Messages postés14Date d'inscriptionvendredi 15 janvier 2016StatutMembreDernière intervention28 février 2016 14 févr. 2016 à 21:05
Si tu fais des successions de if, même s'ils paraissent a priori exclusifs.
Java estime qu'il est tout à fait plausible que la valeur de tes conditions (les valeurs des cases du tableau dans ton cas) ait pu changer entre l'évaluation de chacune des conditions. En conséquence, pour le compilateur il y a encore des cas potentiels où il n'y a pas de return !
Pour bien faire tu dois mettre des else à tes if et un return dans chaque condition afin d'être sûr de couvrir tout les cas.
14 févr. 2016 à 19:07
Modifié par edodo7 le 14/02/2016 à 20:22
14 févr. 2016 à 20:34
Fait étrange puisque quand j'ai essayé simplement en mettant un else, je n'ai eu aucun problème à la compilation mais à l'éxécution j'ai un message me disant qu'une exception a été lancée.
J'essaye donc avec un if(tableau[0] == tableau[1]) et je me retrouve avec l'erreur que j'avais au départ. Dans les deux cas donc, ça ne fonctionne pas
14 févr. 2016 à 21:05
Java estime qu'il est tout à fait plausible que la valeur de tes conditions (les valeurs des cases du tableau dans ton cas) ait pu changer entre l'évaluation de chacune des conditions. En conséquence, pour le compilateur il y a encore des cas potentiels où il n'y a pas de return !
Pour bien faire tu dois mettre des else à tes if et un return dans chaque condition afin d'être sûr de couvrir tout les cas.
14 févr. 2016 à 22:01