Discussion :

[guitz]

Bonjour,

Soit la structure suivante :

Code :

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

1
2
3
4
5
6
typedef struct matrice *matrice ;
struct matrice {
    int l ;
    int c ;
    double *m;
}

l étant le nbre de ligne et c le nbre de colonnes de la matrice. m est un tableau qui contient les coefficient de celle-ci.

Code :

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

matrice mamatrice;

mamatrice est donc un pointeur sur une matrice et non une matrice ?

Si c'est un pointeur alors une fonction qui créerai une matrice pourrait se coder ainsi ?

Code :

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

1
2
3
4
5
6
7
8
matrice creeMatrice(int  l ,  int c ){
 
    matrice mamatrice;
    mamatrice = malloc(sizeof(matrice));
    mamatrice->m = malloc( sizeof(int) * l * c);
    return mamatrice;
 
}

Donc cette fonction renverrai un pointeur de matrice.

Merci pour votre aide

[Obsidian]

Bonjour,

Oui, tu as compris. C'est tout-à-fait cela, à ceci près qu'il faudra utiliser sizeof(double) plutôt que sizeof(int). Et bien sûr, dans l'absolu, il faudrait aussi vérifier que les deux malloc() n'ont pas échoué.

On en profite pour signaler que masquer un pointeur dans un typedef est une très mauvaise pratique, même si elle est répandue dans les écoles et dans certains frameworks objet.

[guitz]

Merci à toi pour cette réponse ultra rapide

Si je crée une fonction qui supprime une matrice :

Code :

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

1
2
3
4
void detruitMatrice ( matrice m){
    free(m->m);
    free(m);
}

je suppose que le premier free est utile ? Car sinon seul le pointeur m est supprimé mais pas le tableau correspondant ?

[Obsidian]

Si je crée une fonction qui supprime une matrice :

Code :

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

1
2
3
4
void detruitMatrice ( matrice m){
    free(m->m);
    free(m);
}

je suppose que le premier free est utile ? Car sinon seul le pointeur m est supprimé mais pas le tableau correspondant ?

Absolument. Et c'est pour cela aussi qu'il est important de le faire dans cet ordre.

Si on veut être puriste, on met quand même « m->m » à NULL pour être sûr que le pointeur ne référence plus rien. A priori, ça ne sert à rien puisque le pointeur lui-même va être désalloué, mais c'est salvateur lorsque l'on débugue de gros projets et c'est pratique aussi lorsqu'on libère une telle structure en oubliant qu'elle est référencée ailleurs. Ça limite les dégâts au moins pour quelques temps.

[diogene]

Et en plus, masquer les pointeurs empêche que ce genre de faute ne saute aux yeux :

Code :

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

1
2
    matrice mamatrice;
    mamatrice = malloc(sizeof(matrice));

qui devrait être

Code :

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

1
2
3
4
   mamatrice = malloc(sizeof(struct matrice));
// ou
   mamatrice = malloc(sizeof *mamatrice);

[Obsidian]

Et en plus, masquer les pointeurs empêche que ce genre de faute ne saute aux yeux :

Code :

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

1
2
    matrice mamatrice;
    mamatrice = malloc(sizeof(matrice));

Oh, misère ! Je sors me cacher immédiatement.

[guitz]

Merci Obsidian et Dioxidedoxigene.