Problème de typage dans un tableau à deux dimensions.

Bonjour (bonsoir ?) à tous,

J'essayais de faire une matrice de t_pixel, mais gcc me retourne une erreur des plus classiques, sauf que je ne comprend pas où est le problème de typage...

Code:

---

1 2 3

test.c: Dans la fonction «make_matrix» : test.c:104: erreur: incompatible types in assignment

---

Alors en gros parmis les choses importantes à savoir c'est :

Mes types :

Code:

---

1 2 3 4

[...] typedef unsigned char t_pixel[3]; typedef t_pixel **t_pixmap;

---

ma structure s_picture

Code:

---

1 2 3 4 5 6 7 8 9

typedef struct picture { [...] t_pixmap pixmap; int width; int height; FILE *fd; } s_picture;

---

et ma fonction make_matrix bien entendu

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

void make_matrix(s_picture *pic) { int i, j; t_pixel pixel;   (*pic).pixmap = malloc(sizeof (t_pixel *) * (*pic).height); for (i = 0; i < (*pic).height; ++i) { (*pic).pixmap[i] = malloc(sizeof (t_pixel) * (*pic).width); }   for (i = 0; i < (*pic).height; ++i) { for (j = 0; j < (*pic).width; ++j) { fread(&(pixel[B]), sizeof (char), 1, (*pic).fd); fread(&(pixel[G]), sizeof (char), 1, (*pic).fd); fread(&(pixel[R]), sizeof (char), 1, (*pic).fd); (*pic).pixmap[i][j] = pixel; /* ceci est la ligne 104 */ } }

---

C'est peut être l'heure mais je n'arrive pas à trouver où est le problème de typage...

Si quelqun pouvait m'apporter son aide, je ne me permettrait pas la refuser :)
Si jamais vous avez besoin d'autre chose (plus de code ? je pense avoir mit l'essentiel) n'hésitez pas à demander :)

Cordialement,

iLUV

Citation:

---

Envoyé par iluv

Si quelqun pouvait m'apporter son aide, je ne me permettrait pas la refuser :)
Si jamais vous avez besoin d'autre chose (plus de code ? je pense avoir mit l'essentiel) n'hésitez pas à demander :)

---

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

Project : Forums Compiler : GNU GCC Compiler (called directly) Directory : C:\dev\forums2\ -------------------------------------------------------------------------------- Switching to target: default Compiling: main.c main.c:9: error: syntax error before "FILE" main.c:9: warning: no semicolon at end of struct or union main.c:10: warning: type defaults to `int' in declaration of `s_picture' main.c:10: warning: data definition has no type or storage class main.c:12: error: syntax error before '*' token main.c:13: warning: function declaration isn't a prototype main.c: In function `make_matrix': main.c:17: error: `pic' undeclared (first use in this function) main.c:17: error: (Each undeclared identifier is reported only once main.c:17: error: for each function it appears in.) main.c:17: warning: implicit declaration of function `malloc' main.c:27: warning: implicit declaration of function `fread' main.c:27: error: `b' undeclared (first use in this function) main.c:28: error: `G' undeclared (first use in this function) main.c:29: error: `R' undeclared (first use in this function) main.c:32: error: syntax error at end of input Process terminated with status 1 (0 minutes, 2 seconds) 9 errors, 6 warnings

---

Purée, on veut bien filer un coup de main, mais on est pas aidé... Faut que je me tape une demi-heure de rétro-conception ou tu postes du code compilable ?
Pour retrouver le défaut, et un code présentable (on est pas obligé d'écrire du code ilisible...) :

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

#include <stdio.h> #include <stdlib.h>   enum { B, G, R };   typedef unsigned char t_pixel[3];   typedef struct picture { t_pixel **pixmap; int width; int height; FILE *fd; } s_picture;   void make_matrix (s_picture * pic) { int i;   pic->pixmap = malloc (sizeof *pic->pixmap * pic->height); for (i = 0; i < pic->height; ++i) { pic->pixmap[i] = malloc (sizeof *pic->pixmap[i] * pic->width); }   for (i = 0; i < pic->height; ++i) { int j; for (j = 0; j < pic->width; ++j) { t_pixel pixel;   fread (&(pixel[B]), 1, 1, pic->fd); fread (&(pixel[G]), 1, 1, pic->fd); fread (&(pixel[R]), 1, 1, pic->fd); pic->pixmap[i][j] = pixel; /* ceci est la ligne 104 */ } } }

---

OK. Tu veux copier un tableau. C'est sûr ? Dans ce cas, c'est memcpy().
Il n'y a pas de contrôle d'erreur sur malloc() ou fread(), c'est exprès ?

Je pense qu'il a posté le code minimal pour la compréhension de ses types etc... Le but était juste de voir s'il y avait une erreur "évidente" au niveau des typages.

pixmap[i][j] est de type t_pixel, non ? Donc où est le problème de type ?

Citation:

---

Purée, on veut bien filer un coup de main, mais on est pas aidé... Faut que je me tape une demi-heure de rétro-conception ou tu postes du code compilable ?

---

Bah, pour un problème de typage je ne pensais pas qu'il y aurait besoin de plus... Cela dit, pas de problème, voici mon petit test.c et test.h.

Une des raisons pour lesquelles je ne montrais pas le code en entier est parcequ'il n'est pas fini, et en général on poste pour un problème, et les autres découvrent tout un tas de problèmes dont on est conscients (du genre j'utilise des write pour sauvegarder l'image, ou l'absence du contrôle d'erreur sur les malloc... c'est pas pour rien que les fichiers s'appellent test.{c|h}) et finissent par oublier de répondre à la question initiale.

Citation:

---

OK. Tu veux copier un tableau. C'est sûr ? Dans ce cas, c'est memcpy().

---

c'est vraiment adapté ça ?


Cordialement,

iLUV

Primo tu peux pratiqument partout remplacer tes :

Code:

---

1 2

(*pic).

---

par

Code:

---

1 2

pic->

---

Ensuite à ta fameuse ligne 104 :

Code:

---

1 2	**((*pic).pixmap) = pixel;

---

La solution d'Emmanuel est la bonne :

Code:

---

1 2	memcpy ( (const void *)&(pic->pixmap[i][j]), (const void *)&pixel, (size_t)3 );

---

Si j'ai bien compris ta structure....

Citation:

---

Envoyé par souviron34

Ensuite à ta fameuse ligne 104 :

Code:

---

1 2	memcpy ( (const void *)&(pic->pixmap[i][j]), (const void *)&pixel, (size_t)3 );

---

---

Trop complexe et pas bon...

Selon le principe de base :

Code:

---

memcpy (des, src, sizeof des);

---

ca donne :

Code:

---

memcpy (pic->pixmap[i][j], pixel, sizeof pic->pixmap[i][j]);

---

ça marche, merci à tous :)