Mettre les lignes d'un fichier dans un tableau

Version imprimable

je cherche à lire sur stdin toutes les lignes d'une fichier et de les stocker dans un tableaux.
J'ai fait ça mais ça a pas l'air de marcher très bien!

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define NMAXLINE 100
#define NMAXCHAR 100
 
int main()
{
	char row[NMAXCHAR];
	char* lines[NMAXLINE];
	char* test;
	int nbLine=0,i;
	test=fgets(row,NMAXLINE,stdin);
	/*Stockage des lignes*/
        while(test!=NULL)
	{
		lines[nbLine]=row;
		test=fgets(row,NMAXLINE,stdin);
		nbLine++;
	}
        /*Affichage des lignes stockées*/
	for(i=0;i<nbLine;i++)
	{
		fprintf(stdout,lines[i]);
		printf("\n");
	}
	return EXIT_SUCCESS;
}

Q'en pensez vous ?
Rod

Code:

1
2
3
4
5
   for(i=0;i<nbLine;i++)
   {
      fprintf(stdout,lines[i]);
      printf("\n");
   }

est à revoir.

Re: Mettre les lignes d'un fichier dans un tableau

Citation:
Envoyé par rod59
je cherche à lire sur stdin toutes les lignes d'une fichier et de les stocker dans un tableaux.
J'ai fait ça mais ça a pas l'air de marcher très bien!
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 #include <stdio.h> #include <stdlib.h> #include <string.h> #define NMAXLINE 100 #define NMAXCHAR 100 int main() { char row[NMAXCHAR]; char* lines[NMAXLINE]; char* test; int nbLine=0,i; test=fgets(row,NMAXLINE,stdin); /*Stockage des lignes*/ while(test!=NULL) { lines[nbLine]=row; test=fgets(row,NMAXLINE,stdin); nbLine++; } /*Affichage des lignes stockées*/ for(i=0;i<nbLine;i++) { fprintf(stdout,lines[i]); printf("\n"); } return EXIT_SUCCESS; }
Q'en pensez vous ?
Rod
Il n'y a pas création spontanée de memoire ... Si chaque ligne doit être stockée séparément, il faut de l'espace pour chaque ligne. Ici, tu te contentes de remplir un tableau de pointeurs avec l'adresse du même tableau. D'autre part, avec *printf(), il faut utiliser une formateur "%s" lorsque la variable chaine n'est pas explicite. On ne sais pas ce qu'il peut y avoir dedans...

Code:

fprintf(stdout,"%s"lines[i]);

et forcer la sortie si on est pas sûr qu'il y a un '\n' dans la chaine...

Code:

fflush (stdout);

Citation:
Envoyé par troumad
Code:

1 2 3 4 5 for(i=0;i<nbLine;i++) { fprintf(stdout,lines[i]); printf("\n"); }
est à revoir.
Pourquoi ?

19/11/2005, 16h01
troumad

Je ne comprends pas l'utilité de mélanger le fprintf et le printf. Peut-être pour brouiller la lecture. C'est d'ailleur ce que ça m'a fait je n'avais pas remarqué que sa sortie était stdout, j'avais cru que c'était un FILE * quelconque.
19/11/2005, 16h16
rod59
Citation:
Envoyé par Emmanuel Delahaye

Citation:

Envoyé par troumad

Code:

1 2 3 4 5 for(i=0;i<nbLine;i++) { fprintf(stdout,lines[i]); printf("\n"); }

est à revoir.

Pourquoi ?
suis d'accord avec Emmanuel! Pourquoi ca ne va pas?
Mon algo c'est de lire ligne par ligne avec fgets, au fur et à mesure je remplis mon tableau lines et ensuite je le reparcours pour voir si il a bien été rempli.
La 2ème boucle sert juste à vérifier que la 1ère à bien fait son travail.rien de plus.

Sinon :

Citation:

Envoyé par Emmanuel Delahaye

Il n'y a pas création spontanée de memoire ... Si chaque ligne doit être stockée séparément, il faut de l'espace pour chaque ligne. Ici, tu te contentes de remplir un tableau de pointeurs avec l'adresse du même tableau.

Ce que tu ve dire c'est que à chaque tour de boucle je stocke le pointeur d'une chaine que je n'est pas stockée en mémoire?
Faut-il faire un malloc à chaque tour boucle?
19/11/2005, 16h22
Emmanuel Delahaye

Citation:

Envoyé par rod59

Faut-il faire un malloc à chaque tour boucle?

Oui. fgets() se contente de retourner l'adresse passée en paramètre ou NULL en cas de fin de lecture.

Donc ca ca pourrait marcher ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
int main() 
{ 
   char* row; 
   char* lines[NMAXLINE]; 
   char* test; 
   int nbLine=0,i;
   row=(char*)malloc(NMAXCHAR*sizeof(char));
   test=fgets(row,NMAXLINE,stdin);  
   while(test!=NULL) 
   { 
      lines[nbLine]=row;
      row=(char*)malloc(NMAXCHAR*sizeof(char)); 
      test=fgets(row,NMAXLINE,stdin); 
      nbLine++; 
   }

19/11/2005, 16h42
Emmanuel Delahaye

Citation:

Envoyé par rod59

Donc ca ca pourrait marcher ?

sizeof (char) valant 1 par définition, et le cast étant inutile :

Code:

row = malloc(NMAXCHAR);

par contre, attention, malloc() peut échouer...

Ensuite, tu as supprimé le tableau de pointeurs. Pourquoi ? Il faut bien mémoriser l'adresse des blocs alloués, non ?
19/11/2005, 17h01
rod59

bah mon tableau de pointeur c'est lines!
Je l'utilise tjs ?!
Sinon pourquoi malloc peut échouer?
19/11/2005, 17h20
Emmanuel Delahaye

Citation:

Envoyé par rod59

bah mon tableau de pointeur c'est lines!
Je l'utilise tjs ?!

Ok, mal lu. Désolé. Il y a quand même un problème, c'est que tu ne stockes pas le dernier row alloué. C'est dû à ton algo plutôt tordu. Tu n'as probablement besoin de fait 2 fgets(). Un seul suffit si la boucle est mieux écrite...

Citation:

Sinon pourquoi malloc peut échouer?

Par manque de ressource. On ne peut pas allouer de la mémoire indéfiniment...
19/11/2005, 17h33
rod59

Citation:

Envoyé par Emmanuel Delahaye

Il y a quand même un problème, c'est que tu ne stockes pas le dernier row alloué.

Je pense que si!

Emmanuel : On avait déjà eu cette discussion sur un autre topic!
Quand je serai sur le dernière ligne fgets va me le retourner et le prochain appel sera à null.
Donc je ne perd aucun information,je pense,mais je peut me tromper
19/11/2005, 17h38
Emmanuel Delahaye

Citation:

Envoyé par rod59

Citation:

Envoyé par Emmanuel Delahaye

Il y a quand même un problème, c'est que tu ne stockes pas le dernier row alloué.

Je pense que si!

Vu que tu sauvegardes avant le malloc(), je ne vois pas comment... Ou alors tu ajoutes une sauvegarde après la boucle... C'est bien compliqué...

Emmanuel : On avait déjà eu cette discussion sur un autre topic!
Quand je serai sur le dernière ligne fgets va me le retourner et le prochain appel sera à null.
Donc je ne perd aucun information,je pense,mais je peut me tromper

Fait les tests avec un traqueur de mémoire... (efence, valgrind, purify etc. ou mon sysalloc http://emmanuel-delahaye.developpez.com/clib.htm)
19/11/2005, 17h49
rod59

+1
Désolé! Voilà c corrigé

C'est quand mêm un peu du n'importe quoi :

Code:

1
2
3
4
5
6
7
  while(test!=NULL)
   {
      lines[nbLine]=row;
      row=(char*)malloc(NMAXCHAR*sizeof(char));
      test=fgets(row,NMAXLINE,stdin);
      nbLine++;
   }

Tu alloues NMAXCHAR et tu lis NMAXLINE.
Tu devrait peut-être faire ainsi :

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
int main(void)
{
   char* row;
   char* lines[NMAXLINE];
   char* test;
   int nbLine=0,i;
   row=malloc(NMAXCHAR);
    if (row == NULL)
   {
      fprintf(stderr, "Pb alloc memoire\n");
      return EXIT_FAILURE; 
   }
   // ne sert à rien
   // test=fgets(row,NMAXLINE,stdin); 
   while(fgets(row,NMAXCHAR,stdin); !=NULL)
   {
      // On peut faire lines[nbLine++]=row;
      lines[nbLine]=row;
      row=malloc(NMAXCHAR);
      if (row == NULL)
      {
        fprintf(stderr, "Pb alloc memoire\n");
        break;; 
      }
       // Si on n'a pas fait lines[nbLine++]=row;
      // devrait être remonté avant le malloc à cause de la sortie éventuelle
      // de la boucle
      nbLine++;
   } 
   // il ne faudra pas oublier de libérer le dernier row alloué et le tableau Lines
   ....................