Discussion :

[koktel_dfr]

J'ai declaré une chaine de caractere par :

char line[256];

Cette chaine est censée s'écrire dans un fichier mais le problème c'est que celle-ci est constitué de plusieurs sous-chaines concaténées par strcat(). Certaines de ces sous-chaines comprennent des retours à la ligne.

La chaine line est censé etre écrite dans un fichier. Malheureusement cette chaine contient des retours a la ligne. Donc lorsque j'ecris celle-ci dans mon fichier je me retrouve avec plusieurs lignes dans celui-ci. Alors que je veux que la chaine line soit écrite sous la forme d'une seule ligne dans mon fichier.

Je souhaiterais donc savoir comment supprimer tous les retours à la ligne de ma chaine.

Merci d'avance.

[koktel_dfr]

Désolé pour le dérangement...il suffit que je poste pour arriver ensuite à mes fins de moi meme... je ne supprime pas mon topic mais préfère laisser la solution au cas ou certains seraient interessés :

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
 
char *Trim (char *str)
{
      char *ibuf, *obuf;
 
      if (str)
      {
            for (ibuf = obuf = str ; *ibuf; )
            {
                  while (*ibuf && ((isspace (*ibuf)) || (*ibuf == '"')))
                        ibuf++;
                  if (*ibuf && (obuf != str))
                        *(obuf++) = ' ';
                  while (*ibuf && (!isspace (*ibuf)))
                        *(obuf++) = *(ibuf++);
            }
            *obuf = '\0';
      }
      return (str);
}

[Franck.H]

C'est simple comme bonjour... ou presque

Tu utilise strchr en boucle à la recherche d'un caractère '\n', dès que tu en trouves un, tu le remplace par exemple par un espace, exemple:

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
 
int main (void)
{
   char str [] = "Une ligne, \nDeux lignes..., \nTrois lignes !";
   char * p = NULL;
 
 
   printf ("Avant:\n%s\n\n", str);
 
   while ((p = strchr (str, '\n')) != NULL)
   {
      *p++ = ' ';
   }
 
   printf ("Apres:\n%s\n", str);
 
 
   return EXIT_SUCCESS;
}

Résultat à la sortie:

Code :

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

1
2
3
4
5
6
7
8
9
Avant:
Une ligne, 
Deux lignes..., 
Trois lignes !
 
Apres:
Une ligne,  Deux lignes...,  Trois lignes !
 
Press ENTER to continue.

[LinuxUser]

Je pense que tu devrais soit utiliser strncat si toute tes chaines ont un '\n' à la fin et mettre la taille (n-1) de ta chaine en paramètre, soit copier toute ta chaine finale dans une autre chaine en ignorant les '\n'.
Mais supprimer les '\n'sur ta chaine déjà créée, je crois pas que ce soit possible.

[Franck.H]

Mais supprimer les '\n'sur ta chaine déjà créée, je crois pas que ce soit possible.

Bin pourquoi on ne pourrais pas ? Voir mon code plus haut.

[LinuxUser]

Effectivement, je n'ai pas été très malin, je n'avais pas pensé à cela.
C'est beaucoup plus simple ainsi.

[Skyounet]

Tu peux aussi décaler ta chaine chaque fois que tu trouves un \n

Code :

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

1
2
3
4
5
6
7
8
9
10
 
void shift_left(char *string)
{
    size_t i = 0;
 
    for(i = 0; i < strlen(string); i++)
    {
        string[i] = string[i+1];
    }
}

[souviron34]

euh.. Skyrunner t'a été trop vite...

Mais sinon bien sûr l'autre solution (en dehors de celle de Franck.h) est comme l'a mentionné juve de le faire au moment du strcat..

Code :

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

1
2
3
4
if ( chaine[strlen(chaine)-1] == '\n' )
   strcat ( &(chaine[strlen(chaine)-1]), Chaine2 );
else
   strcat ( chaine, Chaine2 );

[LinuxUser]

Je n'ai pas bien compris ta fonction.
J'ai l'impression qu'elle fais bien un décalage vers la gauche, mais que le premier élément est perdu et que '\n' est toujours là.

[Jean-Marc.Bourguet]

Code :

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

1
2
3
4
   while ((p = strchr (str, '\n')) != NULL)
   {
      *p++ = ' ';
   }

Code :

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

1
2
3
4
5
p = str;
while ((p = strchr(p, '\n') != NULL) 
{
   *p++ = ' ';
}

Je sais bien que les optimisations prematurees sont a eviter, mais faire du n^2 quand du n est aussi simple...

[souviron34]

non ce qu'il veut faire c'est chaque fois que tu trouves le '\n', au lieu de l'enelver comme on a montré plus haut, il décale ce qui reste de la chaine de 1 cran, en écrasant donc le '\n'.

Mais c'est nettement plus long que les 2 autres méthodes..

[Franck.H]

Code :

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

1
2
3
4
5
p = str;
while ((p = strchr(p, '\n') != NULL) 
{
   *p++ = ' ';
}

Je sais bien que les optimisations prematurees sont a eviter, mais faire du n^2 quand du n est aussi simple...

Et c'est censé changer quoi ? Je vois pas trop en fait

[souviron34]

Je sais bien que les optimisations prematurees sont a eviter, mais faire du n^2 quand du n est aussi simple...

ou as-tu vu du n^2 ??

strchr est linéaire... et reviens à faire la boucle à travers les caractères...

[souviron34]

Et c'est censé changer quoi ? Je vois pas trop en fait

juste que tu redémarres à zéro à chaque fois car tu avais gardé l'adresse du debut de la chaîne.

tu avais raison Jean-Marc désolé

[Franck.H]

A ouais c'est pas faux, j'avais pas vu ca J'ai rien dit Jean-Marc

[koktel_dfr]

Je pense m'être un peu compliqué la vie avec mon code...

Merci à tous pour votre aide. Cela dit la fonction que j'ai noté plus haut ne fait pas que supprimer les retours à la ligne et comme j'aurais besoins d'elles plus tard autant l'utiliser pour supprimer les retour à la ligne même si elle fait plus que ca.

[Jean-Marc.Bourguet]

ou as-tu vu du n^2 ??

strchr est linéaire... et reviens à faire la boucle à travers les caractères...

Oui. Et appele dans une boucle ou on va passer un nombre de fois proportionnel a la longueur de la chaine (hypothese vraissemblable), on un a comportement quadratique (1+2+3+...n, c'est du quadratique).

[diogene]

La solution de Franck.H est simple mais a l'inconvénient de ne pas supprimer mais de remplacer le '\n' et donc de modifier la ligne (mais peut être que ce n'est pas important dans ce cas).
Si on veutsupprimer les '\n' sur la ligne finale, il faut décaler les caractères.

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
void SupprimerRC( char* chaine)
{
   char * lire = chaine;
   while(*lire != '\0')
   {
      while(*lire == '\n') lire++;
      *chaine = *lire;
      if(*lire != '\0')
      {
         chaine ++;
         lire++;
      }
   }
   *chaine = '\0';
}

[Skyounet]

C'est pour ça que j'ai proposé une fonction qui décale vers la gauche.

Dès que tu trouves un \n tu décales à partir de cette position.

[souviron34]

La solution de Franck.H est simple mais a l'inconvénient de ne pas supprimer mais de remplacer le '\n' et donc de modifier la ligne ...

???

parce que le fait de décaler ne modifie pas la ligne ???