Discussion :

[gotrunkssj]

Bonjour;

Je dispose de 4 tablaux à N élement et je veux tester que ma mémoire a bien été alloué dynamiquement, pr cela je fais :

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
 
int *A=NULL,*B=NULL,*C=NULL,*D=NULL;
 
             A=malloc((N)* sizeof(int));
 
	B=malloc((N)* sizeof(int));
 
	C=malloc((N)* sizeof(int));
 
	D=malloc((N)* sizeof(int));
 
	// test si la mémoire a bien été alloué manuellement
 
	if ( (A == NULL || B == NULL) || ( C==NULL || D==NULL) )
    {
        exit(0);
    }

Par ailleur je dispose egalement d une matrice mat[M][N], et j aimerai effectuer le meme test, mais je ne vois pas comment faire

Code :

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

1
2
3
4
5
6
7
8
9
 
int **mat=NULL;
int i;
mat=malloc(M *sizeof(int*)); 
 
	for (i = 0; i < M ; i++)
	{
	mat[i] =malloc(N * sizeof(int));
	}

Merci d'avance

[Pacorabanix]

salut,

Code :

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

	if ( (A == NULL || B == NULL) || ( C==NULL || D==NULL) )

Tes parenthèses internes sont inutiles et nuisent à la lisibilité, mais ce n'est qu'un détail.


Ensuite pour ton souci, ben je ne vois pas trop de problème :

D'abord tu vérifies si mat a bien été alloué (if (mat != NULL) ) (avant ta boucle for ! )


puis , au choix, tu peux vérifier juste après le malloc dans le for si mat[i] a bien été alloué, ou alors refaire une deuxième boucle pour vérifier tous les mat[i].

Par contre sortir brutalement du programme n'est pas très efficace pour le débuggage, fait au moins un printf pour indiquer où et quoi a posé problème .

[diogene]

Code :

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

1
2
3
4
5
6
7
8
9
10
int **mat=NULL;
int i;
mat=malloc(M *sizeof(int*)); 
if(mat != NULL)	
	for (i = 0; i < M ; i++)
	{
	mat[i] =malloc(N * sizeof(int));
        if(mat[i] == NULL) exit(0);
	}
else exit(0);

[gotrunkssj]

Merci,
Ma question était un peu bete, j'avais pas pensé a raison ainsi !

Une derniere chose : qd on fait un alloc dynamique de mémoire est il vraiment necessaire de liberer de l espace memoire à la fin du programme avec la fction free ? Ou bien qu'une fois le programme terminé l'espace est liberer par lui meme ?

Merci encore

[knuthy]

Merci,
Ma question était un peu bete, j'avais pas pensé a raison ainsi !

Une derniere chose : qd on fait un alloc dynamique de mémoire est il vraiment necessaire de liberer de l espace memoire à la fin du programme avec la fction free ? Ou bien qu'une fois le programme terminé l'espace est liberer par lui meme ?

Merci encore

Une fois que le processus se termine l'espace est automatiquement libéré;
mais il est conseillé de s'habituer à libérer l'espace avec free() ;
car un jour on verra un petit "memory leak" (rarement dans les petits programmes, mais comme j'ai dit, il faut apprendre les bonnes habitudes) http://en.wikipedia.org/wiki/Memory_...mple_C_example

[souviron34]

Avec tes exemples, cela donne :

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
int *A=NULL,*B=NULL,*C=NULL,*D=NULL;

             A=malloc((N)* sizeof(int));
	B=malloc((N)* sizeof(int));
	C=malloc((N)* sizeof(int));
	D=malloc((N)* sizeof(int));

	// test si la mémoire a bien été alloué manuellement
	
	if ( (A == NULL) || (B == NULL) || (C==NULL) || (D==NULL) )
              {
                 /* Si possible les mettrre en sens inverse de l'allocation (pour tenter de VRAIMENT libérer la mémoire) */
                 if ( D != NULL )
                   free (D);
                 if ( C != NULL )
                   free (C);
                 if ( B != NULL )
                   free (B);
                 if ( A != NULL )
                   free (A);
                 exit(0);
              }

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
int **mat=NULL;
int i, j;
mat=malloc(M *sizeof(int*)); 
if(mat != NULL)
  {
     for (i = 0; i < M ; i++)
        {
           mat[i] =malloc(N * sizeof(int));
           if(mat[i] == NULL) 
             {
                if ( i != 0 )
                   for ( j = (i-1) ; j >= 0 ; j-- )
                      free ( mat[j]);

                 exit(0);
             }
        }
  }
else 
     exit(0);

[gotrunkssj]

Est ce faux de faire ainsi a la fin du programme !?

Code :

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

1
2
3
4
5
6
 
 
free(*mat);
 
for(i=0;i<M;i++)
       free(mat[i]);

[souviron34]

oui parce que tu libères mat avant d'avoir libérer chacun des éléments d emat...