Discussion :

[triana]

bonjour,

c'est mon essai d'un programme qui calcule le min d'une matrice mais sa ne fonctionne pas
alors svp pouvez vous me dire ou est l'erreur!!

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
#include<stdio.h>
int mini(int*a[100],int*t)
{int i,j,n,m,min;
for(i=0;i<n;i++)
{min=*a[0];
 {for(j=0;j<m;j++)
 {if (min>*a[j])
min=*a[j];}
printf("le min de la ligne est:%d\n",min);}
t[i]=min;}
printf("la ligne des minimum est");
for(i=0;i<n;i++)
printf("%d",t[i]);}
int main()
/*declaration des variables*/
{int *a[100]; /*les dimensions de la matrice*/
int i/*indice pour parcourir les lignes*/,j/*indices pour parcourir les colonnes*/;
int n /*nbre de lignes de la matrice*/,m; /*nbre de colonnes de la matrices*/
int *t[50]/*vecteur de valeur minimale*/;
int min;
printf("entrer le nombre de ligne de la matrice");
scanf("%d",&n);
printf("entrer le nbre de colonne de la matrice");
scanf("%d",&m);
 
printf("entrer l'element a[i][j]");
for(i=0;i<n;i++)
for(j=0;j<m;j++)
scanf("%d",&a[i][j]);
*mini=*(mini(*a[i][j],*t[i]));
 
printf("la ligne des minimum est:",(mini(*a[j],*t[i]));
getchar();getchar();getchar();}

[Neckara]

Bonjour,

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
#include<stdio.h>
int mini(int*a[100],int*t)
{int i,j,n,m,min;
for(i=0;i<n;i++)
{min=*a[0];
 {for(j=0;j<m;j++)
 {if (min>*a[j])
min=*a[j];}
printf("le min de la ligne est:%d\n",min);}
t[i]=min;}
printf("la ligne des minimum est");
for(i=0;i<n;i++)
printf("%d",t[i]);}
int main()
/*declaration des variables*/
{int *a[100]; /*les dimensions de la matrice*/

Je ne lis pas un code aussi mal indenté.

De plus "sa ne fonctionne pas" veut tout et rien dire.
As-tu une erreur de segmentation, de compilation, de résultats ?
On ne peut pas t'aider si tu nous dis pas quel est ton problème et si tu n'indente pas ton code.

[triana]

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
#include<stdio.h>
int mini(int*a[100],int*t)
{int i,j,n,m,min;
for(i=0;i<n;i++)
  {min=*a[0];
     {for(j=0;j<m;j++)
         {if (min>*a[j])
min=*a[j];}
printf("le min de la ligne est:%d\n",min);}
t[i]=min;}
printf("la ligne des minimum est");
for(i=0;i<n;i++)
    printf("%d",t[i]);}
int main()
/*declaration des variables*/
{int *a[100]; /*les dimensions de la matrice*/
int i/*indice pour parcourir les lignes*/,j/*indices pour parcourir les colonnes*/;
int n /*nbre de lignes de la matrice*/,m; /*nbre de colonnes de la matrices*/
int *t[50]/*vecteur de valeur minimale*/;
int min;
printf("entrer le nombre de ligne de la matrice");
scanf("%d",&n);
printf("entrer le nbre de colonne de la matrice");
scanf("%d",&m);
 
printf("entrer l'element a[i][j]");
  for(i=0;i<n;i++)
  for(j=0;j<m;j++)
       scanf("%d",&a[i][j]);
*mini=*(mini(*a[i][j],*t[i]));
 
printf("la ligne des minimum est:",(mini(*a[j],*t[i]));
getchar();getchar();getchar();}

mercci pour votre remarque

mon probleme c'est que sa ne compile pas, il detecte une erreur lors de l'appel de la fonction

[gangsoleil]

Bonjour,

1/ Un code correctement indente, c'est ca :

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
 
#include<stdio.h>
 
int mini(int*a[100],int*t)
{
        int i,j,n,m,min;
 
        for(i=0;i<n;i++)
        {
                min=*a[0];
                {
                        for(j=0;j<m;j++)
                        {
                                if (min>*a[j])
                                        min=*a[j];
                        }
                        printf("le min de la ligne est:%d\n",min);
                }
                t[i]=min;
        }
        printf("la ligne des minimum est");
 
        for(i=0;i<n;i++)
                printf("%d",t[i]);
}
 
int main()
        /*declaration des variables*/
{
        int *a[100]; /*les dimensions de la matrice*/
        int i/*indice pour parcourir les lignes*/,j/*indices pour parcourir les colonnes*/;
        int n /*nbre de lignes de la matrice*/,m; /*nbre de colonnes de la matrices*/
        int *t[50]/*vecteur de valeur minimale*/;
        int min;
 
        printf("entrer le nombre de ligne de la matrice");
        scanf("%d",&n);
        printf("entrer le nbre de colonne de la matrice");
        scanf("%d",&m);
 
        printf("entrer l'element a[i][j]");
 
        for(i=0;i<n;i++)
                for(j=0;j<m;j++)
                        scanf("%d",&a[i][j]);
 
        *mini=*(mini(*a[i][j],*t[i]));
 
        printf("la ligne des minimum est:",(mini(*a[j],*t[i])) );
                        getchar();getchar();getchar();
}

2/ Il te manque au moins une parenthese a la fin du dernier printf

3/ Avec juste 3 options de compilation, tu t'apercois que tu as des soucis dans ton code :

Code :

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

1
2
3
4
5
6
7
8
9
/tmp> gcc -Wall -Wextra -O2 test.c
test.c: In function 'main':
test.c:45: error: invalid type argument of 'unary *'
test.c:45: warning: passing argument 2 of 'mini' makes pointer from integer without a cast
test.c:45: error: invalid type argument of 'unary *'
test.c:47: warning: passing argument 1 of 'mini' makes pointer from integer without a cast
test.c:47: warning: passing argument 2 of 'mini' makes pointer from integer without a cast
test.c:47: warning: too many arguments for format
test.c:32: warning: unused variable 'min'

4/ Il ne faut pas lire les entrees clavier avec scanf. Tu trouveras des tutoriels sur ce sujet, et sur le C en general, ici : http://c.developpez.com/cours

[pythéas]

Bonjour,

c'est gentil d'avoir ajouté quelques espaces pour faire plaîsir à Neckara mais ça n'a rien à voir avec une indentation correcte (du coup ça n'a pas du marché ) (édit : grillé par gangsoleil qui a fait l'effort de mettre le code en forme)
L'indentation doit rendre le code plus lisible (il existe de nombreux guides de bonnes pratiques sur la question et la plupart des éditeurs font ça pas trop mal) notament permettre de visualiser les différents blocs. N'oublie pas que tu es le(a) premier(e) lecteur(trice) de ton code, par exemple quand tu écris :

Code :

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

1
2
3
4
  for(i=0;i<n;i++)
  for(j=0;j<m;j++)
       scanf("%d",&a[i][j]);
*mini=*(mini(*a[i][j],*t[i]));

Es-tu conscient[e] que la dernière ligne ne sera appelée qu'une seulle fois, d'ailleurs puisqu'on parle de la dernière ligne tu écris "*mini=" alors qu'il n'y a pas de variable "mini" dans le code, quand au tableau "a" il est déclaré comme unidimensionnel, de toute façon même si il avait deux dimentions la fonction "mini" attend un tableau et pas une valeur dans un tableau, et puis n'as tu pas l'impression qu'il y a un peu trop de pointeurs?

Bref, essaye d'écrire ton code plus clairement mais surtout étape par étape : Fais un code qui lit les valeurs entrées par l'utilisateur, puis qui les lit et les mets dans une matrice, puis...
Pondre tout un programme d'un coup et chercher ce qui ne marche pas ne me paraît pas trés judicieux.

Si tu bloques sur quelque chose on sera ravis de t'aider

[ternel]

N'hésite pas à écrire des commentaires disant en français (ou en anglais) ce que va faire le code que tu vas écrire avant, cela permet de structurer plus facilement quand on apprend.