Discussion :

[Blackbelt_29]

Bonjour,

Je souhaite allouer dynamiquement un tableau à deux dimensions.
Du coup j'ai d'abord créé une fonction qui prend en paramètre une hauteur et une largeur et qui renvoie un tableau à deux dimensions dont voici le code :

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
 
 
float ** allocation( int height, int width )
{
   int i,j;
   float ** tab = (float *) malloc( height*sizeof(float*) );  /*Ligne 153*/
   for ( i = 0; i<height; i++ )
   {
      tab[i] = (float* ) malloc( width*sizeof(float) );
      for ( j = 0; j<width; j++ )
      {
          tab[i][j] = 1;
      }
   }
   return tab;
}

Et je l'appel dans le main :

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
 
int main(void){
/*....code déterminant height et width....*/
    float *imageVect=NULL;
    float **tab2=NULL;
    float **matrice=NULL;
    imageVect = (float *) malloc(height*width*nbImage * sizeof(float));
 
 
    /*Appel des fonctions*/
   tab2=allocation(height,width);
    imageVect=LireImageRAW(height,width,nbImage);
    matrice=DecompVect(imageVect,tab2,height,width,positionOffset); /*Ligne 240*/
 
}

Le but de mon programme est de transcrire un vecteur en une matrice en 512*512

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
 
float* DecompVect(float* imageVect,float* tab2,int height,int width,int positionOffset){
  /*Declaration des variables*/
    int i,j;
    /*fin de déclaration des variables*/
 
    for(i=0;i<height;i++){
        for(j=0;j<width;j++){
            tab2[i][j]=imageVect[positionOffset]; /*Ligne 179*/
            printf("%.2f\n",tab2[i][j]);
            positionOffset++;
        }
        printf("\n");
    }
 
 
    return(tab2);
    free(tab2);
}

Mon problème est que lorsque je compile le code voici l'erreur que j'ai :

error: cannot convert 'float*' to 'float**' in initialization => ligne 153
error : invalid types 'float[int]' for array subscript => ligne 179
error: cannot convert 'float**' to 'float*' for argument '2' to 'float* DecompVect((float*,float*,int,int,int)' => ligne 240

Je pense que mes erreurs sont liées à une seule et même erreur (lors de l'allocation du tableau je pense), mais je n'arrive pas à trouver la faute !

Merci de vos réponses !

[Matt_Houston]

float ** tab = (float *)foobar; : là tu essaies de mettre une valeur de type float * dans une variable de type float **, ça ne pourra pas fonctionner.

Ne caste jamais le résultat de malloc et consorts, c'est inutile voire dangereux.


Pourquoi veux-tu utiliser des tableaux à deux dimensions pour manipuler tes images ? Tu es plus à l'aise avec ? Parce que si c'est simplement cela tu peux abstraire l'accès à un pixel donné avec une macro ou une fonction comme ceci (code absolument pas testé) :

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
struct imageSequence {
    unsigned char *dataOriginal;
    float *dataProcessed;
    unsigned int w, h;
    unsigned int bpp; // bits-per-pixel: 8, 16, 24 or 32
    unsigned int count; // number of image matrices
};
 
unsigned char *getPixelOriginalAt(struct imageSequence *img, unsigned int num, int x, int y) {
    assert(num < img->count);
    assert(x >= 0 && (unsigned int)x < img->w && y >= 0 && (unsigned int)y < img->h);
 
    const unsigned int imgSize = img->w * img->h * img->bpp / 8;
    return &img->dataOriginal[num * imgSize + y * img->w * img->bpp / 8 + x * img->bpp / 8];
}
 
float *getPixelProcessedAt(struct imageSequence *img, unsigned int num, int x, int y) {
    assert(num < img->count);
    assert(x >= 0 && (unsigned int)x < img->w && y >= 0 && (unsigned int)y < img->h);
 
    const unsigned int imgSize = img->w * img->h;
    return &img->dataProcessed[num * imgSize + y * img->w + x];
}

Outre la difficulté de manipulation de la double indirection, le souci avec la représentation des images dans des TYPE ** (tableau à deux dimensions dynamique) est que cela ne correspond pas au layout de l'image en mémoire (ligne par ligne de haut en bas, dans l'immense majorité des cas). Il y a donc des conversions à réaliser qui ne sont pas forcément triviales pour un débutant.

[RenardEnjoy]

Bonjour, j'ai pas complètement compris ton code mais j'y ai vu plusieurs erreurs:

A la ligne 153, tu affecte un float * à un float **
Dans les paramètres de la fonction DecompVect, la variable tab2 est un float * alors que partout ailleurs dans le code c'est un float **
Ce qui fait qu'à la ligne 179 tu utilise un tableau à une dimension comme un tableau à deux dimensions.


Cordialement

EDIT: Matt_Houston j'avais pas vu ton poste...

[Blackbelt_29]

Merci pour vos réponses Matt_Houston et RenardEnjoy !

Pourquoi veux-tu utiliser des tableaux à deux dimensions pour manipuler tes images ? Tu es plus à l'aise avec ?

En fait mon image est sous forme de vecteur [hauteur*largeur] il faut donc pour que je puisse traiter l'image la transcrire en image en deux dimension (je dois appliquer un filtre médian sur cette image)

du coup j'ai changé certaine chose dans mes fonctions et j'ai toujours des erreurs dans la fonction allocation :

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
float ** allocation( int height, int width )
{
   int i,j;
   float **tab2=NULL;
   tab2 = malloc( height*sizeof(float*) );  /*Ligne 153*/
   for ( i = 0; i<height; i++ )
   {
      tab2[i] = malloc( width*sizeof(float) ); /*ligne 156*/
      for ( j = 0; j<width; j++ )
      {
          tab2[i][j] = 1;
      }
   }
   return tab2;
}

Voici les erreurs

error:invalid conversion from 'void*' to float**' [-fpermisive] => ligne 153
error:invalid conversion from 'void*' to float**' [-fpermisive] => ligne 156

et j'ai changé les appels dans le main et la déclaration dans la fonction de recomposition du vecteur

Code :

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

float** DecompVect(float* imageVect,float** tab2,int height,int width,int positionOffset)

du coup il me revoit un tableau 2D en prenant en paramètre le vecteur, le tableau 2D créer grâce à l'allocation mémoire...

et dans le main je l'appel de cette façon :

Code :

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

1
2
3
 
    tab2=allocation(height,width);
    tab2=DecompVect(imageVect,tab2,height,width,positionOffset); /*Ligne 240*/

edit: J'ai résolu mon erreur finalement j'avais mal initialisé les malloc !
Je vais voir si le programme fonctionne et si oui je mettrai résolu !

merci !