在这种情况下我应该使用 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_lock或unique_lock ？

Answer 1

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

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

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];
    }
};