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

多线程图片下载程序的修改过程.
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个线程的参数会变成最后产生的线程的参数.使的只下载一个文件!

发布了32 篇原创文章 · 获赞 10 · 访问量 4万+
發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章