Missing return statement et autres joyeusetés

Bonjour à tous.

Je suis en train de me faire la main sur quelques problèmes simples grâce à CodingBat. Je me heurte à un problème pour lequel je ne trouve pas la solution.

Le problème en question

Si je fais le code ci-dessous, 2 cas de tests ne sont pas validés et je ne comprends pas pourquoi :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public String seeColor(String str) { if (str.length() >=3){ if (str.substring(0,3).equals("red")) return "red"; } else if (str.length()>=4) { if (str.substring(0,4).equals("blue")) return "blue"; } return (""); }

---

Par ailleurs, si j'utilise le code ci-dessous, j'ai droit à un "missing return statement" et je ne comprends pas non plus pourquoi :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public String seeColor(String str) { if (str.length() >=3){ if (str.substring(0,3).equals("red")) return "red"; } else if (str.length()>=4) { if (str.substring(0,4).equals("blue")) return "blue"; } else return (""); }

---

C'est sans doute trivial et évident, mais je sèche. Une piste ? Merci d'avance.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

public String seeColor(String str) { if (str.length() >=3){ if (str.substring(0,3).equals("red")) return "red"; else return ? } else if (str.length()>=4) { if (str.substring(0,4).equals("blue")) return "blue"; else return ? } else return (""); }

---

C'était bien trivial et évident, Merci :)

Code:

---

1 2 3 4 5 6 7

public String seeColor(String str) { if (str.length() >=3 && str.substring(0,3).equals("red")) return "red"; else if (str.length()>=4 && str.substring(0,4).equals("blue")) return "blue"; else return (""); }

---

Pour ce genre de problème, je te conseillerai plutôt de passer par des expressions régulières (pour des cas plus compliqués), et pour ce cas-là, par la méthode de la classe String "startsWith(String)".

@see http://download.oracle.com/javase/1....lang.String%29

Merci pour l'information. J'ai donc transformé mon code

Code:

---

1 2 3 4 5 6 7

public String seeColor(String str) { if (str.length() >=3 && str.startsWith("red")) return "red"; else if (str.length() >=4 && str.startsWith("blue")) return "blue"; else return (""); }

---

Perso je fait régulièrement comme cela :

Code:

---

1 2 3 4 5 6 7 8 9 10

public String seeColor(String str) { String ret="";   if (str.length() >=3 && str.startsWith("red")) ret="red"; else if (str.length() >=4 && str.startsWith("blue")) ret="blue";   return ret; }

---

Citation:

---

Envoyé par william44290

Perso je fait régulièrement comme cela :
...

---

Le fait d'intégrer une variable locale n'a-t-il pas un impact sur la consommation mémoire ? Évidemment sur une portion de code aussi petite, ça n'a aucune conséquence, mais ne vaut-il pas mieux éviter les variables temporaires lorsque c'est possible ?

Il n'y a pas franchement de problème de consommation mémoire en java.

Ceci est une affaire de style. robustesse, lisibilité, performance etc .. a chacun de se déterminer.

j'aurrais probablement fait comme cela :

Code:

---

1 2 3 4 5 6 7 8 9 10

public String seeColor(String str) { String ret=""; if (str!=null) { if (str.equals("red")) ret=str; else if (str.equals("blue")) ret=str; } return ret; }

---