Discussion :

[User]

Bonjour à tous !

J'ai un problème avec la fonction str_strip du tuto:

A la fin de la fonction,
quand je mets printf("j %d\n",j);
le programme ne plante pas mais sans le printf ca plante

Code :

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

1
2
3
4
...
printf("j   %d\n",j);  
strip[j]='\0';
...

extrait du code du tuto

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
char *str_strip (const char *string)
{
   char *strip = NULL;
 
   if (string != NULL)
   {   
      strip = malloc (sizeof (*strip) * (strlen (string) + 1));
      if (strip != NULL)
      {
         int i, j;
         int ps = 0;
 
         for (i = 0, j = 0; string[i]; i++)
         {
            if (string[i] == ' ')
            {
               /*if (ps == 0)
               {
                  strip[j] = string[i];
                  ps = 1;
                  j++;
               }*/
            }
            else
            {
               strip[j] = string[i];
               ps = 0;
               j++;
            }
         }
 
         printf("j   %d\n",j);  
         strip[j]='\0';
 
      }
      else
      {
         fprintf (stderr, "Memoire insuffisante\n");
         exit (EXIT_FAILURE);
      }
   }
   return strip;
}

merci à vous !

[crocodilex]

A première vue, cette fonction a l'air correcte.
Montres ton code avec lequel cette fonction pose problème.

[User]

Le code est un peu long:

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
int read_proteines(char * src, entete t[1000])
{
    FILE * fsrc;
    char str[15000];
    char * res; 
    char * ch;
    char ch1[1500];
    char * occ;  
    int l;
    int i;  
    char flag[2];
    
    i=0;

    fsrc = fopen(src, "r");
 
    if (fsrc == NULL)
    {
        perror(src); 
        exit(EXIT_FAILURE);
    }
    
    while ((res=super_gets(str, sizeof str, fsrc))!=NULL) 
    {         

      strncpy(flag, res, 2);
                    

           if (strncmp(flag,"ID",2)==0) 
              {
              l=strlen (res)-1;
              ch=str_sub (res,5,l);
              t[i].ID=ch;
              
              } 
           else if (strncmp(flag,"AC",2)==0) 
              {
              l=strlen (res)-1;
              ch=str_sub (res,5,l);          

              res=super_gets(str, sizeof str, fsrc);
              strncpy(flag, res, 2);  

              while (strncmp(flag,"AC",2)==0)
                 {
                 l=strlen (res)-1;
                 occ=str_sub (res,5,l);
                 strcat (ch, " ");  
                 strcat (ch, occ);

                 res=super_gets(str, sizeof str, fsrc); 
                 strncpy(flag, res, 2); 
                 
                 }

              t[i].AC=ch;
              
              }
 
           else if (strncmp(flag,"DE",2)==0) 
              {
              l=strlen (res)-1;
              ch=str_sub (res,5,l);

              res=super_gets(str, sizeof str, fsrc);
              strncpy(flag, res, 2);  

              while (strncmp(flag,"DE",2)==0)
                 {
                 l=strlen (res)-1;
                 occ=str_sub (res,5,l);
                 strcat (ch, " ");  
                 strcat (ch, occ);

                 res=super_gets(str, sizeof str, fsrc); 
                 strncpy(flag, res, 2); 
                 
                 } 

              t[i].DE=ch;
               
              }

           else if (strncmp(flag,"  ",2)==0)
              {           
              
              strcpy(ch1,"");
              ch1[0]='\0';

              l=strlen (res)-1;
              occ=str_sub (res,5,l); 
              strcat (ch1, occ);

              res=super_gets(str, sizeof str, fsrc);

              strncpy(flag, res, 2);  

              while (strncmp(flag,"  ",2)==0)
                 {
                 l=strlen (res)-1;
                 occ=str_sub (res,5,l); 
                 strcat (ch1, occ);

                 res=super_gets(str, sizeof str, fsrc); 
                 strncpy(flag, res, 2);                 
                 }
                        
              t[i].SQ=str_strip(ch1);
 
              i=i+1;
              }
          
                   
    }

    fclose(fsrc);

return i; 

}

l'instruction est en bleu

(je ne suis pas un expert )

@+

[souviron34]

déjà flag doit être dimensionné à 3 : il faut la place pour le '\0' final....

dans tes strncpy ça déborde...

[Emmanuel Delahaye]

déjà flag doit être dimensionné à 3 : il faut la place pour le '\0' final....

dans tes strncpy ça déborde...

Vu qu'il utlise strncpy() et strncmp(), c'est pas obligatoire... Mais il faut être très strict sur l'usage des fonctions et ne jamais utiliser de fonctions 'chaines', de celles qui attendent un 0 en fin de chaine...

Or, il y a un strcat() sur un ch ! C'est mort !

Donc, OK, il faut bien construire des chaines valides.

(en fait, c'est pire que ça, car ch fait 2 et il fait un strcat() sur de la mémoire qui n'existe pas...)

[User]

Desole pour mon retard,

Emmanuel Delahay,
comme je l'ai mentionne plus haut,
mon code fonctionne parfaitement bien avec le printf au niveau de str_strip mais sans le printf ca plante

pour le flag je vais changer,

en fait tu critiques mais tu ne proposes rien de concret a la place du strcat...

tu ne sais pas te mettre au niveau des gens, tout le monde n est pas aussi expert que toi

[Emmanuel Delahaye]

Le code est un peu long:

Peux-tu donner un exemple de fichier de données à traiter. Que se passe-t-il si il y a plus de 1000 résultats ?

Après investigation rapide, ton code est extrêmement faux, car il utilise des chaines incomplètes, de taille insuffisantes... (comportement indéterminé). En C de base, rien n'est automatique. Il faut gérer les chaines à la main.

Néanmoins, et pour simplifier le travail, j'ai mis au point un système qui permet de gérer les chaines automatiquement.

http://emmanuel-delahaye.developpez.com/clib.htm
Module FSTR (Flexible STRings)

C'est un projet scolaire ?

[Emmanuel Delahaye]

Bonjour à tous !

J'ai un problème avec la fonction str_strip du tuto:

A la fin de la fonction,
quand je mets printf("j %d\n",j);
le programme ne plante pas mais sans le printf ca plante

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
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
48
49
50
51
52
53
54
55
56
57
58
59
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
char *str_strip (const char *string)
{
   char *strip = NULL;
 
   if (string != NULL)
   {
      strip = malloc (sizeof (*strip) * (strlen (string) + 1));
      if (strip != NULL)
      {
         int i, j;
         int ps = 0;
 
         for (i = 0, j = 0; string[i]; i++)
         {
            if (string[i] != ' ')
            {
               strip[j] = string[i];
               ps = 0;
               j++;
            }
         }
         strip[j] = '\0';
 
      }
      else
      {
         fprintf (stderr, "Memoire insuffisante\n");
         exit (EXIT_FAILURE);
      }
   }
   return strip;
}
 
#ifdef TEST
 
#include "ed/inc/prt.h"
#include <assert.h>
 
int main (void)
{
   char const *sa = "Hello world";
 
   char *sb = str_strip (sa);
   if (sb != NULL)
   {
      PRT_S (sb);
 
      free (sb), sb = NULL;
   }
   assert (sb == NULL);
 
   return 0;
}
#endif

fonctionne correctement.
(je me serais contenté d'un return NULL en cas d'erreur d'allocation...)

Code :

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

1
2
3
4
 
sb           = 'Helloworld'
 
Press ENTER to continue.

Le problème est ailleurs.