Почему блокировка (это) {& hellip;} плохая?

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

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

А частное поле обычно является более оптимальным вариантом, поскольку компилятор осуществит ограничения доступа к нему, и это будет инкапсулировать механизм блокировки. Используя this нарушает инкапсуляцию путем представления части реализации блокировки общественности. Также не ясно, что Вы будете получать блокировку на this, если это не было зарегистрировано. Даже тогда доверие документации для предотвращения проблемы является субоптимальным.

Наконец, существует распространенное заблуждение, которое lock(this) на самом деле изменяет объект, передал в качестве параметра, и в некотором роде делает его только для чтения или недоступным. Это ложь . Объект передал в качестве параметра lock, просто служит ключ . Если блокировка уже прикрепляется, что ключ, блокировка не может быть сделана; иначе блокировка позволяется.

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

Выполнение следующие C# кодируют как пример.

public class Person
{
    public int Age { get; set;  }
    public string Name { get; set; }

    public void LockThis()
    {
        lock (this)
        {
            System.Threading.Thread.Sleep(10000);
        }
    }
}

class Program
{
    static void Main(string[] args)
    {
        var nancy = new Person {Name = "Nancy Drew", Age = 15};
        var a = new Thread(nancy.LockThis);
        a.Start();
        var b = new Thread(Timewarp);
        b.Start(nancy);
        Thread.Sleep(10);
        var anotherNancy = new Person { Name = "Nancy Drew", Age = 50 };
        var c = new Thread(NameChange);
        c.Start(anotherNancy);
        a.Join();
        Console.ReadLine();
    }

    static void Timewarp(object subject)
    {
        var person = subject as Person;
        if (person == null) throw new ArgumentNullException("subject");
        // A lock does not make the object read-only.
        lock (person.Name)
        {
            while (person.Age <= 23)
            {
                // There will be a lock on 'person' due to the LockThis method running in another thread
                if (Monitor.TryEnter(person, 10) == false)
                {
                    Console.WriteLine("'this' person is locked!");
                }
                else Monitor.Exit(person);
                person.Age++;
                if(person.Age == 18)
                {
                    // Changing the 'person.Name' value doesn't change the lock...
                    person.Name = "Nancy Smith";
                }
                Console.WriteLine("{0} is {1} years old.", person.Name, person.Age);
            }
        }
    }

    static void NameChange(object subject)
    {
        var person = subject as Person;
        if (person == null) throw new ArgumentNullException("subject");
        // You should avoid locking on strings, since they are immutable.
        if (Monitor.TryEnter(person.Name, 30) == false)
        {
            Console.WriteLine("Failed to obtain lock on 50 year old Nancy, because Timewarp(object) locked on string \"Nancy Drew\".");
        }
        else Monitor.Exit(person.Name);

        if (Monitor.TryEnter("Nancy Drew", 30) == false)
        {
            Console.WriteLine("Failed to obtain lock using 'Nancy Drew' literal, locked by 'person.Name' since both are the same object thanks to inlining!");
        }
        else Monitor.Exit("Nancy Drew");
        if (Monitor.TryEnter(person.Name, 10000))
        {
            string oldName = person.Name;
            person.Name = "Nancy Callahan";
            Console.WriteLine("Name changed from '{0}' to '{1}'.", oldName, person.Name);
        }
        else Monitor.Exit(person.Name);
    }
}

Консоль произвела

'this' person is locked!
Nancy Drew is 16 years old.
'this' person is locked!
Nancy Drew is 17 years old.
Failed to obtain lock on 50 year old Nancy, because Timewarp(object) locked on string "Nancy Drew".
'this' person is locked!
Nancy Smith is 18 years old.
'this' person is locked!
Nancy Smith is 19 years old.
'this' person is locked!
Nancy Smith is 20 years old.
Failed to obtain lock using 'Nancy Drew' literal, locked by 'person.Name' since both are the same object thanks to inlining!
'this' person is locked!
Nancy Smith is 21 years old.
'this' person is locked!
Nancy Smith is 22 years old.
'this' person is locked!
Nancy Smith is 23 years old.
'this' person is locked!
Nancy Smith is 24 years old.
Name changed from 'Nancy Drew' to 'Nancy Callahan'.