Discussion :

[stephaneamadeus]

Bonjour à tous!

J'ai besoin de lire un fichier txt du style:

3
Vega
23.698
Rigel
1256.987
Beltegeuse
54.87

et d'en récupérer les données numériques et les chaines de caractères.
Pour ca j'ai écrit le code suivant:

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>
 
main()
{
   char *tab1[200];
   float tab2[200];
   int n,i;
   float luminosite_etoile;
   char nom_etoile[40];
 
   system("clear");
 
   FILE *donnees;
   donnees=fopen("donnees_exo.txt","r");
 
   /* lecture du fichier de donnees puis fermeture du fichier*/
   fscanf(donnees,"%d",&n);
 
   for(i=0;i<n;++i)
   {
      fscanf(donnees,"%s",nom_etoile);
      fscanf(donnees,"%f",&luminosite_etoile);
      tab1[i]=nom_etoile;
      tab2[i]=luminosite_etoile;
   }
   fclose(donnees);
 
   /* affichages*/
   for(i=0;i<n;++i)
   {
      printf("luminosite%d = %f\n",i+1,tab2[i]);
   }
   printf("\n\n");
 
   for(i=0;i<n;++i)
   {
      printf("nom de l'étoile %d = %s\n",i+1,tab1[i]);
   }
   printf("\n\n%s",tab1[0]);
}

Ca marche pour lire les données numériques et les placer dans le tableau tab2 mais pas pour les chaines de caractères. Il semble que seule la dernière soit lue ("Beltegeuse") et tous les éléments du tableau tab1 sont "Beltégeuse".

Quelq'un a une idée?

Merci.

[zul]

Normal..
Quand tu fais ton fscanf, tu modifie une zone mémoier situé à l'endroit indiqué par ton pointeur. Ensuite tu copie ton pointeur. Mais c'est toujours la même zone mémoire qui est pointée, et fscanf apres fscanf, tu réecris dessus... Au final, tu vois seulement la dernière chose écrite, cad la dernière ligne lue.

Au passage, il serait plus sain d'utiliser fgets que fscanf.

[GyZmoO]

Yo !

J'ajouterai :
1) main renvoie int
2) il faudrait que tu testes que l'ouverture de fichier avec fopen s'est bien passé (on ne sait jamais )

[stephaneamadeus]

Ok je crois avoir compris. Mais que faire alors pour résoudre ce problème. Que proposez vous ?
Pour le test du EOF fichier, je vais le rajouter.

[togzz]

Bonjour,

J'ai un peu le meme probleme que stephaneamadeus.
Je cherche aussi a extraire des données d'un fichier .txt afin de les mettre dans un tableau. Ces données sont de type int et sont decimales.

j'ai taper le code suivant :

fichierson = fopen("fichierson.txt","r");//ouverture du fichier en lecture

for(k=0;k<Ncanal;k++){
fscanf(fichierson,"%d",&kk);
donnee[k]=kk;
}

Le probleme est que lorsque j'affiche le tableau *donnee, il ne contient que des 00, alors que mon fichier txt contient bien des valeurs...


les valeurs dans le fichier txt sont ecris avec un \n entre chaque valeur.


Merci d'avance de vos conseils

[zul]

Quelquechose qui pourrait solutionner le problème ( avec le minimum de correction, ce n'est donc pas forcement la meilleure approche )

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
 
#include<stdio.h>
#include<stdlib.h>
 
int main(void)
{
char *tab1[200];
float tab2[200];
int n,i;
float luminosite_etoile;
char nom_etoile[40];
 
system("clear");
 
FILE *donnees;
 
donnees=fopen("donnees_exo.txt","r");
 
/* lecture du fichier de donnees puis fermeture du fichier*/
 
fscanf(donnees,"%d",&n);
/* Il faudrait peut-etre vérifier que n est inférieur a 200 */
 
for(i=0;i<n;++i)
{
fscanf(donnees,"%s",nom_etoile);
fscanf(donnees,"%f",&luminosite_etoile);
/* alloue de la mémoire pour tab[i] et recopie les données */
tab1[i]= malloc(strlen(nom_etoile)+1);
if ( tab1[i] == NULL ){
   /* gérer l'erreur */
} else {
   strcpy(tab1[i],nom_etoile);
}
tab2[i]=luminosite_etoile;}
 
fclose(donnees);
 
/* affichages*/
 
for(i=0;i<n;++i)
{printf("luminosite%d = %f\n",i+1,tab2[i]);}
 
printf("\n\n");
 
for(i=0;i<n;++i)
{printf("nom de l'étoile %d = %s\n",i+1,tab1[i]);}
 
printf("\n\n%s",tab1[0]);
}

togzzz, je ne pense pas que ton problème soit le même, les entiers sont copiés, on n'ecrase donc pas la zone mémoire sans avoir copié la valeur.

[togzz]

j'avais en fait un probleme de passage d'un argument int a un float....

[stephaneamadeus]

Merci zul, ton code marche nickel !!!
J'aimerais comprendre pourquoi il y a besoin de déclarer le tableau tab1 dynamiquement, puisque je lui réservais déjà de la place mémoire dans les déclarations de variables. Mais si je le fais pas, j'ai bien un segmentation fault.

[Trap D]

Deux remarques sur ce code :

Code :

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

1
2
3
4
5
6
7
8
 
  // il ne faut pas oublier de libérer la mémoire 
  for(i=0;i<n;++i)
    free(tab1[i]);
 
  // il faut retourner une valeur au système
  return 0;
}

Peut-être que ton compilateur accepte strdup, qui fait le malloc et la recopie de la chaine ?