Discussion :

[Amybond]

Quand j'execute mon code apres que j'ai rentrer les info du scanf il se ferme alors qu'il y a un "system("PAUSE");"
Mon code:

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
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
// On affiche le menu
long menu()
{
    long votreChoix = 0;
    printf("===MENU===\n\n");
    printf("Faire les multiplication : \n");
    printf("1. Aleatoirement\n");
    printf("2. De facon croissante\n");
    printf("3. Juste une table en particulier\n");
    printf("4. Terminer\n\n");
    printf("Entrez votre choix : ");
    scanf("%ld", votreChoix);
    printf("\n");
    return votreChoix;
}
// On cree le premier chiffre de la multiplication et le resultat
long chiffre1()
{
     const long MAX = 10, MIN = 1;
     long chiffre1 = 0;
     srand(time(NULL));
     chiffre1 = (rand() % (MAX - MIN + 1)) + MIN;
     return chiffre1;
}
 
long chiffre2()
{
     const long MAX = 10, MIN = 1;
     long chiffre2 = 0;
     srand(time(NULL));
     chiffre2 = (rand() % (MAX - MIN + 1)) + MIN;
     return chiffre2;
}
 
long resultatJuste(long chiffre1, long chiffre2)
{
     long resultatJuste = 0;
     resultatJuste = chiffre1 * chiffre2;
     return resultatJuste;
}
 
//Afficher a l'utilisateur la multiplication et recupere sont entrer
long afficherRecup(long chiffre1, long chiffre2)
{
     long nombreEntrerUtilisateur = 0;
     printf("%ld * %ld = ", chiffre1, chiffre2);
     scanf("%ld", nombreEntrerUtilisateur);
     return nombreEntrerUtilisateur;
}
 
int compare(long chiffre1, long chiffre2, long resultatJuste, long resultatUtilisateur)
{
    if (resultatJuste == resultatUtilisateur)
    {
         printf("Vous avez trouvez le bon resultat. ");
         printf("%ld * %ld = %ld", chiffre1, chiffre2, resultatJuste);
         return 1; 
    }
    else
    {
        printf("Vous n'avez pas trouvez le bon resultat. ");
        return 0;
    }
}
 
//Fonction principale
int main(int argc, char *argv[])
{
    afficherRecup(chiffre1(), chiffre2());
    compare(chiffre1(), chiffre2(), resultatJuste(chiffre1(), chiffre2()), afficherRecup(chiffre1(), chiffre2()));
    system("PAUSE");
    return 0;
}

[crocodilex]

Quand j'execute mon code apres que j'ai rentrer les info du scanf il se ferme alors qu'il y a un "system("PAUSE");"
Mon code:

Il faut apprendre a lire les messages du compilateur. Le 2eme argument de scanf doit etre un pointeur.

Code :

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

1
2
scanf("%ld", &nombreEntrerUtilisateur);

J'ai aussi l'impression que tu ne sais pas ce que tu fais. Je verrais bien le main() comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int main(void)
{
    long nb1, nb2, resOut;
    nb1 = chiffre1();
    nb2 = chiffre2();
    resOut = afficherRecup(nb1, nb2);
    compare(nb1, nb2, resultatJuste(nb1, nb2), resOut);
    system("PAUSE");
    return 0;
}

[_SamSoft_]

C'est vrai que demander via scanf une valeur et ne pas pointer vers la variable dans laquelle l'enregistrer ca me lol

[Amybond]

Ok merci a tous mais j'ai un nouveaux probleme .

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
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
void chiffresMultiplication(long *pointeurChiffre1, long *pointeurChiffre2, long *pointeurResultat, const long MAX, const long MIN)
{
    *pointeurChiffre1 = (rand() % (MAX - MIN + 1)) + MIN;
    *pointeurChiffre2 = (rand() % (MAX - MIN + 1)) + MIN;
    *pointeurResultat = *pointeurChiffre1 * *pointeurChiffre2;
}
 
void compareResultats(long resultat, long nombreUtilisateur, long *pointeurComparaison)
{
    if(resultat == nombreUtilisateur)
         *pointeurComparaison = 1;
    else
         *pointeurComparaison = 0;
}
 
int main()
{
    srand(time(NULL));
    long chiffre1 = 0, chiffre2 = 0, nombreUtilisateur = 0, resultat = 0, comparaison = 0;
    const long MAX = 10, MIN = 1;
    long *pointeurChiffre1 = &chiffre1, *pointeurChiffre2 = &chiffre2;
    long *pointeurResultat = &resultat, *pointeurComparaison = &comparaison;
    chiffresMultiplication(pointeurChiffre1, pointeurChiffre2, pointeurResultat, MAX, MIN);
	printf("%ld * %ld = ", chiffre1, chiffre2);
	scanf("%ld", nombreUtilisateur);
	compareResultats(resultat, nombreUtilisateur, pointeurComparaison);
	if(comparaison)
	{
	    printf("Bonne reponse.");
    }
    else
    {
        printf("Faux.");
    }
    system("PAUSE");
	return 0;	
}

Il ne s'execute pas jusqu'au bout

[exhortae]

Ok merci a tous mais j'ai un nouveaux probleme .

Il ne s'execute pas jusqu'au bout

t'as le scanf pour un long qui se balade sans & dans ton main

scanf("%ld", &nombreUtilisateur);

[souviron34]

en plus dans ton main tu mets une instruction (srand()) avant les déclarations de variables... D'abord je sais pas si ça compile, mais en tous cas c'est de la mauvaise programmation..

[Emmanuel Delahaye]

Quand j'execute mon code apres que j'ai rentrer les info du scanf il se ferme alors qu'il y a un "system("PAUSE");"
Mon code:

Mes commentaires (les erreurs ne sont signalées qu'une fois...) :

• Les fonctions sans paramètres doivent être définies avec (void)
• srand() ne doit être appelée qu'une seule fois au début du programme.
• Les paramètres de main() ne sont pas utilisés. On peut les omettre et les remplacer par void.
• Intéressante organisation du code purement fonctionnelle, malheureusement, l'ordre dans lequel sont évaluées les paramètres des fonctions n'est pas défini. Il dépend de l'implémentation. Il faut donc pas compter dessus et il faut passer par des variables intermédiaires. Le comportement est indéfini.
• BUG ! scanf("%ld", votreChoix); %ld attend l'adresse d'un long et non la valeur d'un long.