Discussion :

[Franck.H]

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 :

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

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

[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 c :

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

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 c :

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

1
2
strncpy(p_new->sz_string, p1, size);
p_new->sz_string[size + 1] = '\0';

[Médinoc]

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

Code :

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

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?

[grim7reaper]

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

Code :

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

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

[Franck.H]

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.

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

Édit: ça c'est moche:

Code c :

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

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 c :

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

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é

[Franck.H]

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

Pour le moment ça fonctionne bien mieux