向cnzdgs提问：关于在定时器中循环，导致Cpu负荷过高的问题

Question

各位大大么：
     我的程序在定时器中循环检测变量，来触发事件的发生。其中定时长度为1秒钟，(我的问题是如何降低CPU的负荷)代码如下：
    void CEpriMgrDlg::OnTimer(UINT nIDEvent)
    {

if (m_bStart)
{    m_wndEvent.SetRedraw(FALSE);
m_Linker.MonitorEventByVar();
m_wndEvent.SetRedraw(TRUE);
Sleep(1000);

}
CDialog::OnTimer(nIDEvent);
    }
其中MonitorEventByVar为函数执行体，（）代码如下
      bool  CLinker ::MonitorEventByVar()
   {

CString  szEventVar,szType,szShopValue,szCmd,szValue,szField,szNameField,szParam, szAction, szResult,szName;
std::map<CString,_TagPoint>::iterator it;
for(it = m_mEventVars.begin(); it != m_mEventVars.end();it++){ //做程序假死处理
/* MSG msg;
PeekMessage(&msg,NULL,0,0,PM_REMOVE);
TranslateMessage(&msg); 
DispatchMessage(&msg);*/
///////////////////////////////////////
       szEventVar=it->first;
   _TagPoint  TempPoint=it->second;
       szType=TempPoint.szType;
   szName=TempPoint.szName;
   std::map<CString,vEnetEvent>::iterator iter=  m_mEvents.find(szType);
   if(iter!= m_mEvents.end())
   {
   vEnetEvent  TempEvent=iter->second;
   int nSize=TempEvent.size();
           for (int i=0;i<nSize;i++)
           {
   szValue=TempEvent.at(i).szValue;
               szField=TempEvent.at(i).szField;
   CString szCurVar=szEventVar+"@"+szField;
 
   //如果szValue为空，则认为变化即发生的事件
   //如果szValue不为空，则认为是需要匹配才认为是发生了的事件
         /*      int nRealValue=GetRealValue(szValue);    if(szValue==""|| (nRealValue!=atoi(szShopValue)))
   continue;*/
   std::map<CString,int>::iterator ItHis=  m_mHistoryVar.find(szCurVar);
   int nHistoryVar=0;
   //如果找到历史数据---则将其取出 与当前数据进行比较
   if (ItHis!=m_mHistoryVar.end())
   {
   nHistoryVar=ItHis->second;
   }
   //如果没有找到历史数据---则将其存储
   else
   {
                   m_mHistoryVar[szCurVar]=atoi(szShopValue);
   }    _variant_t varValue = m_pRtShm->GetRTValue("*", (_bstr_t)szCurVar);
   szShopValue=(PCSTR)(_bstr_t)varValue;
     
    //如果当前值发生变化--则将历史数据更新
    if (i==nSize-1)
    {
       m_mHistoryVar[szCurVar]=atoi(szShopValue);
    }
   int nBit=GetRealValue(szValue);
   int nRsult=atoi(szShopValue)&(0x00000001<<nBit)?1:0;
   int nHsRsult=nHistoryVar&(0x00000001<<nBit)?1:0;
   if(szValue==""|| (nRsult!=1)||(nRsult<=nHsRsult))
   continue;
   szNameField = TempEvent.at(i).szField;
   szParam = TempEvent.at(i).szParam;
   szAction= TempEvent.at(i).szAction;
   szResult= TempEvent.at(i).szResult;
   //事件设置失败！
   if(!NTriggerAction(szName,szAction,szEventVar,szNameField, szParam, szResult))
    return FALSE;
           }

   
   }
      
        Sleep(0);
}
return true;
   }
经过验证MonitorEventByVar 函数已无法在进一步的优化。现在程序运行时CPU负荷一致都持续很高，请问如何解决。我试图用Sleep来降低Cpu的负荷，经验证效果不是很明显。

Answer 1

既然定时为何还要sleep  这样还不如定时时间长些

Answer 2

在任何地方只要你的代码不停的使用CPU，必然造成高CPU，估计你CPU忙时，根本还没有到sleep，所有的一切都时在sleep之前执行的
有几个东西你必须检查：
1. 是不是你真的需要不停的使用CPU，如果你时轮询的代码，改成回调的往往是可以降低CPU占有率的，或者改成线程内部的阻塞调用也可以降低CPU。尽一切可能避免不停的轮询
2。如果上述措施之后还无法降低，必须优化算法
3. 如果算法优化依然无法降低，则说明你的CPU根本处理不过来，需要换更高档的机器记住基本的原则：你使用CPU，CPU就忙。所以只要CPU负荷高，你就该分析你的代码以多高的频率占用cpu。
假如你以1秒定时器来执行，每次定时处理函数需要执行0.8s，就说明你的cpu至少80％被你完完全全使用着，当然会忙死了

Answer 3

谢谢阿荣:你指的回调函数是定时回调函数吗？这个会跟ontime触发 有大的区别吗。关于线程内部的阻塞调用还请赐教！

Answer 4

1 应该不是定时回调函数，而是事件发生后主动调用的处理函数，需要你在事件创建的时候来设置关联
2 线程内部的阻塞就是用WaitForSingleObject或WaitForMultipleObjects来等待事件的发生，这样如果事件没有发生的话线程就一直在等待，就不会占用cpu了

Answer 5

我目前的情况是变量是在实时变化的，没办法来触发事件的发生。WaitForSingleObject来触发什么事件呢？赐教！