hhhhh88

其实看出很多问题，因为没空去验证，说不清楚，不过可以肯定，从设计思想上看，这个程序很难运行很久，如果能运行很久，那是运气。不过前面几个家伙提到的HDC只能有一个线程在同一时间操作，我是同意的，自从windows 95以来，这个就是禁忌，几年前看到梁肇新的书《编程高手箴言》也提到，超级解霸解决了那么多死锁问题，这个也是考虑之一。

dandantcb

TO:nbcool
哦.知道了.
TO 大家:
另外通过前面各位高手的指导,我把我程序总体结构改了下,下面是主体代码,大家看看这会问题多不?还有个问题就是,我发现我程序死掉可能不是我程序的事.我这边把开发板打开 过了一个晚上,今天一看 也死掉了.现在我正研究是我内核定制的事,还是 硬件问题.不过还是请大家帮我看看下面程序的结构,以前感觉我这程序没什么问题了。 可是通过这个帖子,我发现我还有很多不足,希望各位能帮我多找点问题,我会努力改进的

1. DWORD WINAPI Thread_ReadPlayList(LPVOID lpParam)
   
2. {
   
3.         HANDLE        hFileHandle;
   
4.         int nWidth,nHeight;
   
5.         BOOL        bRes;
   
6.         DWORD        dwReadByte;
   
7.         int nTemp,nUsedLabelItem;
   
8. 
   
9.         hFileHandle=CreateFile(PlayListPath,GENERIC_READ,0,NULL,OPEN_EXISTING,FILE_ATTRIBUTE_READONLY,NULL);
   
10.         if(INVALID_HANDLE_VALUE==hFileHandle)
    
11.         {MessageErrorBox(GetLastError(),L"Faile Open File");}
    
12.        
    
13.         bRes=ReadFile(hFileHandle,&nWidth,sizeof(int),&dwReadByte,NULL);
    
14.         if(FALSE==bRes)
    
15.         {MessageErrorBox(GetLastError(),L"Read File");}
    
16.         bRes=ReadFile(hFileHandle,&nHeight,sizeof(int),&dwReadByte,NULL);
    
17.         if(FALSE==bRes)
    
18.         {MessageErrorBox(GetLastError(),L"Read File");}
    
19. 
    
20.         if(nWidth!=g_nScreenWidth||nHeight!=g_nScreenHeight)
    
21.         {
    
22.                 //传送错误信息:节目列表错误，不是当前屏幕的节目列表
    
23.                 MessageBoxW(NULL,L"传输的节目列表文件错误",L"Error",MB_OK);
    
24.                 return 1;
    
25.         }
    
26.         while(!g_bStop)
    
27.         {
    
28.                 bRes=ReadFile(hFileHandle,&nTemp,sizeof(int),&dwReadByte,NULL);
    
29.                 if(FALSE==bRes)
    
30.                         {MessageErrorBox(GetLastError(),L"Read File");}
    
31.                
    
32.                 if(0==dwReadByte)//文件结束
    
33.                 {
    
34.                         bRes=CloseHandle(hFileHandle);
    
35.                         if(FALSE==bRes)
    
36.                                 MessageErrorBox(GetLastError(),L"CloseFile");
    
37.                         {//-----------
    
38.                                 hFileHandle=CreateFile(PlayListPath,GENERIC_READ,0,NULL,OPEN_EXISTING,FILE_ATTRIBUTE_READONLY,NULL);
    
39.                                 if(INVALID_HANDLE_VALUE==hFileHandle)
    
40.                                 {MessageErrorBox(GetLastError(),L"Faile Open File");}
    
41.        
    
42.                                 bRes=ReadFile(hFileHandle,&nWidth,sizeof(int),&dwReadByte,NULL);
    
43.                                 if(FALSE==bRes)
    
44.                                 {MessageErrorBox(GetLastError(),L"Read File");}
    
45.                                 bRes=ReadFile(hFileHandle,&nHeight,sizeof(int),&dwReadByte,NULL);
    
46.                                 if(FALSE==bRes)
    
47.                                 {MessageErrorBox(GetLastError(),L"Read File");}
    
48.                         }//-----------
    
49. 
    
50.                 }
    
51.                 switch(nTemp)
    
52.                 {
    
53.                 case CLOCK_Type:
    
54.                         Thread_Switch_Clock_Type(hFileHandle);               
    
55.                         continue;
    
56.                         break;
    
57.                 case PIC_Type:
    
58.                         Thread_Switch_Pic_Type(hFileHandle);
    
59.                         break;
    
60.                 case LABEL_Type:
    
61.                         Thread_Switch_Label_Type(hFileHandle,nUsedLabelItem);
    
62.                         continue;
    
63.                         break;
    
64.                 case NEWNODE:
    
65.                         Thread_Switch_NewNode(nUsedLabelItem);               
    
66.                 case ENDNODE:
    
67.                         continue;
    
68.                         break;
    
69.                 }
    
70.         }//}}}
    
71.         //检查未释放资源，进行释放
    
72.         return 1;
    
73. 
    
74. void MessageErrorBox(int nError,CString strError)
    
75. {
    
76.         CString str;
    
77.         str.Format(L"%d",nError);
    
78.         MessageBoxW(NULL,str,strError,MB_OK);
    
79. }
    
80. void Thread_Switch_Clock_Type(HANDLE hFileHandle)
    
81. {
    
82.         BOOL bRes;
    
83.         DWORD dwReadByte;
    
84.         g_pClockNode=(Arm_Clock_SetInfo *)malloc(sizeof(Arm_Clock_SetInfo));
    
85.         if(g_pClockNode==NULL)
    
86.                 MessageErrorBox(-1,L"Malloc");
    
87.         bRes=ReadFile(hFileHandle,g_pClockNode,sizeof(Arm_Clock_SetInfo),&dwReadByte,NULL);
    
88.         if(FALSE==bRes)
    
89.                 {MessageErrorBox(GetLastError(),L"Read File");}
    
90.         hClockFont=CreateFontIndirect(&g_pClockNode->logFont);
    
91. }
    
92. void Thread_Switch_Pic_Type(HANDLE hFileHandle)
    
93. {
    
94.         BOOL bRes;
    
95.         DWORD        dwReadByte;
    
96.         g_PlayNode.nType=PIC_Type;
    
97.         if(g_PlayNode.pPointer)
    
98.                 free(g_PlayNode.pPointer);
    
99.         g_PlayNode.pPointer=malloc(sizeof(Arm_Pic_SetInfo));
    
100.                 if(g_PlayNode.pPointer==NULL)
     
101.                                 MessageErrorBox(-1,L"Malloc");
     
102.         bRes=ReadFile(hFileHandle,g_PlayNode.pPointer,sizeof(Arm_Pic_SetInfo),&dwReadByte,NULL);
     
103.         if(FALSE==bRes)
     
104.                 {MessageErrorBox(GetLastError(),L"Read File");}
     
105.         g_PlayNode.PathName=
     
106.                 (TCHAR *)malloc(((Arm_Pic_SetInfo *)g_PlayNode.pPointer)->nPathLength*sizeof(TCHAR));
     
107.         if(g_PlayNode.PathName==NULL)
     
108.                 MessageErrorBox(-1,L"Malloc");
     
109.         bRes=ReadFile(hFileHandle,g_PlayNode.PathName,sizeof(TCHAR)*((Arm_Pic_SetInfo *)g_PlayNode.pPointer)->nPathLength,
     
110.                 &dwReadByte,NULL);
     
111.         if(FALSE==bRes)
     
112.                 {MessageErrorBox(GetLastError(),L"Read File");}
     
113.         PlayPic();
     
114. }
     
115. void Thread_Switch_Label_Type(HANDLE hFileHandle,int &nUsedLabelItem)
     
116. {
     
117.         Arm_Label_Node *pLabel;
     
118.         BOOL        bRes;
     
119.         DWORD        dwReadByte;
     
120.         if(NULL==g_pLabelNode)
     
121.         {
     
122.                 g_pLabelNode=(Arm_Label_Node *)malloc(sizeof(Arm_Label_Node));
     
123.                 if(g_pLabelNode==NULL)
     
124.                         MessageErrorBox(-1,L"Malloc");
     
125.                         pLabel=g_pLabelNode;
     
126.                 }else
     
127.                 {
     
128.                         pLabel=g_pLabelNode;
     
129.                         while(NULL!=pLabel->next)
     
130.                                 pLabel=pLabel->next;
     
131.                         pLabel->next=(Arm_Label_Node *)malloc(sizeof(Arm_Label_Node));
     
132.                         if(g_pLabelNode->next==NULL)
     
133.                                 MessageErrorBox(-1,L"Malloc");
     
134.                         pLabel=pLabel->next;
     
135.                 }
     
136.         pLabel->next=NULL;
     
137.         pLabel->pArm_Label_SetInfo=(Arm_Label_SetInfo *)
     
138.                 malloc(sizeof(Arm_Label_SetInfo));
     
139.         if(pLabel->pArm_Label_SetInfo==NULL)
     
140.                 MessageErrorBox(-1,L"Malloc");
     
141.         bRes=ReadFile(hFileHandle,pLabel->pArm_Label_SetInfo,sizeof(Arm_Label_SetInfo),&dwReadByte,NULL);
     
142.         if(FALSE==bRes)
     
143.                 {MessageErrorBox(GetLastError(),L"Read File");}
     
144.         bRes=ReadFile(hFileHandle,pLabel->tContent,sizeof(TCHAR)*pLabel->pArm_Label_SetInfo->nContentLength,&dwReadByte,NULL);
     
145.         if(FALSE==bRes)
     
146.                 {MessageErrorBox(GetLastError(),L"Read File");}
     
147.         hLabelFont[nUsedLabelItem++]=CreateFontIndirect(&pLabel->pArm_Label_SetInfo->logFont);
     
148. }
     
149. void Thread_Switch_NewNode(int &nUsedLabelItem)
     
150. {
     
151.         BOOL bRes;
     
152.         Arm_Label_Node *pLabel;
     
153.         nUsedLabelItem=0;
     
154.         for(int i=0;i
155.         {
     
156.         if(NULL!=hLabelFont[i])
     
157.         {
     
158.                 bRes=DeleteObject(hLabelFont[i]);
     
159.                 hLabelFont[i]=NULL;
     
160.                 if(FALSE==bRes)
     
161.                         MessageErrorBox(GetLastError(),L"DeleteObject");
     
162.         }
     
163.         }
     
164.         if(NULL!=hClockFont)
     
165.         {
     
166.                 bRes=DeleteObject(hClockFont);
     
167.                 hClockFont=NULL;
     
168.                 if(FALSE==bRes)
     
169.                         MessageErrorBox(GetLastError(),L"DeleteObject");
     
170.         }
     
171.         while(NULL!=g_pLabelNode)
     
172.         {
     
173.                 pLabel=g_pLabelNode->next;
     
174.                 free(g_pLabelNode->pArm_Label_SetInfo);
     
175.                 free(g_pLabelNode);
     
176.                 g_pLabelNode=pLabel;
     
177.         }
     
178.         if(NULL!=g_pClockNode)
     
179.                 free(g_pClockNode);
     
180.                 g_pClockNode=NULL;
     
181.                 g_pLabelNode=NULL;
     
182. }
     
183. void PlayPic()
     
184. {
     
185.         CIImage mage;
     
186.         CString strPath;
     
187.         strPath=PicPathExtent;
     
188.         strPath+=g_PlayNode.PathName;
     
189. 
     
190.         mage.Draw(g_hIImageDC,0,0,g_nScreenWidth,g_nScreenHeight,strPath);
     
191.         int i=0;
     
192.         while(1)
     
193.         {
     
194.                 BitBlt(g_hMiddleDC,0,0,i,g_nScreenHeight,g_hIImageDC,g_nScreenWidth-i,0,SRCCOPY);
     
195. 
     
196.                 SendMessage(g_hWnd,MSG_COPYSCREEN,0,0[color=#FF0000]);//这里用SendMessage,不用
     
197.                                                                 //PostMessage,来解决
     
198.                                                                //一个时刻只有一个地方对HDC进行操作.(个人认
     
199.                                                              //为可以,如果有不妥请指出)[/color]
     
200.                 Sleep(5);
     
201.                 i++;
     
202.                 if(i>=g_nScreenWidth)
     
203.                         break;
     
204.         }
     
205. }
     
206. LRESULT CLED_PlayerDlg::WindowProc(UINT message, WPARAM wParam, LPARAM lParam)
     
207. {
     
208.         // TODO: Add your specialized code here and/or call the base class
     
209.         switch(message)
     
210.         {
     
211.         case MSG_COPYSCREEN:
     
212.                 msg_CopyScreen();
     
213.                 break;
     
214.         }
     
215.         return CDialog::WindowProc(message, wParam, lParam);
     
216. }
     
217. 
     
218. void CLED_PlayerDlg::msg_CopyScreen()
     
219. {
     
220.         Arm_Label_Node *pLabel;
     
221.         HDC hdc=::GetDC(this->m_hWnd);[color=#FF0000]//注意:这里我是用API函数做的， 如果我用MFC 做
     
222.                                            //CDC *dc=this->GetDC();dc->BitBlt(...);ReleaseDC(dc);
     
223.                                       //的话 会有内存泄露不知道为什么,请指教[/color]
     
224.         BitBlt(g_hTopDC,0,0,g_nScreenWidth,g_nScreenHeight,g_hMiddleDC,0,0,SRCCOPY);
     
225.         pLabel=g_pLabelNode;
     
226.         int nItem=0;
     
227.         while(NULL!=pLabel)
     
228.         {
     
229.                 DrawLabel(pLabel,nItem++);                       
     
230.                 pLabel=pLabel->next;
     
231.         }
     
232.         if(NULL!=g_pClockNode)
     
233.                 DrawClock(g_pClockNode);
     
234.         BitBlt(hdc,0,0,g_nScreenWidth,g_nScreenHeight,g_hTopDC,0,0,SRCCOPY);
     
235.         ::ReleaseDC(this->m_hWnd,hdc);
     
236.        
     
237. }

复制代码

程序可以说是顺序执行的了.....但我以后还要添加别的消息响应,所以还是把读文件放到了一个单独的线程里面.