c++ - 在这种情况下我应该使用 lock_guard、scoped_lock 还是 unique_lock？

Question

我已经阅读了许多已经回答的与此相关的问题，但没有一个让我清楚地了解当我有多个作家但只有一​​个读者时应该使用哪个。下面的代码是我所说的一个人为的例子。

struct StateInfo {
    long wd{};
    uint32_t perc{};
};

class Blah 
{
    const int numDevices = getDevices();
    std::shared_mutex sharedMutexSI_;
    vector<StateInfo> stateInfo;
public:

    Blah() : stateInfo(numDevices){};

    void writers(StateInfo &newSi, const int i)
    {
        std::shared_lock<std::shared_mutex> _MULTIPLE(sharedMutexSI_);
        stateInfo[i] = newSi;
    }

    StateInfo reader(const int i)
    {
        std::lock_guard<std::shared_mutex> _EXCLUSIVE(sharedMutexSI_);
        return stateInfo[i];
    }
};

情况是多个编写者可能同时更新 stateInfo 向量，但向量中的项目绝不会与i每个线程唯一的项目相同。单个读取器线程可以随时尝试读取任何向量项。

1. 上面的代码在避免竞争条件方面是否正确？

2. 是lock_guard正确的使用还是我应该使用scoped_lockor unique_lock？

标签： c++c++17mutexlock-guard

Answer 1

总结评论中已经写的内容：

是的，代码是正确的。但是，它可能效率低下，因为它不允许在写入另一个数组元素时从任何数组元素中读取。mutex您可能希望通过对每个数组元素使用 a 来进行更细粒度的同步。

class Blah 
{
    const int numDevices = getDevices();
    std::vector<std::mutex> mutexes;
    std::vector<StateInfo> stateInfo;
public:

    Blah() : mutexes(numDevices), stateInfo(numDevices){}

    void writers(StateInfo &newSi, const int i)
    {
        std::lock_guard<std::mutex> guard(mutexes[i]);
        stateInfo[i] = newSi;
    }

    StateInfo reader(const int i)
    {
        std::lock_guard<std::mutex> guard(mutexes[i]);
        return stateInfo[i];
    }
};