Discussion :

[anas0dev]

Bonjour,
j'ai une structure et je veux créer un tableau de cette structure avec une allocation dynamique et le remplir à partir d'un autre tableau de celle-ci.

voici la structure:

Code :

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

1
2
3
4
5
6
7
 
struct Distance {
	int idVille1;
	int idVille2;
	int distance;
};
typedef struct Distance Dist;

je crée un tableau de cette structure et je le copie dans un autre créer avec un malloc() et je l'affiche.

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
 
int main(int argc, char* argv[]){
 
	Dist a[11] = {	{1, 2, 10},
				{1, 3, 5},
				{1, 4, 16},
				{1, 5, 12},
				{2, 3, 9},
				{2, 4, 10},
				{2, 5, 20},
				{3, 4, 13},
				{3, 5, 15},
				{4, 5, 12},
				{2, 6, 10} };
 
	Dist *b = (Dist*) malloc(11 * sizeof b);
	for(int i=0; i<11; i++){
		b[i] = a[i];
	}
	for(int i=0; i<11; i++){
		printf("%d %d %d\n", b[i].idVille1, b[i].idVille2, b[i].distance);
	}
 
	return 0;
}

mais le problème c'est que le résultat qui est bizzare.

le voila :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
1 2 10
1 3 5
1 4 16
1 5 12
2 3 9
2 4 10
2 5 20
3 1041 0
808525875 807416116 10
909194295 540422449 667697
669497 6 10

et les 4 dernière ligne sont toujours les même à chaque fois.

merci pour votre aide.

[foetus]

C'est ultra normal Quel est pour toi la valeur de sizeof(b) ?

De plus tu n'as pas besoin de fournir la taille de a - le compilateur le fait pour toi - Dist list_distances[] = { /* ... */ };.
Et tant que ta variable tableau n'est pas dégradé en pointeur, sizeof(list_distances) vaut 11 (<- Édit : anas0dev me rappelle que c'est la taille totale et donc il faut diviser par la taille de la structure)

Ta variable de boucle peut être sortie et ainsi définie 1 seule fois
En C, le retour de malloc n'a pas besoin d'être casté


Et n'oublie pas l'appel à free (<- lien cplusplus en anglais)

[anas0dev]

C'est ultra normal Quel est pour toi la valeur de sizeof(b) ?

pour moi la valeur de sizeof(b) est la même que sizeof(Dist*) qui est la taille d'un pointeur, et je viens de le vérifier avec printf() et sa affiche 8, et vu que moi je veux un tableau de Dist se qu'il faut c'est sizeof(*b).

merci pour le tuyau

De plus tu n'as pas besoin de fournir la taille de a - le compilateur le fait pour toi - Dist list_distances[] = { /* ... */ };.

c'est vrai merci pour le rapelle.

Et tant que ta variable tableau n'est pas dégradé en pointeur, sizeof(list_distances) vaut 11.

j'ai affiché la valeur de sizeof(a) et ça vaut 132, c'est la valeur de tout le tableau 4*3*11, je pense que tu voulais plutôt sizeof(a)/sizeof(Dist).

Ta variable de boucle peut être sortie et ainsi définie 1 seule fois
En C, le retour de malloc n'a pas besoin d'être casté


Et n'oublie pas l'appel à free (<- lien cplusplus en anglais)

c'est noté merci

et bonne soirée

[foetus]

vu que moi je veux un tableau de Dist se qu'il faut c'est sizeof(*b).

Ou plus simplement sizeof(Dist) (dans sizeof on peut mettre des types)

[anas0dev]

Merci pour tout vos précieux conseils

[Sve@r]

Code c :

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

1
2
3
for(int i=0; i<11; i++){
	b[i] = a[i];
}

Bonjour

Tout a été dit. Il y a juste cette boucle qu'on peut remplacer par un memcpy(b, a, sizeof a)...