Discussion :

[Gruik]

Salux,

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

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
 
switch(valeur)
{
 case CAS1 :
 case CAS2 :
  if (condition)
  {
    break;
  }
  /* sinon, descend*/
 case CAS3 :
 case CAS4 :
  /* action */
  break;
  /*...*/
}

[Sebou77]

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

Mais sinon oui je dirais que c'est à éviter

[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.

[Emmanuel Delahaye]

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

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
 
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...

[Gruik]

Ouais, c'est bien ce qui me semblait, mercix

[Skyounet]

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

Remarque il s'appelle Gruik

[Gruik]

Le goret ça m'connait

[Emmanuel Delahaye]

Le goret ça m'connait

"Dans le cochon tout est bon !" (Mais j'en mange plus, snif, parait que c'est cancérigène...)

[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 :

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

1
2
3
 
CAS 1 :
CAS 2 :

c'est équivalent à :

Code :

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

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 :

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

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 .....

[Emmanuel Delahaye]

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 :

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

1
2
3
 
CAS 1 :
CAS 2 :

c'est équivalent à :

Code :

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

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...

[souviron34]

Emmanuel Delahaye a écrit :

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

Ben si justement :

Gruik a écrit :

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
 
switch(valeur)
{
 case CAS1 :
 case CAS2 :
  if (condition)
  {
    break;
  }
  /* sinon, descend*/
 case CAS3 :
 case CAS4 :
  /* action */
  break;
  /*...*/
}

Il y a

/* sinon, descend*/

[Emmanuel Delahaye]

Ben si justement :

Ben non

Code :

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

   if (condition)

j'appelle ça du pseudo-code abstrait. Ca peut être n'importe quoi.

[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 :

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

1
2
3
4
5
6
7
8
9
10
11
CAS 1 :
CAS 2 :
      if ( condition )
        {
           ....
           break ;
        }
      break ;
CAS 3 :
CAS 4 :

[Skyounet]

ç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 :

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

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.

[souviron34]

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)
....

[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...

[Skyounet]

bon voilà comment je vois la chose moi

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
 
[...]
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.

[Gruik]

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

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
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;
   /*...*/
 }
/* ... */
}

[Emmanuel Delahaye]

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

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
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.

[thewho]

Bonjour,

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

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
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...