Discussion :

[Maverick27]

Bonjour
Je cherche à créer un tableau 2x2 dont la taille est déterminée à l'exécution donc obligé d'utiliser l'allocation dynamique.
Voici mon code

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
29
#include<stdio.h>
 
int main(void)
{
    int i, j, **matrice;
 
    matrice = (int *) malloc( 3*sizeof(int*));    
 
    for( i=0;i<3;i++ )
         matrice[i] = (int) malloc (3*sizeof(int));        
 
    for(i=0;i<3;i++)
    {
       for(j=0;j<3;j++)
       {            
              matrice[i][j]=0;
              printf(" %d ",matrice[i][j]);           
        }
       printf("\n");       
    }  
 
    for ( i = 0 ; i<3 ; i++)
        free( (void *) matrice[i] );
 
    free( (void *) matrice );
 
    getch();
 return 0;   
}

Je voudrais savoir si cette méthode est bonne et n'occasionne pas de bug. La libération de mémoire est-elle correcte ???
Merci
A+

[Médinoc]

Tu veux dire un tableau 2D ?

• Déjà, tu ne sépares pas assez ton code: L'allocation devrait être dans une fonction séparée qui prend les tailles en paramètre plutôt que de forcer 3x3.
• Ensuite, il y a beaucoup de casts inutiles.
• Puis, ton code ne gère pas l'échec de malloc().

En fait, je te conseille de faire moins d'appel à malloc(). Par exemple, te limiter à une allocation par dimension: http://www.developpez.net/forums/m1803870-7/