Discussion :

[Mike35]

Bonjour à vous

J'ai un petit problème pour remplir un tableau de structure
Je vous explique:
J'ai un fichier texte Zodiaque.txt

Code :

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

1
2
3
4
5
BELIER 81 111
TAUREAU 112 142
GEMEAUX 143 173
CANCER 174 205
...

Je dois remplir ce fichier dans un tableau de structure

Code :

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

1
2
3
4
5
6
typedef struct
{
  char Nom[16];
  int debut;
  int fin;
}S_ZODIAQUE;

voila ma fonction:

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
void Remplir(S_ZODIAQUE zod)
{
  int iInc = 0;
  FILE* desc;
  char *debut, fin;
  des = fopen("Zodiaque.txt","r");
 
  while(iInc != 12)
  {
    fscanf("%s %s %s",zod[iInc].Nom,debut,fin);
    zod[iInc].debut = atoi(debut);
    zod[iInc].fin = atoi(fin);
    iInc++;
  }
}

et dans mon main():

Code :

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

1
2
3
4
5
6
7
int main(void)
{
  S_ZODIAQUE zodiaque[12];
  Remplir(zodiaque);
  printf("%s %d %d",zodiaque[0].Nom,zodiaque[0].debut,zodiaque[0].fin);
  return (0);
}

J'ai un segmentation fault avec le premier champ Nom
Hors avec les autres champs de la structure tout se passe bien

Est ce que quelqu'un peu m'aider
Merci d'avance

[Thierry Chappuis]

Le prototype de ta fonction Remplir() est faux. Si tu veux lui passer le tableau zodiaque, l'argument doit être de type pointeur sur S_ZODIAQUE (il est par ailleurs conseillé de passé la taille du tableau en deuxième argument).

En ce qui concerne l'appel à fscanf(), tu fait la chose suivante:

Code :

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

fscanf("%s %s %s",zod[iInc].Nom,debut,fin);

ou debut et fin sont des pointeurs qui ne pointent pas sur une adresse valide (ils n'ont pas été initialisés et aucune mémoire n'a été allouée). Lorsque tu appelles fscanf() avec debut et fin en argument, tu essaies d'accéder à une adresse invalide, d'où l'erreur de segmentation.

Pourquoi debut et fin sont-ils de type char *, alors que les champs de même nom de la structure S_ZODIAQUE sont de type int?

Thierry

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
6
7
void Remplir(S_ZODIAQUE zod)
{
  char *debut, fin;
 
<...>
 
   fscanf("%s %s %s",zod[iInc].Nom,debut,fin);

'debut' et 'fin' sont des pointeurs non initialisés. Tu passes des valeurs indéterminées à une fonction. Le comportement est indéterminé.

De plus, il manque un paramètre à fscanf() (FILE *)

Ton code est inutilement complexe. Utilise directement "%s %d %d" avec
zod[iInc].Nom, &zod[iInc].debut, &zod[iInc].fin et l'affaire est réglée...