Bitmap et HBITMAP

**ywan42** · 01/03/2007, 09h29

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** · 01/03/2007, 09h43

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** · 01/03/2007, 10h35

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** · 01/03/2007, 11h05

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** · 01/03/2007, 11h34

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** · 01/03/2007, 12h06

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

**ywan42** · 01/03/2007, 12h41

Merci encore de te pencher sur mon probleme Medinoc.

Mais crois-tu que je dois utiliser :
- CreateDIBSection
- CreateCompatibleBitmap
- CreateBitmap

C'est quoi les différence ...

Dois-je utiliser un
- Device Dependant Bitmap
ou un
- Device IndependantBitmap

Ai-je besoin d'un Device Context et pourquoi ?

Merci,
Ywan

**Médinoc** · 01/03/2007, 12h49

Si tu joues avec des BITMAPINFO et/ou avec des fichiers, tu devrais utiliser un DIB (Device-independent bitmap), car les fichiers .bmp sont des DIB.
De plus, les DIB peuvent être sélectionnés dans n'importe quel DC.

Et pour cela, le mieux est d'utiliser CreateDIBSection().
CreateCompatibleBitmap() crée forcément un DDB basé sur ton affichage (32bits si tu es en 32bits, 16bits si tu es en 16bits).

Normalement, tu ne devrais pas avoir besoin d'un DC pour CreateDIBSection(), on n'utilise jamais DIB_PAL_COLORS.

**ywan42** · 01/03/2007, 14h02

Okay je vais essayer puisque ma dll doit comporter les 2 moyens de sauvegarde du bitmap généré.

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
extern "C" __declspec(dllexport) TCouleur RVB (UCHAR Rouge, UCHAR Vert, UCHAR Bleu);
extern "C" __declspec(dllexport) int COMPARE_PIXEL(TCouleur c1, TCouleur c2);

extern "C" __declspec(dllexport) int LOAD_BITMAP_FROM_FILE (char * filename, TBitmap * pBitmap);
extern "C" __declspec(dllexport) void LOAD_BITMAP_FROM_HANDLE (HBITMAP hBmp, TBitmap * pBitmap);

extern "C" __declspec(dllexport) int SAVE_BITMAP_FROM_STRUCT (char * filename, TBitmap * pBitmap);
extern "C" __declspec(dllexport) int SAVE_BITMAP_FROM_HANDLE (char * filename, HBITMAP hBitmap);

extern "C" __declspec(dllexport) HBITMAP CREATE_HANDLE_FROM_BITMAP(TBitmap * pBitmap);

extern "C" __declspec(dllexport) DWORD GENERATE_SHADOW_USING_FILE (T_Param_Shadow * pparam);
extern "C" __declspec(dllexport) DWORD GENERATE_SHADOW_USING_HANDLE (T_Param_Shadow * pparam);

extern "C" __declspec(dllexport) DWORD RESIZE_USING_FILE (T_Param_Resized * pparam);
extern "C" __declspec(dllexport) DWORD RESIZE_USING_HANDLE (T_Param_Resized * pparam);

extern "C" __declspec(dllexport) DWORD BUILD_TRANSPARENT_USING_FILE (T_Param_Transparency * pparam);
extern "C" __declspec(dllexport) DWORD BUILD_TRANSPARENT_USING_HANDLE (T_Param_Transparency * pparam);

... c'est pas très clair pour moi !!! ca fait 2 jours que je debugge et no problem avec les fichiers problem avec les handles !!!
Ywan

**Médinoc** · 01/03/2007, 14h14

Tu sais, en règle générale, on n'utilise pas de majuscules pour des noms de fonctions.
De plus, tes pointeurs devraient être des const char * (LPCSTR), voire des const TCHAR * (LPCTSTR), non ?

Pour le SaveBitmapFromHandle(), tu peux t'inspirer du lien que j'ai passé (encore que, il est un peu inutilement compliqué). Pour le chargement, la fonction LoadImage() charge automatiquement un fichier .bmp dans un DDB.
Sachant ça, tu peux:

soit faire manuellement le chargement du fichier vers un DIB, la BITMAPINFOHEADER est là pour ça
soit charger le fichier dans un DDB et recopier l'image vers un DIB.

Dans les deux cas, je pense que ta structure TBitmap n'a pas besoin de garder en mémoire la structure BITMAPFILEHEADER : Il faut plutôt la créer dynamiquement dans le SaveBitmapFromStruct(), comme dans l'exemple.

**ywan42** · 01/03/2007, 14h37

Oui, le code est peu etre pas tres academique mais dans mon entreprise les fonctions maison sont codées en MAJUSCULES et les fonctions C standard en minuscule pour eviter la confuxion,

Sinon, les pointeurs devraient effectivement etre des const char *.

Sinon, je viens d'essayer CreateDIBSection mais cela ne marche pas !!!

Voila l'image que j'obtiens avec les handles :

[IMG]F:\yle\En_cours\Ombre_nodll\ShadowResized.bmp[/IMG]

je fais
1- un Load Image des images avec

Code :

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

1
2
3
4
hDC = GetDC(hwnd);
LoadImage(hDC, fondBitmapName, IMAGE_BITMAP, 0, 0, LR_LOADFROMFILE);
DeleteDC(hDC);

que je veux redimensionnees par interpolation, ombrées, ou rendre transparentes ...

2 - un appel à la classe Shadow, Resize ou Transparency ... en travaillant sur la structure TBitmap (perso).

3 - Un save pour generer un handle en appelant CREATE_BITMAP_FROM_HANDLE (dont le code est plus haut).
puis
LOAD_BITMAP_FROM_HANDLE (dont le code est plus haut).
puis
SAVE_BITMAP_FROM_STRUCT (éprouvé puisque donne l'image suivante sans traits noirs)

[IMG]F:\yle\En_cours\Ombre_nodll\ShadowResized2.bmp[/IMG]

Voila ...

Ywan

**Médinoc** · 01/03/2007, 14h43

Tu ne peux poster directement une image de ton disque dur.
Sauvegarde-là en PNG et uploade-là avec l'option "pièces jointes" du forum...
T'arrive-t-il de lire la doc des fonctions que tu emploies ?
Il faudra m'expliquer comment tu bosses sur l'image si tu n'utilises pas le handle retourné par LoadImage()...

**Médinoc** · 01/03/2007, 14h51

PS: Au passage, je viens de relire la doc de LoadImage(), tu peux obtenir directement un DIB avec au lieu d'un DDB: Il suffit d'ajouter le flag LR_CREATEDIBSECTION.

**ywan42** · 01/03/2007, 15h29

Tu me conseille don d'utiliser le Device independant bitmap, je suis d'accord puisque je ne cherche pas a les afficher mais seulement les gerer sous formes de fichiers ou de handle ... Je vais relire la doc et reessayer avec CreateDIBSection et LR_CREATEDIBSECTION.

Merci, Ywan

PS : Pieces jointes des images avec handles et avec fichiers

**ywan42** · 01/03/2007, 15h49

Merci beaucoup Medinoc cele marche je met mon code pour ceux que cela interesserait ...

C'etait la premiere fois que je chatais sur un forum ... et je suis ravi de ton aide. J'avais deja recupere de la dioc sur developpez.com mais les gens du forum sont tres competent !!!

A bientot, Ywan

PS code corrige :

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
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
void LOAD_BITMAP_FROM_HANDLE (HBITMAP hBitmap, TBitmap * pBitmap)
{
   BITMAP  bitmap;
 	HDC hDC;
   static 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; // Codes ascii des caracteres 'B' et 'M' en hexa
   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 = new TCouleur[size];

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

}

HBITMAP CREATE_HANDLE_FROM_BITMAP(TBitmap * pBitmap)
{
	HBITMAP hBitmap;
   HDC hDC;

   long width, height, start;

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

   hBitmap = CreateDIBSection(NULL, (BITMAPINFO *)&pBitmap->Entete2, DIB_RGB_COLORS,
                              (LPVOID *)&lpBits, NULL, 0);

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

	return hBitmap;
}

**Médinoc** · 01/03/2007, 15h54

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

Le code ASCII des lettres B et M "en hexa", écris-le véritablement en hexa (0x4D42).
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** · 01/03/2007, 16h06

Okay, merci encore.

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

Bitmap et HBITMAP

Windows

Discussions similaires

Partager

Partager