nombre de ligne d'un fichier

Version imprimable

Bonjour,

voici le code me qui me permet de calculer le nombre des lignes dans un fichier txt :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 int getNbLines(const char *file,long pos) { int nbLines; char line[SIZE_LINE_MAX]; /* line instance in file */ FILE *fp; assert (file != NULL); nbLines = 0; line[0] = '\0'; if ((fp = fopen(file, "r")) != NULL) { if(fseek(fp, pos, SEEK_SET)<0){ return FAILURE; } while (fgets(line, SIZE_LINE_MAX, fp) != NULL){ nbLines++; } fclose(fp); } return nbLines; }
Bon, ma question c'est : Est ce que c'est la meilleur solution, ou il faut passer caractere par carcatere, sachant que je suis sur un environement embarque, d'ou la question sur la consomation de la memoire ?

MERCI

Citation:
Envoyé par cmoibal
Bonjour,

voici le code me qui me permet de calculer le nombre des lignes dans un fichier txt :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 int getNbLines(const char *file,long pos) { int nbLines; char line[SIZE_LINE_MAX]; /* line instance in file */ FILE *fp; assert (file != NULL); nbLines = 0; line[0] = '\0'; if ((fp = fopen(file, "r")) != NULL) { if(fseek(fp, pos, SEEK_SET)<0){ return FAILURE; } while (fgets(line, SIZE_LINE_MAX, fp) != NULL){ nbLines++; } fclose(fp); } return nbLines; }
Bon, ma question c'est : Est ce que c'est la meilleur solution, ou il faut passer caractere par carcatere, sachant que je suis sur un environement embarque, d'ou la question sur la consomation de la memoire ?

MERCI
Pourquoi le fseek() ? Tu ne comptes pas du début ???

Maintenant l'optimisation. Si tu veux privilégier la rapidité je pense que tu pourrais envisager aussi le fread(grosse_zone) + compter le nb de '\n' dans la grosse_zone. Voire même regarder du coté des fonctions bas niveau open()+read()+close()
Et si tu veux privilégier la mémoire, alors if (fgetc(fp) == '\n') nbLines++;

20/02/2008, 17h11
dot-_-net

On utilise jamais un fseek dans un fichier texte c’est ce qu’on m’a apprit + il fallait lire avec fgets et incrémenter le compteur c’est tout

Citation:

Envoyé par cmoibal

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
int getNbLines(const char *file,long pos)
{
 int nbLines;
 char line[SIZE_LINE_MAX]; /* line instance in file  */
 FILE *fp;
 assert (file != NULL);
 nbLines = 0;
 line[0] = '\0';
 if ((fp = fopen(file, "r")) != NULL)
 {
  if(fseek(fp, pos, SEEK_SET)<0){
      return FAILURE;
  }
  while (fgets(line, SIZE_LINE_MAX, fp) != NULL){
   nbLines++;
  }
  fclose(fp);
 }
 return nbLines;
}

si fssek échoue tu ne fermes pas le fichier ?
ton assert il sert à quoi ?

21/02/2008, 00h14
Sve@r

Citation:

Envoyé par dot-_-net

On utilise jamais un fseek dans un fichier texte c’est ce qu’on m’a apprit

Soit on t'a mal appris, soit t'as pas compris. Déjà, dans le monde Unix, le terme "fichier texte" ne veut rien dire car il n'y a que des fichiers. Certains fichiers contiennent des caractères affichables rangés de façon compréhensible par un oeuil humain (ce que tu nommes "fichier texte"), certains autres (une grande majorité) contiennent des caractères divers rangés de façon compréhensible par différents logiciels comme gimp, openoffice. Et d'autres plus spéciaux contiennent des caractères rangés de façon compréhensible par le système lui-même (répertoires, exécutables).
Et donc la fonction fseek() qui demande un déplacement en terme de caractères peut s'appliquer sur n'importe lesquels des fichiers cités précédemment si ces fichiers ont été associés à un pointeur sur FILE par le biais de fopen().

Citation:

Envoyé par dot-_-net

+ il fallait lire avec fgets et incrémenter le compteur c’est tout

Oui, c'est effectivement ce que fait le premier code. Mais la question portait sur l'optimisation à y apporter...
21/02/2008, 09h05
Trap D

La lecture avec fgets n'est pas sûre à 100% puisqu'elle implique la lecture d'un nombre maximum de caractères (sizeof buf), donc si on ne teste pas la présence de '\n' on peut se planter.
La meilleure méthode est d'utiliser getchar et de compter les '\n', ne pas oublier de vérifier la fin de fichier (il se peut que la dernière ligne ne comporte pas de '\n').
21/02/2008, 10h16
Médinoc

Il me semble que fseek() est garanti pouvoir être utilisé sans risque avec des valeurs retournées par ftell().

Et pour retourner au début d'un fichier, rewind() est plus préconisé que fseek() (bien que si ça se trouve, son implémentation utilise fseek()).
21/02/2008, 11h47
diogene

Citation:

Envoyé par Médinoc

Il me semble que fseek() est garanti pouvoir être utilisé sans risque avec des valeurs retournées par ftell().

Et pour retourner au début d'un fichier, rewind() est plus préconisé que fseek() (bien que si ça se trouve, son implémentation utilise fseek()).

Pour les fichiers texte, fseek peut prendre comme argument de position la valeur 0 ou une valeur retournée par ftell. Le mode doit être SEEK_SET.
25/02/2008, 17h16
cmoibal

Merci tous le monde :D !!!