Discussion :

[Kevgeii]

Bonjour,

J'ai une application dans laquelle j'ai voulut changer la couleur par défaut de la boite "A propos de"
Pour cela j'ai utilisé ce code:

Code :

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

1
2
3
4
5
6
7
8
 
HBRUSH CAboutDlg::OnCtlColor(CDC* pDC, CWnd* pWnd, UINT nCtlColor) 
{
	HBRUSH hbr = ::CreateSolidBrush(RGB(230,237,250));
	pDC->SetBkMode(TRANSPARENT);
 
	return hbr;
}

Le problème est que avec cette methode quand j'ouvre la boite des objets GDI sont créés (on les voies dans le taskmgr) mais ne sont jamais libérés.

J'ai essayé cela :

Code :

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

1
2
 
ReleaseDC(pDC);

Mais cela ne fonctionne pas car la boite ne dessine pas et en plus les GDI sont quand même créés.

Comment faire SVP ?

[farscape]

salut,
il faut faire sur la brush

Code :

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

1
2
3
4
 
BOOL DeleteObject(
HGDIOBJ hObject
);

en plus de relacher le dc à la fin.

[Kevgeii]

Je ne peux pas faire cela puisque je doit retourner la brush, alors je ne peux pas le détruir !!!

[pipin]

Bonjour,

Vous n'etes pas obligé de creer le HBRUSH dans la fonction,
par exemple vous pouvez le creer dans le constructeur de la dlg avec
une variable membre et le supprimer dans son destructeur.
Et dans la fonction vous retournez la variable membre.

Cordialement

[farscape]

exact comme le precise pipin il faut que ta HBRUSH soit une donnée membre
de ta classe et sur le destructeur tu fais DeleteObject en testant si le handle est <> de zero .
dans le constructeur je le mets à zero .

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
 
CAboutDlg::CAboutDlg(CWnd* pParent /*=NULL*/)
	: CDialog(CAboutDlg::IDD, pParent)
{
     m_hbr=NULL;
}
CAboutDlg::~CAboutDlg
{
   if( m_hbr) ::DeleteObject(m_hbr);
}
HBRUSH CAboutDlg::OnCtlColor(CDC* pDC, CWnd* pWnd, UINT nCtlColor)
{
   if(! m_hbr) m_hbr = ::CreateSolidBrush(RGB(230,237,250)); 
   pDC->SetBkMode(TRANSPARENT);
   return m_hbr;
}

[Kevgeii]

Oui merci,

Il faut donc déclarer une Brush en varaible membre de la classe

Merci à tous