程序运行一段时间就死机！有代码

ldrxxx

程序运行一段时间就死机！有代码 [复制链接]

最近做一个ＣＥ下的程序，对系统稳定性要求很高，能长时间开机工作（运行几个月应该没问题）.可是我这程序运行１个小时左右就会死掉．前几天是有内存泄露，但我现在把这问题解决了．可是还是不好用．我先说下我的思路：
我这个要求能播放图片，我在程序中设置了两个线程Ａ，Ｂ；Ａ不停的读一个节目播放列表文件，每次读一个播放单元，Ｂ中不停检测时候有播放单元，如果有则启动一个线程Ｃ，然后把Ｃ中的图片ＢＭＰ１，ＢＩＴＢＬＴ到另一个图片上，贴出来，而Ｃ中做的工作是把播放单元的图片读出来，并按一定动作画到图片ＢＭＰ１上，．当Ｃ中动作完成后退出，这个时候Ｂ就把播放单元信息释放掉．等待新的播放单元，而Ａ检测到播放单元释放后会读新的播放单元．
这个就是我的思路．下面是部分代码，大家帮忙找找原因　谢谢

DWORD WINAPI Thread_ReadPlayList(LPVOID lpParam)
{
FILE * f;
f=_wfopen(PlayListPath,L"rb");
if(NULL==f)
{
//传送错误信息:文件打开失败
MessageBoxW(NULL,L"Open file false",L"",MB_OK);
return 1;
}
int nWidth,nHeight;
int nTemp,nUsedLabelItem;
Arm_Label_Node *pLabel;
fread(&nWidth,1,sizeof(int),f);
fread(&nHeight,1,sizeof(int),f);
if(nWidth!=g_nScreenWidth||nHeight!=g_nScreenHeight)
{
//传送错误信息:节目列表错误，不是当前屏幕的节目列表
MessageBoxW(NULL,L"width && Heiht false",L"",MB_OK);
return 1;
}
while(!g_bStop)
{//{{{
while(0!=g_PlayNode.nType)//非空结点
{
Sleep(1);
}
WaitForSingleObject(hMutexNode,INFINITE);
READNEXTINFO:
fread(&nTemp,1,sizeof(int),f);

CString str;
str.Format(L"nTemp=%d",nTemp);

if(feof(f))
{
fseek(f,8,SEEK_SET);
goto READNEXTINFO;
}
switch(nTemp)
{
case CLOCK_Type:
g_pClockNode=(Arm_Clock_SetInfo *)malloc(sizeof(Arm_Clock_SetInfo));
fread(g_pClockNode,1,sizeof(Arm_Clock_SetInfo),f);
hClockFont=CreateFontIndirect(&g_pClockNode->logFont);
break;
case PIC_Type:
g_PlayNode.nType=PIC_Type;
g_PlayNode.pPointer=malloc(sizeof(Arm_Pic_SetInfo));
fread(g_PlayNode.pPointer,1,sizeof(Arm_Pic_SetInfo),f);
g_PlayNode.PathName=
(TCHAR *)malloc(((Arm_Pic_SetInfo *)g_PlayNode.pPointer)->nPathLength*sizeof(TCHAR));
fread(g_PlayNode.PathName,
((Arm_Pic_SetInfo *)g_PlayNode.pPointer)->nPathLength,
sizeof(TCHAR),f);
break;
case LABEL_Type:

if(NULL==g_pLabelNode)
{
g_pLabelNode=(Arm_Label_Node *)malloc(sizeof(Arm_Label_Node));
pLabel=g_pLabelNode;
}else
{
pLabel=g_pLabelNode;
while(NULL!=pLabel->next)
pLabel=pLabel->next;
pLabel->next=(Arm_Label_Node *)malloc(sizeof(Arm_Label_Node));
pLabel=pLabel->next;
}
pLabel->next=NULL;
pLabel->pArm_Label_SetInfo=(Arm_Label_SetInfo *)
malloc(sizeof(Arm_Label_SetInfo));
fread(pLabel->pArm_Label_SetInfo,1,sizeof(Arm_Label_SetInfo),f);
fread(pLabel->tContent,pLabel->pArm_Label_SetInfo->nContentLength,sizeof(TCHAR),f);
hLabelFont[nUsedLabelItem++]=CreateFontIndirect(&pLabel->pArm_Label_SetInfo->logFont);
goto READNEXTINFO;
break;
case NEWNODE:
if(1)
{
nUsedLabelItem=0;
for(int i=0;i {
if(NULL!=hLabelFont)
{
int nRes=DeleteObject(hLabelFont);
hLabelFont=NULL;
if(0==nRes)
MessageBoxW(NULL,L"DeleteObject hLabelFont",L"ERROR",MB_OK);
}
}
if(NULL!=hClockFont)
{
int nRes=DeleteObject(hClockFont);
hClockFont=NULL;
if(0==nRes)
MessageBoxW(NULL,L"DeleteObject hClockFont",L"Error",MB_OK);
}
}
while(NULL!=g_pLabelNode)
{
pLabel=g_pLabelNode->next;
free(g_pLabelNode->pArm_Label_SetInfo);
free(g_pLabelNode);
g_pLabelNode=pLabel;
}
if(NULL!=g_pClockNode)
free(g_pClockNode);
g_pClockNode=NULL;
g_pLabelNode=NULL;
case ENDNODE:
// MessageBoxW(NULL,L"EndNode",L"ReadPlayList",MB_OK);
goto READNEXTINFO;
break;
}
ReleaseMutex(hMutexNode);
}//}}}
//检查未释放资源，进行释放
return 1;
}
DWORD WINAPI Thread_Play(LPVOID lpParam)
{
HDC hdc=::GetDC(g_hWnd);
HANDLE handle;
Arm_Label_Node *pLabel;
while(!g_bStop)
{//{{{
while(0==g_PlayNode.nType)
{
Sleep(1);
}
WaitForSingleObject(hMutexNode,INFINITE);
switch(g_PlayNode.nType)
{
case PIC_Type:
g_bThreadStop=FALSE;
//启动一个图片线程
handle=CreateThread(NULL,0,Thread_PlayPic,NULL,0,NULL);
CloseHandle(handle);
break;
}
while(!g_bStop&&!g_bThreadStop)
{
BitBlt(g_hTopDC,0,0,g_nScreenWidth,g_nScreenHeight,g_hMiddleDC,0,0,SRCCOPY);
pLabel=g_pLabelNode;
int nItem=0;
while(NULL!=pLabel)
{
DrawLabel(pLabel,nItem++);
pLabel=pLabel->next;
}
if(NULL!=g_pClockNode)
DrawClock(g_pClockNode);
BitBlt(hdc,0,0,g_nScreenWidth,g_nScreenHeight,g_hTopDC,0,0,SRCCOPY);
Sleep(5);
}
g_PlayNode.nType=0;
free(g_PlayNode.pPointer);
free(g_PlayNode.PathName);
ReleaseMutex(hMutexNode);
}//}}}
return 1;
}
DWORD WINAPI Thread_PlayPic(LPVOID lpParam)
{
CIImage mage;
CString strPath;
strPath=PicPathExtent;
strPath+=g_PlayNode.PathName;
mage.Open(strPath);
mage.Draw(g_hIImageDC,0,0,g_nScreenWidth,g_nScreenHeight,0);
int i=0;
while(1)
{
BitBlt(g_hMiddleDC,0,0,i,g_nScreenHeight,g_hIImageDC,g_nScreenWidth-i,0,SRCCOPY);
Sleep(5);
i++;
if(i>=g_nScreenWidth)
break;
}
g_bThreadStop=TRUE;
return 1;
}

michael52

DWORD WINAPI Thread_ReadPlayList(LPVOID lpParam)
中的问题：
1、打开文件后，在每个退出点都不关闭。
2、switch(nTemp) 中都不处理 default
3、不检查指针是否有值就直接分配。g_pClockNode=(Arm_Clock_SetInfo *)malloc(sizeof(Arm_Clock_SetInfo))
不要认为你的文件都按照你认为的方式排列，只要有一点顺序问题，你的程序就会crash.
4、居然还有goto, 看得很晕。

建议加入log，然后出问题后看log 就行了

jiangyebula

楼上说的好，那些fopen之类的函数一定要fclose，有配对函数的调用一定要注意。

馨怡

最好是自己加入调试语句，看看问题可能出现在什么地主

wmj0402

重贴一下代码吧，格式都丢了。

zhull1984

哎，格式乱啊！
fread的时候最好一次读完，然后fclose掉。

James_Pan

TO billy14
你说一次读完整个文件???我这个文件大小不知道呀，也许文件很大呢,所以我才一个节目单元的读,不然,如果文件很大，会把内存都用光的,我一次读一个单元的目的就是省内存.
TO jennyvenus
你说格式丢了??什么格式?

postman

俺是说，代码的缩进格式，你应该加上c风格

比如

[code]

就比
int i = 0;

好看。

排排队吃果果

你的文件存储应该是有固定的格式的吧，一次读完再去内存中取数这样程序的运行会快些。我之前读过一个10M的文件，发现一次读完，比要用的时候读程序快很多，当然这只是个人意见！
他说格式丢了，是说你的代码都左对齐了。

JJJwan

啊，理解错了，呵呵！
当然你要是只用到文件的一小部分信息还是一点一点的读好！

Titan

帮楼主重新贴一下代码，
这样大家好看点。哈哈。今天很无聊的说。

DWORD WINAPI Thread_ReadPlayList(LPVOID lpParam)
{
FILE * f;
f=_wfopen(PlayListPath,L"rb");
if(NULL==f)
{
//传送错误信息:文件打开失败
MessageBoxW(NULL,L"Open file false",L"",MB_OK);
return 1;
}
int nWidth,nHeight;
int nTemp,nUsedLabelItem;
Arm_Label_Node *pLabel;
fread(&nWidth,1,sizeof(int),f);
fread(&nHeight,1,sizeof(int),f);
if(nWidth!=g_nScreenWidth ? ?nHeight!=g_nScreenHeight)
{
//传送错误信息:节目列表错误，不是当前屏幕的节目列表
MessageBoxW(NULL,L"width && Heiht false",L"",MB_OK);
return 1;
}
while(!g_bStop)
{//{{{
while(0!=g_PlayNode.nType)//非空结点
{
Sleep(1);
}
WaitForSingleObject(hMutexNode,INFINITE);
READNEXTINFO:
fread(&nTemp,1,sizeof(int),f);
CString str;
str.Format(L"nTemp=%d",nTemp);
if(feof(f))
{
fseek(f,8,SEEK_SET);
goto READNEXTINFO;
}
switch(nTemp)
{
case CLOCK_Type:
g_pClockNode=(Arm_Clock_SetInfo *)malloc(sizeof(Arm_Clock_SetInfo));
fread(g_pClockNode,1,sizeof(Arm_Clock_SetInfo),f);
hClockFont=CreateFontIndirect(&g_pClockNode-> logFont);
break;
case PIC_Type:
g_PlayNode.nType=PIC_Type;
g_PlayNode.pPointer=malloc(sizeof(Arm_Pic_SetInfo));
fread(g_PlayNode.pPointer,1,sizeof(Arm_Pic_SetInfo),f);
g_PlayNode.PathName=
(TCHAR *)malloc(((Arm_Pic_SetInfo *)g_PlayNode.pPointer)-> nPathLength*sizeof(TCHAR));
fread(g_PlayNode.PathName,
((Arm_Pic_SetInfo *)g_PlayNode.pPointer)-> nPathLength,
sizeof(TCHAR),f);
break;
case LABEL_Type:
if(NULL==g_pLabelNode)
{
g_pLabelNode=(Arm_Label_Node *)malloc(sizeof(Arm_Label_Node));
pLabel=g_pLabelNode;
}else
{
pLabel=g_pLabelNode;
while(NULL!=pLabel-> next)
pLabel=pLabel-> next;
pLabel-> next=(Arm_Label_Node *)malloc(sizeof(Arm_Label_Node));
pLabel=pLabel-> next;
}
pLabel-> next=NULL;
pLabel-> pArm_Label_SetInfo=(Arm_Label_SetInfo *)
malloc(sizeof(Arm_Label_SetInfo));
fread(pLabel-> pArm_Label_SetInfo,1,sizeof(Arm_Label_SetInfo),f);
fread(pLabel-> tContent,pLabel-> pArm_Label_SetInfo-> nContentLength,sizeof(TCHAR),f);
hLabelFont[nUsedLabelItem++]=CreateFontIndirect(&pLabel-> pArm_Label_SetInfo-> logFont);
goto READNEXTINFO;
break;
case NEWNODE:
if(1)
{
nUsedLabelItem=0;
for(int i=0;i
{
if(NULL!=hLabelFont[i])
{
int nRes=DeleteObject(hLabelFont[i]);
hLabelFont[i]=NULL;
if(0==nRes)
MessageBoxW(NULL,L"DeleteObject hLabelFont",L"ERROR",MB_OK);
}
}
if(NULL!=hClockFont)
{
int nRes=DeleteObject(hClockFont);
hClockFont=NULL;
if(0==nRes)
MessageBoxW(NULL,L"DeleteObject hClockFont",L"Error",MB_OK);
}
}
while(NULL!=g_pLabelNode)
{
pLabel=g_pLabelNode-> next;
free(g_pLabelNode-> pArm_Label_SetInfo);
free(g_pLabelNode);
g_pLabelNode=pLabel;
}
if(NULL!=g_pClockNode)
free(g_pClockNode);
g_pClockNode=NULL;
g_pLabelNode=NULL;
case ENDNODE:
// MessageBoxW(NULL,L"EndNode",L"ReadPlayList",MB_OK);
goto READNEXTINFO;
break;
}
ReleaseMutex(hMutexNode);
}//}}}
//检查未释放资源，进行释放
return 1;
}
DWORD WINAPI Thread_Play(LPVOID lpParam)
{
HDC hdc=::GetDC(g_hWnd);
HANDLE handle;
Arm_Label_Node *pLabel;
while(!g_bStop)
{//{{{
while(0==g_PlayNode.nType)
{
Sleep(1);
}
WaitForSingleObject(hMutexNode,INFINITE);
switch(g_PlayNode.nType)
{
case PIC_Type:
g_bThreadStop=FALSE;
//启动一个图片线程
handle=CreateThread(NULL,0,Thread_PlayPic,NULL,0,NULL);
CloseHandle(handle);
break;
}
while(!g_bStop&&!g_bThreadStop)
{
BitBlt(g_hTopDC,0,0,g_nScreenWidth,g_nScreenHeight,g_hMiddleDC,0,0,SRCCOPY);
pLabel=g_pLabelNode;
int nItem=0;
while(NULL!=pLabel)
{
DrawLabel(pLabel,nItem++);
pLabel=pLabel-> next;
}
if(NULL!=g_pClockNode)
DrawClock(g_pClockNode);
BitBlt(hdc,0,0,g_nScreenWidth,g_nScreenHeight,g_hTopDC,0,0,SRCCOPY);
Sleep(5);
}
g_PlayNode.nType=0;
free(g_PlayNode.pPointer);
free(g_PlayNode.PathName);
ReleaseMutex(hMutexNode);
}//}}}
return 1;
}
DWORD WINAPI Thread_PlayPic(LPVOID lpParam)
{
CIImage mage;
CString strPath;
strPath=PicPathExtent;
strPath+=g_PlayNode.PathName;
mage.Open(strPath);
mage.Draw(g_hIImageDC,0,0,g_nScreenWidth,g_nScreenHeight,0);
int i=0;
while(1)
{
BitBlt(g_hMiddleDC,0,0,i,g_nScreenHeight,g_hIImageDC,g_nScreenWidth-i,0,SRCCOPY);
Sleep(5);
i++;
if(i> =g_nScreenWidth)
break;
}
g_bThreadStop=TRUE;
return 1;
}

复制代码

xb123

有疑问：
g_hIImageDC是个全局变量，不知是否会被多个线程用到呢？还是只有线程Thread_PlayPic?
从你给的代码中看不出来。
但就算是只有Thread_PlayPic用，由于Thread_Play会也可能会启动多个Thread_Play线程.
我记得MS的文档里提到过，不可以多个线程同时对一个HDC就行操作，否则会引起死锁。

玲珑宝贝

“由于Thread_Play会也可能会启动多个Thread_Play线程”
打错了，应当是
"由于Thread_Play会也可能会启动多个Thread_PlayPic线程"

suizhihen

如果真是这个原因，
既使g_hIImageDC改成局部变量也无法避免这个死锁，因为多调几次GetDC()无济于事，还是同一个GDC Object.

最好是永远只启动一个Thread_PlayPic，所有对g_hIImageDC的操作都以PostThreadMessage的方式给它一个人去做。

200fly

To:chengchenz
Thread_PlayPic代码如下,如果仔细看我代码就会发现,同时最多只会启动一个Thread_PlayPic的.但你说MS文档提示不让多个线程对一个HDC进行操作.可是我现在程序就必须这么做呀.一个Thread_PlayPic里对g_hMiddleDC进行不停绘图,而Thread_Play现成在把这个HDC拷贝到另一个HDC上,不然如何做图片特效呀????
还有就是,我一开始也是怕多个线程访问一个HDC会出现问题.所以换了一个全局的BITMAP对象在每个线程里面在CreateCompatibleDC,在让这个新HDC选择这个BITMAP对象,但是后来发现问题,一查 MSDN上面提示一个HBITMAP不能SelectObject进多个HDC,不然除第一个HDC外,其他的HDC是得不到图片的。所以我才弄的全局的HDC

DWORD WINAPI Thread_PlayPic(LPVOID lpParam)
{
CIImage mage;
CString strPath;
strPath=PicPathExtent;
strPath+=g_PlayNode.PathName;
mage.Open(strPath);
mage.Draw(g_hIImageDC,0,0,g_nScreenWidth,g_nScreenHeight,0);
int i=0;
while(1)
{
BitBlt(g_hMiddleDC,0,0,i,g_nScreenHeight,g_hIImageDC,g_nScreenWidth-i,0,SRCCOPY);
Sleep(5);
i++;
if(i>=g_nScreenWidth)
break;
}
g_bThreadStop=TRUE;
return 1;
}

复制代码

gjb04720650

的确只会有一个Thread_PlayPic;
但是有一点没看明白,你是如何保证g_hMiddleDC被Thread_Play, Thread_PlayPic同时访问时的?
从你的代码上看到,你似乎有一种假设:每5秒钟总是Thread_PlayPic在g_hMiddleDC上先画完,然后Thread_Play再用它.
但这种假设从你现在代码看并不一定成立,虽然在大多数时候确实是这样.
另外,一般窗体都是由主线程创建的(因为一般while(GetMessage())都在主线程里面);所以你的Thread_Play可能还各主线程WM_PAINT存在共用g_hWnd DC冲突的情况.
最好是一个线程只提供数据绘图线程,绘图线程只管绘图.
平时一般都是主线程管UI(包括消息处理和绘图)---用户界面线程,
子线程管监控---工作者线程.
当子线程有数据要重重绘时,SendMessage自定义的消息给窗体,在UI线程里去画.
Thread_Play和Thread_PlayPic很容易并在一起的.而且并一起后也很容易并到主线程(UI线程)里去的(可以在主线程中用SetTimer来定时动画).这样所有的DC访问都只出现在主线程(UI线程)里,不存在任何冲突,也不需要全局DC变量了.
可不可以用以下型式来实现?
窗体消息处理函数:

wndProc()
{
switch(msg)
{
case WM_PAINT:
BeginPaint();
drawMidDCtoPaintDC();
EndPaint();
break;
case WM_MYANIMAL:
i=0;
settimer(5000);
break;
case WM_TIMER:
drawPicToMidDC();//此函数里包括原来在Thread_Play和Thread_PlayPic的所有终图功能.不要在里面创建线程了.
i++;
if(i>=ScreenWidth)
KillTimer();
InvalidateRect(hWnd, NULL, FALSE); //最后一个FALSE可防止画背景引起闪烁.
break;
}
}
DWORD WINAPI Thread_Play(LPVOID lpParam)
{
switch(g_PlayNode.nType)
{
case PIC_Type:
SendMessage(g_hWnd, WM_MYANIMAL,......);
break;
}
}

复制代码

另外1楼所述的情况你也得重视.

ztj55

以上伪码中drawPicToMidDC()是把动画所有要画的东西画到g_hTopDC上.
drawMidDCtoPaintDC()是把g_hTopDC画到Window的PaintDC上.

kangtc86

呵呵，这程序跑起来不死机就奇怪了。chengchenz这几天也确实很无聊。鉴定完毕！

wang1jin

TO chengchenz:
那你伪代码我看了,drawPicToMidDC()是什么?直接顺序执行的?还是里面启动了TIMER???这个代码里面要不停的绘制图片(要一段时间,不如说5秒,而同时还要把,g_hMiddleDC上的东西 COPY到g_hTopDC上,在再窗体上显示出来.不然给人的感觉图片不是动的呀.)你那里没也看到对g_hMiddleDC的同时操作时候的保护....我这里也没想明白,所以就根本没有保护,而且如果加上互斥量的话,可能会使图片看起来一卡一卡的,所以最好是一个就是负责动画,另一个负责COPY...

小弟,就只能想到这方法了.如果您有更好的方法告诉一声.对于你说的所有操作都放到主程序里面.我会试下的.等我试完在把结果告诉大家

ip602

TO nbcool:
看来你看出我这个哪里有问题了.麻烦告诉声.我这里都急坏了谢谢

程序运行一段时间就死机！有代码 [复制链接]

最新回复