probleme initialisation matrice

**sneb5757** · 18/11/2006, 15h59

bonjour j ai un probleme d initiation de matrice.Je vous explique le but est de faire saisir à l utilisateur une matrice en console.Pendant qu il saisit on affiche la matrice qui prend progressivement les valeur qu il entre(ainsi l utilisateur sait ou il entre sa valeur).La matrice est initialisé à 0 . le problème est que lorsque j affiche la matrice initialisé à 0 le programme m affiche n importe quoi.Voici les deux fichier le matrice.h qui contient les operations sur les matrices et le saisi.c qui execute ces fonction.SI vous voulez plus de details precisez moi mais je pense etre clair:

matrice.h:

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
 
#include <stdio.h>
#include <stdlib.h>
#define NB_ELEM 20
 
struct  matrice
{
       int tab[NB_ELEM][NB_ELEM];
       int nbl;
       int nbc;
};
 
typedef struct matrice matrice;
 
 
 
 
 
 void init_mat(matrice *mat)
 {   
        int i,j,l,c;
 
    printf("saisir le nombre de ligne\n");
     scanf("%d",&l);
     printf("saisir le nombre de colonne\n");
     scanf("%d",&c);
      printf("%d\n",l);
      (*mat).nbl=l;
      (*mat).nbc=c;
 
     for(i=0;i<(*mat).nbl;i++)
     {
 
 
                     for (j=0;j<(*mat).nbc;j++)
                     {
 
                      (*mat).tab[i][j]=0;
                      }
 
     }
 
 
}
 
void affiche_mat(matrice * mat)
{
     int i,j;
 
 
                 for(i=0;i<(*mat).nbl;i++)
 
     {
                     for (j=0;j<(*mat).nbc;j++)
                     {
                      printf("%d",mat->tab[i][j]);
                      }
                      printf("\n");
     } 
}
 
 void saisir_mat(matrice  *mat)
{
int i,j;
for(i=0;i<(*mat).nbl;i++)
{
                  for (j=0;j<(*mat).nbc;j++)
                     {
                      scanf("%d",&(*mat).tab[i][j]);
                      printf("à saisi");
                      system("cls");
                      affiche_mat(mat);
 
                      }                      
 
}
}

saisi.c

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
 
#include <stdio.h>
#include <stdlib.h>
#define NB_ELEM 20
 
#include "matrice.h"
 
 
 
 
main()
{
matrice  mat;
     init_mat(&mat);
     printf("fin init\n");
     system("pause");
     affiche_mat(&mat);
     saisir_mat(&mat);
     affiche_mat(&mat);
     system("pause");
 
}

**kamouminator** · 18/11/2006, 16h24

il faudrai passer un pointeur sur mat aux fonctions qui modifient la matrice !
dans ton cas, elles font une copie de mat et modifient la copie !

**sneb5757** · 18/11/2006, 16h29

ah zut j ai deja bouffé plein de tuto sur les pointeur et je comprend pas trop je fais comment?

En C++ je sais qu il faut mettre un & devant les variable modifiable mais la ça a pas l air de marcher:-(

**kamouminator** · 18/11/2006, 16h35

oui:

Code :

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

init_mat(&mat);

mais il faut changer la fonction d'initialisation en concequence....
dans la declaration deja:

Code :

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

matrice * mat

et à chaque utilisation de mat, tu remplace mat par (*mat) ou alors remplacer mat. par mat->

tu pe aussi le faire pour les autres fonctions, c'est meme mieux, y'aura mon de copies

mais revoies les poiteurs !!!! si tu veux apprendre le C, c'est indispensable de BIEN comprendre leur fonctionnement...

**sneb5757** · 18/11/2006, 16h45

voila j ai fait les modifes mais gros plantage la je crois que j ai pas trop compris.j ai fais les modifs dansd le premiers post.

**Emmanuel Delahaye** · 18/11/2006, 16h49

Envoyé par sneb5757

matrice.h:

Code :

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

1
2
3
4
 
 void init_mat(matrice *mat)
 {   
        int i,j,l,c;

On ne met pas de définition d'objets ni de code (sauf inline) dans un header. Jamais.

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

En plus, si c'est avoir de genre d'incohérences

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

1
2
3
4
5
6
7
#include <stdio.h> #include <stdlib.h> #define NB_ELEM 3 #include <stdio.h> #include <stdlib.h> #define NB_ELEM 20

ça craint vraiment...

Il faudrait aussi corriger ça :

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
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums2\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:71: warning: return type defaults to `int'
main.c:71: warning: function declaration isn't a prototype
main.c: In function `main':
main.c:72: warning: unused variable `i'
main.c:72: warning: unused variable `j'
main.c:88: warning: control reaches end of non-void function
Linking console executable: C:\dev\forums2\console.exe
Process terminated with status 0 (0 minutes, 6 seconds)
0 errors, 5 warnings

Pour le reste, c'est simple. Pour qu'une fonction puisse modifier un objet, il faut lui passer son adresse via un pointeur du même type. C'est tout.

Code :

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

1
2
3
4
 
   T x;
 
   f (&x);

avec

Code :

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

f (T *p);

Ah, j'avais pas vu encore vu le main(). Tu veux une taille programmable ?. OK. Il faut utiliser malloc(). C'est une FAQ (tableau dynamique à 2 dimensions). La structure devient (je recommande size_t pour cet usage) :

http://emmanuel-delahaye.developpez....tes.htm#size_t

Code :

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

1
2
3
4
5
6
struct matrice
{
   int **tab;
   int nbl;
   int nbc;
};

Fait de ton mieux et poste ton code si tu es en panne.

**kamouminator** · 18/11/2006, 16h51

dans ta fonction saisir mat....
c'est pas

Code :

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

affiche_mat(&mat);

mais:

Code :

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

affiche_mat(mat);

vu que tu a passé en argument de saisir mat un ponteur sur mat....
ecoutes, tu ferais mieux de revoir tes cours sur les pointeurs avant hein !

tu as aussi declaré deux NB_ELEM deifferents !

**sneb5757** · 18/11/2006, 16h59

pour le header j ai effectivement eu un doute mais sur ce forum on m avait donné le lien d un projet et le mec avait mis son code dans un header donc je pensais qu on avait "droit".

L incoherence a été vu est modifié mais j ai pas pensé a corrigé sur le forum

**Emmanuel Delahaye** · 18/11/2006, 17h03

Envoyé par sneb5757

pour le header j ai effectivement eu un doute mais sur ce forum on m avait donné le lien d un projet et le mec avait mis son code dans un header donc je pensais qu on avait "droit".

Redonne le lien...

C'est techniquement possible, mais çäy mal.

**sneb5757** · 18/11/2006, 17h17

vla le lien

http://www.developpez.net/forums/sho...d.php?t=235691

j ai trouvé la ligne qui me fait planter:

Code :

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

1
2
3
4
 
 
 (*mat).nbl=l;
  (*mat).nbc=c;

j essaie de la corriger avec ce que tu m as dis j ai mis le malloc j ai encore quelque probleme je cherche et je reposte

**kamouminator** · 18/11/2006, 17h27

tu melanges, c'est dans les declarations de fonction que je t'ai dis de mettre
matrice * mat
pas dans le main !
en le declarant ainsi dans le main, tu n'as pas besoin de mettre le & !

avant d'utiliser les pointeurs il faut les comprendre, sinon tu risque de faire de grosses betises !
un petit rappel:

Code :

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

matrice * mat

signifie: mat est un poiteur sur la structure matrice, le pointeur contien donc l'adresse de la matrice...

Code :

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

&mat

signifie qu'on recupere l'adresse de mat

Code :

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

*mat

signifie qu'on recupere ce qu'il y a dans l'adresse contenue dans le pointeur...

**sneb5757** · 18/11/2006, 17h46

le pire c est que je sais a quoi correspond chacune de ces notation le fait est que c est surtout un manque de pratique.

j ai modifié le code je garde le tableau statique(je ferai le tableau dynamique apres avoir fait avec tableau statique j avance petit à petit).Cette fois plus d erreur mais pas pas d affichage de matrice

**kamouminator** · 18/11/2006, 17h47

montre ce que tu as ecris !

**sneb5757** · 18/11/2006, 17h49

j ai modifié mon premier post .

**kamouminator** · 18/11/2006, 17h57

dans init mat, pourquoi tu mes le nombre de colones dand c et le nombre le ligne dans l ???
c'est dans (*mat).nbc et (*mat).nbl qu'il faut les mettre ! vu que c'est eux que tu utilise comme condition dans tes boucles for

ou alors dans les for

tu met for(i=0;i<l;i++)
et for(j=0;j<c;j++)

**kamouminator** · 18/11/2006, 18h02

d'ailleur tu as fais une erreure ici:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for(i=0;i<(*mat).nbl;i++)
     {
 
 
                     for (j=0;(*mat).nbl;j++)
                     {
 
                      (*mat).tab[i][j]=0;
                      }
 
     }

si tu veux utiliser nbl et nbc, dans la deuxieme boucle il faut mettre nbc et pas nbl !!et il faudra mettre avant:

Code :

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

1
2
(*mat).nbl=l;
(*mat).nbc=c;

**sneb5757** · 18/11/2006, 18h04

ah vi pourtant j ai relu deux fois pour voir si c etait pas une erreur conne de ce style.Si j ai pas capté ça c est qu il est temps de faire une petite pause parce que la je vais accumuler les conneries

en plus dans le saisi_mat j avais oublié un "&" dans le scanf.Merci beaucoup en tout cas à toi et emmanuel
je commence a comprendre un peu mieux mais je vais faire moult exercices sur les pointeurs

**kamouminator** · 18/11/2006, 18h08

ps: un main c se declare comme ca:

int main(void)

avec un return 0;
à la fin !

et...bonnes revisions ^^

**Emmanuel Delahaye** · 18/11/2006, 18h09

Envoyé par sneb5757

je commence a comprendre un peu mieux mais je vais faire moult exercices sur les pointeurs

Commence par des choses plus simples que les tableaux à 2 dimensions. Essaye déjà avec une... Une fois que tu auras compris certains principes, tu pourras envisager des choses plus complexes...

**sneb5757** · 18/11/2006, 18h15

ah je ne savais pas qu un main se declarait comme ça à liut ils nous l ont appris comme je l ai ecris.

Bein j ai pas trop le choix pour els tableaux a 2 dimension c est pour mon projet.

Vi je sais si tu regarde mon code de la premier page dans la fonction affichage j ai utilisé "->"

probleme initialisation matrice

C

Discussions similaires

Partager

Partager