Discussion :

[cmoibal]

Bonjour,

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

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

[Sve@r]

Bonjour,

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

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

[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

[Sve@r]

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

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

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

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

[diogene]

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.

[hegros]

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