Lecture d'une matrice depuis un fichier

**benois123** · 21/10/2013, 08h35

Bonjour, je suis débutant en C et j'ai ce problème j'ai pas pu le fixer
Si quelqu'un pouvait m'aider je serais reconnaissant

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
78
79
80
81
82
83
84
85
86
87
88
89
 
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#define max(a,b)((a)>(b)?(a):(b))
 
typedef int Matrice [100][100];
typedef int rempduree[100][100];
 
void remplir_matrice_duree(rempduree red)
{
    int i,j,C[100];
    Matrice m;
    for(i=1;i<11;i++)
    { j=1;
     while ((m[i][j]==0)&(j<=i))
      {
	   j++;
      }
       C[i]=j;
    }
  for(j=1;j<11;j++)
   {
    for(i=j;i<11;i++)
     if(C[i]<=j)
       {
           if (i>j)
           {
            red[i][j]=4*(j-max(C[i],C[j]))+2;
           }
           else
            {
            //if (i==j)
            //red[j][j]==4*(j-C[j]);
            //printf("%d\n",red[j][j]);
            }
       }
    }
 
}
void ecrire(rempduree red)
{
    int i,j;
    for(j=1;j<11;j++)
   {
    for(i=j;i<11;i++)
        printf("%d",red[i][j]);
        printf("\n");
   }
 
}
int duree (int i, int j)
{int a;
    if(i>j)
    {
      a=rempduree[i][j];
      return a;
    }
    else
    if (i==j)
    {
     a=rempduree[j][j];
     return a;
    }
}
 
int main(void)
{
    Matrice A;
	int  m, n;
	rempduree redtab;
    /* ouvrir le fichier Matrices.dta sur le disque réseau pour la lecture */
	FILE * aLire = fopen("C......\\matrice.txt", "r") ;
	 if(aLire == NULL)
    {
       printf("Le fichier n'existe pas");
    }
	 else
 
	 {  fscanf(aLire,"%d %d\n", &m, &n);
            lire (aLire, A, m, n);
            remplir_matrice_duree(redtab);
            ecrire(redtab);
	 }
 
 
 
return 0;
}

voila l'erreur lors de la compilation

In function remplir_matrice_duree
33 statement with no effect
In function 'duree':
55 erreur: expected expression before 'rempduree'
61 erreur: expected expression before 'rempduree'
In function 'main':
80 attention: implicit declaration of function 'lire'
In function 'duree':
64 contrôle a atteint la fin non void de la fonction

**Franck.H** · 21/10/2013, 09h03

33 statement with no effect

Code :

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

1
2
3
4
5
6
           else
            {
            //if (i==j)
            //red[j][j]==4*(j-C[j]);
            //printf("%d\n",red[j][j]);
            }

Bin peut être parce que tu as un else qui ne sert à rien

55 erreur: expected expression before 'rempduree'
61 erreur: expected expression before 'rempduree'

Je vois que ta fonction retourne void mais tu cherches tout de même à affecter une valeur à a en retour de la fonction donc le compilateur ne comprend pas trop ce que tu veux faire.

80 attention: implicit declaration of function 'lire'

Elle est où ta fonction lire ?

64 contrôle a atteint la fin non void de la fonction

Ta fonction duree retourne un int, tu retournes des valeurs dans les if (ce qui n'est pas vraiment une bonne pratique) mais en fin de fonction il n'y a pas de return donc fait-le à la fin de ta fonction et non dans tes conditions

Voilà voilà

Invité · 21/10/2013, 14h59

Bonjour,

Envoyé par Franck.H

Bin peut être parce que tu as un else qui ne sert à rien

J'ai plutôt tendance à dire que c'est la ligne commentée :

Code :

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

red[j][j]==4*(j-C[j]);

qui a un '=' en trop. On fait donc une comparaison dont on ne récupère pas le résultat, plutôt qu'une assignation.

Envoyé par Franck.H

Je vois que ta fonction retourne void mais tu cherches tout de même à affecter une valeur à a en retour de la fonction donc le compilateur ne comprend pas trop ce que tu veux faire

Non la fonction retourne bien un int, le problème vient du fait que rempduree :

Code :

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

typedef int rempduree[100][100];

n'est pas une variable mais un type, tout comme Matrice, je présume que tu voulais plutôt écrire Matrice rempduree; ? (certains rempduree seront à remplacer par Matrice dans ton code actuel).

Remarque : tu n'as pas besoin de la variable temporaire a dans ta fonction duree() : return rempduree[i][j]; directement.

Fonction ecrire() : attention à l'indentation :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void ecrire(rempduree red)
{
   int i,j;
   for(j=1;j<11;j++)
   {
        for(i=j;i<11;i++)
            printf("%d",red[i][j]);
        printf("\n"); /* n'est pas inclus dans le for précédent */
   }
 
}

**benois123** · 21/10/2013, 15h16

Merci pour vos réponses
J'ai en fait un autre question
pour la fonction durée je veux utiliser le tableau red retourné par la fonction remplir_matrice_duree mais je ne veux pas faire passé le tableau red en paramètre de la fonction durée.
Y'a t-il une autre solution ?? veuillez m'expliquer s'il vous plait ?? je serai reconnaissant .
Voici le 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
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
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
 
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#define max(a,b)((a)>(b)?(a):(b))
 
typedef int Matrice [100][100];
typedef int rempduree[100][100];
rempduree red;
void lire(FILE * aLire, Matrice mat)
 {
		  int i, j ;
		  for (i = 1; i <11; i++)
            {
			  for (j =1 ; j <11; j++)
               fscanf(aLire, "%d", & mat[i] [j]);
			   fscanf(aLire, "\n");
		    }
}
 
void remplir_matrice_duree(rempduree red,Matrice m)
{
    int i,j,C[100];
    int nb;
    printf("matrice donnee\n");
    for (i = 1 ; i <11; i++)
    {
       for (j = 1;j<11; j++)
 
           printf("%5.2d", m[i][j]);
           printf("\n");
    }
    //printf("salut\n");
    for(i=1;i<11;i++)
    { j=1;
     while ((m[i][j]==0)&(j<=i))
      {
	   j++;
      }
       C[i]=j;
    }
    for(i=1;i<11;i++)
    {
        printf("%d\n",C[i]);
    }
  for(j=1;j<11;j++)
   {
    for(i=j;i<11;i++)
     if(C[i]<=j)
       {
           if (i>j)
           {
            nb=4*(j-max(C[i],C[j]))+2;
            red[i][j]=nb;
           }
           else
            {
            if (i==j)
            nb=4*(j-C[j]);
            red[j][j]=nb;
            //printf("%d\n",red[j][j]);
            }
       }
    }
    for (i = 1 ; i <11; i++)
    {
       for (j = 1;j<11; j++)
 
           printf("%5.2d", red[i][j]);
           printf("\n");
    }
 
}
void ecrire(rempduree red)
{
    int i,j;
    for(j=1;j<12;j++)
   {
    for(i=j;i<12;i++)
        printf("%d",red[i][j]);
        printf("\n");
   }
 
}
void writeMatrixtoFile(Matrice mat)
{
    int i,j;
    for (i = 1 ; i <11; i++)
    {
       for (j = 1;j<11; j++)
 
           printf("%5.2d", mat[i][j]);
           printf("\n");
    }
 
}
int duree (int i, int j)
{
    int a;
    rempduree red;
    if(i>j)
    {
     a=red[i][j];
    }
    else
    if (i==j)
    {
     a=red[j][j];
    }
    return a;
}
 
int main(void)
{
    Matrice A;
	int  m, n;
	int d;
	rempduree redtab;
    /* ouvrir le fichier test sur le disque réseau pour la lecture */
	FILE * aLire = fopen("C: ......\\test.txt", "r") ;
	 if(aLire == NULL)
    {
       printf("Le fichier n'existe pas");
    }
	 else
	 {
	        fscanf(aLire,"%d %d\n", &m, &n);
            lire (aLire, A);
            writeMatrixtoFile(A);
            remplir_matrice_duree(redtab,A);
            d=duree(9,9);
            printf("valeur de d est %d\n",d);
//            ecrire(redtab);
	 }
 
return 0;
}

ma solution ici quand je passe red comme un paramètre local ne me donne pas
la durée exacte, pas de valeur .

Invité · 21/10/2013, 15h20

Pourquoi ne pas vouloir faire passer ta matrice en paramètre ?
C'est pourtant la seule solution hormis l'utilisation d'une variable globale (ici rempduree red; ligne 9) qui est déconseillée.

**Franck.H** · 21/10/2013, 15h21

J'avoue... j'ai un peu biglé sur quelques lignes, pas fait trop attention mais au travail je regarde entre deux ligne de code VB.Net ou C#

**benois123** · 21/10/2013, 20h32

Merci pour vos réponses
en fait, Winjerome je veux pas passer le tableau en paramètre car je vais appelé la fonction durée plusieurs fois, donc vaut mieux sans tableau, les indices i et j seulement.

Invité · 21/10/2013, 20h41

Si tu te soucies des performances, c'est juste l'adresse qui est passée

Comment passer un tableau en paramètre à une fonction ?

**benois123** · 21/10/2013, 20h54

J'ai pas bien compris est ce que tu peux mieux m'expliqué ??

Invité · 21/10/2013, 20h56

Envoyé par benois123

je veux pas passer le tableau en paramètre car je vais appelé la fonction durée plusieurs fois, donc vaut mieux sans tableau, les indices i et j seulement.

C'est bien parce que tu penses que cela va copier entièrement ton tableau à chaque appel ? => Ce qui n'est pas le cas.
Ou je n'ai pas compris pourquoi...

**benois123** · 07/11/2013, 20h28

Salut a tous,
en fait j'ai un autre problème, je veux lire un grand fichier de taille 100*100
et avec les modification apporté au programme concernant la taille de fichier
la compilation se passe bien , le problème c'est que lors de l'exécution le programme se bloque il affiche une fenetre qui dit
nomdepg a céssé de fonctionner
pui quand j'appuie sur un boutton il affiche dans le console comme ça

Code :

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

1
2
3
 
Process returned -1073741819 (0xC0000005)   execution time : 28.387 s
Press any key to continue.

voici le code après modification

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
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
 
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#define max(a,b)((a)>(b)?(a):(b))
 
typedef int Matrice [100][100];
typedef int rempduree[100][100];
 
void lire(FILE * aLire, Matrice mat)
 {
		  int i, j ;
		  for (i = 1; i <101; i++)
            {
			  for (j =1 ; j <101; j++)
               fscanf(aLire, "%d", & mat[i] [j]);
			   fscanf(aLire, "\n");
		    }
}
 
void remplir_matrice_duree(rempduree red,Matrice m)
{
    int i,j,C[100];
    int nb;
    printf("matrice donnee\n");
    for (i = 1 ; i <101; i++)
    {
       for (j = 1;j<101; j++)
 
           printf("%5.2d", m[i][j]);
           printf("\n");
    }
    //printf("salut\n");
    for(i=1;i<101;i++)
    { j=1;
     while ((m[i][j]==0)&(j<=i))
      {
	   j++;
      }
       C[i]=j;
    }
    for(i=1;i<101;i++)
    {
        printf("%d\n",C[i]);
    }
  for(j=1;j<101;j++)
   {
    for(i=j;i<101;i++)
     if(C[i]<=j)
       {
           if (i>j)
           {
            nb=4*(j-max(C[i],C[j]))+2;
            red[i][j]=nb;
           }
           else
            {
            if (i==j)
            nb=4*(j-C[j]);
            red[j][j]=nb;
            //printf("%d\n",red[j][j]);
            }
       }
    }
    for (i = 1 ; i <101; i++)
    {
       for (j = 1;j<101; j++)
 
           printf("%5.2d", red[i][j]);
           printf("\n");
    }
 
}
 
void writeMatrixtoFile(Matrice mat)
{
    int i,j;
    for (i = 1 ; i <101; i++)
    {
       for (j = 1;j<101; j++)
 
           printf("%5.2d", mat[i][j]);
           printf("\n");
    }
 
}
int temps(int i, int j, rempduree red)
{
    int a;    
    if(i>j)
    {
     a=red[i][j];
    }
    else
    if (i==j)
    {
     a=red[j][j];
    }
    return a;
}
int nbtlibre (rempduree red)
{
    int i,j,nblib=0;
    for (i = 1 ; i <101; i++)
    {
       for (j = 1;j<101; j++)
           if(red [i][j]!=0)
           {
               if(red[j][j]==0 && red[i][j-1]==0)
               {
                 printf("valeur de red[%d][%d]=%d\n",i,j,red[i][j]);
                 nblib++;
                }
           }
    }
    return nblib;
}
int main(void)
{
    Matrice A;
	int  m, n;
	int d;
	rempduree redtab;
    /* ouvrir le fichier test sur le disque réseau pour la lecture */
	FILE * aLire = fopen(".........", "r");
	 if(aLire == NULL)
    {
       printf("Le fichier n'existe pas");
    }
	 else
	 {
	        fscanf(aLire,"%d %d\n", &m, &n);
            lire (aLire, A);
            writeMatrixtoFile(A);
            remplir_matrice_duree(redtab,A);
            d=temps(10,10,redtab);
            printf("valeur de d est %d\n",d);
 
	 }
return 0;
}

Est ce quelqu'un peut m'aider je serai très reconnaisssant

Invité · 07/11/2013, 20h57

Tu as défini ton type Matrice comme étant un tableau int [100][100], tes boucles for sont fausses :

Code :

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

1
2
3
for (i = 0 ; i <100 ; i++)
{
    for (j = 0 ; j <100 ; j++)

N'oublie pas que les indices en C commencent à 0 et vont jusqu'à N-1.

Je vois ensuite que tu sembles lire la taille des données de ton fichier avec la ligne

Code :

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

fscanf(aLire,"%d %d\n", &m, &n);

Mais tu ne sembles pas les utiliser par la suite ?

**benois123** · 08/11/2013, 14h14

merci Winjerome
En cas ou je veux débuter par 1 j'essais ça en faisant une variable globale pour être utilisé dans tout le programme
les modifications sont apportés ci-dessous

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
 
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#define max(a,b)((a)>(b)?(a):(b))
#define n 51
 
typedef int Matrice [n][n];
typedef int rempduree[n][n];
 
void lire(FILE * aLire, Matrice mat)
 {
		  int i, j ;
		  for (i =1; i <n+1; i++)
            {
			  for (j =1 ; j <n+1; j++)
 
		    }
}

avec la taille de la matrice 50*50
le programme plante, il affiche seulement une fenêtre noire
j'ai pas pu savoir où est le probléme

**Bktero** · 08/11/2013, 15h05

Plusieurs techniques pour savoir où ça marche pas :

mettre des traces sur la console
utiliser un débogueur pour executer en mode pas à pas pour voir ou ça plante
utiliser des outils d'analyse mémoire comme valgrind ou drmemory

Voir :
http://www.developpez.net/forums/d44...debugger-code/
http://www.developpez.net/forums/d12...ows-and-linux/

Invité · 08/11/2013, 15h47

As-tu compris ceci ou pas ?

Envoyé par Winjerome

N'oublie pas que les indices en C commencent à 0 et vont jusqu'à N-1.

Ton code montre le contraire en tout cas.

Je réexplique donc :
Tu as : typedef int Matrice [n][n]; : une matrice de n x n éléments (ou 51x51 ici).

Pour parcourir en C un tel tableau, on le fait de 0 à n-1 selon chaque dimension
Autrement dit ici les indices 0, 1, 2, ... 49, 50, et PAS 1, 2, ..., 50, 51.

Pour Matrice mat; on a :

Code :

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

1
2
3
4
5
6
7
8
// 1ère colonne   2ème colonne         (n-1)ème colonne    n-ième colonne
    mat[0][0]      mat[0][1]     . . .   mat[0][n-2]         mat[0][n-1]   // première ligne 
    mat[1][0]      mat[1][1]     . . .   mat[1][n-2]         mat[1][n-1]   // deuxième ligne 
         .             .                      .                  .
         .             .                      .                  .
         .             .                      .                  .
  mat[n-2][0]     mat[n-2][1]    . . .   mat[n-2][n-2]      mat[n-2][n-1]  // (n-1)ème ligne 
  mat[n-1][0]     mat[n-1][1]    . . .   mat[n-1][n-2]      mat[n-1][n-1]  // n-ième ligne

Soit pour n=51 :

Code :

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

1
2
3
4
5
6
7
 mat[0][0]  mat[0][1]  . . . mat[0][49]    mat[0][50]
 mat[1][0]  mat[1][1]  . . . mat[1][49]    mat[1][50]
       .        .                .             .
       .        .                .             .
       .        .                .             .
mat[49][0] mat[48][1] . . . mat[49][49]  mat[49][50]
mat[50][0] mat[50][1] . . . mat[50][49]  mat[50][50]

Bien compris cette fois-ci ?
Tenter d'accéder à la colonne mat[i][n] ou ligne mat[n][j] est une erreur ! Tu dépasses la taille de ton tableau.

Lecture d'une matrice depuis un fichier

C

Discussions similaires

Partager

Partager