Probleme de malloc

Version imprimable

24/03/2007, 12h21
L8O8L

Probleme de malloc

Salut a tous,

Je dois rendre un programme pour la semaine prochaine en C.
Mais j'ai un problème , j'explique :

Je suis sous Vista donc je ne peux utiliser que Visual C++ qui a été mis a jour pour Vista (sinon y'a des problemes de compilation :evilred: :furax: )
Je veux tester ce programme sous une autre machine et la ca foire, j'essaye encore sous une autre et ca refoire;
En faisant plusieurs test je me rend compte que ca bloque sur un malloc

Est-ce que c'est possible que l'ordi bloque tout seul sur un malloc?
En plus c'est pour faire une copie donc j'avais déjà fait ce même malloc avant.
24/03/2007, 12h44
Franck.H

Bin montre ton code ... là c'est un peu dur de t'aider.

ok...

en fait dans un .c j'ai ca:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
TYPE** InitialiserEchiquier(Coordonnees* taille_tab){			
 
..........
 
echiquier = (TYPE**)malloc(sizeof(TYPE*)*x);
		if (echiquier == NULL){							
			fprintf(stderr,"!! . ..Erreur d'allocation (colonnes).. . !!\n\n");
			exit(1);
		}
		for ( i = 1 ; i <= x ; i++){							
			echiquier[i] = (TYPE *)malloc(sizeof(TYPE)*y);			
			if (echiquier[i] == NULL){							
				fprintf(stderr,"!! . ..Erreur d'allocation (ligne %d).. . !!\n\n",i);
				exit(1);
			}
		}
....
 
    }
 
	return echiquier;
}

ce malloc marche mais appres j'ai cette fonction :

Code:

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
TYPE** CopieEchiquier(TYPE** echiquier,Coordonnees taille_tab){
	TYPE** copie;
	int i;
	int j;
 
	copie = (TYPE**)malloc(sizeof(TYPE*)*taille_tab.x); // apparament (si mes tests sont exact... ) c'est la que ca bloque
		if (copie == NULL){
			printf("!! . ..Erreur d'allocation (colonnes).. . !!\n\n");
			exit(1);
		}
		for ( i = 1 ; i <= taille_tab.x ; i++){
			copie[i] = (TYPE *)malloc(sizeof(TYPE)*taille_tab.y);
			if (copie[i] == NULL){
				printf("!! . ..Erreur d'allocation (ligne %d).. . !!\n\n",i);
				exit(1);
			}
		}
		for ( i = 1 ; i <= taille_tab.x ; i++){
			for ( j = 1 ; j <= taille_tab.y ; j++){
				copie[i][j].coord.x=echiquier[i][j].coord.x;
				copie[i][j].coord.y=echiquier[i][j].coord.y;
				copie[i][j].representation=echiquier[i][j].representation;
			}
		}
 
 
		return copie;
}

24/03/2007, 13h19
Franck.H

Franchement, je reste perplexe quant à l'endroit où ton programme plante, il est déjà rare qu'un malloc échoue donc cela pourrait très bien venir d'autre part, à tester mais sans le programme complet je ne peut t'en dire plus pour le moment.

Juste un conseil pour améliorer la lisibilité et la maintenance de ton code pour un malloc, prennons:

Code:

echiquier = (TYPE**)malloc(sizeof(TYPE*)*x);

fait plutôt:

Code:

echiquier = malloc (x * sizeof (** echiquer));

de même pour allouer les indices de ton tableau:

Code:

echiquier[i] = (TYPE *)malloc(sizeof(TYPE)*y);

fait:

Code:

echiquier[i] = malloc (y * sizeof (* echiquier));

Tu peux remarquer que j'ai supprimé les cast car en effet, ils sont inutiles en C sauf si tu veux que ton programme soit compilable par un compilateur C++ ;)
24/03/2007, 13h24
L8O8L

Ca marche en metant autre chause que des types dans le sizeof ??
J'avais appris comme ca.
Mais je trouve ca bizarre aussi que ca foire aux malloc...

peut etre que c'est un autre probleme je vais retester mais quand je met un printf apres la ligne ou je croyais que ca merdais, il n'affiche pas a l'ecran.

Sinon on est d'accord que le probleme c'est sur copie par sur echiquier.
(enfin le probleme sur Code::Blocks... c'est bizarre ca...?)
24/03/2007, 14h02
salseropom

Citation:

Envoyé par Franck.H

Code:

echiquier[i] = malloc (y * sizeof (* echiquier));

ce ne serait pas plutôt

Code:

echiquier[i] = malloc (y * sizeof (* echiquier[i]));
24/03/2007, 14h04
L8O8L

Aparament ca depend des machines
Donc on va mettre 'resolu' et voila..
24/03/2007, 14h15
Franck.H

Citation:

Envoyé par salseropom

ce ne serait pas plutôt

Code:

echiquier[i] = malloc (y * sizeof (* echiquier[i]));

Je n'ai jamais testé cette approche mais ma solution fonctionne !
24/03/2007, 14h18
Franck.H

Citation:

Envoyé par L8O8L

Ca marche en metant autre chause que des types dans le sizeof ??
J'avais appris comme ca.

Oui biensûr que ca fonctionne, un pointeur pointe sur une donnée d'un type mais c'est que du point de vue maintenance, la solution que je te propose est nettement mieux.

Imagine en utilisant ta méthode, le type change, tu sera obligé de revoir toutes tes allocations pour changer le type dans tes sizeof alors qu'avec la mienne, juste à changer le type dans la déclaration du pointeur ;)

Voilà le gros avantage ;)