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

**ladygtk** · 09/03/2007, 11h41

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 :

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

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

**souviron34** · 09/03/2007, 11h47

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

Code :

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

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

**Emmanuel Delahaye** · 09/03/2007, 11h53

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

**Médinoc** · 09/03/2007, 11h53

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"

**souviron34** · 09/03/2007, 11h55

Envoyé par Médinoc

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

lol

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

**Médinoc** · 09/03/2007, 12h01

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

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

**souviron34** · 09/03/2007, 12h12

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

Code :

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

1
2
 
ligne[strlen(ligne)-1] = '\0' ;

**Médinoc** · 09/03/2007, 12h15

Non, car si tu as tapé plus que le fgets() accepte, il n'y aura pas de \n dans le buffer.

**souviron34** · 09/03/2007, 12h38

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

**Thierry Chappuis** · 09/03/2007, 12h40

Envoyé par souviron34

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

Code :

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

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.

Thierry

**Thierry Chappuis** · 09/03/2007, 12h42

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

**ladygtk** · 09/03/2007, 12h59

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.

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

**souviron34** · 09/03/2007, 13h00

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 :

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

1
2
3
4
5
6
7
8
 
char Ligne[5000] ;
 
while ( fgets ( Ligne , 5000, fich) != NULL )
{
   Ligne[strlen(Ligne)-1] = '\0' ;
    .....
}

**Trap D** · 09/03/2007, 13h26

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

**Emmanuel Delahaye** · 09/03/2007, 14h25

Envoyé par souviron34

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

Code :

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

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 :

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

**Emmanuel Delahaye** · 09/03/2007, 14h27

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.

**Emmanuel Delahaye** · 09/03/2007, 14h30

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 ?

**souviron34** · 09/03/2007, 14h51

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

**jowo** · 09/03/2007, 16h26

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.