Discussion :

[iluv]

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 :

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

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 :

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

1
2
3
4
 
[...]
typedef unsigned char   t_pixel[3];
typedef t_pixel         **t_pixmap;

ma structure s_picture

Code :

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

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 :

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
 
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

[Emmanuel Delahaye]

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 :

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
 
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 :

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
 
#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 ?

[NiuAge]

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 ?

[iluv]

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.

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

c'est vraiment adapté ça ?


Cordialement,

iLUV

[souviron34]

Primo tu peux pratiqument partout remplacer tes :

Code :

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

1
2
 
(*pic).

par

Code :

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

1
2
 
pic->

Ensuite à ta fameuse ligne 104 :

Code :

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

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

La solution d'Emmanuel est la bonne :

Code :

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

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

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

[Emmanuel Delahaye]

Ensuite à ta fameuse ligne 104 :

Code :

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

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 :

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

   memcpy (des, src, sizeof des);

ca donne :

Code :

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

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

[iluv]

ça marche, merci à tous