错误性病列表

出于某种原因，我不断收到在ErrorHandler.h 以下错误，为什么大小功能缺失的参数？错误性病列表

'的std ::名单< _Ty> ::大小'：函数调用缺少参数列表;使用 '&的std ::列表< _Ty> ::大小' 来创建一个指针成员

'std::_List_iterator<_Mylist> std::list<_Ty>::erase(std::_List_const_iterator<_Mylist>,std::_List_const_iterator<_Mylist>)' : cannot convert parameter 1 from 'int' to 'std::_List_const_iterator<_Mylist>' 

    'std::_List_iterator<_Mylist> std::list<_Ty>::erase(std::_List_const_iterator<_Mylist>)' : cannot convert parameter 1 from 'int' to 'std::_List_const_iterator<_Mylist>' 


// in errorhandler.h 

    class ErrorHandler{ 
     std::list<unsigned int> m_ErrorList; 
    public: 
     ErrorHandler(){ } 
     ~ErrorHandler(){ } 
     void ForceShutdown(){ free(&m_ErrorList); } 
     void Add(int errCode){ m_ErrorList.push_back(errCode); } 
     unsigned int GetLastError(){ if(m_ErrorList.size!=0)return m_ErrorList.back(); } 
     void Remove(int pos){ if(m_ErrorList.size!=0)m_ErrorList.erase(pos); } 
     void RemoveRange(int start,int end){ if(m_ErrorList.size!=0)m_ErrorList.erase(start,end); } 

    }; 


// in criticalsection.h 
    class CriticalSection{ 
     long m_nLockCount; 
     long m_nThreadId; 
     typedef CRITICAL_SECTION cs; 
     cs m_tCS; 
    public: 
     CriticalSection(){ 
      ::InitializeCriticalSection(&m_tCS); 
      m_nLockCount = 0; 
      m_nThreadId = 0; 
     } 
     ~CriticalSection(){ ::DeleteCriticalSection(&m_tCS); } 
     void Enter(){ ::EnterCriticalSection(&m_tCS); } 
     void Leave(){ ::LeaveCriticalSection(&m_tCS); } 
     void Try(); 
    }; 
    class LockSection{ 
     CriticalSection* m_pCS; 
     ErrorHandler * m_pErrorHandler; 
    public: 
     LockSection(CriticalSection* pCS,ErrorHandler* pErrorHandler){ 
      m_pCS = pCS; 
      m_pErrorHandler = pErrorHandler; 
      if(!m_pCS)m_pErrorHandler->Add(0x1AE1); // 0x1AE is code prefix for critical section header 
      if(m_pCS)m_pCS->Enter(); 
     } 
     ~LockSection(){ 
      if(!m_pCS)m_pErrorHandler->Add(0x1AE2); 
      if(m_pCS)m_pCS->Leave(); 
     } 
    };

来源

2011-10-06 NT_SYSTEM

不，pop_back不返回的最后一个元素。这是为了防止意外错误。你必须通过back()明确得到最后一个元素。如果你想在没有阅读的情况下弹出几个，这种方式也更快。这也适用于所有其他标准C++库容器。

看你的警告，它看起来像你也有无法删除。对于列表它可能会非常棘手：

void Remove(int pos){ 
    std::list<unsigned int>::const_iterator iter = m_ErrorList.begin(); 
    //no need to check the size, advance will throw an exception if pos is invalid 
    std::advance(iter, pos); 
    m_ErrorList.erase(iter); 
}

来源

2011-10-06 21:52:03

'pop_back（）'不返回的最后一个元素，因为它必须通过价值的话，这可能是低效的，虽然C++ 11的举动建设将表面上解决这个问题。 –

@JonPurdy：这不仅仅是效率，这是关于异常的正确性。如果复制ctor抛出异常，则返回正在弹出的对象的pop将丢失该对象。 –

@JerryCoffin：另一件值得考虑的好事。虽然，为什么复制构造函数首先会抛出更多的关注。 –

您使用列表方法严重：

if(m_ErrorList.size!=0)

size是一个方法，所以你需要调用它（带括号）：

if(m_ErrorList.size()!=0)

请注意size是list慢;您可能要实现GetLastError函数是这样的：

unsigned int GetLastError(){ if(!m_ErrorList.empty())return m_ErrorList.back(); }

m_ErrorList.erase(pos);

擦除需要一个迭代器，而不是一个整数。因此，您最好使用

std::list::iterator it=m_ErrorList.begin(); 
std::advance(it, pos); 
m_ErrorList.erase(it);

请注意，这也不是一个特别有效的方法。

BTW，检查你需要list;一个vector可能会更好地为您服务。

来源

2011-10-06 21:53:14 jpalecek

我不认为'std :: list :: iterator'重载'operator +'。我认为他必须使用'std :: advance' –

@MooingDuck：true。这使得笨拙更加明显。 – jpalecek

错误性病列表

回答

相关问题