Что использовать вместо Interlocked.Equals

Question

У меня есть код устаревшего кода, который использует Interlocked.Equals для сравнения значений. Значения могут быть двумя типами bool или могут сравнивать массив структур с NULL. Resharper жалуется на Interlocked.Equals, говорящий «доступ к статическому члену типа через производный тип». Я знаю, что Equals не является членом класса Interlocked, а скорее является членом класса объектов. Сравнение происходит в потоке, поэтому я предполагаю, что исходный кодер хотел выполнить сравнение как атомную операцию, следовательно, использование блокировки. Поскольку object.Equals не является атомарным, что является правильным, потокобезопасным, способ делать подобные сравнения? Примечание. Большинство данных являются статическими, некоторые из них являются статическими нестабильными.

Answer 1

Одиночные чтения булевых или объектных ссылок являются атомарными. Итак, если вы сравниваете одно общее значение с постоянной или локальной переменной, не требуется «блокировка». Как сказал Джон, вам нужно будет использовать Interlocked.CompareExchange, чтобы убедиться, что вы читаете последнее значение, если только общие переменные не являются volatile.

Если оба сравнения объединены, тогда вам понадобится фактическая блокировка. Не существует способа атомно сравнить два общих значения AFAIK.

Update:

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

Если вы думаете об этом, что бы вы сделали с результатом сравнения? Это не имеет смысла; как только вы получите результат, это может быть неправильно. Блокировка должна быть задержана дольше, чем только для сравнения, чтобы сравнение было полезным для чего-либо.

Answer 2

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

Вы можете использовать Interlocked.CompareExchange, чтобы убедиться, что вы читаете последнее значение.

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