Discussion :

[ywan42]

Bonjour,

Je travaille actuellement sur les bitmaps en C et API Windows.

Je cherche à obtenir le tableau de pixels (RGB) à partir du handle (HBITMAP) et inversement.

Mais mon code me génère des scans lines noires !!! Comment puis-je résoudre ce problème ?

Voici mon code atuel :

+ Pour obtenir le tableau a partir du HBITMAP :

Code :

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

1
2
3
4
5
6
7
8
 
start = 0;
for (ligne = 0; ligne < height; ligne++)
{
retour = SetDIBits(hDC, hBitmap, ligne, 1, &lpBits[start], 
(BITMAPINFO *)&pBitmap->Entete2, DIB_RGB_COLORS);
start += width*sizeof(TCouleur)+alignement;
}

+ Pour obtenir le HBITMAP a partir du tableau :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
start = 0;
for (ligne = 0; ligne < height; ligne++)
{
GetDIBits(hDC, hBitmap, ligne, 1, &lpBits[start], 
(BITMAPINFO *)&pBitmap->Entete2, DIB_RGB_COLORS);
 
memcpy(&pBitmap->pixels[ligne*width], &lpBits[start], 
width*sizeof(TCouleur));
 
start += width*sizeof(TCouleur)+alignement;
}

Ou est mon erreur ? Je sèche depuis 2 jours !!!

Merci d'avance de votre aide.
Ywan

[Médinoc]

Déjà, c'est bizarre, il me semblait que GetDIBits() servait à extraire les pixels du HBITMAP, et SetDIBits() à les injecter...

Et au passage, tu peux jeter un coup d'oeil à cet exemple de code :
Storing an Image

[ywan42]

Merci de m'avoir repondu ... c'est la premiere fopis que j'utilise un forum pour debugger un programme.

J'ai regardé avec attention le code sue tu m'as conseillé pour stocker un bitmap. De mon côté mes algos pour créer mon tableau de bitmap sont okay car je peux les stocker dans un fichier .bmp sans problème, seulement je voudrais fournir une dll qui passe par des handle et non des fichiers aussi je cherche a transformer mon array en HBITMAP et effectivement j'ai utilisé SetDIBits et GetDIBits mais il m'insere des lignes noires RGB = (0,0,0).

+ Voici donc mon code pour chharger le bitmap a partir du handle :

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
void LOAD_BITMAP_FROM_HANDLE (HBITMAP hBitmap, TBitmap * pBitmap)
{
   BITMAP  bitmap;
 	HDC hDC;
   long int size, width, height, start;
   int ligne, reste, alignement;
   UCHAR * lpBits;

   GetObject(hBitmap, sizeof(BITMAP), (LPVOID)&bitmap);
	width = bitmap.bmWidth;
   height = bitmap.bmHeight;
   size = width*height;

   reste = (width*sizeof(TCouleur))%4;
   if (reste > 0)
   {
      alignement = 4-reste;
   }
   else
   {
   	alignement = 0;
   }

   pBitmap->Entete1.bfType = 19778; 
   pBitmap->Entete1.bfSize = sizeof (BITMAPFILEHEADER) + sizeof (BITMAPINFOHEADER) + size*sizeof(TCouleur);
   pBitmap->Entete1.bfReserved1 = 0;     // Toujours zero pour les Bitmaps
   pBitmap->Entete1.bfReserved2 = 0;
   pBitmap->Entete1.bfOffBits = sizeof (BITMAPINFOHEADER) + sizeof(BITMAPFILEHEADER);
   pBitmap->Entete2.biSize = sizeof (BITMAPINFOHEADER);
   pBitmap->Entete2.biWidth = width;
   pBitmap->Entete2.biHeight = height;
   pBitmap->Entete2.biPlanes = 1;
   pBitmap->Entete2.biBitCount = 24;  //Codage sur 3 octets (24bits) de la couleur
   pBitmap->Entete2.biCompression = BI_RGB;
   pBitmap->Entete2.biSizeImage = (width*sizeof(TCouleur)+alignement)*height;
   pBitmap->Entete2.biXPelsPerMeter = 0;
   pBitmap->Entete2.biYPelsPerMeter = 0;
   pBitmap->Entete2.biClrUsed = 0;
   pBitmap->Entete2.biClrImportant = 0;

   pBitmap->pixels = (TCouleur *) malloc (size*sizeof(TCouleur));

   lpBits = (UCHAR *)malloc(height*(width*sizeof(TCouleur)+alignement)*sizeof(UCHAR));


   hDC = GetDC(NULL);

   start = 0;
   for (ligne = 0; ligne < height; ligne++)
   {
		GetDIBits(hDC, hBitmap, ligne, 1, &lpBits[start], (BITMAPINFO *)&pBitmap->Entete2, DIB_RGB_COLORS);

      memcpy(&pBitmap->pixels[ligne*width], &lpBits[start], width*sizeof(TCouleur));

      start += width*sizeof(TCouleur)+alignement;
   }

	ReleaseDC(NULL, hDC);
   DeleteDC(hDC);

   free(lpBits);

}

+ et mon code pour creer le handle a partir du bitmap :

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
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
HBITMAP CREATE_HANDLE_FROM_BITMAP(TBitmap * pBitmap)
{
	HBITMAP hBitmap;
   HDC hDC;

   long width, height, start, retour;

   int ligne, reste, alignement;
   static int ret;
   char car[4] = {0, 0, 0, 0};
   UCHAR * lpBits;

	width = pBitmap->Entete2.biWidth;
   height = pBitmap->Entete2.biHeight;

   reste = (width*sizeof(TCouleur))%4;
   if (reste > 0)
   {
      alignement = 4-reste;
   }
   else
   {
   	alignement = 0;
   }

   lpBits = (UCHAR *)malloc(height*(width*sizeof(TCouleur)+alignement)*sizeof(UCHAR));

   start = 0;
   for (ligne = 0; ligne < height; ligne++)
   {
		memcpy(&lpBits[start], &pBitmap->pixels[ligne*width], width*sizeof(TCouleur));
      memcpy(&lpBits[start+width*sizeof(TCouleur)], car, alignement*sizeof(UCHAR));
      start += width*sizeof(TCouleur)+alignement;
   }

   hDC = GetDC(NULL);
   hBitmap = CreateCompatibleBitmap(hDC, width, height);

   start = 0;
   for (ligne = 0; ligne < height; ligne++)
   {
		retour = SetDIBits(hDC, hBitmap, ligne, 1, &lpBits[start], (BITMAPINFO *)&pBitmap->Entete2, DIB_RGB_COLORS);
      /*
      if (retour == 0)
      {
         LPVOID lpMsgBuf;

         FormatMessage(FORMAT_MESSAGE_ALLOCATE_BUFFER | FORMAT_MESSAGE_FROM_SYSTEM,
                       NULL,
                       GetLastError(),
                       MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT), // Default language
                       (LPTSTR) &lpMsgBuf,
                       0,
                       NULL);

         MessageBox( NULL, (char *)lpMsgBuf, "GetLastError", MB_OK|MB_ICONINFORMATION );
         LocalFree( lpMsgBuf );
      }
      */
      start += width*sizeof(TCouleur)+alignement;
   }

   ReleaseDC(NULL,hDC);
   DeleteDC (hDC);

   //free(lpBits);

	return hBitmap;
}

Merci d'avance de passer un peu de temps sur mon program,
Ywan.

[Médinoc]

Pourquoi utiliser un BITMAPFILEHEADER ? Je ne vois pas où tu as besoin d'un fichier.bmp...

PS: TBitmap, c'est un truc de Borland, ou une classe à toi ? (quand je vois TCouleur, je me pose la question...

[ywan42]

TBitmap et TCouleur sont des structures perso ... mais elles respectent le standard bmp :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
typedef struct SCouleur
{
 unsigned char Rouge,Vert,Bleu;
} TCouleur;

typedef struct SBitmap
{
	BITMAPFILEHEADER Entete1;
	BITMAPINFOHEADER Entete2;
	TCouleur * pixels;
} TBitmap;

aussi Entete2 est du type BITMAPINFO (Microsoft).

Ywan

[Médinoc]

C'est le standard bmp seulement pour 24 ou 32 bits.
Et plutôt que TCouleur, tu devrais utiliser directement des structures RGBQUAD (bitmap 32 bits uniquement dans les deux cas).

Mais tu ne devrais pas avoir besoin de BITMAPFILEHEADER pour un bitmap en mémoire. BITMAPFILEHEADER, comme son nom l'indique, c'est pour les fichiers.
La fonction CreateDIBSection() n'a besoin que d'une BITMAPINFO pour fonctionner...

[Médinoc]

Je n'ai pas vraiment eu le temps d'examiner le code en détail, mais quelques remarques sur la forme:

1. Le code ASCII des lettres B et M "en hexa", écris-le véritablement en hexa (0x4D42).
2. Si les noms des fonctions ne sont pas imposés, plutôt que CREATE_, tu devrais les nommer comme des fonctions de conversion: BITMAP_XXX_FROM_YYY() avec XXX et YYY pouvant être HANDLE, FILE ou STRUCT...

[ywan42]

Okay, merci encore.

Ywan
(http://ywan.lesoeur.free.fr)