Discussion :

[AI_user02000]

Bonsoir, je travaille actuellemnt sur le jeu du pendu et j'avoue que je n'y comprends plus rien. A chaque fois que j'excécute mon programme, apres avoir entrer la lettre qu'on m'a demandé dans la fonction lirecaractere(), le programme affiche segmention fault core dumped. Je vous montre la partie main du programme car je me doute que le probleme se trouve là-bas.

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
#include "pendu.h"
 
 
int main()
{
    printf("*****Bienvenue dans le jeu du pendu***** \n   Saurez-vous trouver le mot cache?\n");
 
 
 
    char malettre; //correspond à la lettre que l'utilisateur va entrer
    int i;
    int tab_car[TAILLE_MAX]; //correspond au tableau contennant les differentes lettres du mot caché
 
 
 
    FILE* fichier = NULL;
    fichier=fopen("/home/fagnan/Documents/Pendu/Pendu.txt", "r");
 
    if(fichier!=NULL)
    {
    malettre=lirecaractere();
 
    do
    {
        for(i=0;i<TAILLE_MAX;i++)
        {
                tab_car[i]= fgetc(fichier);
        }
 
 
    }while(fgetc(fichier)!=EOF);
 
 
         for(i=0;i<6;i++)
            {
                comparercaractere(malettre,'tab_car[i]');
            }
 
        fclose(fichier);
 
    }
 
    else{printf("Impossible de trouver le fichier");
 
    }
 
 
 
}

Merci.

[LittleWhite]

Bonjour,

Dans ce genre de cas, pour mieux comprendre l'erreur, il faut utiliser un débogueur. Voici un tutoriel expliquant comment le prendre en main, car c'est le deuxième ami du programmer (le premier étant le compilateur).

Je vous montre la partie main du programme car je me doute que le probleme se trouve là-bas.

Dommage, car je trouve que vous n'avez pas assez d'arguments pour une telle supposition.

Cette ligne est fausse :

Code :

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

comparercaractere(malettre,'tab_car[i]');

Je suppose que la version correcte devrait être :

Code :

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

comparercaractere(malettre,tab_car[i]);

afin de comparer avec le contenu du tableau et non avec une chaîne contenant le texte "tab_car[i]".

[pjlan]

Bonjour,

je rajouterai en plus de LittleWhite que ta boucle même si elle ne pose peut être pas de problème dans ton cas est un peu tordue dans le sens où si ton fichier est vide tu ne t'en rendra compte qu'a la fin et il y aura 'EOF' dans tout les caractères de ton tableau.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
    do
    {
        for(i=0;i<TAILLE_MAX;i++)
        {
                tab_car[i]= fgetc(fichier);
        }
 
 
    }while(fgetc(fichier)!=EOF);

Tu pourrais tout mettre dans une seule boucle for ou while.

avec while:

Code :

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

1
2
3
4
5
6
7
8
 
int car;
i=0;
while( (car = fgetc(fichier) )!= EOF && i < TAILLE_MAX )
{
  tab_car[i] = car;
  i++;
}

avec for:

Code :

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

1
2
3
4
5
6
 
int car;
for(i=0; (car = fgetc(fichier)) != EOF && i < TAILLE_MAX;i++ )
{
  tab_car[i] = car;
}

[Sve@r]

Bonjour

je rajouterai en plus de LittleWhite que ta boucle même si elle ne pose peut être pas de problème dans ton cas est un peu tordue dans le sens où si ton fichier est vide tu ne t'en rendra compte qu'a la fin et il y aura 'EOF' dans tout les caractères de ton tableau.

Tu n'as pas bien lu. Sa lecture foirera même si le fichier n'est pas vide à cause de la boucle interne qui lit quoi qu'il arrive TAILLE_MAX caractères.
Ainsi seule la dernière itération de la boucle while() sera conservée dans tab_car pour le meilleur et pour le pire.

Sinon ta solution est effectivement bien plus correcte