Discussion :

[BatuBou]

Bonsoir a vous,

Comment libérer l'espace mémoire occupée par la chaîne de caractère 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
16
17
18
19
char *CopieLigne (const char *s)
{
 int k = 0;
 char *res = malloc(sizeof(char)*strlen (s));
 while (s[k]!='\n')
  {
   res[k] = s[k];
   k++;
  }
 return res;
}
 
int main (void)
{
 char *origine = "Bonjour\nComment allez vous?\n";
 char *res = CopieLigne (origine);
 printf("%s\n",res);
 return 0;
}

De plus valgrind me donne se message

==4725== Conditional jump or move depends on uninitialised value(s)
==4725== at 0x401E215: strlen (mc_replace_strmem.c:246)
==4725== by 0x4088134: puts (in /lib/tls/i686/cmov/libc-2.3.6.so)
==4725== by 0x8048423: main

Merci pour votre aide

[Emmanuel Delahaye]

Comment libérer l'espace mémoire occupée par la chaîne de caractère res?
<...>
De plus valgrind me donne se message

Code :

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

1
2
3
4
5
 
==4725== Conditional jump or move depends on uninitialised value(s)
==4725==    at 0x401E215: strlen (mc_replace_strmem.c:246)
==4725==    by 0x4088134: puts (in /lib/tls/i686/cmov/libc-2.3.6.so)
==4725==    by 0x8048423: main

Merci pour votre aide

Déjà, tu peux corriger ça :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `CopieLigne':
main.c:4: warning: implicit declaration of function `malloc'
main.c:4: warning: implicit declaration of function `strlen'
main.c: In function `main':
main.c:15: warning: initialization discards qualifiers from pointer target type
main.c:17: warning: implicit declaration of function `printf'
main.c:19:2: warning: no newline at end of file
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 0 seconds)
0 errors, 5 warnings

http://emmanuel-delahaye.developpez....tm#cfg_compilo

ensuite un simple free() de res dans le main suffit, sauf complications que je n'aurais pas vues...

Quelques corrections :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
char *CopieLigne (const char *s)
{
   int k = 0;
   /* -ed- de la place pour le 0 final */
   char *const res = malloc (strlen (s) +1);
   while (s[k] != '\n')
   {
      res[k] = s[k];
      k++;
   }
 
    /* -ed- le 0 final... */
      res[k] = 0;
   return res;
}
 
int main (void)
{
   char const *origine = "Bonjour\n" 
                          "Comment allez vous?\n";
   char *res = CopieLigne (origine);
   if (res != NULL)
   {
      printf ("'%s'\n", res);
      free (res), res = NULL;
   }
   return 0;
}

Pose des questions si tu ne comprends pas.

[BatuBou]

Merci beaucoup pour votre reponse.

J'ai vraiment besoin de rajouter les options de compilation pr mon gcc

Bonne soirée a vous

[Pacorabanix]

Merci beaucoup pour votre reponse.

J'ai vraiment besoin de rajouter les options de compilation pr mon gcc

Bonne soirée a vous

-Wall (c-à-d afficher simplement tous les warnings) comme option est deja fortement conseillé

[BatuBou]

oui je compile tout le temps avec -Wall -ansi... Mais il m'a pas affiché tous ces warning lol.
C'est pour ca que je vais rajouter les autres options

[gl]

ensuite un simple free() de res dans le main suffit, sauf complications que je n'aurais pas vues...

Quelques corrections :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
char *CopieLigne (const char *s)
{
   int k = 0;
   /* -ed- de la place pour le 0 final */
   char *const res = malloc (strlen (s) +1);
   while (s[k] != '\n')
   {
      res[k] = s[k];
      k++;
   }
 
    /* -ed- le 0 final... */
      res[k] = 0;
   return res;
}
 
int main (void)
{
   char const *origine = "Bonjour\n" 
                          "Comment allez vous?\n";
   char *res = CopieLigne (origine);
   if (res != NULL)
   {
      printf ("'%s'\n", res);
      free (res), res = NULL;
   }
   return 0;
}

Pose des questions si tu ne comprends pas.

Personellement, j'aurais aussi rajoute un test sur la valeur de res (et donc sur le resultat de malloc) avant de l'utiliser.
En outre, arreter la boucle lorsque la fin de chaine est atteinte me semble preferable pour gerer le cas ou il n'y a pas de '\n' dans la chaine initiale et eviter un debordement de buffer.

[Emmanuel Delahaye]

Personellement, j'aurais aussi rajoute un test sur la valeur de res (et donc sur le resultat de malloc) avant de l'utiliser.
En outre, arreter la boucle lorsque la fin de chaine est atteinte me semble preferable pour gerer le cas ou il n'y a pas de '\n' dans la chaine initiale et eviter un debordement de buffer.

Oui, j'ai juste corrigé les bugs de bases. La fonction est améliorable, c'est sûr.

[Black-Hawk]

Déjà, tu peux corriger ça :

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

char *CopieLigne (const char *s)
{
   int k = 0;
   /* -ed- de la place pour le 0 final */
   char *const res = malloc (strlen (s) +1);
   while (s[k] != '\n')
   {
      res[k] = s[k];
      k++;
   }

    /* -ed- le 0 final... */
      res[k] = 0;
   return res;
}

int main (void)
{
   char const *origine = "Bonjour\n" 
                          "Comment allez vous?\n";
   char *res = CopieLigne (origine);
   if (res != NULL)
   {
      printf ("'%s'\n", res);
      free (res), res = NULL;
   }
   return 0;
}

A quoi ça sert de mettre res à NULL ??
C'est en prévision du cas où on ne prendrait pas soin d'initialiser un futur malloc qui retomberait au même endroit ??

Il y a pas des situations où cela est superflu de nettoyer les variables.

Les programmeurs professionnels le font systématiquement ???

[Melem]

Ca sert à indiquer que le pointeur ne pointe plus sur une région valide. Cela permet d'écrire ultérieurement par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
if (res == NULL)
{
    /* res est libre, on va lui donner du boulot */
    ...
    res =  <...>;
    ...
}
else
{
    /* res est encore occupe pour le moment, vaut mieux ne pas le deranger */
    ...
}

[Black-Hawk]

Ok bon c'est utile selon les codes qu'on implémente.

Mais bon je vais commençer quand même à prendre ça comme une habitude systématique .

Ce qui est sûr , c'est que ça fera pas de mal à mon code


PS : je crois que tu t'es trompé dans ton if , tu as inversé les conditions .

[Melem]

Merci.

Mais bon je vais commençer quand même à prendre ça comme une habitude systématique .

Ce qui est sûr , c'est que ça fera pas de mal à mon code

Oui ça fait partie des bonnes habitudes de programmation, d'ailleurs c'est très souvent utile.

[Emmanuel Delahaye]

A quoi ça sert de mettre res à NULL ??
C'est en prévision du cas où on ne prendrait pas soin d'initialiser un futur malloc qui retomberait au même endroit ??

Parce que c'est une Bonne Pratique

http://emmanuel-delahaye.developpez.....htm#pointeurs

Il y a pas des situations où cela est superflu de nettoyer les variables.

Certainement.

Les programmeurs professionnels le font systématiquement ???

Moi, je le fais.

[Black-Hawk]

Très intéressant ton site qui doit sûrement regorger de pleins d'autres bonnes pratiques.

Je le mets dans mes favoris.

Merci de partager tout ça !!