多线程图片下载程序的修改心得.

多线程图片下载程序的修改过程.
1.在void CThreadURLDlg::OnSave()里共产生16个线程:
for (int i=0; i<isel; i++)
{
m_param.strPath=sFilePath+"//"+strFileName.GetAt(i);
m_param.strURL=m_list.GetItemText(i,1);
m_param.hwnd=this->m_hWnd;//原始程序未初始化!
CWinThread* pThread = AfxBeginThread(SearchUrlThread,&m_param,THREAD_PRIORITY_NORMAL);
Sleep(10);//
//afxDump<< m_param.strURL<<"..."<<"/n";
}
Sleep(10)一句非常重要,这条语句的意思是叫当前线程睡眠10毫秒,其实质是让新产生的线程有运行时间.否则新产生线程的参数可能不对,
现象是strURL的值会重复,不但导致少下载一个或几个,还会在写文件时出现共享错误.
在没有Sleep()的时候,由于文件名会重复一次或几次,多线程共用函数SearchUrlThread(LPVOID pParam)中
CFile outFile (strFile, CFile::modeCreate|CFile::modeReadWrite);必须添加CFile::shareDenyNone,否则共享冲突.也就是有一个以上的
线程在写同一个文件.
附带的是OnSave()里条件判断值不对(i<isel-1),少产生一个线程.
2.内存泄漏问题严重,需要通过 delete HttpSendState; 及delete myHttpFile;解决
3.MYTHRURL m_param中的成员hwnd没初始化,见上.
typedef struct THRURL
{
CString strURL;
CString strPath;
HWND hwnd;
}MYTHRURL;

4.SearchUrlThread()
UINT SearchUrlThread(LPVOID pParam)
{
MYTHRURL *m_pParam=(MYTHRURL*)pParam;//!
CString url=m_pParam->strURL; //!
CString path=m_pParam->strPath; //!
....
}
以上3句打'!'的语句有特殊意义,不能直接用人口参数pParam,必须在栈内产生临时变量,以便线程切换时能保持正确的参数.
否则16个线程的参数会变成最后产生的线程的参数.使的只下载一个文件!