Discussion :

[Personne.c]

Bonjour,

J'ai un petit problème lorsque que je veux afficher une matrice; la taille est reçue en paramètre de la fonction, fonction qui sert à créer puis à initialiser toutes les cases à zéro.
Je fais appel à la fonction à partir du main et j'envoie des variables de type int contenues dans une structure.
le fichier compile sans erreur mais m'affiche un "segmentation fault" lors de l'execution.
j'ai fais plusieurs tests avec printf mais je n'arrive pas à trouver ou est mon erreur (qui est surement idiote)...

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
int     **creer_matrice(int col, int line)
 88 {
 89     int i;
 90     int j;
 91     int **tab = malloc(sizeof(int) * ((line + 1) * (col + 1)));
 92     printf("val de line : %d\n", line); 
 93     printf("val de col : %d\n", col);
 94         
 95     i = 0;
 96     j = 0;
 97     while(i < col)
 98     {
 99         while (j < line)
100         {
101             tab[i][j] = 0;
102             printf("%d", tab[i][j]);
103             j++;
104         }   
105         printf("\n");
106         i++;  
              j = 0;
107     }   
108     return (tab);
109 }

[ternel]

Code :

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

int **tab = malloc(sizeof(int) * ((line + 1) * (col + 1)));

ceci déclare un double pointeur de int, mais ne lui donne qu'une adresse obtenue pour un pointeur simple.

il faut choisir.
et pourquoi avoir des +1?

il te faudrait idéalement des fonctions telles que:

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
 
int* matrix_create(int cols, int lines, int cols) {
    return malloc(sizeof(int) * lines * cols);
}
 
void matrix_create(int *m) {
    return free(m);
}
 
int matrix_get(int const* m, int cols, int line, int col) {
    return m[line*cols+col];
}
 
int* matrix_get_ref(int const* m, int cols, int line, int col) {
    return m+(line*cols+col); /* ou &m[line*cols+col] */
}

[Personne.c]

Il faudrait donc que je cast mon malloc pour lui indiquer que c'est un double pointeur ?

Code :

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

1
2
int **tab;
tab = (int**)malloc(sizeof(int) * ((line + 1) * (col + 1)));

Les "+1" c'est juste pour créer une matrice avec une taille plus grande que celle qui mesure (line, col). Ce n'est pas clair en effet, je vais les retirer et les mettre directement lorsque j'appelle la fonction.


J'aimerais une matrice à deux dimensions, les fonctions que tu me proposes me semble servir uniquement pour une seule dimension.




Merci pour ton aide.

[neuneutrinos]

Ça dépend comment tu vois ton tableau à 2 dimensions.

Code :

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

1
2
3
4
5
6
7
8
 
//Je décide de tout stocker dans une ligne ma matrice à deux dimension.
//la case [x][y] aura comme indice [y*TAILLE_X+x]
 
int* creerMatrice(int tailleX,int tailleY)
{
 return calloc(tailleX*tailleY,sizeof(int));//un calloc remplit la zone alloué de 0, ou un malloc et une boucle de remplisage
}

Dans ce code, je n'ai pas fais de '+1' car ça n'a pas trop de sens dans cette fonction, autant incrémenter dans les paramètres de la fonction !

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
//Je décide d'utiliser 2 dimensions
int ** creerMatrice(int tailleX,int tailleY)
{
 int i;
 //int** -> tableau de int* et int* ->tableau de int ==> int** tableau de int*int
 int ** mat=malloc(sizeof(int*)*tailleX);//tu peux echanger tailleX et tailleY si tu le souhaite ça reste une matrice :p, cependant il faut en tenir compte au moment de la lecture
 for(i=0;i<tailleX;i++)//si tu n'as pas le droit au boucle for, tu t'adapte ;)
 {
  mat[i]=calloc(tailleY,sizeof(int));
 }
 return mat;
}

EDIT:
Tu peux même créer un type Matrice contenant la matrice et ses dimensions !

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
typedef struct
{
 int tailleX,tailleY;
 int** mat;//pour le cas où on choisit de le faire à 2 dimensions ;)
}Matrice;
 
//et les fonction
Matrice* creerMatrice(int tailleX,tailleY);
void libererMatrice(Matrice* m);
void afficherConsoleMatrice(const Matrice* m);

[Personne.c]

Je comprends mieux. Disons que je vois un tableau à deux dimensions comme un tableau avec la première ligne contenant des pointeurs vers chaque colonnes, ça me permet de mieux visualiser (mentalement) le fonctionnement. Mais ce n'est peut être pas le cas pour tout le monde.

Du coup ton second code me parait beaucoup plus clair que le premier (que j'essayerais de comprendre également) et me permet de comprendre beaucoup plus facilement.
il faut donc malloc la ligne contenant les pointeurs vers les colonnes puis malloc chaque colonne. C'est génial comme idée.


Edit : je n'ai besoin d'utiliser une seule matrice (une seule fois), donc je ne pense pas que ce soit nécessaire de faire une structure (je vais quand même y réflechir) mais merci pour l'idée.

Merci encore pour votre aide, encore quelques mois et j'espère avoir le niveau nécessaire pour vous aider en cas de problème.

[ternel]

Bien sur que si, tu as besoin d'une structure.
Une structure permet de clarifier le code, de réunir ce qui va ensemble.
Et même d'apporter des garanties d'intégrité, surtout si la forme de la structure n'est pas donnée dans le .h qui définit les fonctions l'utilisant, mais seulement dans le .c. C'est comme cela que fonctionne la structure FILE, dont aucun développeur n'a a savoir ce qu'ell contient précisément.

Nous serons heureux de ton aide!

[Personne.c]

Effectivement, créer une structure me permettra d'utiliser les fonctions manipulant les matrices beaucoup plus facilement, notamment pour free la mémoire utilisée (ce qui réglera surement mon problème de "segmentation fault" qui s'affiche une fois sur x, lorsque je lance mon programme plusieurs fois de suite).
Vous m'avez convaincu, merci encore.