Discussion :

[gerald3d]

Bonjour à toutes et à tous.

Je désire copier une structure allouée dynamiquement. Jusqu'à présent j'allouais un nouveau bloc mémoire puis copiais une à une les variables de la source vers la copie:

Code :

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

1
2
3
4
5
6
mastructure *source;
mastructure *copie;
 
copie->a=source->a;
copie->b=source->b;
...

J'ai besoin de rapidité pour mon application. Ce matin m'est venu une idée farfelue mais qui me semble être correcte. Je vous la pose donc en attendant vos commentaires éclairés.

En admettant que toutes les données de mastructure ne soient que des données simples (pas de liste chaînée par exemple) puis-je faire une copie du bloc mémoire entier de source directement dans copie comme ca:

Code :

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

1
2
3
copie=(mastructure*)malloc(sizeof(mastructure));
 
copie=memcpy (copie, source, sizeof(mastructure));

En opérant comme ca vais-je retrouver tout dans l'ordre?

[diogene]

Oui, mais pour copier une structure dans une autre, il suffit d'écrire struct1 = struct2

[jeremya]

En admettant que toutes les données de mastructure ne soient que des données simples (pas de liste chaînée par exemple) puis-je faire une copie du bloc mémoire entier de source directement dans copie comme ca:

Code :

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

1
2
copie=(mastructure*)malloc(sizeof(mastructure));
copie=memcpy (copie, source, sizeof(mastructure));

En opérant comme ca vais-je retrouver tout dans l'ordre?

Exact !

[nicolas.sitbon]

Oui, mais pour copier une structure dans une autre, il suffit d'écrire struct1 = struct2

exemple :

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

int main (void)
{
    struct maStruct { int a; int b; };
    struct maStruct * struct1 = NULL;
    struct maStruct * struct2 = NULL;

    struct1 = malloc (sizeof *struct1);
    struct2 = malloc (sizeof *struct2);

    struct1->a = 2;
    struct1->b = 4;

    *struct2 = *struct1;

    printf ("%d %d\n", struct2->a, struct2->b);
    return 0;
}

[gerald3d]

Oui, mais pour copier une structure dans une autre, il suffit d'écrire struct1 = struct2

Non. tu ne fais que copier le pointeur de la struct2. Si tu désalloues struct2 struct1 pointe sur n'importe quoi aprés.

Moi j'ai vraiment besoin d'une vraie copie.

[jeremya]

Oui, mais pour copier une structure dans une autre, il suffit d'écrire struct1 = struct2

Non !

En considérant que source pointe vers une structure.

Code :

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

1
2
3
mastructure *source;
mastructure *copie
copie = source;

En faisant copie=source, la structure pointée vers source ne sera pas copiée. Le pointeur copie pointera simplement vers la même structure que source.

[gerald3d]

exemple :

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

int main (void)
{
    struct maStruct { int a; int b; };
    struct maStruct * struct1 = NULL;
    struct maStruct * struct2 = NULL;

    struct1 = malloc (sizeof *struct1);
    struct2 = malloc (sizeof *struct2);

    struct1->a = 2;
    struct1->b = 4;

    *struct2 = *struct1;

    printf ("%d %d\n", struct2->a, struct2->b);
    return 0;
}

Je ne pensais pas que *struct2 = *struct1; suffirait à copier tous les éléments de la structure !
Mon code va se simplifier. Merci à vous tous.

[nicolas.sitbon]

C'est ce qu'à expliqué diogene :

Oui, mais pour copier une structure dans une autre, il suffit d'écrire struct1 = struct2

[gerald3d]

Je n'ai pas compris sa réponse comme ca. Au temps pour moi si c'était le fond de sa pensée.

[nicolas.sitbon]

Je n'ai pas compris sa réponse comme ca. Au temps pour moi si c'était le fond de sa pensée.

Pas de soucis, c'est bien pour ce la que j'ai donné un exemple.
Cordialement. (pense à la balise résolu)

[jeremya]

C'est ce qu'à expliqué diogene

En précisant que quand il écrit struct1=struct2, struct1 et struct2 sont des structures et non pas des ponteurs de structures, sa pensée aurait été plus claire.

Code :

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

1
2
3
4
5
6
7
 
maStruct struct1;
maStruct struct2 = struct1;
 
maStruct* pstruct1 = &struct1;
maStruct* pstruct2 = malloc (sizeof maStruct);
*pstruct2 = *pstruct1;

[diogene]

J'avais pourtant bien écris "copier les structures" pas les pointeurs de structures.
nicolas.sitbon : tu as raison, on ne donne jamais trop de détails.

[jeremya]

J'avais pourtant bien écris "copier les structures" pas les pointeurs de structures.

Exact, mais le problème initial concernait des pointeurs de structure.

Je désire copier une structure allouée dynamiquement.