Discussion :

[sahid]

Bonjour,

j'essai de lire une chaine de caractere à partir d'un fichier avec la fonction fgets,
jusque là, tout marche bien.

le probleme est que quand j'affiche cette chaine de caractere il y a un retour chario à la fin, et ca me gene pour le formatage de celle-ci

exemple

Code :

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

1
2
 
fprintf (stdout, "%s grammes.", maChaine);

merci !

[Gruik]

Bein oui, fgets met le \n dans le buffer
T'as plus qu'a l'enlever à la main.

Code :

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

1
2
3
4
 
assert(strlen(buff) > 0 && buff[strlen(buff) - 1] == '\n');
 
buff[strlen(buff) - 1] = '\0';

Edit : l'instruction ci dessus n'est correcte que dans le cas où fgets a bel et bien mis un \n dans le buffer (c'est pas le cas pour la derniere ligne lue)

[sahid]

oh, merci

j'etais en train de me casser la tete, a faire ce que tu as fais en une ligne ... lol

[Emmanuel Delahaye]

Edit : l'instruction ci dessus n'est correcte que dans le cas où fgets a bel et bien mis un \n dans le buffer (c'est pas le cas pour la derniere ligne lue)

Ou si buff est trop petit pour la ligne lue. Cette méthode simpliste est incorrecte. Ne pas l'utiliser.

[sahid]

Code :

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

1
2
3
4
5
6
7
8
9
10
 
static void search(char *chaine)
{
    char *p = strchr(chaine, '\n');
 
    if (p)
    {
        *p = 0;
    }
}

pourquoi une fonction static ?

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
6
7
8
9
10
 
static void search(char *chaine)
{
    char *p = strchr(chaine, '\n');
 
    if (p)
    {
        *p = 0;
    }
}

pourquoi une fonction static ?

Pourquoi pas ! Si elle n'est pas exportée, static, sinon, on la met dans un fichier séparé, on crée un header avec un prototype séparé... procédure habituelle que je recommande. On peut même aller plus loin en la mettant dans sa bibliothèque personnelle...

Mais cette technique n'est pas formidable. Il faut au moins profiter du fait qu'on a détecté le '\n' ou non pour retourner cette information, car il peut y avoir des traitements conséquents à faire...

Je fais plutôt comme ceci :

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
 
#include <string.h>
 
static int search (char *chaine)
{
   int found = 0;
    char *p = strchr(chaine, '\n');
 
    if (p != NULL)
    {
        *p = 0;
        found = 1;
    }
   return found;
}

[Gruik]

Ou si buff est trop petit pour la ligne lue. Cette méthode simpliste est incorrecte. Ne pas l'utiliser.

C'était qu'une heuristique pour repondre au probleme posé qui était "y a un \n genant en fin de chaine"

[sahid]

tout marche bien,

par contre j'ai un warning, au niveau du strlen ...

Code :

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

1
2
 
main.c:66: warning: incompatible implicit declaration of built-in function ‘strlen’

Code :

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

1
2
3
4
5
6
 
void
nettoie_chaine (char *chaine)
{
  chaine[strlen (chaine) - 1] = '\0';
}

il y a un moyen de faire quelque chose ?

[Gruik]

#include <string.h>

[Emmanuel Delahaye]

tout marche bien,

Code :

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

1
2
3
4
5
6
 
void
nettoie_chaine (char *chaine)
{
  chaine[strlen (chaine) - 1] = '\0';
}

il y a un moyen de faire quelque chose ?

Je t'ai indiqué une méthode correcte. Celle-ci est incorrecte. Il faut tester la présence du \n avant de le retirer, sinon, on vire un caractère utile...