Problème de restitution de texte

Version imprimable

:salut:

Je travaille actuellement sur ma bibliothèque de chaînes de caractères (enfin de nouveau du temps !!). Je suis sur une fonction qui coupe le texte en lignes distinctes en utilisant comme repères de découpage, les saut de ligne et a terme, le retour chariot. Chaque ligne se retrouve donc être une chaîne de caractères appartenant à un buffer C_Str_buffer (une liste chaînées d'objects C_Str).

Voici ma sortie:

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
#include <stdio.h>
#include <stdlib.h>
 
@#include "c_str.h"
 
@
@void cb (void * p_data, void * p_user_data)
{
   (void)p_user_data;
   printf ("%s", C_Str_get ((C_Str *)p_data));
}
 
int main (void)
{
   C_Str * str = C_Str_new_from_file ("main.c");
   C_Str_buffer * buff = C_Str_split_lines (str);
 
   if (str && buff)
   {
      C_Str_buffer_foreach(cb, (void*)buff, NULL);
 
      C_Str_buffer_destroy (&buff);
      C_Str_destroy (&str);
   }
 
   return EXIT_SUCCESS;
}

Le problème se situe toujours sur les mêmes lignes. Pour le moment je teste que le caractère d'échappement \n car je sais qu"il n'y a que celui là dans mon code. Voici la fonction de traitement:

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
37
38
39
40
41
42
43
44
45
46
47
DECLSPEC C_Str_buffer * C_Str_split_lines (C_Str * const self)
{
   C_Str_buffer * p_buf = NULL;
 
   if (self && self->sz_string)
   {
      char * p1 = self->sz_string;
      char * p2 = p1;
 
      p_buf = C_Str_buffer_new ();
 
      if (p_buf)
      {
         size_t size = 0;
 
         while (*p2 != '\0')
         {
            size++;
 
            if (*p2 == LF)
            {
               C_Str * p_new = C_Str_new_len (size + 1);
 
               if (p_new)
               {
                  int i = 0;
 
                  do
                  {
                     p_new->sz_string[i++] = *p1;
                  }
                  while (p1++ != p2);
 
                  p_new->sz_string[size + 1] = '\0';
                  size = 0;
 
                  C_Str_buffer_add(p_buf, p_new);
               }
            }
 
            p2++;
         }
      }
   }
 
   return p_buf;
}

Je ne vois franchement pas d'où viens le problème, il me faut surtout un oeil neuf ;)

:merci:

07/07/2014, 16h29
grim7reaper
Salut,

Est-ce que tu pourrais fournir un SSCCE (que l'on puisse au moins compiler et exécuter le code) s'il te plaît?
(j'ai bien essayé de récupérer le code ici, mais C_Str_buffer n'est pas défini).

Édit: ça c'est moche:
Code:

1 2 3 4 5 6 do { p_new->sz_string[i++] = *p1; } while (p1++ != p2); p_new->sz_string[size + 1] = '\0';
Autant faire un
Code:

1 2 strncpy(p_new->sz_string, p1, size); p_new->sz_string[size + 1] = '\0';
07/07/2014, 16h57
Médinoc
Bonjour,
Ça, ça sent très mauvais:
Code:

1 2 3 4 5 C_Str * p_new = C_Str_new_len (size + 1); ... p_new->sz_string[size + 1] = '\0';
De plus, pourquoi p1 et p2 ne sont-ils pas des pointeurs const?
07/07/2014, 16h59
grim7reaper
Citation:
Envoyé par Médinoc

Bonjour,
Ça, ça sent très mauvais:

Code:

1 2 3 4 5 C_Str * p_new = C_Str_new_len (size + 1); ... p_new->sz_string[size + 1] = '\0';
Non, à priori ça va quand tu regardes le code de C_Str_resize (qui va vraiment faire l'allocation).
Enfin si j'ai bien suivi en survolant rapidement le code (je garanti rien...)
07/07/2014, 17h05
Franck.H

1 pièce(s) jointe(s)
Citation:

Envoyé par grim7reaper

Est-ce que tu pourrais fournir un SSCCE (que l'on puisse au moins compiler et exécuter le code) s'il te plaît?

Voir en pièce jointe (projet C::B Linux) ! Désolé de ne pas l'avoir mis mais j'avais mis uniquement le code concerné par le problème car ça devient un projet de plus en plus gros.

Citation:

Envoyé par grim7reaper

(j'ai bien essayé de récupérer le code ici, mais C_Str_buffer n'est pas défini).

Oui bin normal, c'est une version en cours de dev donc..
Citation:
Envoyé par grim7reaper

Édit: ça c'est moche:

Code:

1 2 3 4 5 6 do { p_new->sz_string[i++] = *p1; } while (p1++ != p2); p_new->sz_string[size + 1] = '\0';

Autant faire un

Code:

1 2 strncpy(p_new->sz_string, p1, size); p_new->sz_string[size + 1] = '\0';
Je sais je sais, c'est en cours de dev, il n'y a encore rien de terminé :aie:
07/07/2014, 17h17
Franck.H

Citation:

Envoyé par grim7reaper

Non, à priori ça va quand tu regardes le code de C_Str_resize (qui va vraiment faire l'allocation).
Enfin si j'ai bien suivi en survolant rapidement le code (je garanti rien...)

En fait Médinoc n'avait pas spécialement tort, le size + 1 et le '\0' à ce même emplacement alors qu'il y a un espace vide entre le dernier caractère et la fin de chaîne car effectivement mon C_Str_resize appelé par C_Str_new_len ajoute déjà 1 à la taille pour justement le zéro final.. Ca fait un bail que j'ai pas touché à cette lib, je l'ai quand même démarré en 2005 et fait déjà énormément de modifications et ajouts en tous genres :aie:

Pour le moment ça fonctionne bien mieux ;)

:merci: