[RESOLVED] about fonts

[Cambalinho]

on my label class:

private:
      CHOOSEFONT chFont;
public:
CHOOSEFONT getFont()
    {
        return chFont;
    }

    void setFont(const CHOOSEFONT font)
    {
        chFont=font;
        clrTextColor=chFont.rgbColors;
        HFONT hFont=CreateFontIndirect(chFont.lpLogFont);
        SendMessage(hwnd, WM_SETFONT,(WPARAM) hFont, TRUE);//setting the control font for resize the control
        if(blnAutoSize==true)//the autosize is true
            setAutoSize(true);
    }

and see how i show the font dialog(using the label font):

CHOOSEFONT ShowSelectFont(const CHOOSEFONT cf1)
{
    CHOOSEFONT cf = {sizeof(CHOOSEFONT)};
    cf=cf1;
    HWND hwnd=GetForegroundWindow();

    cf.Flags = CF_EFFECTS | CF_INITTOLOGFONTSTRUCT | CF_SCREENFONTS;
    cf.hwndOwner = hwnd;
    ChooseFont(&cf);
    return cf;
}

the lpLogFont, CHOOSEFONT class member, it's a pointer. the problem is the font dialog isn't showed. what i belive is that i lose the pointer between the getFont() and ShowSelectFont(). what anyone can tell me?

[VladimirF]

1. CHOOSEFONT is a structure, NOT a pointer.
2. You need to check the return value from ChooseFont(&cf). If it is 0 (FALSE), you should call CommDlgExtendedError(); to see what was wrong.
3. You didn't show how you create CHOOSEFONT structure in the first place. Did you set its lStructSize member? To what?

[Cambalinho]

1. CHOOSEFONT is a structure, NOT a pointer.
2. You need to check the return value from ChooseFont(&cf). If it is 0 (FALSE), you should call CommDlgExtendedError(); to see what was wrong.
3. You didn't show how you create CHOOSEFONT structure in the first place. Did you set its lStructSize member? To what?

1 - the CHOOSEFONT don't but the lpLogFont member is
3 - thanks. i didn't notice that :P
now the code works fine.

anotherthing: please see these function:

CHOOSEFONT ShowSelectFont()
{
    HWND hwnd=GetForegroundWindow();
    CHOOSEFONT cf = {sizeof(CHOOSEFONT)};
    LOGFONT* lf = new LOGFONT;
    HDC hdc=GetDC(hwnd);

    HFONT hlf =(HFONT) GetCurrentObject(hdc,OBJ_FONT);

    GetObject(hlf,sizeof(LOGFONT),lf);
    cf.Flags = CF_EFFECTS | CF_INITTOLOGFONTSTRUCT | CF_SCREENFONTS;
    cf.hwndOwner = hwnd;
    cf.lpLogFont = lf;
    cf.rgbColors =GetTextColor(hdc);
    ChooseFont(&cf);
    ReleaseDC(hwnd,hdc);
    cf.lpLogFont = lf;
    return cf;
}

the lf use 'new', but i'm not using the 'delete'. so it's a memory leak. but honestly i can't use the 'delete', or i lose the lf data. can you correct me these?

[VictorN]

1 - the CHOOSEFONT don't but the lpLogFont member is
3 - thanks. i didn't notice that :P
now the code works fine.

anotherthing: please see these function:

CHOOSEFONT ShowSelectFont()
{
    HWND hwnd=GetForegroundWindow();
    CHOOSEFONT cf = {sizeof(CHOOSEFONT)};
    LOGFONT* lf = new LOGFONT;
    HDC hdc=GetDC(hwnd);

    HFONT hlf =(HFONT) GetCurrentObject(hdc,OBJ_FONT);

    GetObject(hlf,sizeof(LOGFONT),lf);
    cf.Flags = CF_EFFECTS | CF_INITTOLOGFONTSTRUCT | CF_SCREENFONTS;
    cf.hwndOwner = hwnd;
    cf.lpLogFont = lf;
    cf.rgbColors =GetTextColor(hdc);
    ChooseFont(&cf);
    ReleaseDC(hwnd,hdc);
    cf.lpLogFont = lf;
    return cf;
}

the lf use 'new', but i'm not using the 'delete'. so it's a memory leak. but honestly i can't use the 'delete', or i lose the lf data. can you correct me these?

But why do you pass the CHOOSEFONT from a function as the return value?
Why not just pass it to the function as a parameter?
Like (reference)

BOOL ShowSelectFont(CHOOSEFONT& chooseFont )

or (pointer)

BOOL ShowSelectFont(CHOOSEFONT* chooseFont )

[Cambalinho]

But why do you pass the CHOOSEFONT from a function as the return value?
Why not just pass it to the function as a parameter?
Like (reference)

BOOL ShowSelectFont(CHOOSEFONT& chooseFont )

or (pointer)

BOOL ShowSelectFont(CHOOSEFONT* chooseFont )

honestly, by my experience, is more easy my way. but thinking more... i belive it's a opinion... it depends on programmer way

[2kaud]

Think RAII

[Cambalinho]

Think RAII

i'm sorry... what means RAII?

[2kaud]

Resource Acquisition Is Initialisation. See http://stackoverflow.com/questions/2...alization-raii and many other internet articles. If you program with c++ this is something with which you really should be familiar.

[VictorN]

i'm sorry... what means RAII?

A very fast search with Google shows you https://www.google.ch/search?q=RAII&..._sm=0&ie=UTF-8 where every from the first 10 items explains you what it is!

[Cambalinho]

i never ear about RAII, now i more or less understand it.
but i continue with my last question\confusion: why i can't use 'delete'?
the rule is: when we use the 'new' we must use the 'delete', when isn't needed. but i can't, in these case, delete the lf or i lose the data

[2kaud]

i never ear about RAII, now i more or less understand it.
but i continue with my last question\confusion: why i can't use 'delete'?
the rule is: when we use the 'new' we must use the 'delete', when isn't needed. but i can't, in these case, delete the lf or i lose the data

So you don't understand RAII. The new and the corresponding delete go in a class - with delete in the destructor so memory is not 'lost'.

[Cambalinho]

So you don't understand RAII. The new and the corresponding delete go in a class - with delete in the destructor so memory is not 'lost'.

understood... thanks for all

[Cambalinho]

correct me anotherthing about fonts color and control backcolor : when i put the child control transparent(regions in pixel way) a see 1 color arround the letters(not a backcolor)... why these happens?

[Igor Vartanov]

Font has nothing to do with color.

SetTextColor function
SetBkColor function
SetBkMode function

[Cambalinho]

Font has nothing to do with color.

SetTextColor function
SetBkColor function
SetBkMode function

i continue with that white color
what i can change is instead be white, been black. but i can't avoid that color