Discussion :

[nomane64]

Bonsoir,

Je voudrais que vous m'aidez a ecrire un prog de calcul mental avec un menu permettant de faire les 4 options sur des nombres entre 0 et 10 genre aléatoirement. Une fois l'opération sélectionnée, on demande à l'utilisateur combien de calculs il veut faire et on lui pose le nombre de questions correspondant ce nombre.
Ensuite, on retourne au menu principal. Le menu doit permettre de quitter le programme.

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
#include<stdio.h>  
#include<stdlib.h>  // les biblioteques interessant
#include<math.h>    
#include <time.h>              
main(){
       int k,n,x,a,b,bonnereponse,mareponse,compteurnote;// k commence de premier operation jusqu'à n, n:le nombre d'operations. (a, b) sont des variable aléatoires. mareponse: le reponse d'utilisateur.bonnereponse:la bonne reponse.
       float note;//la note est un nomre réel
 
       printf("\nchoisir le nombre d'operation \n");
       scanf("%d", &n);  //n:représente le numero de l'opération à choisir
       printf("\nChoisir une opération parmi les quatre opérations:\n 1) addition(+)\n 2) soustraction(-)\n 3) multiplication(*)\n 4) division(/)\n");
       scanf("%d",&x);   //x:représente une opération
       compteurnote=0; // initialisation du compteurnote
 
       if(x==1){  // (+)
                      for(k=1;k<=n;k++){
                                          a=(aleatoire())%11;
                                          b=aleatoire()%11;
                                          bonnereponse=a+b;
                                          printf("\n%d + %d = ?\n", a,b);
                                          scanf("%d", &mareponse);
                                          if(mareponse==bonnereponse){
                                                                           printf("CORRECT\n");
                                                                           compteurnote=compteurnote +1;
                                          }
                                          else {
                                                printf("FAUX !! La réponse CORRECTE est %d\n",bonnereponse);
                                          }
                       }
        }
 
       else if(x==2){ //(-) 
                           for(k=1;k<=n;k++){
                                          a=(aleatoire())%11;
                                          b=aleatoire()%11;
                                          bonnereponse=a-b;
                                          printf("\n%d - %d = ?\n", a,b);
                                          scanf("%d", &mareponse);
                                          if(mareponse==bonnereponse){
                                                                           printf("CORRECT\n");
                                                                           compteurnote=compteurnote +1;
                                          }
                                          else {
                                                printf("FAUX !! La réponse CORRECTE était %d\n",bonnereponse);
                                          }
                       }
        }
 
        else if(x==3){ // (*)
                           for(k=1;k<=n;k++){
                                          a=(aleatoire())%11;
                                          b=aleatoire()%11;
                                          bonnereponse=a*b;
                                          printf("\n%d * %d = ?\n", a,b);
                                          scanf("%d", &mareponse);
                                          if(mareponse==bonnereponse){
                                                                           printf("CORRECT\n");
                                                                           compteurnote=compteurnote +1;
                                          }
                                          else {
                                                printf("FAUX !! La réponse CORRECTE était %d\n",bonnereponse);
                                          }
                       }
        }
 
        else if(x==4){ // (/)
                           for(k=1;k<=n;k++){
                                          a=(aleatoire())%11;
                                          b=(aleatoire()%10)+1; //le nombre b doit être différent de 0
 
                                          bonnereponse=(float)a/b;
                                          printf("\n%d / %d = ?\n", a,b);
                                          scanf("%d", &mareponse);
                                          if(mareponse==bonnereponse){
                                                                           printf("CORRECT\n");
                                                                           compteurnote=compteurnote +1;
                                          }
                                          else {
                                                printf("FAUX !! La réponse CORRECTE était %d\n",bonnereponse);
                                          }
                       }
        }
        else {
              printf("\nvotre type d'opération incorrect !!!\n");
        }
 
        note=(float)compteurnote*100/(float)n; // calcule de la note sur 100
        printf("\nVotre note est: %f sur 100\n\n", note);
} //ferme la fonction main()
 
int aleatoire (){ // une fonction qui donne un nombre aléatoire
 
              static int first = 0;
 
 
              if (first==0){
                              srand (time (NULL));  
                              first=1;
              }
              return (rand());                     
}

Je vous remercie

[ram-0000]

Quel est ton problème avec ce code ?

[Médinoc]

Alors, par où je commence ?

• C'est indenté comme l'as de pique
• Déclaration de main() incorrecte
• Utilisation de variables dont le nom ne fait qu'une seule lettre
• Fonction aleatoire() utilisée avant toute déclaration
• Tu devrais utiliser switch au lieu de ces if successifs
• Retour de main() absent
• main() de trois kilomètres, pas assez décomposé.

[Médinoc]

Voici ton code, reformaté, reséparé, et corrigé:

Code C :

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
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <time.h>
 
int aleatoire () { /* une fonction qui donne un nombre aléatoire */
	static int bFirst = 0;
 
	if(!bFirst) {
		srand((int)time(NULL));  
		bFirst=1;
	}
	return rand();
}
 
void operation(int x, int *pCompteurnote)
{
	int a = aleatoire() % 11;
	int b = aleatoire() % 11;
	int bonnereponse=0, mareponse=0;
	if(x==1) {        /* Addition(+) */
		bonnereponse = a+b;
		printf("\n%d + %d = ?\n", a,b);
	} else if(x==2) { /* Soustraction (-) */
		bonnereponse = a-b;
		printf("\n%d - %d = ?\n", a,b);
	} else if(x==3) { /* Multiplication (*) */
		bonnereponse=a*b;
		printf("\n%d * %d = ?\n", a,b);
	} else if(x==4) { /* Division (/) */
		bonnereponse=a/b; /*Le cast en float ne sert à rien ici.*/
		printf("\n%d / %d = ?\n", a,b);
		scanf("%d", &mareponse);
	}
	scanf("%d", &mareponse);
	if(mareponse==bonnereponse) {
		printf("CORRECT\n");
		(*pCompteurnote)++;
	} else {
		printf("FAUX !! La réponse CORRECTE est %d\n", bonnereponse);
	}
}
 
int main(void) {
	int k; /*commance de premier operation jusqu'à n*/
	int n; /*le nombre d'operations*/
	int x,compteurnote;
	double note;/*le nete est un nomre réel*/
 
	printf("\nchoisir le nombre d'operation \n");
	scanf("%d", &n);  /*n:représante le nombre des opération à choisir*/
	printf("\nChoisir une opération prmi les catres opérations:\n 1) addition(+)\n 2) soustraction(-)\n 3) multiplication(*)\n 4) division(/)\n");
	scanf("%d",&x);   /*x:représente une opération*/
	compteurnote=0; /* initialisation du compteurnote */
 
	if(x>=1 && x<=4) {
		for(k=1 ; k<=n ; k++) {
			operation(x, &compteurnote);
		}
	} else {
		printf("\nvotre type d'opération incorrect !!!\n");
	}
 
	note = (double)compteurnote*100.0/(double)n; /* calcule de la note sur 100 */
	printf("\nVotre note est: %f sur 100\n\n", note);
	return 0;
} /* ferme la fonction main() */

Par contre, j'ai laissé ton orthographe déplorable telle quelle...

[nicolas.sitbon]

Quite à utiliser le C99, autant l'utiliser à fond :

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
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <time.h>
#include <stdint.h>
#include <inttypes.h>
 
static void compute (char const operation, uint8_t * const note)
{   
   int a = rand () % 11;
   int b = rand () % 11;
   int bonnereponse = 0;
   int mareponse = 0;
   int ret = 0;
 
   switch (operation)
   {
      case '+':
         bonnereponse = a + b;
         break;
      case '-':
         bonnereponse = a - b;
         break;
      case '*':
         bonnereponse = a * b;
         break;
      case '/':
         bonnereponse = a / b;
         break;
   }
 
   do
   {
      printf ("%d %c %d = ?\n", a, operation, b);
      ret = scanf ("%d", &mareponse);
      scanf ("%*[^\n]"), getchar ();
   }
   while (ret != 1);
 
   if (mareponse == bonnereponse)
   {
      puts ("CORRECT");
      (*note) ++;
   }
   else
   {
      printf ("FAUX !! La réponse CORRECTE était %d\n", bonnereponse);
   }
}
 
int main (void)
{
   uint8_t nb_operation = 0;
   char operation[2] = {0};
   uint8_t note = 0;
   int ret = 0;
 
   srand ((unsigned) time (NULL));
 
   do
   {
      puts ("choisir le nombre d'operation");
      ret = scanf ("%" SCNu8, &nb_operation);
      scanf ("%*[^\n]"), getchar ();
   }
   while (ret != 1);
 
   do
   {
      puts ("operation :+, -, * ou /");
      ret = scanf ("%1[-+*/]", operation);
      scanf ("%*[^\n]"), getchar ();
   }
   while (ret != 1);
 
   for (uint8_t loop = nb_operation; loop != 0; loop--)
   {
      compute (*operation, &note);
   }
 
   printf ("Votre note est: %g sur 100\n", note * 100.0 / nb_operation);
   return 0;
}

Attention, je ne gère pas les overflows!!!

[Emmanuel Delahaye]

Quite à utiliser le C99, autant l'utiliser à fond :

Code :

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

1
2
3
4
 
int main (void)
{
   uint8_t nb_operation = 0;

peut être, mais avec discernement. uint8_t n'est pas portable. Comme tous les types de tailles fixes, il n'existe que sur les plateformes qui le supportent.

Sur certains DSP, un char fait 16, voire 32-bit...

De plus, mon MinGW ne connait pas 'SCNu8' (normal, j'étais pas en C99). En fait, c'est OK avec -std=c99.

Enfin, quel intérêt ici, d'utiliser un type de largeur fixe, fusse-t-il de 8-bit ? Un int suffit largement et est tout à fait portable.

[Médinoc]

Quite à utiliser le C99

Huh? Oo