C / C++ / MFC

Re: How i can Save Cimage file+ shapes & text on it?

1-Jul-10 5:59

hbm is a handle to a bitmap. The Windows type is HBITMAP.

humais1-Jul-10 19:38

humais

1-Jul-10 19:38

Runtime error: tht hbm is not initialized....?

Re: How i can Save Cimage file+ shapes & text on it?

Alan Balkany6-Jul-10 3:23

Drawing on 8bpp grayscale bitmap (unmanaged)

6-Jul-10 3:23

Use CreateCompatibleBitmap (): http://msdn.microsoft.com/en-us/library/dd183488%28VS.85%29.aspx[^]

In your OnPaint method, you have a device context, which can be used to create a compatible bitmap. You can then draw on this bitmap, then attach it to a CImage object and save it.

Alan Balkany29-Jun-10 4:52

29-Jun-10 4:52

Hi all,

I've been attempting to draw on an 8bpp grayscale bitmap without success. Here are some of my attempts. Maybe someone can point out what I'm doing wrong.

===================================================
Attempt 1: Create, select, and draw:

In constructor:

CBitmap bm;
bm.CreateBitmap (200, 200, 1, 8, NULL);

In OnDraw:

CDC *mdc=new CDC ();
HGDIOBJ tmp = mdc->SelectObject(bm);

Result: tmp is NULL, indicating failure.

===================================================
Attempt 2: CreateDIBSection

In constructor:

HBITMAP hbm;
BITMAPINFOHEADER bih;
BITMAPINFO bi;
HANDLE hb;
CDC* myDc = new CDC ();
HDC hdc = myDc->GetSafeHdc ();
void* bits;
RGBQUAD rq [256];

	initBi ();
	hbm = CreateDIBSection (hdc, &bi, DIB_RGB_COLORS, &bits, NULL, 0);

...

void CEightBitDrawingView::initBi()
{
	bih.biSize = sizeof (BITMAPINFOHEADER);
	bih.biWidth = 200;
	bih.biHeight = -200;
	bih.biPlanes = 1;
	bih.biBitCount = 8;
	bih.biCompression = BI_RGB;
	bih.biSizeImage = 0;
	bih.biXPelsPerMeter = 14173;
	bih.biYPelsPerMeter = 14173;
	bih.biClrUsed = 0;
	bih.biClrImportant = 0;

	memset ((void *) rq, 0, 256 * sizeof (RGBQUAD));

	bi.bmiHeader = bih;
	bi.bmiColors = rq;
}

Result: This doesn't even compile because the BITMAPINFO bmiColors member is defined as:

RGBQUAD          bmiColors[1];

so won't accept more than one RGB color. In fact, nothing I assign to this member compiles! (Could they possibly make it any more complex!?)

Any suggestions would be appreciated! Thanks!

===================================================

Code-o-mat29-Jun-10 5:46

29-Jun-10 5:46

Alan Balkany wrote:
CDC *mdc=new CDC ();
HGDIOBJ tmp = mdc->SelectObject(bm);

- do you create the DC anywhere or you just instantiate it? You need to create the DC, a CDC instance alone isn't enough. Where does bm come from? Acording to what you said bm would be a local variable in the constructor, so how would it get to OnDraw?

About the RGBQUAD thing:

Alan Balkany wrote:
RGBQUAD bmiColors[1];

this means that an array of RGBQUADs containing a variable number of colors will follow directly the BITMAPINFO structure. I think the biClrUsed member would indicate this in the struct (see the documentation for the structure).
You have to allocate enough memory to contain the bitmap info struct AND the palette after it, you can set the members in the palette -if you like- thorough the bmiColors array. Something like:

BITMAPINFO *bmi = (BITMAPINFO *)malloc(sizeof(BITMAPINFO) + sizeof(RGBQUAD) * 255);
...fill the bmiHeader ...
bmi->bmiCOlors[0] = firstColor;
bmi->bmiCOlors[1] = secondColor;
...

> The problem with computers is that they do what you tell them to do and not what you want them to do. <
> "It doesn't work, fix it" does not qualify as a bug report. <
> Amazing what new features none of the programmers working on the project ever heard of you can learn about when reading what the marketing guys wrote about it. <

Alan Balkany29-Jun-10 5:53

29-Jun-10 5:53

Code-o-mat, thanks for clearing up the mystery about bmiColors!

I just created a new DC because (it appears that) existing DCs will default to the color depth of my display. If this isn't correct, what's the right way to obtain a DC to draw on an 8bpp bitmap?

(The bitmap is actually declared in the class, and was just included in the constructor's code to show the type.)

Code-o-mat29-Jun-10 6:07

29-Jun-10 6:07

I needed to do this some time ago and as far as i can remember i went the DIB section way myself, creating a palette that went from 0,0,0 to 255,255,255. If you create a screen-compatible DC and then select your bitmap into it i think you should be able to draw on it.

Alan Balkany29-Jun-10 8:21

29-Jun-10 8:21

Thanks for the help, Code-o-mat! I can now generate bitmaps, but they're coming out incorrectly.

Short version: I created an 8bpp bitmap, then I BitBlt it to the screen and save it into a file. The copy on the screen has color, which suggests it's not just 8bpp. The version saved to the file is 8bpp, but it's totally black, where it should have at least *some* of the drawing.

Question: How can I draw on an 8bpp grayscale bitmap and save it to a file? Can anyone see what I'm doing wrong?

========================================================
The Details

Declarations:

CBitmap bm;

HBITMAP hbm;
BITMAPINFO* bi = (BITMAPINFO *)malloc(sizeof(BITMAPINFO) + sizeof(RGBQUAD) * 256);
BITMAPINFOHEADER bih;

Constructor:

initBi ();
hbm = CreateDIBSection (hdc, bi, DIB_RGB_COLORS, &bits, NULL, 0);

...

void CEightBitDrawingView::initBi()
{
	bih.biSize = sizeof (BITMAPINFOHEADER);
	bih.biWidth = 200;
	bih.biHeight = -200;
	bih.biPlanes = 1;
	bih.biBitCount = 8;
	bih.biCompression = BI_RGB;
	bih.biSizeImage = 0;
	bih.biXPelsPerMeter = 14173;
	bih.biYPelsPerMeter = 14173;
	bih.biClrUsed = 0;
	bih.biClrImportant = 0;

	bi->bmiHeader = bih;

	for (int i = 0;   i < 256;   i++)
	{
		bi->bmiColors [i].rgbRed = i;
		bi->bmiColors [i].rgbGreen = i;
		bi->bmiColors [i].rgbBlue = i;
	}
}

OnDraw:

void CEightBitDrawingView::OnDraw(CDC* pDC/*pDC*/)
{

	CClientDC dc(this);
	CDC *mdc = GetDC ();

	HGDIOBJ tmp = mdc->SelectObject(hbm);

	mdc->FloodFill (0, 0, RGB (255, 255, 255));
	// Draw box:
	CBrush* br = new CBrush (RGB (255, 0, 255));
	CRect rect (30, 60, 130, 160);
	mdc->FillRect (&rect, br);

	mdc->TextOutW (50, 50, L"Testing...");

		CImage image;					
		//image.Attach ((HBITMAP)bm.m_hObject);
		image.Attach (hbm);
		HRESULT hr = image.Save(_T("C:\\test.tif"), Gdiplus::ImageFormatTIFF);


	GetClientRect(&rect);
	dc.BitBlt(0,0,rect.right,rect.bottom, mdc, 0,0,SRCCOPY);
	mdc->SelectObject (tmp);

	delete br;
}

Code-o-mat29-Jun-10 9:37

29-Jun-10 9:37

Am not sure where you are going wrong, i did a quick proggie to check and it produced a correct bitmap:

BITMAPINFO *bmi = (BITMAPINFO *)malloc(sizeof(BITMAPINFOHEADER) + sizeof(RGBQUAD) * 256);
BITMAPINFOHEADER &bih(bmi->bmiHeader);

bih.biSize = sizeof (BITMAPINFOHEADER);
bih.biWidth         = 200;
bih.biHeight        = -200;
bih.biPlanes        = 1;
bih.biBitCount      = 8;
bih.biCompression   = BI_RGB;
bih.biSizeImage     = 0;
bih.biXPelsPerMeter = 14173;
bih.biYPelsPerMeter = 14173;
bih.biClrUsed       = 0;
bih.biClrImportant  = 0;

for (int I = 0; I < 255; I++)
{
  bmi->bmiColors[I].rgbBlue = bmi->bmiColors[I].rgbGreen = bmi->bmiColors[I].rgbRed = (BYTE)I;
  bmi->bmiColors[I].rgbReserved = 0;
}

void *Pixels;
HBITMAP hbmp = CreateDIBSection(NULL, bmi, DIB_RGB_COLORS, &Pixels, NULL, 0);

//fill the bitmap so it wil display a gradient to check if the palette is correctly used...
BYTE *Pix = (BYTE *)Pixels;
for (int Y = 0; Y < abs(bih.biHeight); Y++)
  for (int X = 0; X < bih.biWidth; X++, Pix++)
    *Pix = (BYTE)Y;

free(bmi);

CImage Image;
Image.Attach(hbmp);
Image.Save(_T("r:\\test.bmp"), Gdiplus::ImageFormatBMP);

DeleteObject(hbmp);

Code-o-mat29-Jun-10 9:46

29-Jun-10 9:46

I also added this:

HDC dc    = ::GetDC(NULL);
HDC memdc = CreateCompatibleDC(dc);
::ReleaseDC(NULL, dc);

HGDIOBJ Ofont = ::SelectObject(memdc, GetStockObject(DEFAULT_GUI_FONT));
HGDIOBJ Obmp  = ::SelectObject(memdc, hbmp);

::SetTextColor(memdc, RGB(128, 128, 128));
::SetBkColor(memdc, RGB(64, 64, 64));
::TextOutA(memdc, 2, 2, "Hello world", 11);

::SelectObject(memdc, Ofont);
::SelectObject(memdc, Obmp);
DeleteDC(dc);

between free(bmi); and CImage Image;, this works fine too, i get the text in grays.

Alan Balkany29-Jun-10 9:47

29-Jun-10 9:47

Thanks again Code-o-mat! This gives me a lot to experiment with!

Code-o-mat29-Jun-10 9:55

29-Jun-10 9:55

No probs, i hope it helps, good luck.

Alan Balkany1-Jul-10 4:50

1-Jul-10 4:50

It worked!! Thanks so much! A big 5 points for you! (I'd give you more if I could.)

Code-o-mat1-Jul-10 5:57

Shaving a section from a dialog.

1-Jul-10 5:57

Great

, glad i could help, thanks for the points. Smile | :)

Spawn@Melmac29-Jun-10 4:46

29-Jun-10 4:46

Ok I know this is a bit of a cheat but my UI is split into 3rds with the idea being I can hide the bits of the UI.

Now I can slice off the right hand side easily enough as follows.

...
	CRect rect;
	GetWindowRect(rect);
...
		rect.right += CONSOLE_SIZE;
		MoveWindow(rect, 1);
...

The problem I am having is how do I cut the dialog's window down so that only the middle 3rd is displayed? I have had a look at CRgn with the SetWindowRgn but I am getting nowhere.

Any suggestions would be appreciated.

Many thanks

Alan

Code-o-mat29-Jun-10 5:13

29-Jun-10 5:13

You could try making your dialog a child of another dialog. You'd have a popup dialog and inside its client area a child dialog with its controls, thus you can shift this child dialog inside the popup's client area to show the part you wish.

On a sidenote: i don't know what kind of controls you have on your dialog, but if you use the keyboard, you can tab around controls even if they are not visible because they are "outside" of the visible client area. So if you tab onto a button you can press it by using SPACE even though you don't see it.

David Crow29-Jun-10 5:32

David Crow

29-Jun-10 5:32

Code-o-mat wrote:
...but if you use the keyboard, you can tab around controls even if they are not visible...

Unless they are also disabled, then they get skipped over.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"Man who follows car will be exhausted." - Confucius

Code-o-mat29-Jun-10 5:34

29-Jun-10 5:34

Of course...also if they are hidden by ShowWindow(SW_HIDE), my point was, just by "not being seen" doesn't mean "not there at all" Smile | :)

Spawn@Melmac29-Jun-10 5:51

29-Jun-10 5:51

I take your point but there are no input controls on the area being hidden.

Put simply, as a user of the system you only get to see the bits you have permission to see. All, Left+Middle or Right+Middle.

Alan

Code-o-mat29-Jun-10 5:58

29-Jun-10 5:58

All right, just trying to help. Smile | :)

David Crow29-Jun-10 5:37

David Crow

29-Jun-10 5:37

I find the easiest way to make such a dialog is to add a hidden 1x1 static control to the dialog where the expansion will take place. In your case, you'd need two. Then when the "shrink" button is clicked, you'd shrink the dialog's left and right edges up to those hidden controls. When the "expand" button is clicked, you'd grow the dialog's left and right edges back to their original position.

Spawn@Melmac29-Jun-10 5:55

29-Jun-10 5:55

hmmm... sounds a very messy way of cutting off the left hand edge.

The object is to show All, Left+Middle or Right+Middle.

Alan

David Crow29-Jun-10 6:11

David Crow

29-Jun-10 6:11

Spawn@Melmac wrote:
hmmm... sounds a very messy way of cutting off the left hand edge.

Not messy at all. In fact it is very clean and quite common to grow/shrink dialogs in this fashion.

Spawn@Melmac wrote:
The object is to show All, Left+Middle or Right+Middle.

Even though you initially stated, "...how do I cut the dialog's window down so that only the middle 3rd is displayed", what I described will work just fine for all seven possibilities.

Spawn@Melmac29-Jun-10 6:43

29-Jun-10 6:43

I knew I had missed an option. Yes it is All, Left+Middle, Right+Middle or Middle.

Thank you for the suggestion.

Alan