贴上一段10个语句的函数,大家看看有什么问题

[xxfxiazai]

小弟乃VC初学者,望大家帮助

void CzimuView::draw(char zimu, int pointx) 
{
	CDC* ppdc=GetDC ();
	CBitmap bmp;
	pdc.CreateCompatibleDC (NULL);
	bmp.CreateCompatibleBitmap (ppdc,20,20);
	CBitmap *pdolbit=pdc.SelectObject (&bmp);
	pdc.FillSolidRect (0,0,20,20,RGB(20,200,230));
	//pdc.DrawEdge (CRect(0,0,20,20),EDGE_BUMP,BF_RECT);
	pdc.Rectangle (0,0,20,20);
	pdc.TextOutW (0,0,(static_cast<CString>(zimu)),1);
	this->bmplist .AddHead (&bmp);
	ppdc=0;
}

[xxfxiazai]

另:
pdc 为全局 CDC

[ahao]

资源泄漏得一塌糊涂
加上保存栈对象指针

[jyh_baoding]

好象有问题

[快乐鹦鹉]

pdc.CreateCompatibleDC (NULL);
==pdc哪里来的？，参数为什么是NULL？而不是ppdc
bmp是临时变量，加到bmpList也没有用。出了这个函数就废了。
这个函数想干啥？下次调用时就会出毛病了。

[xxfxiazai]

[Quote=引用 4 楼 happyparrot 的回复:]
pdc.CreateCompatibleDC (NULL);
==pdc哪里来的？，参数为什么是NULL？而不是ppdc
bmp是临时变量，加到bmpList也没有用。出了这个函数就废了。
这个函数想干啥？下次调用时就会出毛病了。
[/Quote]
pdc是函数外定义的全局变量
把BMP的地址加进去也不行的吗.
确实是下次调用就出错了,楼上大虾能否再解释解释
谢谢

[快乐鹦鹉]

光是地址有啥用啊。地址指向的东西在出了这个函数后就被系统回收了，也许立刻被其它系统给用掉了。就像你光有一个家庭住址，可现在那个地方都归其它国家管了，原来的房子早就被砸了。你这时候还说那是你家，谁还认啊。
至于下次出错，你这个pdc在这个函数中反复创建为兼容dc，却从来没有释放过。要能够成为兼容dc，那么它自己的HDC就必须为0。显然你这个不是。

[xxfxiazai]

哦.明白了,你说的变量被回收我明白了.
但是关于DC我还是不太明白
该怎么改,这段代码才能重复运行

[Lin]

问题有2：
1、存储局部变量CBitmap bmp的指针，超出变量作用域使用，迟早会出问题；
2、GetDC请与ReleaseDC配套使用，避免ppdc=NULL这类危险代码，如：
ReleaseDC(ppdc);
ppdc=NULL;

[qq14923349]

散分块没了，只能来接分，见谅