Discussion :

[tsb01]

Bonjour,

Je travaille sur un exercice de programmation procédurale.
Je voudrai avoir votre avis pour perfectionner un bout de code qui me semble perfectible.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
int main()
 
{
   enum Afficher a;
   Menu();
 
   while (a != KEY_ESC)
   {
      a = entree();
      launch(a);
 
   }
   return 0;

C'est une boucle infinie et j'ai l'impression qu'il est possible d'aborder cela sous un autre angle.
Merci
Enum.rar

[Sve@r]

Bonjour

C'est une boucle infinie et j'ai l'impression qu'il est possible d'aborder cela sous un autre angle.

Ben ici quand a vaut "KEY_ESC" tu appelles quand-même "launch(a)" une fois avant de sortir et je ne pense pas que ce soit voulu...

Code c :

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

1
2
3
4
5
while (1) {
      a = entree();
      if (a == KEY_ESC) break;
      launch(a);
}

[Bousk]

Pourquoi pas avec une boucle for ? for (a = entree(); a != KEY_ESC; a = entree()) launch(a);

[Guesset]

Bonjour,

Dans le genre concision et illisibilité croissantes , on peut aussi proposer :

Code :

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

while((a = entree()) != KEY_ESC ) launch(a);

Salutations