cnzdgs：再帮我解决一下问题

[edisonli]

昨天那个线程退出问题还是没有解决，我把下载重新做了一下，用了网上的HttpDownload下载类，但是还是不能解决线程退出问题，产生"内存不能为read"的错误，我知道你是高手，麻烦你帮我解决一下
代码如下
CWinThread *g_pMyThr;
BOOL m_blFlag;
BOOL CUpdateDlg::OnInitDialog()
{
CDialog::OnInitDialog();
TE_InitLibrary();
return TRUE;
}
void CUpdateDlg::OnOK()
{
// TODO: Add extra validation here
m_blFlag=true;
g_pMyThr=AfxBeginThread(DownloadThread,(void *)this);
}
UINT CUpdateDlg::DownloadThread(void *pArg)
{
CUpdateDlg *pMainWnd = (CUpdateDlg *)pArg;
while(pMainWnd->m_blFlag)
{
pMainWnd->down.Download("http://www.ienno.com/upfile/user/lzr/1/update/1.exe","d:\\1.exe");
}
return 0;
}

void CUpdateDlg::OnCancel()
{
// TODO: Add extra cleanup here
m_blFlag = FALSE;
tagMSG msg;
while(PeekMessage(&msg, NULL, 0, 0, PM_REMOVE))
{
DispatchMessage(&msg);
if(WAIT_OBJECT_0 == WaitForSingleObject(g_pMyThr->m_hThread, 100))
{
delete g_pMyThr;
g_pMyThr = NULL;
break;
}
}
TE_CleanupLibrary();
}

[cnzdgs]

AfxBeginThread创建的CWinThread对象默认是自动释放的，把delete g_pMyThr去掉。

[edisonli]

去掉了还是不行
UINT CUpdateDlg::DownloadThread(void *pArg)
{
CUpdateDlg *pMainWnd = (CUpdateDlg *)pArg;

while(pMainWnd->m_blFlag)
{
pMainWnd->down.Download("http://www.ienno.com/upfile/user/lzr/1/update/1.exe","d:\\1.exe");
}
AfxMessageBox("退出循环");
return 0;
}
退出的时候“退出循环”这个对话框根本都显不出来就报错了

[edisonli]

cnzdgs:我把源码传到网上，我知道你是好人，你帮我看看吧，谢谢你了
http://www.ienno.com/ienno.rar

[cnzdgs]

我不方便调试你的代码，你再这样修改试试：
void CUpdateDlg::OnCancel()
{
m_blFlag = FALSE;
while(TRUE)
{
tagMSG msg;
if (PeekMessage(&msg, NULL, 0, 0, PM_REMOVE))
DispatchMessage(&msg);
if (WAIT_OBJECT_0 == WaitForSingleObject(g_pMyThr->m_hThread, 100))
{
g_pMyThr = NULL;
break;
}
}
TE_CleanupLibrary();
}

[cnzdgs]

void CUpdateDlg::OnCancel()
{
HANDLE hThread = g_pMyThr->m_hThread;
m_blFlag = FALSE;
while(TRUE)
{
tagMSG msg;
if (PeekMessage(&msg, NULL, 0, 0, PM_REMOVE))
DispatchMessage(&msg);
if (WAIT_OBJECT_0 == WaitForSingleObject(hThread, 100))
{
g_pMyThr = NULL;
break;
}
}
TE_CleanupLibrary();
}

[ninehoolu]

楼上确是是好人，呵呵，无意间路过~

[edisonli]

没有抱错了，但是线程确没有停止，还是在继续运行

[stivenjia]

那肯定的了！
都
while(1)
{

}了如果不死循环都见鬼了

[edisonli]

没有死循环啊，我有m_blFlag = FALSE;

[edisonli]

if (WAIT_OBJECT_0 == WaitForSingleObject(hThread, 100))
{
这里面的没有执行
}

[liumeiqinyang]

WAIT_OBJECT_0 == WaitForSingleObject(hThread, 100) 应该在线程函数的循环里检查的，在oncancel里设置信号量，你弄反了

[cnzdgs]

再这样试试：
if (WAIT_TIMEOUT != WaitForSingleObject(hThread, 100))

[Derekfan]

cnzdgs 厉害

[wjx_0_2001]

cnzdgs 的确有实力

人品也很有实力