3

Смотрю на код из статьи и не могу понять, в чём смысл использования Interlocked.Increment внутри lock. Зачем он там?

public class Node
{
    public const int InvalidId = -1;
    private static int s_idCounter;

    private int m_id;

    public int Id
    {
        get
        {
            if (m_id == InvalidId)
            {
                lock (this)
                {
                    if (m_id == InvalidId)
                    {
                        m_id = Interlocked.Increment(ref s_idCounter);
                    }
                }
            }

            return m_id;
        }
    }
}

1 ответ 1

3

Представьте, что у вас несколько экземпляров Id в многопоточном приложении и если они используются в разных потоках (каждый - в своем), оба потока могут оказаться одновременно внутри lock, т.к. для блокировки будут использованы разные объекты (this же). Поэтому использование Interlocked здесь не лишнее и оправдано.

Использование Interlocked было бы не нужно, если бы для всех/разных экземпляров для блокировки был использован один какой-то специально созданный статический объект, но это привело бы к некоторой потере скорости при одновременном доступе к разным экземплярам Id.

8
  • "но это привело бы к некоторой потере скорости при одновременном доступе к разным экземплярам Id" - почему привело бы? Здесь же всё равно lock, только другой. Логично либо два Interlocked'а, либо нормальный lock, а тут какой-то гибрид.
    – Qwertiy
    25 дек 2017 в 10:05
  • Потому что все потоки, не зависимо от того с разными экземплярами Id или с одним и тем же они работают, ждали бы, пока один из них выйдет из lock. А здесь нужно будет подождать только лишь при использовании того же самого экземпляра. А почему гибрид - надо спрашивать у автора, но, вроде, выглядит нормально и хорошо читается, да и объектов не нужно дополнительных заводить 25 дек 2017 в 10:12
  • Получается, Interlocked.Increment намного эффективнее, чем пять операций внутри lock'а (сравнить, инкрементировать (прочитать, увеличить, записать), присвоить)?
    – Qwertiy
    25 дек 2017 в 10:21
  • @Qwertiy, ну лучше ведь заблокировать всех на 3 операции, и лишь некоторых на 5, чем абсолютно всех на 5? :) Хотя, конечно, какой вариант покажет себя лучше зависит от контекста, если много потоков работают с одним и тем же экземпляром - то будет быстрее работать код с одной блокировкой, но если в основном идет работа с разными экземплярами, то этот код будет быстрее, я так думаю... 25 дек 2017 в 10:23
  • 1
    @Qwertiy два Interlocked'а сделали бы возможным пропуск id, иногда важно этого избежать. Впрочем, автор той статьи выбрал lock по более простой причине - он же измерял производительность лока... 25 дек 2017 в 11:23

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.