procédure Saisie matrice

Version imprimable

Voir 40 message(s) de cette discussion en une page

Bonjour à tous

je suis nouvelle dans ce domaine de développement en C, je veux écrire une procédure Saisie qui permet la saisie d’une matrice d’entiers V de taille M x N. M et N sont inférieurs ou égales à une constante donnée MAX = 100

j'ai trouvé deux problèmes :
1* au niveau de test N,M<= Max
2* au niveau remplissage Matrice
voici mon code

Code:

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>
#include <stdlib.h>
 
#define Max 100
 
void saisie(int **V,int *m,int *n)
{   int i,j;
    printf("Saisie dimension du matrice \n") ;
    //do{
    printf("Donner nombre de ligne= ");
    scanf("%d",m);
    printf("Donner nombre de colonne= ");
    scanf("%d",n);
    //}while ((n>Max)&&(m>Max));
    // allocation dynamique de la taille de mat
    V=(int **)malloc((*m)*(sizeof(int)));
    printf("Saisie élément du matrice \n") ;
    for(i=0;i<(*m);i++)
    {
         V[i] = (int*) malloc((*n) * sizeof(int));
 
        for(j=0;j<(*n);j++)
        {
        printf("Donner valeur de case [%d,%d]= ",i+1,j+1);
        scanf("%d",&V[i][j]);
        }
    }
}
int main()
{   int M[10][10];
    saisie(M,3,3);
    return 0;
}

29/12/2014, 12h02
Médinoc
D'ici, je vois deux problèmes:
- Tu confonds tableau statique et dynamique:
  - Tu passes un tableau statique à ta fonction, dans laquelle tu alloues un tableau dynamiquement
  - Les type int[10][10] et int** ne sont pas compatibles.
- Tu tentes de modifier un paramètre pointeur de pointeur, mais tu ne prends pas celui-ci par pointeur: Pour faire ça, il te manque un niveau d'indirection.
Donc, première chose à savoir: Quel type de tableau veux-tu utiliser? Un tableau 2D statique de dimensions 10*10, ou un tableau 2D dynamique de dimensions déterminées à l'exécution?
29/12/2014, 15h29
alias2015_29

tableau statique
a l’utilisateur de faire entrer le nombre de ligne et colonne qu'il veux
29/12/2014, 16h15
Médinoc

Et si l'utilisateur entre des tailles supérieures à la taille statique, le programme doit refuser, c'est ça?
29/12/2014, 16h19
alias2015_29

oui
j'ai mis la boucle de répéter je sais pas juste ou faux c'est pour ça j'ai mis en commentaire
29/12/2014, 16h59
Médinoc

Dans ce cas, tu n'as pas d'allocation dynamique à faire.
29/12/2014, 19h47
alias2015_29

je met do while ou j'enleve ?
29/12/2014, 19h49
Médinoc

Tu le mets (tu peux même mettre toute l'entrée des dimensions dans une fonction à part) et tu enlèves les appels à malloc.

Code:

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
#include <stdio.h>
#include <stdlib.h>
 
#define Max 10
 
void saisie(int V[10][10],int m,int n)
{
    int i,j;
    printf("Saisie dimension du matrice \n") ;
    do{
    printf("Donner nombre de ligne= ");
    scanf("%d",&m);
    printf("Donner nombre de colonne= ");
    scanf("%d",&n);
    }while ((n>Max)&&(m>Max));
 
    printf("Saisie élément du matrice \n") ;
    for(i=0;i<m;i++)
    {
        for(j=0;j<n;j++)
        {
        printf("Donner valeur de case [%d,%d]= ",i+1,j+1);
        scanf("%d",&V[i][j]);
        }
    }
}
void affichage(int MAT[10][10],int m, int n)
{   int i,j;
    printf("Affichage du matrice \n") ;
    for(i=0;i<m;i++)
    {   for(j=0;j<n;j++)
        {
        printf("[%d,%d]= %d ",i+1,j+1,MAT[i][j]);
        }
        printf("\n");
    }
}

c'est bon le code ?

29/12/2014, 20h57
Médinoc

C'est pas trop mal, mais tu devrais prendre n et m par pointeurs dans ta fonction de saisie.
De plus, il faudrait ajouter de la gestion d'erreur pour scanf(), qui est une fonction très capricieuse (mal l'utiliser peut résulter en une boucle infinie!)
30/12/2014, 00h09
edgarjacobs

Hello,

Il y a à mon sens deux erreurs dans la boucle do....while lors de l'input:
1) Le test dans le while est faux: ce n'est pas && qu'il faut mettre, mais ||: répéter la boucle tant qu'une ou les deux données sont > Max
2) Ce test ne tient pas compte d'une entrée <= 0, qui donnera pour cette donnée un résultat vrai dans la condtion du while

Bon courage & a+

Edgar;

Probleme dans Procedure Saisie Matrice

Re-bonjour,

j'ai changé dans procédure saisie et j'ai ajouté de main pour tester mais ca marche pas non plus :(
Code:

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 void saisie(int V[10][10],int *m,int *n) { int i,j; printf("Saisie dimension du matrice \n") ; do{ printf("Donner nombre de ligne= "); scanf("%d",m); printf("Donner nombre de colonne= "); scanf("%d",n); }while ((*n>Max)||(*m>Max)||(*n<1)||(*m<1)); printf("Saisie élément du matrice \n") ; for(i=0;i<*m;i++) { for(j=0;j<*n;j++) { printf("Donner valeur de case [%d,%d]= ",i+1,j+1); scanf("%d",&V[i][j]); } } } int main(void) { int l,c; int MAT[100][100]; saisie(MAT,l,c); return 0; }
warning: passing argument 1 of 'saisie' from incompatible pointer type [enabled by default]| !!!!!

30/12/2014, 12h44
Médinoc

Oh, pour l'amour de fuck... :roll:

Code:

saisie(MAT,&l,&c);

Merci pour ta réponse Mais SVP sans les mots de :? ( l'amour de fuck)

Les deux procédures saisie et affichage fonctionne bien et juste :D

Maintenant je voudrais bien implémenter une procédure Extraction_Ligne qui permet d’extraire la ligne i d’une matrice d’entiers V de taille M x N dans un tableau T

voici mon essai j’espère qu'il est juste ? que pensez vous?
Code:

1 2 3 4 5 6 7 8 9 10 11 void Extraction_ligne(int MAT[Max][Max],int l,int *Tab[Max]) { int i,j,k=0; for (i=0;i<10;i++) // i parcours les lignes { if (i=l) for (j=0;j<10;j++) //j parcours les colonnes Tab[k]=MAT[l][j]; k++; } }

01/01/2015, 13h35
Médinoc

Pourquoi int* Tab[MAX]?
01/01/2015, 13h43
alias2015_29

pour dire que le tableau est une variable de sortie et qui change de valeur au début il est vide puis il sera plein :?
01/01/2015, 14h06
Invité
Bonjour,

L'intention est louable... mais un tantinet erronée :P

int *Tab[Max] représente un tableau de pointeurs sur int et non un pointeur sur un tableau de int.
La syntaxe pour passer un pointeur sur un tableau de Max int utilise des parenthèses comme ceci :

Code:

int (*Tab)[Max]

puis (*Tab)[i] pour accéder aux éléments.

Mais c'est inutile ici, car les tableaux sont passés par adresse (celle de la première case) ; int Tab[Max] tout court est amplement suffisant ici pour pouvoir modifier les cases du tableau d'origine.
Voir la FAQ Comment passer un tableau en paramètre à une fonction ? à ce sujet.

Remarques par rapport au code de ton message d'avant :
- Attention à la comparaison : if (i==l), sinon tu affectes l à i, puis vérifies si le résultat est non nul. Je ne saurais te conseiller d'utiliser au moins les options -Wall -Wextra à la compilation, ou d'écouter un peu mieux ton compilateur ;)
- Le parcours selon les lignes i est inutile : tu traites juste la ligne l.
- La variable k est inutile : utilise j à la place.
- Enfin, d'où vient ce 10 ? Ne serait-ce pas plutôt Max ?

Merci pour vos remarques,

j'ai pas fais attention à ces fautes, je l'est corrigé et j'ai ajouté un bot de code pour voir le contenu de tableau qui contient la ligne extraite :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 void Extraction_ligne(int MAT[Max][Max],int l,int Tab[Max]) { int i=0,j,k=0; while (i!=l) { i++; } for (j=0;j<Max;j++) { Tab[k]=MAT[l][j]; k++; } } void aff (int T[Max]) {int i; for (i=0;i<Max;i++) printf("** %d**",T[i]); }
Mais il affiche des nombres non de la matrice ??

03/01/2015, 09h38
Invité
Cette partie :
Code:

1 2 3 4 while (i!=l) { i++; }
(autant faire i = l; :aie:) ne sert strictement à rien : tu as l !
Et k peut être remplacé par j :
Code:

1 2 3 4 for (j=0;j<Max;j++) { Tab[j]=MAT[l][j]; }
Par contre rien dans ce que tu montres ne justifie des valeurs erronées, cela doit venir d'autre part.

À noter que les noms de variable tout en majuscules sont habituellement utilisés pour les constantes #define de compilation, et les noms commençant par une majuscule pour les types (bon, certains préfèreront rajouter des t_).
03/01/2015, 12h23
alias2015_29

1 pièce(s) jointe(s)
au niveau de main j'ai mis pour tester les deux procédures extraction ligne et afficher le tableau en sortie.
Code:

1 2 3 Extraction_ligne(MAT,2,T); aff(T);
avec le code de procédure affiche tableau
Code:

1 2 3 4 5 6 void aff(int T[Max]) {int i; for (i=0;i<Max;i++) printf("** %d**",T[i]); }
il m'affiche ce résultat dans éxecution
Pièce jointe 165246

Voir 40 message(s) de cette discussion en une page