Чем плох lock(this) в C# и чем его заменить

Обновлено: 31.10.2026 · 2 мин чтения

Введение

При разработке программного обеспечения на языке C#, одним из распространенных способов управления параллельностью является использование блокировок. Однако, существует определенный антипаттерн, который может привести к серьезным проблемам - lock(this).

Определение проблемы

Чтобы понять проблему, посмотрим, как выбор объекта блокировки влияет на остальной код.

Контекст блокировки

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

public class MyClass
{
    private object lockObject = new object();

    public void SomeMethod()
    {
        lock (this)
        {
            // Критическая секция
        }
    }

    public void AnotherMethod()
    {
        lock (lockObject)
        {
            // Другая критическая секция
        }
    }
}

Возможность дедлоков

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

public class DeadlockExample
{
    private object lockObject1 = new object();
    private object lockObject2 = new object();

    public void Method1()
    {
        lock (lockObject1)
        {
            // Критическая секция
            lock (lockObject2)
            {
                // Другая критическая секция
            }
        }
    }

    public void Method2()
    {
        lock (lockObject2)
        {
            // Критическая секция
            lock (lockObject1)
            {
                // Другая критическая секция
            }
        }
    }
}

Альтернативные решения

Безопасный вариант - отдельный закрытый объект, который служит только для блокировки.

Использование private объекта блокировки

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

public class BetterLockExample
{
    private object lockObject = new object();

    public void Method1()
    {
        lock (lockObject)
        {
            // Критическая секция
        }
    }

    public void Method2()
    {
        lock (lockObject)
        {
            // Другая критическая секция
        }
    }
}

Использование Monitor

Monitor предоставляет более гибкий и безопасный способ управления блокировками, позволяя избежать некоторых проблем, связанных с использованием lock.

public class MonitorExample
{
    private object lockObject = new object();

    public void Method1()
    {
        Monitor.Enter(lockObject);
        try
        {
            // Критическая секция
        }
        finally
        {
            Monitor.Exit(lockObject);
        }
    }

    public void Method2()
    {
        Monitor.Enter(lockObject);
        try
        {
            // Другая критическая секция
        }
        finally
        {
            Monitor.Exit(lockObject);
        }
    }
}

Заключение

Использование lock(this) в C# может привести к серьезным проблемам с параллельностью и безопасностью кода. Для избежания этих проблем следует использовать более безопасные и эффективные подходы, такие как использование приватных объектов блокировки или Monitor. Это позволит разработчикам создавать более надежные и эффективные приложения.