Да, вам следует заблокировать, если этот словарь является общим ресурсом для нескольких потоков. Это гарантирует, что вы получите правильное значение, и другой поток не сможет изменить значение на полпути во время вашего вызова Lookup.

Как уже упоминалось здесь :

Использование TryGetValue () без блокировки небезопасно. Словарь временно находится в состоянии, которое делает его непригодным для чтения, пока другой поток пишет словарь. Словарь будет реорганизовываться время от времени по мере увеличения количества записей в нем. Когда вы читаете в точное время, когда происходит реорганизация, вы рискуете найти неправильное значение для ключа, когда корзины обновлены, но еще не введены значения.

ОБНОВЛЕНИЕ: взгляните на часть «Безопасность потоков» на этой странице .

Да, вам необходимо заблокировать словарь для доступа в многопоточной среде. Запись в словарь не является атомарной, поэтому он может добавить ключ, но не значение. В том случае, когда вы получаете к нему доступ, вы можете получить исключение.

Как и во многих тонких вопросах программирования, ответ таков: не обязательно.

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

Тем не менее, полноценный lockможет быть не лучшим способом, в зависимости от объема трафика, который получает ваш словарь. Попробуйте, ReaderWriterLockSlimесли вы используете .NET 3.5 или выше.

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

if (myDictionary.TryGetValue(key, out value))
{
}

и отдельный поток делает это:

myDictionary.Remove(key);

TryGetValueМожет случиться так, что поток, который выполняет определение, определяет, что элемент находится в словаре, но прежде чем он сможет извлечь элемент, другой поток удаляет его. В результате поток, выполняющий поиск, либо выдаст исключение, либо TryGetValueвернет, trueно valueбудет nullили, возможно, объектом, не соответствующим ключу.

Это только одна вещь, которая может случиться. Нечто подобное может произойти, если вы выполняете поиск в одном потоке, а другой поток добавляет значение, которое вы пытаетесь найти.

Если вы используете .Net 4, вы можете заменить его на ConcurrentDictionary, чтобы сделать это безопасно. В пространстве имен System.Collection.Concurrent есть и другие похожие коллекции, предпочтительные, когда вам нужен многопоточный доступ .

Не используйте блокировку «по-своему», если это вариант для вас.

Используйте новый ConcurrentDictionary<TKey, TValue>объект, и вы можете забыть о необходимости делать какие-либо блокировки.

Да, вы должны заблокировать, если у вас есть многопоточные обновления в этом словаре. Проверьте этот великий пост для деталей: словарь «Безопасный поток» (TKey, TValue)

Но с тех пор как появился ConcurrentDictionary <>, вы можете использовать его либо через .NET 4, либо с помощью Rx в 3.5 (он содержит System.Threading.dll с реализацией для новых многопоточных коллекций)