这段代码是一个有效的关键部分包装类吗

Question

我实际上有两个问题，这是第一个问题。

使用我在两个不同站点找到的代码，我编写了这两个关键部分包装类。

它会起作用吗？

#ifndef CRITICALSECTION_H
#define CRITICALSECTION_H
#include "windows.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;
public:
    LockSection(CriticalSection* pCS){
        m_pCS = pCS;
        if(m_pCS)m_pCS->Enter();
    }
    ~LockSection(){
        if(m_pCS)m_pCS->Leave();
    }
}

/*

Safe class basic structure;

class SafeObj
{
     CriticalSection m_cs;

public:
    void SafeMethod()
    {
        LockSection myLock(&m_cs);
        //add code to implement the method ...

    }
};



*/
#endif

第二个问题。在浏览此处时，我注意到作者做了不包括

::Initialize、Delete、Enter、Leave关键部分。这些不是班级正常运作所需要的吗？或者我错过了什么？

Answer 1

LockSection 类使用 RAII 调用 进入并离开。创建对象时，将调用 Enter。当对象被销毁（通过超出范围）时，将调用 Leave。

Initialize 和 Delete 由 CriticalSection 类的构造函数和析构函数调用。

学RAII，学好。这是你的朋友。

Answer 2

我来回答第一个问题。第二个问题涉及不同站点的代码，您一次只能问一个问题。也许其他人会回答第二个问题。

您包含的代码将正常工作。成员m_nLockCount 和m_nThreadId 未使用且不需要。 Try() 方法没有实现，但您可以将其删除。

就个人而言，如果未分配 m_pCS，我会引发异常。这是一个明显的错误情况。默默地继续并假装资源受到保护是明显的危险。