스레드로부터 안전한 방법으로 포인터 반환

Question

스레드 안전 컬렉션 (ThingList라고 함)이 있다고 가정하고 다음 함수를 추가하고 싶습니다. 객체를 것이다 해제/분명히

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

RAII 잠금 :

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

그러나 이렇게

, 나는 이런 식으로 뭔가를해야 할 것입니다 호출 코드에 대한 스레드 안전에 대한 책임을 위임했습니다 try/catch/unlocks를 단순화/제거 할 수 있지만 호출자가 잊어 버리기는 쉽습니다.

내가 검토 한 결과 몇 가지 대안이 있습니다 대신 포인터의 값으로

• 반환 살아라 - 당신이 필요하지 않는 한 잘
• 기능 ThingList::setItemBar(string name, int value)를 추가 것 수정 - 좋은, 그러나 이들은 경향이 있습니다 proliferate
• 생성시 목록을 잠그고 파기시 다시 잠금을 해제하는 포인터와 같은 객체를 반환합니다. 이것이 좋은지 나쁜지 잘 모르겠다면 ...

이 문제를 해결하는 올바른 방법은 무엇입니까?

Answer 1

'올바른 접근 방식'이 없습니다. 그것은 당신의 어플리케이션의 요구에 달려 있습니다.

가능하면 값으로 값을 반환하거나 호출자가 원하는대로 수행 할 수있는 복사본을 반환하십시오.

위의 변형은 수정 가능한 복사본을 반환 한 다음 수정 된 개체를 원자 적으로 목록에 다시 병합하는 방법을 제공합니다. 예 :

Thing t = myThingList.getThing(key); 
t.setFoo(f); 
t.setBar(b); 
myThingList.merge(t);  // ThingList atomically updates the appropriate element 

그러나 여러 스레드가 동일한 개체를 업데이트하려고하면 문제가 발생할 수 있습니다.

"포인터와 같은 객체"아이디어는 멋지지만 어떤 잠금 장치가 어딘가에 릴리스되지 않을 때 찾기 힘든 버그로 이어질 것이라고 생각합니다.

나는 모든 잠금/잠금 해제 코드를 ThingList 내에 유지하려고하므로 ThingList::set... 기능이 아마도 내가하는 일일 것입니다.

Answer 2

저장 및 반환 부스트 :: shared_ptr의의

당신은 액세스하는 동안 고정해야하지만 당신은