为什么这段代码运行后，程序会死掉？

[zqy1999]

我在视图类中定义了一个响应菜单命令的函数，目的是要画一个不断变大的矩形，并且在函数的执行过程中，程序要能继续响应操作。代码如下(很多其他的功能我已经简化、或删除掉了）：

void CTTView::OnStartRun()
{
CDC* pDC ;
pDC = this->GetDC();

CBrush brush( RGB(125,135,135) );
CBrush* OldBrush = pDC->SelectObject( &brush );

int j = 5000000;//做延时用

for( int i = 0; i <= 100; i++ )
{
//使程序能继续响应外部事件
if( PeekMessage(&msg, NULL, 0, 0, PM_REMOVE ) )
{
::TranslateMessage( &msg );
::DispatchMessage( &msg );
}

//画矩形（x1, y1, x2, y2已经在程序的其他地方定义）
pDC->Rectangle( x1, y1, x2, y2 );
x2 += 1;
y2 += 1;

while( j-- );//延时
j = 5000000;
}

pDC->SelectObject( OldBrush );
this->ReleaseDC( pDC );///!!!!!!!这行代码好象有问题。

}
程序运行起来后，我点击这个菜单命令，在这个函数运行到一半还未结束的时候，执行关闭程序的操作，则系统弹出一个异常错误对话框。当我去掉this->ReleaseDC( pDC )（函数中的最后一行代码）这句代码后，则程序就不发生错误了。
在下不知道到底是什么原因，导致这个严重的错误，希望各位兄弟指教。

[leech42]

这样子那行代码去掉不就行了.

[yu_hl]

这样：
void CTTView::OnStartRun()
{
CBrush brush( RGB(125,135,135) );

int j = 5000000;//做延时用

for( int i = 0; i <= 100; i++ )
{
//使程序能继续响应外部事件
if( PeekMessage(&msg, NULL, 0, 0, PM_REMOVE ) )
{
::TranslateMessage( &msg );
::DispatchMessage( &msg );
}

CDC* pDC ;
pDC = this->GetDC();

CBrush* OldBrush = pDC->SelectObject( &brush );
//画矩形（x1, y1, x2, y2已经在程序的其他地方定义）
pDC->Rectangle( x1, y1, x2, y2 );
x2 += 1;
y2 += 1;

while( j-- );//延时
j = 5000000;

pDC->SelectObject( OldBrush );
this->ReleaseDC( pDC );///!!!!!!!这行代码好象有问题。
}

}

[zqy1999]

我想知道为什么？真诚的请教各位，谢谢！

[zcpro]

那是因为你点关闭后，窗口就被销毁了，然后你调用this->ReleaseDC( pDC );当然会出错。
你可以这样改一下：
if(GetSafeHwnd())
ReleaseDC(pDC);

[zqy1999]

zcpro:
还是不行啊？

[snsins]

用线程做吧

[zcpro]

if(GetSafeHwnd() && pDC)
ReleaseDC(pDC);
再试试

[ccnuxjg]

agree with zcpro(zcpro)
那是因为你点关闭后，窗口就被销毁了，然后你调用this->ReleaseDC( pDC );当然会出错。

if( PeekMessage(&msg, NULL, 0, 0, PM_REMOVE ) )
{
::TranslateMessage( &msg );
::DispatchMessage( &msg );
}

关闭程序的消息也在这里处理，所以当你点击关闭按钮后，没法执行后面的
this->Release(pDC)

[zqy1999]

这样：if(GetSafeHwnd() && pDC)
ReleaseDC(pDC);
还是不行。我不想用多线程。

[blue_flash]

方法不对，使用线程吧，
还有重画的机制部队
this->ReleaseDC( pDC );///这句必须删掉

[zcpro]

你对WM_CLOSE单独处理一下吧
if( PeekMessage(&msg, NULL, 0, 0, PM_REMOVE ) )
{
if(msg == WM_CLOSE)
{
//这里放你的资源释放代码,如 ReleaseDC(pDC);
//再重发一次close消息
PostMessage(WM_CLOSE,...);
return;
}
::TranslateMessage( &msg );
::DispatchMessage( &msg );
}

[fhbkyo]

先删除画刷然后释放环境变量

[zqy1999]

这些方法都不行，到底是怎么回事呢？

[awanghero]

你可用跟踪调试，看看在什么地方放掉了资源
比如窗口不再显示，你再调用一些与之相关的操作，系统会报一些异常的信息，通常在这些调用之前,mfc会调用一些，比如以下的代码
ASSERT(WIDOWs_HANDLE),你可以利用这些代码，找出真正的原因所在！
调试的方法是，逐步缩小错误范围。
第一遍找出出错的那个函数调用
第二遍跟踪进去
。。。
直到找到好个ASSERT宏
good luck!!!!!

[ju_feng]

我运行没问题啊？！

是你程序别的地方的问题吧！

[zqy1999]

不会吧？在你的机器上不死？
虽然现在我也没有能找到原因，不过对于上面几位朋友的帮助真的非常非常感谢！