Discussion :

[LlufRuS]

Bonjour à tous,

je planche un petit peu sur une histoire de recopie de mémoire (qui je dois le préciser ne fonctionne pas...)

J'ai la structure suivante

Code :

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

1
2
3
4
typedef struct ElementListePile{
  void *attribut;
  struct ElementListePile *suivant;
} ElementPile;

et souhaite recopier la valeur mémoire d'un void* dans mon "attribut".

j'utilise actuellement cette façon (je ne test pas le malloc sur l'attribut mais de la mémoire lui est bien allouée).

Code :

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

1
2
3
4
5
ElementPile *nouveau_element;
  if ((nouveau_element = (ElementPile *) malloc (sizeof (ElementPile))) == NULL)
    return -1;
  nouveau_element->attribut = malloc(sizeof(size));
  nouveau_element->attribut = memmove(nouveau_element->attribut, donnee, sizeof(size));

Or le memmove ne fait strictement rien.. et je ne comprend pas pourquoi.
NB : "donnee" est un void* correspondant à la réference d'un double sur ce problème
"size" vaut 8

Si quelqu'un est capable de m'éclairer, je l'en remercie d'avance.

++,
Adrien

[LlufRuS]

... sizeof(size) ...
Besoin de dormir un peu je crois

[BainE]

bonjour,

Code :

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

1
2
3
4
5
6
 
ElementPile *nouveau_element;
if ((nouveau_element = (ElementPile *) malloc (sizeof (ElementPile))) == NULL)
return -1;
nouveau_element->attribut = malloc( size );
nouveau_element->attribut = memmove(nouveau_element->attribut, donnee, 8 );

sizeof( size ) : tu veux la taille d un int ou d un long ou le nombre contenu dedans "size" ?

Est tu sur que donnee n est pas NULL ?
pourquoi ne pas utiliser memcpy ? (memmove est plus gourmand en ressource, mais memcpy les zones ne doivent pas se chevaucher).

Comment sais tu que memmove ne fait rien ?

P.S. : pense a la balise code la prochaine fois.

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
ElementPile *nouveau_element;
  if ((nouveau_element = (ElementPile *) malloc (sizeof (ElementPile))) == NULL)
    return -1;
  nouveau_element->attribut = malloc(sizeof(size));
  nouveau_element->attribut = memmove(nouveau_element->attribut, donnee, sizeof(size));

Or le memmove ne fait strictement rien.. et je ne comprend pas pourquoi.
NB : "donnee" est un void* correspondant à la réference d'un double sur ce problème
"size" vaut 8

Si quelqu'un est capable de m'éclairer, je l'en remercie d'avance.

memmove() fait bien la recopie, mais les infos de taille sont fausses. de plus, il n'est utile de toucher au pointeur :

Code :

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

1
2
3
 
  nouveau_element->attribut = malloc(size);
  memmove (nouveau_element->attribut, donnee, size);

Nota : pourquoi memmove() ? Son usage est requis quand il y a recouvrement. Ce n'est pas le cas ici. Tu peux utiliser sans crainte memcpy() (plus efficace).

Code :

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

1
2
3
 
  nouveau_element->attribut = malloc(size);
  memcpy (nouveau_element->attribut, donnee, size);

Enfin, je suis étonné que tu recopies des données. Est-ce bien nécessaire ? Si elles sont persistantes (chaine, statique, allouée, locale de haut niveau), il suffit de mémoriser l'adresse, ou alors de passer (et de mémoriser) une information 'faire copie' ou non... Seul l'utilisateur le sait...