Указатель на значение в std :: map

Question

У меня есть std :: map, которая используется несколькими потоками для хранения данных. Карта объявляется так:

std::map<int, Call> calls; 

Из каждого потока, я должен получить блокировку мьютекса, получить указатель или ссылку на объект, принадлежащий к этой теме, а затем снять блокировку мьютекса. Я могу изменить объект после этого, потому что каждый объект используется только одним потоком. Как только нить погибнет, соответствующая пара на карте также будет удалена.

Я хотел бы знать, как лучше всего реализовать это. Я думал о двух способах:

1) Я знаю, что это одно может выглядеть безобразно с ума, но до сих пор

std::map<int, Call> calls; 
... 

{ 
    mutex.lock(); 
    Call* callptr = &calls[id]; 
    mutex.unlock(); 

    // use callptr 
} 

или 2) Я думаю, что это один выглядит более разумным

std::map<int, std::auto_ptr<Call> > calls; 

... 

{ 
    mutex.lock(); 
    std::auto_ptr<Call> callptr = map[id]; 
    mutex.unlock(); 

    // use callptr 

    mutex.lock(); 
    map[id] = callptr; 
    mutex.unlock(); 
} 

Нить фактически создаются в другой dll, и у меня нет кода для этого. Эта DLL, которую я пишу сейчас, импортируется этой DLL и используется. Поэтому он должен быть реализован только с помощью std :: map, но может ли кто-нибудь сказать мне, подходит ли один из этих методов или есть способы сделать его более стабильным.

Благодаря

Answer 1

Вы должны использовать итераторы:

mutex.lock(); 

std::map<int, Call>::iterator callptr = calls.find(id); 
callptr->second.foo(); 
... 

mutex.unlock(); 

Ваше первое решение с указателями является проблематичным, поскольку время жизни объекта в карте является неопределенным - он может быть перемещен, когда дерево перебалансированы когда элементы вставлены или удалены.

Ваше второе решение не будет работать вообще, потому что std::auto_ptr не отвечает требованиям, предъявляемым к mapped_type из std::map - в основном потому, что его конструктор копирования и operator= фактически не копировать. Вероятно, вы не получите ошибку компилятора, но во время выполнения вы получите очень странное поведение.

Answer 2

Не используйте auto_ptr в контейнерах STL. DO NOT. Разработчикам действительно необходимо попробовать и сделать там ошибку компиляции. Стандартные контейнеры имеют тенденцию копировать вещи вокруг. Это очень неправильно с auto_ptr.

Answer 3

По мне Thread Local storage - лучший вариант для вас.

Если вам нужны данные, относящиеся к потоку, вы можете использовать локальное хранилище потоков и полностью исключить необходимость блокировки карты и мьютекса.