Discussion :

[JohnNC]

Bonjour, je voudrais savoir comment je peux sortir d'un block "if" sans passer par return, comme un break dans une boucle :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
private void saisir(Composant comp) {
 if (comp.nom.equals("unComp") {
  if ( comp.valeur == null ) {
   break; // retourner à la racine de la méthode
  }
  else {
   // Traitement
  }
 }
 if (comp.nom.equals("UnAutreComp") {
  if ( comp.valeur == autreTest ) {
   break; // idem
  }
  else {
   // Traitement
  }
 }
 // Traitement à faire dans tous les cas
}

Suis-je clair ?

[raph707]

pas vraiment, non !

[franck06]

Bonjour, je voudrais savoir comment je peux sortir d'un block "if" sans passer par return, comme un break dans une boucle :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
private void saisir(Composant comp) {
 if (comp.nom.equals("unComp") {
  if ( comp.valeur == null ) {
   break; // retourner à la racine de la méthode
  }
  else {
   // Traitement
  }
 }
 if (comp.nom.equals("UnAutreComp") {
  if ( comp.valeur == autreTest ) {
   break; // idem
  }
  else {
   // Traitement
  }
 }
 // Traitement à faire dans tous les cas
}

Suis-je clair ?

mmmmmmmmhhh ..... non
si la condition est fausse tu rentres pas ds le if
donc je vois pas trop, c'est pour eviter le else qui suit ?

[JohnNC]

Bah pas vraiment, en fait je n'avais pas de "else", car les "//Traitement" sont assez volumineux, et je voulais éviter de rajouter des else un peu partout :/ (qui finissent en plus, avec l'indentation, par rendre le code plus vraiment lisible).
Je cherchais donc un autre moyen, au cas où. Mais je viens de penser aussi que je peux extraire le "//Traitement" dans une méthode, ce qui m'allègerait un peu le code au passage.

[_Mac_]

Mais je viens de penser aussi que je peux extraire le "//Traitement" dans une méthode, ce qui m'allègerait un peu le code au passage.

Bon réflexe !

[gifffftane]

En ce genre de cas tu peux procéder en séparant la détection de la condition de son traitement, si je suis clair.

Un exemple appliqué à ton cas :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
private void saisir(Composant comp) {
 boolean conditionok;
 
 conditionok = true;
 if (comp.nom.equals("unComp") {
  if ( comp.valeur == null ) {
   conditionok = false;
  }
 }
 if (comp.nom.equals("UnAutreComp") {
  if ( comp.valeur == autreTest ) {
   condtionok = false;
 }
 if (conditionok)
   // Traitement.
 // Traitement à faire dans tous les cas
}

Surprise ! Tu vois qu'il n'y a plus de else.

Par contre cela oblige à faire gaffe à la série des tests, et ainsi tu arrives vite à un modèle chaîne de responsabilités.

[STB_Fk]

Bonjour,

tu peux aussi utiliser la négation de tes conditions et faire des "and" dans tes conditions pour n'avoir plus que 2 if

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
private void saisir(Composant comp) {
 if ((comp.nom.equals("unComp")&&(comp.valeur != null)) {
   // Traitement
  }
 }
 if ((comp.nom.equals("UnAutreComp"))&&(comp.valeur != autreTest)) {
   // Traitement
  }
 }
 // Traitement à faire dans tous les cas
}

J'imagine que les 2 traitements sont défférents, sinon tu peux encore rajouter des "and" --> &&, mais c'est un peu barbare pour la lecture, tout en ayant un avantage, si la 1ere condition est fausse tu ne fait pas les autres.

Fk