Discussion :

[space_56]

bonjour,
j'ai un problème avec ce programme, je voudrais remplir une matrice et l'afficher.
est-ce que quelqu'un pourrait corriger ce code pour qu'il fonctionne, svp ?
merci.

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
30
31
32
33
34
35
36
37
38
39
 
#include <stdio.h>
#include <stdlib.h>
 
int a, b;
int i,j;     
 
int remplir_mat(int *l, int *c)
{
     int **mat;
     for(i=0;i<*l;i++)
     {for(j=0;j<*c;j++)
     {printf("ligne %d, colonne %d : ", i+1, j+1);
     scanf("%d", &mat[i][j]);
     }
     }
     return mat;
}
 
void affiche_mat(int **mat, int l, int c)
{
 
     for(i=0;i<l;i++){
     for(j=0;j<c;j++)
     { printf("%d ", &mat[i][j]);
     }
     printf("\n");
     }
}
 
int main()
{
    int mat[10][10];
    N=2;
    M=2;
     printf("%d %d", a, b);
    remplir_mat(&a, &b);
    affiche_mat(mat, a, b);
}

[Emmanuel Delahaye]

j'ai un problème avec ce programme, je voudrais remplir une matrice et l'afficher.

Beaucoup de fautes dans ce code... Tu devrais déjà revoir comment on passe un tableau à 2 dimensions à une fonction :

http://emmanuel-delahaye.developpez.....htm#param_tab

est-ce que quelqu'un pourrait corriger ce code pour qu'il fonctionne, svp ?

Euh, non. Ca, c'est ton boulot... Commence par supprimer ces globales inutiles et par bien configurer ton compilateur :

http://emmanuel-delahaye.developpez....tm#cfg_compilo

[space_56]

tu peux me dire où sont les erreurs de ligne, stp ?
dans ce code les tableaux sont-ils bien transmis ?

[Emmanuel Delahaye]

tu peux me dire où sont les erreurs de ligne, stp ?
dans ce code les tableaux sont-ils bien transmis ?

Tu ne vois pas que int **mat n'a aucun sens ici ? Il faut utiliser ce que j'ai indiqué, c'est à dire int mat[10][10] ou int mat[][10] et ne pas oublier de passer les dimensions aux fonctions.

[Nykoo]

Je m'étais amusé à gérer des matrices avec un programme c. On m'avait aidé sur HFR. Je peux toujours poster le code auquel j'étais arrivé, ça pourra toujours aider. Mais je sais pas si c'est dans l'esprit du forum. Donc si ça peut t'aider space_56

[space_56]

je ne comprends pas, ça ne marche toujours pas.

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
30
31
32
33
34
35
36
37
38
39
40
41
42
 
#include <stdio.h>
#include <stdlib.h>
 
int N, M;
int i,j;     
 
int remplir_matrice(int *l, int *c)
{
     int mat[10][10];
     /*scanf("%d %d", l, c);
     printf("%d %d", *l, *c);*/
     //mat = (int**) malloc((*l) * sizeof(int*)); 
     for(i=0;i<*l;i++)
     {for(j=0;j<*c;j++)
     {printf("ligne %d, colonne %d : ", i+1, j+1);
     scanf("%d", &mat[i][j]);
     }
     }
     return mat;
}
 
void affiche_mat(int mat[][], int l, int c)
{
 
     for(i=0;i<l;i++){
     for(j=0;j<c;j++)
     { printf("%d ", 1);
     }
     printf("\n");
     }
}
 
int main()
{
    int mat[10][10];
    N=2;
    M=2;
    printf("%d %d", N, M);
    remplir_matrice(&N, &M);
    affiche_mat(mat, N, M);
}

[Nykoo]

Pourquoi tu déclares tes variables N,M,i,j en globale? Surtout i, j c'est bizzare.

Ta fonction remplir_matrice doit renvoyer un int alors que tu renvoies un pointeur mat. Il faut donc rajouter un *après le int du prototype de ta fonction pour lui indiquer qu'elle doit renvoyer un pointeur vers un int et non pas un int.

Pourquoi tu passes tes paramètres (N,M) par pointeurs?

[Thierry Chappuis]

je ne comprends pas, ça ne marche toujours pas.

Parce que tu codes au hasard. Premièrement, tu devrais enlever tes globales. Puis, dans ta fonction remplir_matrice(), mat est tableau de 10 tableaux de 10 ints (ou plus simplement dit une matrice de 10 x 10 ints). Tu ne peux pas retourner mat comme ça, et pour deux raisons.

1. Le type de retour est faux! Ce n'est pas int, mais un pointeur sur un tableau de 10 ints. Ce qui nous donnerait le prototype suivant:
   

   Code :

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

   1
   2
    
   int (*(remplir_matrice(int *l, int *c)))[10];

   cryptique, non?
   
2. mat est un tableau local à ta fonction. La mémoire allouée est libérée dès la sortie de la fonction. Ainsi, l'adresse retournée par la fonction devient aussitôt invalide.

De plus, le prototype de affiche_mat() doit être:

Code :

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

void affiche_mat(int mat[][10], int l, int c);

où mat est de type pointeur sur tableau de 10 ints.

Thierry