Set bitmap into CStatic object

Question

I'm trying to add one bitmap into a picture box. This is what I've tried:

void DlgError::Define_Image()
{
    // I generate all requiered elements.
    CBitmap bitmap;

    // Depending on the type of the message, I define the image to show.
    switch (Message_Type)
    {
        case Error: bitmap.LoadBitmap(IDB_ERROR); break;
        case Warning: break;

        case Information:
        default: bitmap.LoadBitmap(IDB_INFORMATION); break;
    }

    // I set the new picture.
    m_picture.ModifyStyle(0, SS_BITMAP);
    m_picture.SetBitmap(bitmap);
}

I don't know why it's not working. I have the same code I've in some forums. Can anybody help me? I have to define some extra styles?

The picture box has Bitmap as its type (Can be hanged in Picture control properties)

SOLUTION

I must say that the solution is a combination of both answers. Finally, I define one local variable and I use the next code in order to do what I want.

void DlgError::Define_Image()
{
    // Depending on the type of the message, I define the image to show.
    switch (Message_Type)
    {
        case Error: bitmap.LoadBitmap(IDB_ERROR); break;
        case Warning: break;

        case Information:
        default: bitmap.LoadBitmap(IDB_INFORMATION); break;
    }

    // I set the new picture.
    m_picture.ModifyStyle(SS_ENHMETAFILE, SS_BITMAP);
    m_picture.SetBitmap(bitmap);
}

Thanks everyone who helped me!

Try adding SS_CENTERIMAGE style. – EylM Nov 25 '19 at 12:27 — EylM, Nov 25 '19 at 12:27
@EyIM Still not working... – RookieCoder Nov 25 '19 at 14:22 — RookieCoder, Nov 25 '19 at 14:22
Use SS_TYPEMASK not SS_ENHMETAFILE – xMRi Nov 26 '19 at 08:07 — xMRi, Nov 26 '19 at 08:07

score 3 · Answer 1 · answered Nov 25 '19 at 13:19

3

Are you sure that LoadBitmap succeeded?

Isn't m_picture already created with the matching style if not your code might fail.

SS_BITMAP is a number. If you want to change the tsyle of the static control, you need to remove the old style:

m_picture.ModifyStyle(SS_TYPEMASK,SS_BITMAP);

Otherwise you get side effects. Check the style with Spy++.

answered Nov 25 '19 at 13:19

xMRi

14,982
3
26
59

I tried what you say but still appears as a solid gray rect. I tried to ensure that the method succed using: `ENSURE(m_picture.SetBitmap(bitmap));` and it throws me an exception. – RookieCoder Nov 25 '19 at 14:16
Is the handle inside bitmap not NULL? – xMRi Nov 25 '19 at 15:14
Thanks for your help! It was really helpfull! – RookieCoder Nov 25 '19 at 16:10

score 3 · Accepted Answer · answered Nov 25 '19 at 15:00

3

You are using the CBitmap object as a local variable in the Define_Image function. Once the function goes out of scope, the bitmap is destroyed by the destructor of CBitmap object (there are no ref-counts here).

You need to detach the object and then pass the HBITMAP handle to SetBitmap of CStatic.

Fortunatly, only a single line is required:

 m_picture.SetBitmap((HBITMAP)bitmap.Detach());

answered Nov 25 '19 at 15:00

EylM

5,967
2
16
28

1

Thanks for teach me about the destruction of the object! Your answer was really helpfull – RookieCoder Nov 25 '19 at 16:09

Set bitmap into CStatic object

2 Answers2