优化程序
今天很自觉地把以前写的MMI对话程序优化了一下。
以前在CDigMMI::OnInitDialog里做了很多初始化工作，其中有些初始化还花费比较长的时间。这样的话，打开MMI窗口就会变得很慢，特别是跟MMI通信失败的情况下，整个窗口都挂死状态。
优化：当检测到CAM存在的时候设置个时钟，然后在时钟消息处理函数里检测什么时候MMI到来，到了后再KillTimer 及更新MMI 窗口控件列表的显示。
相当于把一些需要长时间等待的事情放到另外一个线程里做，这样的话就不会影响到主线程。MMI窗口的打开和关闭变得比较顺畅。
希望以后在动手写代码之前先仔细地想一想，稍微做一些规划及注重下效率。
顺便附上一点代码
BOOL CDigMMI::OnInitDialog()
{
CDialog::OnInitDialog();
UCHAR ucSetCmd[512];
UCHAR ucStatus[1024];
//if cam is available, set a timer to check if mmi arrived,then to fill dialog context.
if(TBS_CamStatus())
{
ucSetCmd[0]=PCMSG_ENTER_MENU;
TBS_ci_MMI_Pstatus(ucSetCmd,ucStatus);
SetTimer(1, 1000, NULL);
}
//initDisplayEdit();
return TRUE; // return TRUE unless you set the focus to a control
}
void CDigMMI::OnTimer(UINT_PTR nIDEvent)
{
OutputDebugString("********OnTimer ");
if(initDisplayEdit() != S_OK)
{
return;
}
else
{
KillTimer(1);
}
CDialog::OnTimer(nIDEvent);
}
void CDigMMI::OnBnClickedExit()
{
//initDisplayEdit();
UCHAR ucSetCmd[512];
UCHAR ucStatus[1024];
int i=0;
if(TBS_CamStatus())
{
ucSetCmd[0] = PCMSG_ENTER_MENU;
TBS_ci_MMI_Pstatus(ucSetCmd,ucStatus);
while((initDisplayEdit() != S_OK)&&(i<5))
{
i++;
Sleep(1000);
}
}
}
HRESULT CDigMMI::initDisplayEdit()
{
int i=0;
CString strMmiInfo;
UCHAR ucStatus[1024];
UCHAR ucSetCmd[512];
HRESULT hr = S_FALSE;
//for menu
int ilength =0;
int ichoiceNb =0;
CString strTitle;
CString strSubTitle;
CString strBottom;
CString strItem;
CString strTemp;
CString strItemNew;
int ipos=0;
int ilastpos=0;
CString strAppInfo;
// to_log
char buff[1024]="";
DWORD dwStyle;
dwStyle = m_editCmd.GetStyle();
// PCMSG_ANSWER
if(dwStyle & ES_PASSWORD)
{
// cancel password style
m_editCmd.SetPasswordChar(NULL);
}
ucSetCmd[0]=PCMSG_GET_MMI;
TBS_ci_MMI_Pstatus(ucSetCmd,ucStatus);
if(ucStatus[4] ==PCMSG_NULL)
return hr;
/*
for(i=0;i<5;i++)
{
if(ucStatus[4] !=PCMSG_NULL) // commandTag !=0
{
break;
}
Sleep(2000);
TBS_ci_MMI_Pstatus(ucSetCmd,ucStatus);
}
*/
// get mmi menu and analyze it then set infs to UI
if(ucStatus[4] ==PCMSG_MENU)
{
//get length
ilength = ucStatus[6]*256 + ucStatus[5];
//get choice_nb
ichoiceNb = ucStatus[7];
//get title text()
while((*(ucStatus+8+ipos)!=0)&&(ipos<ilength))
ipos++;
strTitle = ucStatus + 8;
ipos++;
//get sub-title text()
ilastpos=ipos;
while((*(ucStatus+8+ipos)!=0)&&(ipos<ilength))
ipos++;
strSubTitle = ucStatus + ilastpos+8;
ipos++;
//get bottom text()
ilastpos=ipos;
while((*(ucStatus+8+ipos)!=0)&&(ipos<ilength))
ipos++;
strBottom = ucStatus + ilastpos+8;
//get each choice_nb text
strItem ="";
for(int itext=0;itext<ichoiceNb;itext++)
{
ipos++;
for(ilastpos=ipos;ipos<ilength;ipos++)
{
if(*(ucStatus+8+ipos) ==0)
{
//strItem = ucStatus + ilastpos+8;
break;
}
}
strTemp.Format("%d",itext+1);
strItemNew = ucStatus + ilastpos+8;
strItem = strItem + strTemp +": "+ strItemNew +"/r/n";
}
// set menu infs to UI
m_textMain.SetWindowText(strTitle);
m_textSub.SetWindowText(strSubTitle);
m_textCmd.SetWindowText(strBottom);
m_editDisplay.SetWindowText(strItem);
}
// get application info
ucSetCmd[0]=PCMSG_APPLICATION_INFO;
for(int i=0;i<5;i++)
{
TBS_ci_MMI_Pstatus(ucSetCmd,ucStatus);
if(ucStatus[4] !=PCMSG_NULL) // commandTag !=0
{
break;
}
}
if(ucStatus[4] ==PCMSG_APPLICATION_INFO)
{
// application type
if(*(ucStatus+7)==0x01)
{
m_appType.SetWindowText("CA");
}
else if(*(ucStatus+7)==0x02)
{
m_appType.SetWindowText("EPG");
}
else
{
m_appType.SetWindowText("NA");
}
// application Info
strAppInfo = ucStatus+12;
m_appInfo.SetWindowText(strAppInfo);
}
return S_OK;
}