Problème avec malloc et autre

Version imprimable

Bonjour,

cela fait bien 10 ans que je n'ai pas fait de C. Mais pour répondre à qq'un sur le forum, j'ai réécrit un petit bout de 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
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
#include <stdio.h>
#include <stdlib.h>
 
 
typedef struct _Matrice {
	int nbColonnes;
	int nbLignes;
	int **data;
} Matrice;
 
void showMatrice(Matrice *m) {
	int i,j,value;	
	int *ligne;
	printf(">> showMatrice\n");
	for (i=0;i<m->nbLignes;i++) {
		ligne = m->data[i];
		for (j=0;j<m->nbColonnes;j++) {
			value = *(ligne+j*sizeof(int));
			printf("%d ",value);
		}
		printf("\n");
	}
	printf("<< showMatrice\n");
}
 
Matrice * creerMatrice(int nbColonnes, int nbLignes) {
	Matrice * m = (Matrice *)malloc(sizeof(Matrice));
	m->nbColonnes = nbColonnes;
	m->nbLignes = nbLignes;
	m->data = (int **)malloc(nbLignes*sizeof(int *));
	int i,j;
	for (i=0;i<nbLignes;i++) {
	   m->data[i] =  (int*)malloc(nbColonnes*sizeof(int)); 
	   for (j=0;j<nbColonnes;j++)
	   	*(m->data[i]+j*sizeof(int)) = (int)7;   
	}	
	return m;
}
 
void libereMatrice(Matrice *m) {
	int i;
	for (i=0;i<m->nbLignes;i++) {
	   free(m->data[i]);	    	  
	}	
	free(m->data);
	free(m);
}
 
int main (int argc,char **argv)
{
	Matrice *m;	
 
	m = creerMatrice(2,2);
	showMatrice(m);
	libereMatrice(m);
 
 
	return 0;
}

Lors de l'exécution, les chiffres affichés en "fin" de tableau sont totalement farfelus.

Où est le hic ?

Merci d'avance

01/05/2007, 16h59
LinuxUser

Il y a un probleme à la ligne : value = *(ligne+j*sizeof(int));
pourquoi ne pas mettre simplemet value = m->data[i][j];

En fait en multiliant par sizeof(int) tu fausse la valeur, car tu te deplace plus que tu ne le devrais. Pourqoui ne pas mettre value = *(ligne+j); tout court.
01/05/2007, 17h05
MSM_007

Le programme est plus simple que ça , utilisé directement m->data[i][j] dans la lecture et l'affichage

Citation:

for (i=0;i<nbLignes;i++) {
m->data[i] = (int*)malloc(nbColonnes*sizeof(int));
for (j=0;j<nbColonnes;j++)
m->data[i][j] = i*10+j;
}

Citation:

for (i=0;i<m->nbLignes;i++) {
for (j=0;j<m->nbColonnes;j++) {
printf("\t%d ",m->data[i][j]);
}
printf("\n");
}
01/05/2007, 17h11
ego

ok, cela fonctionne bien maitenant.

Peux-tu me dire d'où vient mon erreur même si mon code est trop compliqué ?

Merci

EDIT : ok, j'ai finalement compris.
Le *sizeof(int) dans mes déplacements de pointeur fausse tout. Je n'ai pas besoin de cela.

Merci à vous

Citation:

Envoyé par ego

ok, cela fonctionne bien maitenant.

Peux-tu me dire d'où vient mon erreur même si mon code est trop compliqué ?

Le *sizeof(int) dans mes déplacements de pointeur fausse tout. Je n'ai pas besoin de cela.

Exactement : ton code corrigé :

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
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
 
#include <stdio.h>
#include <stdlib.h>
typedef struct _Matrice
{
   int nbColonnes;
   int nbLignes;
   int **data;
}
Matrice;
 
void showMatrice (Matrice * m)
{
   int i, j, value;
   int *ligne;
   printf (">> showMatrice\n");
   for (i = 0; i < m->nbLignes; i++)
   {
      ligne = m->data[i];
      for (j = 0; j < m->nbColonnes; j++)
      {
         /* -ed- horrible !
            value = *(ligne + j * sizeof (int));
          */
         value = ligne[j];
         printf ("%d ", value);
      }
      printf ("\n");
   }
   printf ("<< showMatrice\n");
}
 
Matrice *creerMatrice (int nbColonnes, int nbLignes)
{
   Matrice *m = malloc (sizeof *m);
   m->nbColonnes = nbColonnes;
   m->nbLignes = nbLignes;
   m->data = malloc (nbLignes * sizeof *m->data);
   int i, j;
   for (i = 0; i < nbLignes; i++)
   {
      m->data[i] = malloc (nbColonnes * sizeof *m->data[i]);
      for (j = 0; j < nbColonnes; j++)
      {
         /* -ed- Horrible !
          *(m->data[i] + j * sizeof (int)) = (int) 7;
          */
 
         m->data[i][j] = i + j;
      }
   }
   return m;
}
 
void libereMatrice (Matrice * m)
{
   int i;
   for (i = 0; i < m->nbLignes; i++)
   {
      free (m->data[i]);
   }
   free (m->data);
   free (m);
}
 
int main (void)
{
   Matrice *m = creerMatrice (3, 4);
   showMatrice (m);
   libereMatrice (m);
   return 0;
}

02/05/2007, 18h29
ego

ouhai, c'est comme ça que j'ai finalement réécrit le code.
Ah les années, ça jouent des tours :roll: