Discussion :

[Pirodactil]

Je suis un débutant et je voudrais savoir comment trouver vous ce mini programme archi débutant . Les critiques sont les biens venus.

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
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
#include <stdio.h>
#include <stdlib.h>
 
int menu()
{
int identifier = 0;
 
while (identifier != 37)
{
    printf("\n=== Mot de Passe ===\n\n");
    printf("Identifez-vous : ");
    scanf("%d", &identifier);
}
  int choix = 0;
 
while (choix < 1 || choix > 5)
{
    printf("\n\nOperation :\n\n");
    printf("1. Addition\n");
    printf("2. Soustraction\n");
    printf("3. Multiplication\n");
    printf("4. Division\n");
    printf("5. Quitter\n\n");
    printf("Votre choix ?\n");
    scanf("%d", &choix);
    printf("\n\n");
}
return choix;
}
int main ( int argc, char** argv )
{
 
 
int operation;
int resultat = 0, nombre1 = 0, nombre2 = 0;
int Menu = 1;
 
do
{
 
switch (operation = menu())
 
{
case 1:
printf("Addition :\n\n");
printf("Nombre 1 : ");
scanf("%d", &nombre1);
printf("Nombre 2 : ");
scanf("%d", &nombre2);
resultat = nombre1 + nombre2;
printf("%d + %d = %d\n", nombre1, nombre2, resultat);
break;
 
case 2:
printf("Soustraction :\n\n");
printf("Nombre 1 : ");
scanf("%d", &nombre1);
printf("Nombre 2 : ");
scanf("%d", &nombre2);
resultat = nombre1 - nombre2;
printf("%d - %d = %d\n", nombre1, nombre2, resultat);
break;
 
case 3:
printf("Multiplication :\n\n");
printf("Nombre 1 : ");
scanf("%d", &nombre1);
printf("Nombre 2 : ");
scanf("%d", &nombre2);
resultat = nombre1 * nombre2;
printf("%d * %d = %d\n", nombre1, nombre2, resultat);
break;
 
case 4:
printf("Division :\n\n");
printf("Nombre 1 : ");
scanf("%d", &nombre1);
printf("Nombre 2 : ");
scanf("%d", &nombre2);
resultat = nombre1 / nombre2;
printf("%d / %d = %d\n", nombre1, nombre2, resultat);
break;
 
case 5:
printf("Appuyer sur une touche pour continuer...");
break;
 
default:
printf("Vous n'avez pas rentre un nombre correct.");
break;
}
 
printf("\n\n");
 
 
system("PAUSE");
printf("Revenir au menu?\n");
printf("1-Oui\n");
printf("2-Non\n");
scanf("%d",&Menu);
 
}while(Menu != 0);
 
return 0;
}

Comme vous l'avais vu le mot de passe est 37

ps: Désolé pour les fautes d'ortho

[Sve@r]

Je suis un débutant et je voudrais savoir comment trouvez-vous ce mini programme archi débutant . Les critiques sont les bienvenues.

Petite question avant toute chose: ce code fonctionne-t-il chez-toi sans souci ? Tu peux tout saisir comme tu en as envie ? Il n'y a pas par exemple un des deux nombres ou un choix qui se retrouve avec une valeur sans que le système t'ait demandé de la rentrer ?

[droggo]

Heo,

Je suis un débutant et je voudrais savoir comment trouvez-vous ce mini programme archi débutant . Les critiques sont les bienvenues

Ce serait plutôt

Je suis un débutant et je voudrais savoir comment vous trouvez ce mini programme archi débutant . Les critiques sont les bienvenues

[Médinoc]

Critique: Si quelqu'un tape n'importe quoi d'autre que des chiffres, ton programme ne va pas aimer.

[lilington]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
while (choix < 1 || choix > 5)
{
    printf("\n\nOperation :\n\n");
    printf("1. Addition\n");
    printf("2. Soustraction\n");
    printf("3. Multiplication\n");
    printf("4. Division\n");
    printf("5. Quitter\n\n");
    printf("Votre choix ?\n");
    scanf("%d", &choix);
    printf("\n\n");
}

sinon , ton while fait l'inverse de ce que tu veux. en C le while continue tant que l'expression =1. prenons par exemple le cas ou choix = 2 pour une soustraction on a while (2<1 || 2>5) ce qui donne while(0||0) ce qui donne while(0) donc on quite la boucle sans faire la soustraction.
je crois que tu voulais dire:

Code :

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

1
2
3
while(choix>0&&choix<6)  
//ou encore 
while(choix>=1&&choix<=5)

[Médinoc]

^Non non, cette partie du code est correcte: On pose la question tant qu'on n'a pas une réponse valide.

[bpy1401]

Bonjour Pirodactil

Est tu sur que cela compile correctement.

Pour moi, il est interdit de faire une déclaration de variable locale après du code (ligne 14 de ton code). ce n'est pas ANSI C.

Remplace

Code :

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

int menu()

par

Code :

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

int menu(void)

je préfère

Code :

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

1
2
3
 
   operation = menu();
   switch (operation)

au lieu de

Code :

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

1
2
 
  switch (operation = menu())

Car je n'aime pas les affectation dans un test.

Tu peux utiliser getch() au lieu de system("PAUSE");

et pour finir

Code :

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

1
2
3
4
5
6
printf("Revenir au menu?\n");
printf("1-Oui\n");
printf("2-Non\n");
scanf("%d",&Menu);
 
}while(Menu != 0);

0 n'est pas une option de ton menu

Cordialement

[Médinoc]

^ getch() n'est pas standard (bon OK, system("pause") non plus)

[gangsoleil]

Bonjour,

Lire les entrees clavier avec scanf est un probleme : si l'utilisateur rentre une lettre au lieu d'un nombre, le programme plante.

Regarde les tutoriaux sur "comment faire des entrees/sorties"

Il faut aussi que tu indentes ton code.

[Sve@r]

Pour moi, il est interdit de faire une déclaration de variable locale après du code (ligne 14 de ton code). ce n'est pas ANSI C.

Salut

Sisi, c'est maintenant autorisé depuis, je crois, C99...

Remplace

Code :

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

int menu()

par

Code :

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

int menu(void)

Là c'est une notation C++. C'est autorisé aussi de la mettre en C mais ce n'est pas une obligation.

je préfère

Code :

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

1
2
3
 
   operation = menu();
   switch (operation)

au lieu de

Code :

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

1
2
 
  switch (operation = menu())

Car je n'aime pas les affectation dans un test.

Chacun voit midi à sa porte (surtout quand il est midi )

Accessoirement, l'une ou l'autre écriture produisent exactement le même code à la compilation. Donc autant c'est vrai que dans un while ((carac=fct()) != valeur_precise) c'est pratique de mettre l'affectation dans le test, autant c'est vrai que ici ta proposition apporte plus de clarté sans rien enlever en rapidité.

[Obsidian]

Sisi, c'est maintenant autorisé depuis, je crois, C99...

Ça reste mal ! :-)

En C++, je veux bien admettre qu'il y a quelques cas où ça se justifie. En C, c'est beaucoup plus contestable…

[lilington]

Ça reste mal ! :-)

En C++, je veux bien admettre qu'il y a quelques cas où ça se justifie. En C, c'est beaucoup plus contestable…

Je le fais en tout cas tres souvent dans des fonctions car je fait si les parametre de la fonction sont pas correct je sort tout de suite sans rien declarer, tiens un exemple:

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
char * function(const char *arg)
{
   if(arg==NULL)
         return NULL;
   if( initLeTruc(arg)!=0)
         return NULL;
   /* maintenant je declare on grand tableau et mon tas de gaspillage 
memoire que je suis sure d'utiliser*/
   char *ptr=malloc(...);
    if (machintruc_ptr)
           return NULL;
   char tbl[4096];
   float interm[512];
 
  return ptr;
}

et un cas encore plus interressant c'est quand on a un test qui est bon disons 10%des cas pourquoi declarer une variable qui sera utiliser que 10%du cas? ex:

Code :

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

1
2
3
4
5
6
if (cas_rare)
{
    float valeurs[16];
    ...
    //faire un truc avec valeurs
}

bon maintenant il y a un truc que je sais pas c'est si le float valeurs[16]; sera execute quoi qu'il en soit ou bien si il n'est execute que si le car_rare est arrive.
dans le premier cas effectivement ca devient inutile de le declarer la.

[Sve@r]

et un cas encore plus interressant c'est quand on a un test qui est bon disons 10%des cas pourquoi declarer une variable qui sera utiliser que 10%du cas? ex:

Code :

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

1
2
3
4
5
6
if (cas_rare)
{
    float valeurs[16];
    ...
    //faire un truc avec valeurs
}

bon maintenant il y a un truc que je sais pas c'est si le float valeurs[16]; sera execute quoi qu'il en soit ou bien si il n'est execute que si le car_rare est arrive.
dans le premier cas effectivement ca devient inutile de le declarer la.

Ton exemple était déjà autorisé dès le début du C en 1968. En effet, une variable peut se déclarer au début de tout bloc (et non, comme on le croit généralement, au début de la fonction)
Et donc la variable ne sera créée dans la pile que si le cas rare arrive.

[lilington]

a merci c'est bien ce que je pensais (sans preuve).
d'ou l'utilite de pouvoir declarer quand c'est utile plutot que de tout declarer au debut meme si c'est pas sur qu'on utilise(meme si c'est plus jolie pour les yeux).

[gangsoleil]

La meme code, sans declaration au milieu de la fonction, mais qui respecte le fait de n'allouer que lorsque c'est necessaire, en C 89 (et probablement avant) :

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
char * function(const char *arg)
{
    char * return_value;
    if(arg==NULL) {
        return_value = NULL;
    }
    else {
        if( initLeTruc(arg)!=0) {
            return_value = NULL;
        }
        else {
            /* maintenant je declare on grand tableau et mon tas de gaspillage memoire que je suis sure d'utiliser*/
            char *ptr=malloc(...);
 
            if (ptr == NULL) {
                return_value = NULL;
            }
            else {
                char tbl[4096];
                float interm[512];
 
                /* code */
                return_value = resultat;
            }
        }
    }
    return return_value
}

[lilington]

Je suis d'accord c'est possible c'est meme plus ou moins beau.
mais voila, ca fait beaucoup de if-else. si dans mon code j'ai deja 2 voir 3 niveau de if-else ca va finir par faire profond.
sinon dans l'idee quand la fonction est simple je pense qu'effectivement ta presentation est preferable.

[bpy1401]

Bonjour à tous,

En voila une discussion intéressante.

Je suis d'accord c'est possible c'est même plus ou moins beau.
mais voila, ca fait beaucoup de if-else. si dans mon code j'ai deja 2 voir 3 niveau de if-else ca va finir par faire profond.
sinon dans l'idee quand la fonction est simple je pense qu'effectivement ta presentation est preferable.

Tout dépend du type de logiciel que l'on souhaite développer. je travail dans l'informatique embarqué, et chez nous nous devons respecter les règles MISRA C

http://www.misra-c.com/Activities/MI...0/Default.aspx

C'est très spécifique à l'embarqué (et principalement automobile). Sur les bouts de code que j'ai vu je pourrais dire ceci.

1 -Dans une fonction, on ne peut avoir qu'un seul "return", la dernière solution pour moi est donc la meilleure.

2- les variables ne doivent être déclarées que dans la partie de code ou elles seront utilisées. ==> utilisation du
{
déclaration
code
{
déclaration
code
}
}

2- void fonction() {} doit être void fonction(void) car les paramètres doivent être clairement définis (ou pas de paramètre).

Ces règles ont été éditées suite à des bugs détectés sur la calculateurs, certains de ces bugs sont du aux compilateurs, d'autres à des erreurs de frappe ou mauvaise écriture de code.

Sur PC , les compilateurs sont éprouvés, et il est possible que ces règles ne soient pas très utiles, mais je les appliquent toujours même sur PC.

Cordialement.

[LIST]

[lilington]

interressant.