stl::map和stl::vetctor线程安全解决方案

[forster]

再多线程下处理同一map和vector
读是否线程安全？
除了在写操作前后cirticalsection之外有无成熟简易的解决方案？

[lsldd]

map本身含有静态变量，在多线程下使用一定要加锁。
CirticalSection已经是最简单的互斥了。。。不知道还有什么更简单的解决方案。。。

[老邓]

不安全。加个临界区检测。

class CriticalSection
{
public:
    CriticalSection() { ::InitializeCriticalSection(&m_cs); }
    virtual ~CriticalSection() { ::DeleteCriticalSection(&m_cs); }
    void Lock() { ::EnterCriticalSection(&m_cs); }
    void Unlock() { ::LeaveCriticalSection(&m_cs); }
    operator PCRITICAL_SECTION() { return &m_cs; }

protected:
    CRITICAL_SECTION m_cs;
};

class Locker
{
public:
    Locker(PCRITICAL_SECTION cs) : m_p(cs) { ::EnterCriticalSection(m_p); }
    ~Locker() { ::LeaveCriticalSection(m_p); }

protected:
    PCRITICAL_SECTION m_p;
};

使用时：

// 维护需要处理消息的所有对象指针，只有指针存在，才能发送消息
static std::set<const void*> g_objPtr;
static CriticalSection g_cs;

void EvtObjMgr::Add(const void* objPtr)
{
    Locker lock(g_cs);
    g_objPtr.insert(objPtr);
}

void EvtObjMgr::Remove(const void* objPtr)
{
    Locker lock(g_cs);
    g_objPtr.erase(objPtr);
}

[do_fork]

针对同一对象的访问，只要有写操作，几乎都不是线程安全的

[forster]

比如map是不是读写都要加锁？

[老邓]

[Quote=引用 5 楼 forster 的回复:]
比如map是不是读写都要加锁？
[/Quote]
是的。

bool EvtObjMgr::IsExist(const void* objPtr)
{
    Locker lock(g_cs);
    return g_objPtr.find(objPtr) != g_objPtr.end();
}

[jackyjkchen]

[Quote=引用 5 楼 forster 的回复:]
比如map是不是读写都要加锁？
[/Quote]
如果你要兼容老的STL，干脆全加上，因为STL对于多线程没有提要求。

[老邓]

我给的示例代码，可以不使用全局的静态变量，改用普通的成员变量。
我那个类因为是全局类静态类，所以才使用了静态变量。

static std::set<const void*> g_objPtr;
static CriticalSection g_cs;

[winnuke]

线程安全是指针对某些类的方法的调用的吧。
你若是多个线程读写一个vector对象，那叫线程同步问题。
crt里有些函数以前是线程不安全的，因为其实现依赖了一些静态和全局变量。但是只是说这个函数线程不安全。如果说一个类线程不安全应该是指单独的线程内使用单独的该类的对象是否安全吧？

[thinkboy234]

学习。。。

[forster]

[Quote=引用 8 楼 loaden 的回复:]
我给的示例代码，可以不使用全局的静态变量，改用普通的成员变量。
我那个类因为是全局类静态类，所以才使用了静态变量。
C/C++ codestatic std::set<constvoid*> g_objPtr;static CriticalSection g_cs;
[/Quote]

用起来还是比较方便，我在想怎么把所有的函数都加上。。

[老邓]

[Quote=引用 11 楼 forster 的回复:]
引用 8 楼 loaden 的回复:
我给的示例代码，可以不使用全局的静态变量，改用普通的成员变量。
我那个类因为是全局类静态类，所以才使用了静态变量。
C/C++ codestatic std::set <constvoid*> g_objPtr;static CriticalSection g_cs;


用起来还是比较方便，我在想怎么把所有的函数都加上。。
[/Quote]
建议针对每个容器实例用一个锁。
以免出现奇怪的死锁现象。

[老邓]

另外，加锁的代价是比较大的。
所以要设计好代码。
不要频繁的加锁。

[forster]

函数巨多打算打算声明成fuc（..,Locker l = Locker（m_cs））;看看效果

[forster]

成员函数做默认参数需要静态，，
改成静态又链接错误了。。。
Error 4 error LNK2001: unresolved external symbol "public: static class CriticalSection CSocketServer::m_cs" (?m_cs@CServer@@2VCriticalSection@@A) Server.obj

[wangxipu]

线程安全需要用户负责，否则STL也管的太多了些……

其实完全可以将STL 容器包装一个类，此类继承一个如上的CS同步类或对象

[cattycat]

还是用window提供的加锁机制吧，Mutex,Semaphore或CRITICAL_SECTION 。
用这些实现类似读写锁机制性能上比较好。

[老邓]

[Quote=引用 15 楼 forster 的回复:]
成员函数做默认参数需要静态，，
改成静态又链接错误了。。。
Error 4 error LNK2001: unresolved external symbol "public: static class CriticalSection CSocketServer::m_cs" (?m_cs@CServer@@2VCriticalSection@@A) Server.obj

[/Quote]
静态成员需要在类定义的外面声明的，比如：

CriticalSection CSocketServer::m_cs;

[老邓]

[Quote=引用 17 楼 cattycat 的回复:]
还是用window提供的加锁机制吧，Mutex,Semaphore或CRITICAL_SECTION 。
用这些实现类似读写锁机制性能上比较好。
[/Quote]
CRITICAL_SECTION是最快的。
其他内核锁（事件、互斥体），每进一次内核，都需要上千个CPU周期。

[love514425]

> 学习.

[永远的风筝]

3楼回答的第一部分已经足够了啊