Discussion :

[Invité]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[william44290]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ("");
}

[Invité]

C'était bien trivial et évident, Merci

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ("");
}

[Hizin]

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

[Invité]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ("");
}

[william44290]

Perso je fait régulièrement comme cela :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
}

[Invité]

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 ?

[william44290]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
}