C++でスレッドセーフなメソッドを実装する適切な方法11

Question

私は、さまざまな種類のツール（GCC、LEX、YACC、...）を表すクラスを持っています。 各インスタンスにはツールを表す型が与えられ、特別な構成が許可されます。

デフォルトの設定を処理するために、マップとベクトルのデフォルト値を格納しています。私は、そのクラスは、すべてのコンテキストで使用できるようにしたいと、それは私が次のことを実施し、レースを回避すること、スレッドセーフでなければならないであろう。

int Tool::addType(std::string typeName, 
     std::string typeFlagName) 
    throw (GP::Exceptions::LockAcquisitionException) 
{ 
    static std::timed_mutex mutex; 
    std::unique_lock<std::timed_mutex> lock{mutex}; 

    int typeId = 0; 

    if (lock.try_lock_for(std::chrono::microseconds(100))) 
    { 
     int typeId = typeNames.size(); 
     typeNames[typeId] = typeName; 
     typeFlagNames[typeId] = typeFlagName; 
    } 
    else 
    { 
     throw GP::Exceptions::LockAcquisitionException{"Unable to generate new type ID within 100 microseconds."}; 
    } 
    return typeId; 
} 

これは良い解決策である場合、または場合、私は知りたいのですが私は何かが恋しい。 これで問題が解決しない場合は、もう少し冗長な解決策がありますか？

Answer 1

どうしてですか？

class tool 
{ 
    std::atomic<int> index_; 
    std::array<std::pair<std::string, std::string>, 2048> types_; // Should be more than enough room. 

    int addType(std::string typeName, std::string typeFlagName) 
    { 
     int id = index++; 

     if(id >= types_.size()) 
      throw GP::Exceptions{"To many types."}; 

     types_[id] = std::make_pair(typeName, typeFlagName); 

     return id; 
    } 

}; 

あなたはのstd ::ベクトルを使用してビット賢く、この作ると、それは大きなサイズのために再割り当てする必要がある場合にのみ、ロックを持つことができます。

注：throw（）節はC++ 11では非推奨です。

Answer 2

私の提案は次のようになります。

class tool 
{ 
    std::mutex myMutex; 

    std::vector<std::string> typeNames; 

    int addType(std::string typeName) 
    { 
     std::lock_guard myLock(myMutex); 
     typeNames.push_back(typeName); 
     return typeNames.size()-1; // 
    } 

    // Only hold lock when needed 
    void longComplexFunction(int f) 
    { 
     // Compute as much as possible before 
     int complexMagic = veryLongFunction(f); 

     { 
      std::lock_guard myLock(myMutex); 
      typeNames[complexMagic] += "s"; 
     } 
    } 

}