Discussion :

[marilou]

bonjour !

je suis en train de faire un programme sous dev c++, qui permet d'entrer au clavier une matrice dont les dimensions sont inconnues au départ. on doit donc entrer les valeurs des dimensions puis celles des composants de la matrice.

voilà la fonction concernée. elle est incluse dans un main "test" disons ou je rentre les valeurs m et n qui sont les dimensions de ma matrice.

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
  int ** EntreeMatrice(int m, int n)
    {   int i=0;
        int j=0;
        int **c;
 
 
    c=(int**)malloc(m*sizeof(int*));
    *c =(int*)malloc(n*sizeof(int));
 
      while (i<m)
      {j=0;
         while(j<n)
          {
           printf("Entrez au clavier l'element(entier positif)[%d][%d]\n",i+1,j+1);
            scanf("%d",((*(c+i))+j));  */PROBLEME?*/
               j++;
           }
          i++;
        }
 
      printf("la matrice est:\n");
      i=0;
      j=0;
 
       while (i<m)
      {j=0;
           while(j<n)
          {  printf("%d",(*(*(c+i))+j));
             j++;
           }
          printf("\n");
          i++;
      }
 
      return c;
    }

lorsque j'execute ce programme la fenêtre d'execution bug à differents moments. je peux parfois rentrer certaines valeurs ( exemple, pour une matrice de dimension 2,2 , je ne peux pas entrer les éléments [2][1] et [2][2], mais par contre, la première ligne a l'air de marcher...) et je me vois obligée de terminer l'execution. lorsque j'essaye de débugger, il se trouve que c'est a la ligne que j'ai indiquée que le probleme se pose ... pourquoi la boucle commence-t-elle a s'enclencher? quelle est mon erreur de syntaxe ?

merci par avance de votre indulgence et de votre réponse

[DarkMolo]

Salut,
Le problème est au niveau de l'allocation de mémoire pour la matrice.

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
int tab[4][10];
//c'est quoi en fait, c'est 4 tableau de 10 élèments
//donc pour le faire dynamiquement il faut
int **tab;
tab=malloc(sizeof(int)*4);
for(i=0;i<4;i++)
   tab[i]=malloc(sizeof(int)*10);
 
//et pas ce que t'as mis
tab=(int**)malloc(4*sizeof(int*));
*tab =(int*)malloc(10*sizeof(int));

Tu pourrais donc faire des modifs sur ton code pour qu'il ressemble à ceci:

Code C :

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
int ** EntreeMatrice(int m, int n)
{
        int i=0;
        int j=0;
        int **c;
 
 
    c=malloc(m*sizeof(int*));            //<---
 
      while (i<m)
      {j=0;
         c[i] =malloc(n*sizeof(int));          //<---
         while(j<n)
          {
           printf("Entrez au clavier l'element(entier positif)[%d][%d]\n",i+1,j+1);
            scanf("%d",((*(c+i))+j));  */PROBLEME?*/
               j++;
           }
          i++;
        }
 
       //... 
 
      return c;
}

PS: Tu vas te faire gueuler dessus, tous les modos ici n'aiment pas voir des scanf.

[marilou]

oh milles mercis !!
l'algo marche parfaitement avec ces modifications.

mais pour le coup, mes connaissances sur le c en prennent un coup.. en effet, il me semblait qu'avec malloc, il était necessaire de preciser le type pointé puisqu'il etait (void*) et qu'il m'est deja arrivé que le programme ne ce compile pas a cause d'un oubli du genre

p= malloc(blabla*sizeof(int));

avec p pointeur sur un entier,
alors que le programme marchait quand on remplacait par

p=(int*)malloc.. etc

navrée par ailleurs du scanf ... je dois etre un peu old school. ... plutot "school" en fait : )

merci encore.

[Médinoc]

Dans les vieux compilos, il y avait des problèmes avec void*.

La seule autre raison pour laquelle ça compilerait mal sans cast, c'est si un gogo oublie d'inclure <stdlib.h>.
Le problème, c'est que si on "corrige" ça en rajoutant le cast, ça compile toujours aussi mal, car on ne fait que supprimer le symptome, pas la cause...

[Thierry Chappuis]

Dans les vieux compilos, il y avait des problèmes avec void*.

La seule autre raison pour laquelle ça compilerait mal sans cast, c'est si un gogo oublie d'inclure <stdlib.h>.

Ou alors si on utilise pas mégarde un compilateur C++...

Thierry

[Emmanuel Delahaye]

mais pour le coup, mes connaissances sur le c en prennent un coup.. en effet, il me semblait qu'avec malloc, il était necessaire de preciser le type pointé puisqu'il etait (void*) et qu'il m'est deja arrivé que le programme ne ce compile pas a cause d'un oubli du genre

p= malloc(blabla*sizeof(int));

avec p pointeur sur un entier,
alors que le programme marchait quand on remplacait par

p=(int*)malloc.. etc

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

[Thierry Chappuis]

PS: Tu vas te faire gueuler dessus, tous les modos ici n'aiment pas voir des scanf.

Personne n'a rien contre scanf, si cette fonction est utilisée correctement, et c'est bien là la difficulté... scanf est bien plus complexe que ne le pensent souvent les débutants.

Thierry