Zurückgeben von Zeigern in einer thread-sicheren Weise

Question

Angenommen, ich habe eine Thread-sichere Sammlung von Things (nennen Sie es eine ThingList), und ich möchte die folgende Funktion hinzufügen.

Thing * ThingList::findByName(string name) 
{ 
    return &item[name]; // or something similar.. 
} 

Aber auf diese Weise habe ich die Verantwortung für die Thread-Sicherheit an den anrufenden Code delegiert, die so etwas tun hätte:

try 
{ 
    list.lock(); // NEEDED FOR THREAD SAFETY 
    Thing *foo = list.findByName("wibble"); 
    foo->Bar = 123; 
    list.unlock(); 
} 
catch (...) 
{ 
    list.unlock(); 
    throw; 
} 

Offensichtlich ein RAII Sperren/Entsperren Objekt würde vereinfachen/entfernen Sie das try/catch/entsperrt, aber es ist immer noch einfach für den Anrufer zu vergessen.

Es gibt ein paar Alternativen, die ich angeschaut habe:

• Return Thing von Wert, anstelle eines Zeiger - fein, es sei denn Sie müssen modifizieren, um die Sache
• hinzufügen Funktion ThingList::setItemBar(string name, int value) - fein, aber diese neigen dazu, proliferieren
• Zurückgeben eines zeigerartigen Objekts, das die Liste bei der Erstellung gesperrt und entsperrt es wieder bei der Zerstörung. Nicht sicher, ob dies eine gute/schlechte Übung ist ...

Was ist der richtige Ansatz dafür?

Answer 1

Es gibt keinen "richtigen Ansatz"; Es hängt von den Bedürfnissen Ihrer Anwendung ab.

Wenn möglich, die Dinge nach Wert zurückgeben, oder eine Kopie zurückgeben, die der Anrufer tun kann, was auch immer sie wollen.

Eine Variante des Obigen besteht darin, eine änderbare Kopie zurückzugeben und dann eine Möglichkeit zur Verfügung zu stellen, ein modifiziertes Objekt in die Liste einzufügen. So etwas wie:

Dies kann jedoch Probleme verursachen, wenn mehrere Threads versuchen, das gleiche Objekt zu aktualisieren.

Die "pointerähnliche Objekt" Idee klingt cool, aber ich vermute, es würde zu schwer zu findenden Bugs führen, wenn irgendwo keine Locks freigegeben werden.

Ich würde versuchen, alle sperren/entsperren Code innerhalb ThingList, so dass die ThingList::set... Funktionen sind wahrscheinlich, was ich tun würde.

Answer 2

speichern und zurück boost :: shared_ptr s

Sie haben während der Zugang zu sperren, aber sie sind sicher nach dem Entsperren