Discussion :

[CechD]

Salut à tous !

J'essaye de créer un jeu de calcul mental pour m'entraîner (quand j'ai vu ce que l'app store propose.... ).

Du coup j'ai crée un premier jet et lors de l'exécution un calcul apparaît, je donne la réponse et ensuite deux calculs apparaissent à la suite.

Voici un bout de mon code :

Code :

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

1
2
3
4
5
6
7
 while(conteur != 10){
        operation(niveau);
        printf("\nReponse : ");scanf("%d",&reponse);
        printf("\nScore : %d",score);
        if(reponse == operation(niveau)){score+=1;}
        conteur+=1;
    }

et puis :

Code :

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

1
2
3
4
5
6
7
8
9
int operation (int niveau){
    int x; x=rand()%3;
    switch (x){
        case 0 : return addition(niveau); break;
        case 1 : return soustraction(niveau); break;
        case 2 : return multiplication(niveau); break;
        case 3 : return division(niveau); break;
    }
}

Qu'en pensez-vous ? Je vous tiens au courant pour la suite ! Merci !

[Sve@r]

Qu'en pensez-vous ?

Bonjour
Je ne sais pas trop. Qu'est-ce qu'on en pense par rapport à quoi ? A ton niveau ? Ben si t'es débutant c'est pas mal. Si t'es confirmé alors tu peux certainement faire mieux. Remplacer par exemple tes opérations switchées par carrément un pointeur de fonctions. Tu stockes tes fonctions dans un tableau de pointeurs puis à chaque tour tu appelles la fonction[x] du tableau. Voilà par exemple ce que je pourrais en penser...

[BufferBob]

salut,

accessoirement le conteur c'est celui qui narre une histoire, celui qui compte c'est le compteur, c'est un détail mais bon...

[CechD]

salut,

accessoirement le conteur c'est celui qui narre une histoire, celui qui compte c'est le compteur, c'est un détail mais bon...

Tu m'as tué, je m'incline je vais modifier cela.

[Sve@r]

Code :

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

1
2
3
4
5
6
7
8
9
int operation (int niveau){
    int x; x=rand()%3;
    switch (x){
        case 0 : return addition(niveau); break;
        case 1 : return soustraction(niveau); break;
        case 2 : return multiplication(niveau); break;
        case 3 : return division(niveau); break;
    }
}

Tu m'as tué, je m'incline je vais modifier cela.

Oui et aussi (en dehors de l'inutilité de cette variable "x" puisque tu peux directement switcher le résultat de l'opération; et des break puisque tu quittes la fonction avant de les atteindre) demande-toi si le case 3 sera atteint un jour...

[CechD]

Bonjour

Salut !

Tu stockes tes fonctions dans un tableau de pointeurs puis à chaque tour tu appelles la fonction[x] du tableau.

Super idée, je connais les pointeurs, je pense que je peux le faire. Et ben finalement mon programme plante. Je vais faire une pause puis reattaquer voici où j'en suis :


Partie opération :

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
int power(a,b){
    if((a==0)&&(b==0)){return 0;}
    else if(b==0){return 1;}
    else if(b==1){return a;}
    return a*power(a,b-1);
}
 
int** arithmetique(niveau){
    int *addition = calloc(sizeof(int),3);
    addition[0] = rand()%(niveau*100); addition[1] = rand()%(niveau*100); addition[2] = addition[0]+addition[1];
 
    int* soustraction = calloc(sizeof(int),3);
    soustraction[0] = rand()%(niveau*100); soustraction[1] = rand()%(niveau*100); soustraction[2] = soustraction[0]-soustraction[1];
 
    int* multiplication = calloc(sizeof(int),3);
    multiplication[0] = rand()%(power(10,niveau)); multiplication[1] = rand()%(power(10,niveau)); multiplication[2] = multiplication[0]*multiplication[1];
 
    int* division = calloc(sizeof(int),3);
    while((division[0]%division[1] == 0)&&(division[1]!=0)){
    division[0] = rand()%(power(10,niveau)); division[1] = rand()%(power(10,niveau)); division[2] = division[0]/division[1];
    }
 
    int** arith = calloc(sizeof(int*),4);
    arith[0] = addition; arith[1] = soustraction; arith[2] = division; arith[3] = multiplication;
    return arith;
 
}
 
char operateur(int operation){
    if(operation == 0){return '+';}
    if(operation == 1){return '-';}
    if(operation == 2){return '*';}
    if(operation == 3){return '/';}
}
 
void free_arith(int ** arith){
    int i; for(i=0;i<2;i++) {free(arith[i]);}
    free(arith);
}

Partie partie :

Code :

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

1
2
3
4
5
6
7
8
 while(compteur != 10){
        operation = rand()%4;
        printf("\n%d %c %d = ",arithmetique(niveau)[operation][0],operateur(operation),arithmetique(niveau)[operation][1]);
        printf("\nReponse : ");scanf("%d",&reponse);
        printf("\nScore : %d",score);
        if(reponse == arithmetique(niveau)[operation][2]){score+=1;}
        compteur+=1;
    }

[CechD]

demande-toi si le case 3 sera atteint un jour...

Salut ! Je suis d'accord avec vous de A à Z. Mais maintenant elle n'est plus dans mon code (je l'avais mise y'a longtemps).

[Sve@r]

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
 
int** arithmetique(niveau){
    int *addition = calloc(sizeof(int),3);
    addition[0] = rand()%(niveau*100); addition[1] = rand()%(niveau*100); addition[2] = addition[0]+addition[1];
 
    int* soustraction = calloc(sizeof(int),3);
    soustraction[0] = rand()%(niveau*100); soustraction[1] = rand()%(niveau*100); soustraction[2] = soustraction[0]-soustraction[1];
 
    int* multiplication = calloc(sizeof(int),3);
    multiplication[0] = rand()%(power(10,niveau)); multiplication[1] = rand()%(power(10,niveau)); multiplication[2] = multiplication[0]*multiplication[1];
 
    int* division = calloc(sizeof(int),3);
    while((division[0]%division[1] == 0)&&(division[1]!=0)){
    division[0] = rand()%(power(10,niveau)); division[1] = rand()%(power(10,niveau)); division[2] = division[0]/division[1];
    }
 
    int** arith = calloc(sizeof(int*),4);
    arith[0] = addition; arith[1] = soustraction; arith[2] = division; arith[3] = multiplication;
    return arith;
 
}

Mille fois trop compliqué !!! Pourquoi vouloir stocker les opérandes ???
Te suffit de stocker la fonction puis, quand tu reçois les valeurs à opérer tu appelles la fonction en lui passant les valeurs (sans les stocker)

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
#include <stdio.h>
 
int add(int x, int y)
{
	return x+y;
}
 
int sub(int x, int y)
{
	return x-y;
}
 
int mult(int x, int y)
{
	return x*y;
}
 
int main()
{
	typedef struct {
		char *libelle;
		char operator;
		int (*fct)(int , int);
	} t_operation;
	t_operation tabOP[]={
		{"addition", '+', add},
		{"soustraction", '-', sub},
		{"multiplication", '*', mult},
		{NULL, 0, NULL},
	};
	t_operation *pt_op;
 
	int x=10, y=12;
 
	for (pt_op=tabOP; pt_op->fct != NULL; pt_op++)
		printf("%d %c %d = %d\n", x, pt_op->operator, y, (*pt_op->fct)(x, y));
}

[CechD]

Merci Sve@r pour vos conseils.

Je travail sur votre code .

[Matt_Houston]

Y'a de l'idée ! On voit que tu essaies de mettre en place des constructions avec les briques que tu connais, ça ne fait pas toujours sens mais c'est un bon entraînement. Le projet que tu as choisi est intéressant, lorsque tu seras plus à l'aise tu pourras même éventuellement le compléter pour qu'il génère des expressions mathématiques (mettons « 18 * 9 + 7 ^ 3 ») et chronomètre l'utilisateur !

Et puis j'aime bien la manière que tu as de nommer tes threads.