break de switch conditionnel

Version imprimable

Salux,

Je me demandais si ceci etait possible, et si oui si c'était à eviter :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
switch(valeur)
{
 case CAS1 :
 case CAS2 :
  if (condition)
  {
    break;
  }
  /* sinon, descend*/
 case CAS3 :
 case CAS4 :
  /* action */
  break;
  /*...*/
}

07/02/2007, 09h48
Sebou77

Jamais testé mais oui je dirais que c'est possible, tu as testé ? :)

Mais sinon oui je dirais que c'est à éviter :?
07/02/2007, 09h53
Skyounet

Pareil moi je dis que c'est possible, je vois pas pourquoi on pourrait pas.

Après c'est vrai niveau lisibilité c'est pas tip-top.
07/02/2007, 10h02
Emmanuel Delahaye
Citation:
Envoyé par Gruik

Je me demandais si ceci etait possible, et si oui si c'était à eviter :

Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 switch(valeur) { case CAS1 : case CAS2 : if (condition) { break; } /* sinon, descend*/ case CAS3 : case CAS4 : /* action */ break; /*...*/ }
Joli ! Faut que je mette ma page à jour :

http://emmanuel-delahaye.developpez.com/goret.htm

Ca mérite un bon 7 ou 8...
07/02/2007, 10h32
Gruik

Ouais, c'est bien ce qui me semblait, mercix
07/02/2007, 10h42
Skyounet

Citation:

Envoyé par Emmanuel Delahaye

http://emmanuel-delahaye.developpez.com/goret.htm

Remarque il s'appelle Gruik :aie: :aie:
07/02/2007, 10h51
Gruik

Le goret ça m'connait :king:
07/02/2007, 11h18
Emmanuel Delahaye

Citation:

Envoyé par Gruik

Le goret ça m'connait :king:

"Dans le cochon tout est bon !" (Mais j'en mange plus, snif, parait que c'est cancérigène...)
07/02/2007, 11h38
souviron34
Désolé de vous contredire, mais je ne crois pas que ça marche...

Quand le switch est évalué, il donne un entier...

Quand on fait
Code:

1 2 3 CAS 1 : CAS 2 :
c'est équivalent à :
Code:

1 2 if ( (valeur == 1) || (valeur == 2) )
Donc aucune raison d'avoir à l'intérieur de ça un valeur=3 ou valeur=4...

Par contre, faire :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 CAS1 : CAS2 : CAS3 : CAS4 : if ( (valeur == 1) || (valeur ==2) ) { ....... break ; } ...... break ;
oui ça ça marche .....
07/02/2007, 11h56
Emmanuel Delahaye
Citation:
Envoyé par souviron34

Désolé de vous contredire, mais je ne crois pas que ça marche...

Quand le switch est évalué, il donne un entier...

Quand on fait

Code:

1 2 3 CAS 1 : CAS 2 :

c'est équivalent à :

Code:

1 2 if ( (valeur == 1) || (valeur == 2) )

Donc aucune raison d'avoir à l'intérieur de ça un valeur=3 ou valeur=4...
Certes, mais la condition n'était pas précisée...
07/02/2007, 12h17
souviron34
Citation:

Emmanuel Delahaye a écrit :

Certes, mais la condition n'était pas précisée...

Ben si justement :
Citation:
Gruik a écrit :

Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 switch(valeur) { case CAS1 : case CAS2 : if (condition) { break; } /* sinon, descend*/ case CAS3 : case CAS4 : /* action */ break; /*...*/ }
Il y a

Citation:

/* sinon, descend*/
07/02/2007, 12h21
Emmanuel Delahaye

Citation:

Envoyé par souviron34

Ben si justement :

Ben non

Code:

if (condition)

j'appelle ça du pseudo-code abstrait. Ca peut être n'importe quoi.
07/02/2007, 12h30
souviron34
ça n'empêche rien, Emmanuel...

le if est à l'intérieur du CAS1,2

Et donc ça revient à ce que je disais.. On ne peut JAMAIS dans ce cas-là avoir CAS3,4 après le if ...

ou alors il faudrait faire :
Code:

1 2 3 4 5 6 7 8 9 10 11 CAS 1 : CAS 2 : if ( condition ) { .... break ; } break ; CAS 3 : CAS 4 :
07/02/2007, 12h33
Skyounet
Citation:
Envoyé par souviron34

ça n'empêche rien, Emmanuel...

le if est à l'intérieur du CAS1,2

Et donc ça revient à ce que je disais.. On ne peut JAMAIS dans ce cas-là avoir CAS3,4 après le if ...

ou alors il faudrait faire :

Code:

1 2 3 4 5 6 7 8 9 10 11 CAS 1 : CAS 2 : if ( condition ) { .... break ; } break ; CAS3 : CAS4 :
Euh c'est
case CAS1
case CAS2

avec CAS1, CAS2 des #define je suppose.
Si la condition n'est pas évalué, ben on rentre bien dans les case CAS3 et CAS4.
07/02/2007, 12h38
souviron34

Citation:

Skyrunner a écrit :

case CAS1
case CAS2

avec CAS1, CAS2 des #define je suppose.
Si la condition n'est pas évalué, ben on rentre bien dans les case CAS3 et CAS4.

ça ne change rien , c'est sur le SWITCH ...
Comme je disais c'est un if ... Donc du moment où tu rentres dans

if (cas1 ou cas2)

tu ne peux pas être dans le cas

if (cas3 ou cas4)
....
07/02/2007, 12h43
souviron34

et de plus je me demande si le compilateur accepteras (je pense que non) d'avoir un case suivi de code suivi d'un case sans avoir de break entre les 2...

bon voilà comment je vois la chose moi
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 [...] switch(valeur) { case CAS1: case CAS2: { if(valeur2 == MASUPERVALEUR) { break; } } case CAS3: case CAS4: { /* code */ } } [...]
Tu compares le switch à un if mais connais-tu exactement la structure utilisée derrière ? Comment le compilo interprète le tout ? Pourquoi alors est-on obligé de mettre des break à la fin de chaque case ?

Ici c'est crade mais correct.

Bon et sinon, la solution pas crade ce serait ceci?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
 
static void faire (void)
{
 /* ... */
}
 
void func(void)
{
 /* ... */
 switch(valeur)
 {
  case CAS1 :
  case CAS2 :
   if (!(condition))
   {
     faire();
   }
   break;
 
  case CAS3 :
  case CAS4 :
   faire();
   break;
   /*...*/
 }
/* ... */
}

Citation:

Envoyé par Gruik

Bon et sinon, la solution pas crade ce serait ceci?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
 
static void faire (void)
{
 /* ... */
}
 
void func(void)
{
 /* ... */
 switch(valeur)
 {
  case CAS1 :
  case CAS2 :
   if (!(condition))
   {
     faire();
   }
   break;
 
  case CAS3 :
  case CAS4 :
   faire();
   break;
   /*...*/
 }
/* ... */
}

Oui, par exemple.

Bonjour,
Citation:
Envoyé par Gruik
Bon et sinon, la solution pas crade ce serait ceci?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 static void faire (void) { /* ... */ } void func(void) { /* ... */ switch(valeur) { case CAS1 : case CAS2 : if (!(condition)) { faire(); } break; case CAS3 : case CAS4 : faire(); break; /*...*/ } /* ... */ }
sauf que ça ne correspond plus au problème: si on est entré dans une des cas CAS1 ou CAS2, que le test sur condition échoue ou non, on ne poursuivra pas dans CAS3...

Citation:

thewho a écrit :

sauf que ça ne correspond plus au problème: si on est entré dans une des cas CAS1 ou CAS2, que le test sur condition échoue ou non, on ne poursuivra pas dans CAS3...

Et si, puisqu'il répète la fonction faire() en inversant la condition....

Mais on peut aussi faire :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 switch ( valeur ) { case CAS1 : case CAS2 : case CAS3 : case CAS4 : if ( (valeur == CAS1) || (valeur == CAS2) ) { if ( condition ) { ..... break ; } } ....... break ; }

07/02/2007, 13h34
Skyounet

Citation:

Envoyé par thewho

Bonjour,

sauf que ça ne correspond plus au problème: si on est entré dans une des cas CAS1 ou CAS2, que le test sur condition échoue ou non, on ne poursuivra pas dans CAS3...

Euh si.

Si la condition est fausse il appelle la fonction faire().
Si valeur = CAS3 il appelle aussi faire()
07/02/2007, 19h42
gl
Citation:

Envoyé par souviron34

et de plus je me demande si le compilateur accepteras (je pense que non) d'avoir un case suivi de code suivi d'un case sans avoir de break entre les 2...

Si c'est valide (la pratique est contestable, mais c'est un autre probleme).

Sinon concernant le rapprochement entre switch et if, ce n'est pas exactement vrai.
Dans un switch, on commence l'execution sur le case correspondant a la valeur teste (ou default: si aucune valeur ne correspond) et on le deroule jusqu'a rencontre un break (ou un return).

Ainsi dans le code
Code:

1 2 3 4 5 6 7 8 9 10 switch(var1) { case 1: faire1(); case 2: faire2(); break; ... }
si var1 vaut 1, les deux fonctions faire1() et faire2() sont appele.
08/02/2007, 01h21
souviron34
Je suis d'accord avec toi que cela permet de le faire, mais ce n'est pas vraiment recommandé, ni pour clarté du code, ni pour correspondre à l'esprit du switch, qui est quand même de bien séparer les cas..

D'autre part, quand tu dis

Citation:

Sinon concernant le rapprochement entre switch et if, ce n'est pas exactement vrai

si c'est vrai. C'est juste un if multiple imbriqué..

Dans ton exemple :
Code:

1 2 3 4 5 6 7 8 9 10 if ( (var == 1) || (var ==2) ) { faire1(); if ( var == 2 ) faire2(); } else { }
08/02/2007, 19h15
gl
Citation:

Envoyé par souviron34

mais ce n'est pas vraiment recommandé, ni pour clarté du code

Effectivement (quoique dans certains cas ca puisse se defendre)

Citation:

Envoyé par souviron34

ni pour correspondre à l'esprit du switch, qui est quand même de bien séparer les cas..

Je n'ai pas la pretention de pretendre connaitre les intentions des createurs du C ou des normalisateurs, mais je n'en suis pas aussi persuader que toi.
Citation:
Envoyé par souviron34

D'autre part, quand tu dis

Citation:

Sinon concernant le rapprochement entre switch et if, ce n'est pas exactement vrai

si c'est vrai. C'est juste un if multiple imbriqué..

Dans ton exemple :

Code:

1 2 3 4 5 6 7 8 9 10 if ( (var == 1) || (var ==2) ) { faire1(); if ( var == 2 ) faire2(); } else { }
Le fait que l'on puisse remplacer un switch par des if est une evidence, mais ne signifie pas qu'un switch n'est qu'un if.

En outre dans ton explication premiere sur le if et le switch, tu pretendais qu'un switch n'etait qu'un if et que la construction d'un case comme mon exemple n'etait pas possible. Je cite:

Citation:

ça ne change rien , c'est sur le SWITCH ...
Comme je disais c'est un if ... Donc du moment où tu rentres dans

if (cas1 ou cas2)

tu ne peux pas être dans le cas

if (cas3 ou cas4)
....
09/02/2007, 00h06
souviron34

Citation:

gl a écrit :

En outre dans ton explication premiere sur le if et le switch, tu pretendais qu'un switch n'etait qu'un if et que la construction d'un case comme mon exemple n'etait pas possible. Je cite:

Eh oui désolé, mais comme 1) je n'ai jamais eu de cours de C (j'ai toujours appris "sur le tas", car je suis physicien), 2) je code et je vois du code C depuis 17 ans 3) j'ai moi-même codé plus d'1 million de lignes de code en C 4) je suis passé à travers du code C "officiel" (dont le code de XWindows, le code de HTTP) en 17 ans je n'ai JAMAIS vu de telle utilisation. Après ta remarque j'ai donc été piocher sur le net et j'ai vu que OUI, ça pouvait exister.. Mea Culpa :oops: C'est peut-être enseigné dans les cours, mais je ne connais personne dans mon milieu qui penserait même à l'utiliser comme ça...

Mais visiblement ça n'est pas franchement utilisé, et je reviens encore sur la clarté....

case ça veut dire cas .... Donc si c'est pour faire des conditions, autant se servir d'un if.. Et si c'est des cas, eh bien alors on peut grouper les cas oui, mais GROUPER est là le maître-mot, ce me semble.....
10/02/2007, 16h30
gl

Citation:

Envoyé par souviron34

mais je ne connais personne dans mon milieu qui penserait même à l'utiliser comme ça...

Ce qui se comprends fort bien.

Citation:

Envoyé par souviron34

Mais visiblement ça n'est pas franchement utilisé, et je reviens encore sur la clarté....

Non c'est assez rare. Mais ca arrive de le rencontrer et parfois, assez rarement, c'est meme un choix (si c'est correctement documente bien entendu) defendable.

Citation:

Envoyé par souviron34

case ça veut dire cas .... Donc si c'est pour faire des conditions, autant se servir d'un if.. Et si c'est des cas, eh bien alors on peut grouper les cas oui, mais GROUPER est là le maître-mot, ce me semble.....

L'utilisation des switch ... case comme dans mon exemple n'est ni plus ni moins l'extension du regroupement de cas.