Discussion :

[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 :

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
 
#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

[troumad]

Code :

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

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

est à revoir.

[Emmanuel Delahaye]

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 :

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
 
#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 :

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

   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 :

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

   fflush (stdout);

[Emmanuel Delahaye]

Code :

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

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

est à revoir.

Pourquoi ?

[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.

[rod59]

Code :

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

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 :

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?

[Emmanuel Delahaye]

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.

[rod59]

Donc ca ca pourrait marcher ?

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
 
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++; 
   }

[Emmanuel Delahaye]

Donc ca ca pourrait marcher ?

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

Code :

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

      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 ?

[rod59]

bah mon tableau de pointeur c'est lines!
Je l'utilise tjs ?!
Sinon pourquoi malloc peut échouer?

[Emmanuel Delahaye]

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...

Sinon pourquoi malloc peut échouer?

Par manque de ressource. On ne peut pas allouer de la mémoire indéfiniment...

[rod59]

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

[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)

[rod59]

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

[Trap D]

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

Code :

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

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 :

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
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
   ....................