Discussion :

[Fuxor]

Bonjour à tous,

Je suis nouveau sur le forum, mais cela fais déjà un petit temps que je scrute le site pour toutes ces tonnes d'informations

J'ai un travail a rendre en C, je l'ai terminer mais je voudrais savoir ce que vous pensez du code ? ce qu'il y aurait a revoir, modifier,...



ENNONCE
1/ créer une fonction pile ou face ( vrai ou faux )

2/ creer une fonction qui recoit le nombre d'essais, un vecteur de tirage réalisé par l'ordinateur et un vecteur de proposition du joueur et, retourne le nombre de fois ou l'utilisateur a trouvé le bon résultat ( le nombre de succes)

3/ecrire une fonction qui recoit le nombre d'essais, le vecteur de tirage et le vecteur de proposition, affiche l'ecran ci dessous et retourne le choix de l'utilisateur. ( c'est le menu qui est deja dans mon pgm ca )

4/un programme principal qui gere les parties et affiche le resultat complet avant de demarrer une nouvelle partie ou de quitter le programme.

Contraintes:

-utiliser la pré-définition de toutes les constantes
-utiliser le type de boucle le mieux adapté a chaque cas.
-utiliser le type de variable le plus adéquat. ( optimiser l'espace memoire)

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
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <time.h>
 
// Fonction qui renvoie le nombre de succès du joueur
int nbsucces (int essai , char ordi[] ,char joueur[])
{
    int pourcent=0;
    int nbsucces=0;
    if(joueur==ordi)
    {
        printf("\n\t GAGNE\n");
        nbsucces=nbsucces+1;
    }
    else
    {
        printf("\n\t PERDU\n");
        nbsucces=nbsucces+0;
    }
    return nbsucces;
 
    pourcent=(nbsucces/essai)*100;
}
 
// Fonction rand qui génère aléatoirement un 1 ou un 0 pour créer respectivement un face ou un pile.
char pileface (void)
{
    int essai=0;
    char P , F;
    char joueur;
 
    printf("\n\t Pile ou Face");
    printf("\n\t P = pile et F = Face.\n");
    scanf("%d", &joueur);
    int j=0;
    do
    {
        if(joueur!=P || joueur!=F)
        {
            printf("\n\t P = pile et F = Face.\n");
            scanf("%d", &joueur);
        }
 
 
        srand(time(NULL));
        char ordi = rand() % 2;
        int i=0;
 
        if(ordi==0)
        {
            ordi= P;
            printf("La piece est tombee sur PILE.\n");
            }
        else
        {
            ordi= F;
            printf("La piece est tombee sur FACE.\n");
        }
        essai++;
    }
    while(j!=1);
	int nbsucces (int essai , char ordi[] ,char joueur[]);
}
 
// Fonction pour l'affichage
void affiche(int essai, char ordi[],char joueur[])
{
    char choix , N , L , Q;
 
	printf("\t \t Pile ou Face \n");
	printf("\t \t ============ \n \n");
 
	printf("Partie en cours : \n");
	printf("\t Nombre d'essais : %d\n", essai);
	printf("\t Pourcentage de reussite du joueur : %d\n", pourcent);
 
	prtinf("Menu :\n");
	printf("\t (N) \t Demarrer une Nouvelle partie\n");
	printf("\t (L) \t Lancer La piece\n");
	printf("\t (Q) \t Quitter\n");
	printf("\t \t \t \t \t Choix ?\n");
	scanf("%d",&choix);
 
    if(choix=N)
    {
 
    }
    if(choix=L)
    {
       char joueur (void);
    }
    if(choix=Q)
    {
        system('CLS');
    }
 
 
 
}
 
// Fonction qui gère les parties et affiche le résultat complet avant de
// démarrer une nouvelle partie ou de quitter le prgramme.
void afficheresultat (int essai, char ordi[], char joueur[])
{
    printf("\t \t Pile ou Face \n");
	printf("\t \t ============ \n \n");
 
    printf("Resultat de la derniere partie : ( P = Pils et F = Face)\n\n");
    printf("---------------------------------------------------------------------------\n");
    printf("|Tentative   |   1 |   2 |   3 |   4 |   5 |   6 |   7 |   8 |   9 |   10 |\n");
    printf("---------------------------------------------------------------------------\n");
    printf("|Tirage      |  %d |  %d |  %d |  %d |  %d |  %d |  %d |  %d |  %d |  %d  |\n", ordi);
    printf("|Proposition |  %d |  %d |  %d |  %d |  %d |  %d |  %d |  %d |  %d |  %d  |\n", joueur);
    printf("---------------------------------------------------------------------------\n\n");
 
 
 
}
 
int main()
{
    printf("\n\n\n\t\t\t BIENVENU DANS LE JEU DU PILE OU FACE !!\n\n");
 
    void affiche(int essai, char ordi[],char joueur[]);
 
 
}

Un gd merci à vouss

[fregolo52]

Il y a plein (trop) d'erreurs !!

Tu n'as pas de moyen de tester (compiler) ton code ?

[jeroman]

Salut

Il aurait fallu que tu tentes de compiler pour avoir un début de réponse. Mon compilateur m'a hurlé dessus.

C'est impossible de tout corriger, il y a beaucoup trop d'incohérences.

Quelques pistes, dans le désordre :

Fonction "nbsucces" :
_ Tu incrémentes "nbsucces" à chaque appel de la fonction. Or, cette variable est initialisée à 0 à chaque fois.
_ Tu veux effectuer un calcul ("pourcent"), mais le résultat est perdu à la fin de la fonction. De toute façon, le "return" zappe cette ligne.
_ Les résultats sont perdus à chaque fois, car les objets sont créés à chaque fois lors de l'appel à la fonction
_ Tu divises deux entiers. Le résultat ne sera pas celui attendu.
_ Même nom pour ta fonction et la variable. Pas top.

Les deux autres fonctions :
_ "srand(time(NULL));" doit être appelé une seule fois et au début du programme.
_ Quand on utilise "%d" avec "scanf", la fonction attend un pointeur sur "int".
_ Dans

Code :

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

if(joueur!=P || joueur!=F)

"P" et "F" ne sont pas initialisées.
_ Paramètres de mauvais type (char <-> char*) lorsque tu veux appeler une fonction.

D'une manière générale :
_ Il y a des variables non déclarées ou déclarées ailleurs.
_ Tu déclares des fonctions dans des fonctions. Je pense que tu confonds l'appel de fonction et la déclaration de fonction.
_ On ne sait pas qui a gagné ou perdu. Il faudrait plutôt (par ex) créer un tableau et stocker le résultat dedans (indice 0 -> joueur 1 , indice 1 -> joueur 2).
_ Certaines fonctions sont censées renvoyer qqchose, mais ne renvoient rien.
_ ...