Un mini programme de la part d'un débutant.

**giggs** · 25/10/2006, 23h42

Salut à tous, je débute en C, et j'ai conçu un mini programme permettant à l'utilisateur d'effectuer des opérations sur deux nombres qu'il rentre (en gros une calculette). Le code n'est pas du tout optimisé, et je souhaiterai avoir votre avis sur celui-ci et si il est possible de l'optimiser...

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
 
#include <iostream.h>
#include <stdio.h>
 
main()
{
int a;
int b;
char signe;
printf("Bienvenue sur mon premier vrai programme en C.\n Ce programme est une calculette, vous pourrez donc effectuer plusieurs\n operations sur deux nombres\n");
printf("commencez par choisir le type d'operation a effectuer en tapant soit le signe: * ou / ou - ou +");
signe = getchar();
switch(signe)
{
case'+':printf("vous avez tape +\n");
	signe=1;
	break;
case'-':printf("vous avez tape -\n");
	signe=2;
	break;
case'/':printf("vous avez tape /\n");
	signe=3;
	break;
case'*':printf("vous avez tape *\n");
	signe=4;
	break;
default:printf("desole, ce que vous avez tape n'est pas prevu...\n");
	break;
}
printf("vous allez maintenant devoir rentrer le premier nombre\n");
if (signe==1)
{
	printf("tapez maintenant le permier chiffre:\n");
	scanf("%d",&a);
	printf("tapez maintenant le second chiffre:\n");
	scanf("%d",&b);
	printf("le resultat est:%d\n",a+b);
}
else if(signe==2)
{
	printf("tapez maintenant le permier chiffre:\n");
	scanf("%d",&a);
	printf("tapez maintenant le second chiffre:\n");
	scanf("%d",&b);
	printf("le resultat est:%d\n",a-b);
}
else if(signe==3)
{
	printf("tapez maintenant le permier chiffre:\n");
	scanf("%d",&a);
	printf("tapez maintenant le second chiffre:\n");
	scanf("%d",&b);
	printf("le resultat est:%d\n",a/b);
}
else if(signe==4)
{
	printf("tapez maintenant le permier chiffre:\n");
	scanf("%d",&a);
	printf("tapez maintenant le second chiffre:\n");
	scanf("%d",&b);
	printf("le resultat est:%d\n",a*b);
}
		return 0;
}

**millie** · 25/10/2006, 23h52

Juste une première remarque, tu déclares ta fonction main comme suit :

Code :

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

main()

Donc ne retourne rien, mais pourtant, tu fais à la fin : return 0;

Normalement, on fait plutôt comme ça :

Code :

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

1
2
3
4
5
6
int main()
{
...
 
 return EXIT_SUCCESS;
}

Tu as fait un switch/case au début, tu peux continuer après également. Tu as oublié le cas où c'est égal à signe==0 (si la personne tappe un mauvais caractère)

Deuxième remarque, tu inclues la fichier d'entête : #include <iostream.h>, ce n'est pas un fichier de la bibliothèque C standard (mais C++, et dans ce cas, c'est mal inclue).

**Emmanuel Delahaye** · 26/10/2006, 00h02

Envoyé par giggs

Salut à tous, je débute en C, et j'ai conçu un mini programme permettant à l'utilisateur d'effectuer des opérations sur deux nombres qu'il rentre (en gros une calculette). Le code n'est pas du tout optimisé, et je souhaiterai avoir votre avis sur celui-ci et si il est possible de l'optimiser...

Il y'a de l'idée, mais la réalisation est inutilement complexe.

Je te conseille de renommer 'signe' en 'operateur' et de lui donner le type int. Ca évitera au compilateur de générer du code de conversion inutile.

Ensuite, il n'est recommandé de changer la valeur d'une variable. Si elle a reçu une valeur qui est celle d'un caractère représentant un opérateur (+, - * etc.), il n'y a aucune raison de la modifier par la suite en lui donnant une valeur 1, 2 ou 3 qui n'a aucune signification. Il suffit d'utiliser la valeur initiale dans le switch et le code reste clair, lisible et maintenable.

Plutôt que cette bardée de if-else-if, qui est correct et utile dans certains cas, il est préférable ici d'utiliser un switch-case qui convient parfaitement à une variable et à des constantes de type int.

Enfin, pour les saisies, il est préférable d'éviter getchar() tout seul et scanf() au profit de fgets() suivit d'une fonction de nettoyage.

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

dernière minute : ton code fait une division entière. Le resultat est donc naturellement peu précis. Tu pourrais garder l'esprit 'division entière' en affichant aussi le reste de la division.

Voilà pour les remarques préliminaires. Maintenant, je vais tester ton code.

Code :

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

#include <iostream.h>

est une abomination. En C, ça n'existe pas (ici, on fait du C, à supprimer). En C++ (la porte en face), non plus, depuis 1998.

Code :

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

main()

Le type retourné doit êre explicite depuis 1999. Il est conseillé d'utiliser la forme 'prototypale' qui exprime clairement qu'il n'y a pas de paramètres.

Code :

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

int main(void)

Code :

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

printf("Bienvenue sur mon premier vrai programme en C.\n Ce programme est une calculette, vous pourrez donc effectuer plusieurs\n operations sur deux nombres\n");

Eviter les ignes trop longues. Une meilleure présentation permet d'éviter ça, et comme pas hasard, rend le code subitement plus lisible :

Code :

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

1
2
3
4
 
   printf ("Bienvenue sur mon premier vrai programme en C.\n"
           " Ce programme est une calculette, vous pourrez donc effectuer plusieurs\n"
           " operations sur deux nombres\n");

Je passe sur les incohérences 'nombre' vs 'chiffres'...

Dans un premier temps, je conseille 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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
 
#include <stdio.h>
 
int main (void)
{
   printf ("Bienvenue sur mon premier vrai programme en C.\n"
           " Ce programme est une calculette, vous pourrez donc effectuer plusieurs\n"
           " operations sur deux nombres\n\n");
 
   printf ("commencez par choisir le type d'operation a effectuer\n"
           " en tapant soit l'un des operateurs suivants : * / - +\n");
   {
      int operateur = getchar ();
 
      switch (operateur)
      {
      case '+':
         {
            int a;
            int b;
            printf ("vous avez tape %c\n", operateur);
            printf ("tapez maintenant le premier operande:\n");
            scanf ("%d", &a);
            printf ("tapez maintenant le second operande:\n");
            scanf ("%d", &b);
            printf ("%d %c %d = %d\n", a, operateur, b, a + b);
         }
         break;
 
      case '-':
         {
            int a;
            int b;
            printf ("vous avez tape %c\n", operateur);
            printf ("tapez maintenant le premier operande:\n");
            scanf ("%d", &a);
            printf ("tapez maintenant le second operande:\n");
            scanf ("%d", &b);
            printf ("%d %c %d = %d\n", a, operateur, b, a - b);
         }
         break;
 
      case '/':
         {
            int a;
            int b;
            printf ("vous avez tape %c\n", operateur);
            printf ("tapez maintenant le premier operande:\n");
            scanf ("%d", &a);
            printf ("tapez maintenant le second operande:\n");
            scanf ("%d", &b);
            printf ("%d %c %d = %d, reste %d\n", a, operateur, b, a / b, a % b);
         }
         break;
 
      case '*':
         {
            int a;
            int b;
            printf ("vous avez tape %c\n", operateur);
            printf ("tapez maintenant le premier operande:\n");
            scanf ("%d", &a);
            printf ("tapez maintenant le second operande:\n");
            scanf ("%d", &b);
            printf ("%d %c %d = %d\n", a, operateur, b, a * b);
         }
         break;
 
      default:
         printf ("desole, ce que vous avez tape n'est pas prevu...\n");
      }
   }
   return 0;
}

Evidemment, un meilleur codage permettrait d'éviter ces horribles redondances...

Il est facile de tester l'appartenance d'un caractère à un ensemble de caractères (une chaine, en fait) avec strchr()...

Ca permettrait à peu de frais de 'factoriser' la saisie qui est strictement identique...

Dans le switch-case, il n'y aurait plus que le calcul et l'affichage du résultat...

**giggs** · 26/10/2006, 00h08

Merci d'avoir prit le temps de me répondre, c'est vrai maintenant je réalise que les if-else sont inutiles, j'aurai pu tout mettre dans le switch tout betement...
pour le iostream.h, en fait c'est mon compilateur qui l'a mis automatiquement, donc c'est pour ça une petite erreur...

**millie** · 26/10/2006, 00h12

Envoyé par giggs

en fait c'est mon compilateur qui l'a mis automatiquement,

Ca me semble particulièrement suspect, un compilateur ne fait que lire le fichier source pour le traduire et ne modifie en rien le fichier....

Tu parles peut être d'un EDI (environnement de développement intégré), dans ce cas, si tu lui as demandé de faire un projet en langage C et qu'il a inclu ça, il faut sérieusement penser à le changer...

**giggs** · 26/10/2006, 00h17

Non, en fait j'ai visual c++ 6.0 et je ne sais pas pourquoi il l'a rajouté, je suis certain de ne rien avoir mis à part le stdio.h parce que c'est à peu près le seul que je connais bien et que j'utilise correctement... Enfin de toute manière, le plus important c'est que j'ai compris comment optimiser ce code, merci encore.

**Gnux** · 26/10/2006, 10h54

Envoyé par giggs

Non, en fait j'ai visual c++ 6.0 et je ne sais pas pourquoi il l'a rajouté, je suis certain de ne rien avoir mis à part le stdio.h parce que c'est à peu près le seul que je connais bien et que j'utilise correctement... Enfin de toute manière, le plus important c'est que j'ai compris comment optimiser ce code, merci encore.

Tu devrais utiliser un vrai EDI comme Code::Blocks

**Franck.H** · 26/10/2006, 11h42

Envoyé par Gnux

Tu devrais utiliser un vrai EDI comme Code::Blocks

+1

C'est plus adapté, VC++ 6 ou .Net semble bien à première vu mais si tu veux faire du C Standard, oublies-les

Pour le C Standard tu as Code::Blocks (qui est d'ailleurs portable sur plusieurs OS et au mieux, si tu veux te rapprocher plus de Linux, tu peut avoir CygWin