Discussion :

[L8O8L]

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 )
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.

[Franck.H]

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

[L8O8L]

ok...

en fait dans un .c j'ai ca:

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
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 :

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
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;
}

[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 :

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

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

fait plutôt:

Code :

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

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

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

Code :

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

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

fait:

Code :

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

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++

[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...?)

[Franck.H]

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

[salseropom]

Code :

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

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

ce ne serait pas plutôt

Code :

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

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

[L8O8L]

Aparament ca depend des machines
Donc on va mettre 'resolu' et voila..

[Franck.H]

ce ne serait pas plutôt

Code :

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

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

Je n'ai jamais testé cette approche mais ma solution fonctionne !