qu'est ce qu'une chaîne vide?

Version imprimable

Voir 40 message(s) de cette discussion en une page

Bonjour à tous,
j'ai une portion de code qui vise à afficher à l'écran, toutes les lignes d'un fichier qui répondent à deux conditions:

la chaîne lue ne contient pas à la chaîne appelée chaine

la chaîne lue n'est pas la chaîne vide

Ce que je traduis en test:
Code:

1 2 3 4 5 if (fgets(ligne_a_lire,TAILLE_BUFF,pfichier)!=NULL) {// une ligne existe if ((strstr(ligne_a_lire,chaine)==NULL)||(strcmp(ligne_a_lire,"")!=0) { printf("\nligne lue: %s \n",ligne_a_lire); }//fin du if
Vu que ça ne donnait pas les bons résultats j'ai séparé mes deux conditions en deux tests.
Le premier test strstr se passe bien, mais le test de comparaison avec la chaîne vide reste infructueux.
"" est-il une mauvaise formulation d'une chaîne vide lue dans un fichier?
P.S: même résultat avec " \0".
Merci bien

09/03/2007, 10h47
souviron34
D'abord il te faut un ET et non pas un OU
Ensuite comparer une chaine à vide est absurde..
Code:

1 2 3 if ((strstr(ligne_a_lire,chaine)==NULL) && (strlen(ligne_a_lire) > 0))
09/03/2007, 10h53
Emmanuel Delahaye
Citation:
Envoyé par ladygtk

j'ai une portion de code qui vise à afficher à l'écran, toutes les lignes d'un fichierqui répondent à deux conditions:

la chaîne lue ne contient pas à la chaîne appelée chaine

la chaîne lue n'est pas la chaîne vide
Mauvaise terminologie menant à du code faux.

Un fichier texte est organisé en lignes, c'est à dire en séquences de caractères terminées par un '\n'.

Rappel : il n'y a pas de 0 dans un fichier texte.

Reformulation :
- la ligne lue ne contient pas à la chaine appelée chaine
- la ligne lue n'est pas une ligne vide
Ca devrait être beaucoup plus clair comme ça...

La bonne rédaction des spécification, c'est essentiel. Evidemment, ce n'est jamais enseigné...

Remarque concernant l'implémentation. Je rappelle que fgets() prend le '\n' (quand il peut...)
09/03/2007, 10h53
Médinoc

:faq:
fgets() enregistre aussi le \n tapé.
Il faut purger la ligne lue avant de faire les comparaisons.

http://emmanuel-delahaye.developpez....s.htm#fichiers
sous-sous-titre "gestion des fins de ligne"
09/03/2007, 10h55
souviron34

Citation:

Envoyé par Médinoc

:faq:
fgets() enregistre aussi le \n tapé.
Il faut purger la ligne lue avant de faire les comparaisons.

lol :D oui mais là tu rentres dans des subtilités qui visiblement sont pas encore de mise....

Bon, alors un petit copier-coller-mutiler du code en question pour faire simple ?

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
32
33
34
35
36
#include <stdio.h>
#include <string.h>
 
int PurgerLigneEtStdin(char *ligne)
{
   /* chercher le '\n' */
   char *p = strchr(ligne, '\n');
 
   if (p != NULL)
   {
      /* si on l'a trouve, on l'elimine. */
      *p = '\0';
   }
   else
   {
      /* Ici, on a choisi d'ignorer les autres caracteres. */
 
      int c;
      while ((c = fgetc(stdin)) != '\n' && c != EOF)
      {
      }
   }
}
 
/* Et ça s'utilise ainsi: */
int main(void)
{
   char buf[40];
 
   printf("Entrer une ligne (%d caracteres max) :\n", (int)(sizeof(buf)-1));
   fgets(buf, sizeof(buf), stdin);
   PurgerLigneEtStdin(buf);
 
   printf("Vous avez saisi \"%s\".\n", buf);
   return 0;
}

09/03/2007, 11h12
souviron34
il y a encore plus simple, pour le fgets ... ;)
Code:

1 2 ligne[strlen(ligne)-1] = '\0' ;
09/03/2007, 11h15
Médinoc

Non, car si tu as tapé plus que le fgets() accepte, il n'y aura pas de \n dans le buffer.
09/03/2007, 11h38
souviron34

je parlais pas d'une saisie mais d'une lecture dans un fichier, comme le PO mentionnait....
09/03/2007, 11h40
Thierry Chappuis
Citation:
Envoyé par souviron34

il y a encore plus simple, pour le fgets ... ;)

Code:

1 2 ligne[strlen(ligne)-1] = '\0' ;
Non, il faut tester au préalable que la ligne n'a pas été tronquée par fgets().

EDIT: OUPS! Pardon, Médinoc, j'avais pas lu ta réponse. :oops:

Thierry
09/03/2007, 11h42
Thierry Chappuis

Citation:

Envoyé par souviron34

je parlais pas d'une saisie mais d'une lecture dans un fichier, comme le PO mentionnait....

Et la différence ici, elle est où? Tu peux toujours avec une ligne qui dépasse la taille de ton tampon, à moins de déterminer au préalable la taille maximale des lignes de ton fichier.

Thierry
09/03/2007, 11h59
ladygtk

merci à tous

En effet moi la taille du paramètre 1 de fgets est de taille fixe, alors c'est pour ça que j'utilise la solution de Médinoc pour éliminer le retour à la ligne.
Dans ma recherche, en effet, il m'affichait quand même les lignes vides. Pour me rendre compte de ce qui était lu en tant que ligne vide, javais mis des cotes dans le printf autour de la ligne lue, et la dernière cote passait à ligne. Grâce à vous maintenant j'ai compris pourquoi alors grand merci.

Citation:

D'abord il te faut un ET et non pas un OU
Ensuite comparer une chaine à vide est absurde..

Code :

if ((strstr(ligne_a_lire,chaine)==NULL)
&& (strlen(ligne_a_lire) > 0))

OOOoops, c'est vrai c un et logique qu'il fallait. Merci.
Sinon, je fais une comparaison avec la ligne vide car ma ligne ayant une taille fixe je ne pouvais pas passer par des considérations de taille de chaîne.;)

Voilà ça marche impec.

Sinon rien à voir, quelles instructions permettent de passer à la ligne dans un fichier.Parce que j'ai une idée un peu grottesque du style: ligne avec mémoire allouée dynamiquement, je récupère la taille et je fais un fssek pour me déplacer au premier carctère de la prochaine ligne. Mais ça me paraît...:?
09/03/2007, 12h00
souviron34
Citation:

Envoyé par mujigka

Et la différence ici, elle est où? Tu peux toujours avec une ligne qui dépasse la taille de ton tampon, à moins de déterminer au préalable la taille maximale des lignes de ton fichier.

Thierry

En général, quand tu lis un fichier, tu connait en gros la longueur maximale , et même si tu sur-dimensionnes , pour 99,9999 % des cas c'est bon.
Code:

1 2 3 4 5 6 7 8 char Ligne[5000] ; while ( fgets ( Ligne , 5000, fich) != NULL ) { Ligne[strlen(Ligne)-1] = '\0' ; ..... }
09/03/2007, 12h26
Trap D

Tout est dans le "en général" ...

Citation:
Envoyé par souviron34
il y a encore plus simple, pour le fgets ... ;)
Code:

1 2 ligne[strlen(ligne)-1] = '\0' ;
Ben non. C'est pas parce que c'est simple que c'est correct.

Si il n'y a pas de '\n', tu supprimes un caractère utile. De plus, tu ne purges rien et le prochaine saisie est non blocante. Le syndrome du scanf() du Diable est revenu...

Le code publié par Medinoc n'est pas le fruit du hasard. C'est au contraire la synthèse d'années de reflexion et de remarques sur le sujet... Et ce n'est pas moi qui l'ai inventé, il provient plus ou moins de [f.]c.l.c.

Personnellement, je prends le problème à la source et je réécris la fonction de saisie de ligne à base de fgetc(). C'est ce que j'enseigne à mon élève actuel (15 ans, débutant), et il s'en sort très bien.
Pour le moment, il m'a sorti ça :
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 #include <stdio.h> #include <stdlib.h> int main(void) { int x = 0, i = 0; char chaine[20]; chaine[0] = '\0'; do { x = fgetc(stdin); if (i < 19 && x != '\n') { chaine[i] = x; i++; } } while (x != '\n'); chaine[i] = '\0'; printf("Voici la chaine:\n"); printf("'%s'\n", chaine); return 0; }
Il y a une ligne inutile, mais c'est pas grave.

Prochaine étape, la mise en fonction. J'attends sa réponse...

09/03/2007, 13h27
Emmanuel Delahaye

Citation:

Envoyé par souviron34

je parlais pas d'une saisie mais d'une lecture dans un fichier, comme le PO mentionnait....

Aucune différence, donc même problème.
09/03/2007, 13h30
Emmanuel Delahaye

Citation:

Envoyé par souviron34

En général, quand tu lis un fichier, tu connait en gros la longueur maximale<...>

Yo ! Tu as déjà lu des fichiers HTML avec des lignes de plusieurs dizaines de milliers de char ? Il y aura toujours un risque de débordement.

Pourquoi ne pas écrire du code généraliste ?
09/03/2007, 13h51
souviron34

Citation:

Envoyé par Emmanuel Delahaye

Yo ! Tu as déjà lu des fichiers HTML avec des lignes de plusieurs dizaines de milliers de char ? Il y aura toujours un risque de débordement.

Pourquoi ne pas écrire du code généraliste ?

Je veux bien, c'est sûr on peut ps être contre la vertu .. Quoique ;) ..
mais je parlais de la lecture d'un fichier normal, en général.
Et enfin pour les HTML comme ça, oui j'en ai vu, et je pense que c'est une connerie...
09/03/2007, 15h26
jowo

Citation:

Envoyé par souviron34

Et enfin pour les HTML comme ça, oui j'en ai vu, et je pense que c'est une connerie...

Pourquoi une connerie, si le code HTML est généré par un programme.

Je ne vois aucune raison d'introduire des retours de chariot dans le code HTML généré. Le code peut être sur une seule ligne même si sa longueur est 20'000 caractères.
09/03/2007, 15h29
souviron34

Citation:

Envoyé par jowo

Pourquoi une connerie, si le code HTML est généré par un programme.
.

eh eh eh :mrgreen: et qu'il l'a fait le programme ??? :P

Je dis que faire une ligne de 50000 caractères de TEXTE, c'est pas de la programmation terrible, c'est tout..

Voir 40 message(s) de cette discussion en une page