这段截屏代码有内存泄漏问题么?

[chengyumao]

CopyScreenToBitmap(LPRECT lpRect) //lpRect 代表选定区域
{
HDC hScrDC, hMemDC;
// 屏幕和内存设备描述表
HBITMAP hBitmap,hOldBitmap;
// 位图句柄
int nX, nY, nX2, nY2;
// 选定区域坐标
int nWidth, nHeight;
// 位图宽度和高度
int xScrn, yScrn;
// 屏幕分辨率
// 确保选定区域不为空矩形
if (IsRectEmpty(lpRect))
return NULL;
//为屏幕创建设备描述表
hScrDC = CreateDC(_TEXT("DISPLAY"), NULL, NULL, NULL);
if (hScrDC==0) MessageBox(_T("CreateDC Fail"),NULL,MB_OK);
//为屏幕设备描述表创建兼容的内存设备描述表
hMemDC = CreateCompatibleDC(hScrDC);
if (hMemDC==0) MessageBox(_T("CreateCompatibleDC Fail"),NULL,MB_OK);
// 获得选定区域坐标
nX = lpRect->left;
nY = lpRect->top;
nX2 = lpRect->right;
nY2 = lpRect->bottom;
// 获得屏幕分辨率
xScrn = GetDeviceCaps(hScrDC, HORZRES);
yScrn = GetDeviceCaps(hScrDC, VERTRES);
//确保选定区域是可见的
if (nX < 0)
nX = 0;
if (nY < 0)
nY = 0;
if (nX2 > xScrn)
nX2 = xScrn;
if (nY2 > yScrn)
nY2 = yScrn;
nWidth = nX2 - nX+1;
nHeight = nY2 - nY+1;
// 创建一个与屏幕设备描述表兼容的位图
hBitmap=CreateCompatibleBitmap(hScrDC,nWidth,nHeight);
if (hBitmap==0) MessageBox(_T("hBitmap Create Fail"),NULL,MB_OK);
// 把新位图选到内存设备描述表中
hOldBitmap=(HBITMAP)SelectObject(hMemDC,hBitmap);
// 把屏幕设备描述表拷贝到内存设备描述表中
BitBlt(hMemDC,0,0, nWidth,nHeight,hScrDC, nX, nY, SRCCOPY);
//得到屏幕位图的句柄
hBitmap=(HBITMAP)SelectObject(hMemDC,hOldBitmap);
//清除
if (DeleteDC(hScrDC)==0) MessageBox(_T("hScrDC Fail"),NULL,MB_OK);
if (DeleteDC(hMemDC)==0) MessageBox(_T("hMemDC Fail"),NULL,MB_OK);
// 返回位图句柄
return hBitmap;

这是网上搜来的一段截屏代码,在我的程序里每秒调用一次....大概一个小时左右开始出错......但始终没看出哪里有问题啊.........请高手指教....Orz...

[chengyumao]

附上调用的函数内容
OnBnClickedButtonGetrepair()
{
CRect rect;
float val=0.0;
CString StrNow;
rect.SetRect(2,393,129,429);
HBITMAP Hlocalmap;
Hlocalmap = CopyScreenToBitmap(rect);
//val = Dllfun(Hlocalmap);<---------调试的时候注释掉了......仍然出错...
}
曾经想过是否Hlocalmap 需要 在函数最后 DeleteObject 但总是失败...........大家帮忙看看啊....谢谢了....

[chengyumao]

出错时首先是
hMemDC = CreateCompatibleDC(hScrDC);
if (hMemDC==0) MessageBox(_T("CreateCompatibleDC Fail"),NULL,MB_OK);
然后
hBitmap=CreateCompatibleBitmap(hScrDC,nWidth,nHeight);
if (hBitmap==0) MessageBox(_T("hBitmap Create Fail"),NULL,MB_OK);
最后
if (DeleteDC(hMemDC)==0) MessageBox(_T("hMemDC Fail"),NULL,MB_OK);
......估计hScrDC = CreateDC(_TEXT("DISPLAY"), NULL, NULL, NULL);时就有问题了...但没看到出错的CreateDC Fail消息窗口...........

[GGYYBOY]

mark

[jhs1982419]

句柄没有释放,有可能造成内存泄露.

[chengyumao]

...这个...hOldBitmap还好说.....但那个hBitmap是要返回的对像...如果释放了不是没有返回值了??..................

[lyg_zy]

hScrDC = CreateDC(_TEXT("DISPLAY"), NULL, NULL, NULL);
改为
GetDC()
最后
ReleaseDC()试试

你的问题应该是GDI资源泄露。不是内存泄露。

[lyg_zy]

另外把dc,memdc,bitmap都提为类成员变量。不要每次都去创建。这样，太影响效率了。

[hediant]

句柄hBitmap和hOldBitmap没有释放，请使用DeleteObject (hBitmap)。

另外是否有内存泄漏，调试器会给予提示的，请打开该选项；还可以在任务管理器的性能中查看内存是否持续增长。

[hediant]

如果楼主是每秒钟刷新一次，那要创建多少个新的Bitmap的句柄啊？！如果实在需要这样，建议把内容存在硬盘上，然后每次都释放句柄。

lyg_zy(学无止境,宁静致远) ，GDI资源泄漏，实质上也是内存泄漏的一种吧：）

[cpio]

打开Windows任务管理器，把GDI对象列选上

运行你的程序，看那个值是不是一直在增加，当增加到很大时就出错

如果是这样，不是有些H对象没有删除