Réinitialiser un tableau de caractères pour un tp de jeu du pendu

Version imprimable

Bonjour. Je serai brève.

Pour initialiser mon mot au départ :

char temp[26]={0};

Comment fais-je, quand la personne demande à rejouer, pour le réinitialiser avant de relancer la boucle ?

J'ai essayé plein de trucs... Moi pas trouvé le bon.

Si je ne le fais pas, et que le mot tiré au sort suivant est plus court que le mot précédent, j'ai des tas de bugs.

Sinon mon tp est fini et tourne bien... Faut juste que je modularise (presque tout est dans le main actuellement)...

Je vous mets le code pour vos critiques acerbes.

Code:

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "fonctions.c"
 
int main(){
    char s='s';
    char ligne[26]={0},mottire[26]={0},temp[26]={0},curlet=0,rejoue='O';
    int vies,i,flag=0,pif,total=0,score=0;
 
 
 
    do {
            s='s';
            vies=5;
            i=0;
            total++;
            pif=alea();
            FILE *fichier = fopen("liste.txt","r");;
            if (fichier == NULL){
               printf("Le fichier ne peut etre ouvert.");
               return 1;
            }
            while (fgets(ligne,26,fichier)!=NULL){
                i++;
                if(i==pif){
 
                    strcpy(mottire,ligne);
                }
 
            }
            fclose(fichier);
            i=0;
 
            for(i=0;i<(strlen(mottire)-1);i++){
                temp[i]='*';
            }
            temp[i]='\n';
 
            i=0;
            //tant que ni gagne ni perdu
            //boucle do while (faut bien rentrer d'abord)
            do{
                    printf("\n\nMOT A TROUVER : %s\n",temp);
                    printf("Il vous reste %d vie%c\n",vies,s);
                    printf("Proposez une lettre (pas d'accent) : ");
 
                    flag=0;
                    curlet=lirelettre();
 
                    for(i=0;i<strlen(temp);i++){
                        if (curlet==mottire[i]){
                            temp[i]=curlet;
                            flag=1;
                        }
                    }
                    if(flag==0){
                        vies--;
                    }
                    if(vies==1){
                        s=' ';
                    }
 
                }
            while((strcmp(temp,mottire)!=0)&&(vies>0));
            if(vies<=0){
                printf("\n\nDommage, vous avez perdu.\n");
                printf("Il fallait trouver %s\n\n",mottire);
                printf("\nVotre score : %d/%d\n",score,total);
                printf("\n\nVoulez-vous rejouer ? (o/n)");
                rejoue=lirelettre();
            }
            if(strcmp(temp,mottire)==0){
                score++;
                printf("\n\nBravo ! Le mot etait : %s",mottire);
                printf("\nVotre score : %d/%d\n",score,total);
                printf("Voulez-vous rejouer ? (o/n)");
                rejoue=lirelettre();
            }
            if(rejoue=='N'){
                printf("\nVotre score : %d/%d\n",score,total);
                printf("Au revoir et a bientot !!!\n\n");
            }
            else{
                //reinitialise le mot d'ou ma question...
            }
        }
 
    while (rejoue=='O');
    return 0;
}

15/11/2009, 11h18
Franck.H

:salut:

Code:

memset (temp, 0, sizeof (temp));
15/11/2009, 11h29
Dendrite

Salut Franck.

et merci ça semble tourner tout rond cette fois.

(Ce qui ne vous dispense pas de vos remarques à une novice en C).
15/11/2009, 11h44
Franck.H

Oui j'aurais une petite remarque mais pas moins importante, je n'ai pas lu le code cependant, une meilleure présentation/indentation serait sympa, ici le code n'est pas très très lisible et peut en rebuter plus d'un moi y compris ;) :aie:
15/11/2009, 11h46
Dendrite

Indentation ? L'est pas bonne mon indentation ?

Sinon, je me lance dans la modularisation de ce pas. Est-ce de cela que tu parles ?

Je mettrai mes commentaires dans les fonctions.

Oops, une erreur d'indentation corrigée :

Code:

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "fonctions.c"
 
int main(){
    char s='s';
    char ligne[26]={0},mottire[26]={0},temp[26]={0},curlet=0,rejoue='O';
    int vies,i,flag=0,pif,total=0,score=0;
 
 
 
    do {
            s='s';
            vies=5;
            i=0;
            total++;
            pif=alea();
            FILE *fichier = fopen("liste.txt","r");;
            if (fichier == NULL){
               printf("Le fichier ne peut etre ouvert.");
               return 1;
            }
            while (fgets(ligne,26,fichier)!=NULL){
                i++;
                if(i==pif){
 
                    strcpy(mottire,ligne);
                }
 
            }
            fclose(fichier);
            i=0;
 
            for(i=0;i<(strlen(mottire)-1);i++){
                temp[i]='*';
            }
            temp[i]='\n';
 
            i=0;
            //tant que ni gagne ni perdu
            //boucle do while (faut bien rentrer d'abord)
            do{
                printf("\n\nMOT A TROUVER : %s\n",temp);
                printf("Il vous reste %d vie%c\n",vies,s);
                printf("Proposez une lettre (pas d'accent) : ");
 
                flag=0;
                curlet=lirelettre();
 
                for(i=0;i<strlen(temp);i++){
                    if (curlet==mottire[i]){
                        temp[i]=curlet;
                        flag=1;
                    }
                }
                if(flag==0){
                    vies--;
                }
                if(vies==1){
                    s=' ';
                }
 
            }
            while((strcmp(temp,mottire)!=0)&&(vies>0));
 
            if(vies<=0){
                printf("\n\nDommage, vous avez perdu.\n");
                printf("Il fallait trouver %s\n\n",mottire);
                printf("\nVotre score : %d/%d\n",score,total);
                printf("\n\nVoulez-vous rejouer ? (o/n)");
                rejoue=lirelettre();
            }
            if(strcmp(temp,mottire)==0){
                score++;
                printf("\n\nBravo ! Le mot etait : %s",mottire);
                printf("\nVotre score : %d/%d\n",score,total);
                printf("Voulez-vous rejouer ? (o/n)");
                rejoue=lirelettre();
            }
            if(rejoue=='N'){
                printf("\nVotre score : %d/%d\n",score,total);
                printf("Au revoir et a bientot !!!\n\n");
            }
            else{
               memset (temp, 0, sizeof (temp));
               memset (mottire, 0, sizeof (mottire));
            }
        }
 
    while (rejoue=='O');
    return 0;
}

15/11/2009, 11h51
Franck.H
Citation:

Envoyé par Dendrite

Indentation ? L'est pas bonne mon indentation ?

Sinon, je me lance dans la modularisation de ce pas. Est-ce de cela que tu parles ?

Je mettrai mes commentaires dans les fonctions.

Non, mais des espaces entre les différentes déclaration de variables, des lignes vides aux bons endroits pour entrecouper un peu le code, des indentations moins longues (3 à 4 caractères suffisent).

A noter également que les bons prototype de main sont au choix:
Code:

1 2 3 int main (void) /* ou */ int main (int argc, char ** argv)
Voilà ;)
15/11/2009, 11h54
Dendrite

Merci chef.

Faut donc que j'évite d'utiliser le tab pour indenter ? :cry:

Ou que je change les paramètres de mon tab sur mon clavier Mac (je sais pas si c'est possible).
15/11/2009, 12h49
Médinoc

Citation:

Envoyé par Dendrite

Merci chef.

Faut donc que j'évite d'utiliser le tab pour indenter ? :cry:

Ou que je change les paramètres de mon tab sur mon clavier Mac (je sais pas si c'est possible).

Vue la gueule de ton code, tu n'utilises déjà pas le tab pour indenter, ou bien ton éditeur est mal réglé, parce qu'ici, tout ce qu'on voit, c'est des espaces.
Je te conseille d'utiliser des vraies tabs pour indenter. Ainsi, même si sur le forum ça aura l'air un peu grand (vu que les navigateurs hard-codent les tabs à huit caractères), il suffira à n'importe quel membre d'un copier-coller vers son éditeur favori pour avoir à nouveau des tabs à son goût...
15/11/2009, 12h54
Dendrite

Citation:

Faut donc que j'évite d'utiliser le tab pour indenter ?

Citation:

Vue la gueule de ton code, tu n'utilises déjà pas le tab pour indenter,

Citation:

Je te conseille d'utiliser des vraies tabs pour indenter.

:roi:
15/11/2009, 20h07
orfix

Désolé je ne vais plus loin que :

Code:

#include "fonctions.c"

il ne faut inclure que des fichier d'entête (header).
16/11/2009, 15h10
Dendrite

Citation:

Envoyé par ssmario2

Désolé je ne vais plus loin que :

Code:

#include "fonctions.c"

il ne faut inclure que des fichier d'entête (header).

Donc mon main n'inclut que des fichiers.h...

Mais les autres fichiers.c du coup ? Tu peux expliquer ?

ex :

dans main, je mets un include fonctions.h
et dans fonctions.c ? Rien ? Ou à nouveau include fonctions.h ?
17/11/2009, 19h48
orfix

Citation:

Envoyé par Dendrite

Donc mon main n'inclut que des fichiers.h...

Pas que ton main tout les fichier *.c

Je suppose que le fichier fonctions.h contient les prototypes de fonctions définit dans fonctions.c partant de là tout fichier utilisant ces fonctions (y compris fonctions.c) devras inclure fonctions.h