3

У меня в коде много блокировок. Хотел немного сократить код и написал небольшой велосипед. Скажите пожалуйста будет ли это работать должным образом?

public static class LockingCall
{
    public static readonly object LockObject = new object();

    public static void Invoke(Action action)
    {
        lock (LockObject)
        {
            action();
        }
    }

    public static T Invoke<T>(Func<T> function)
    {
        lock (LockObject)
        {
            return function();
        }
    }
}
8
  • 4
    Вопрос некорректен. Что такое "правильно"?
    – vitidev
    5 апр 2016 в 6:42
  • @vitidev ну то что действие или функция будут выполняться только в одном потоке. По моему это очевидно (то что я хотел этим сказать). Например если я хочу извлечь элемент из очереди которая используется несколькими потоками и я вместо громоздкого lock-а напишу так: LockingCall.Invoke(() => tasks.Dequeue())
    – Developer
    5 апр 2016 в 6:44
  • ну оно будет работать как ожидаете. но сам подход в корне неверен. Так у вас общая блокировка на все вызовы подобного рода. По сути вы своими руками делаете так называемый GIL (Global Interpreter Lock). Блокировать нужно очередь в момент Dequeue
    – vitidev
    5 апр 2016 в 6:48
  • 2
    Разница в том, что блокируя очередь на месте вы блокируете только эту очередь для тех, кто ее использует. А блокируя вашим способом вы блокируете любой метод вызванный с помощью вашего LockingCall.Invoke() общей блокировкой для всех вызовов. То есть у вас будет глобальная блокировка, где все методы которые попадут в ваш Invoke() просто станут в очередь, хотя у них и нет общих ресурсов за которые нужно бороться. Хотите улучшить - передавайте объект блокировки, LockingCall.Invoke(() => tasks.Dequeue(), tasks), тогда это будет метод хелпер с правильной блокировкой.
    – vitidev
    5 апр 2016 в 6:57
  • 1
    И не "в одном потоке" (это к контекстам синхронизации), а просто не параллельно.
    – vitidev
    5 апр 2016 в 6:58

1 ответ 1

4

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

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

Ну и конечно по возможности использовать другие методы в виде потокобезопасных коллекций и async/await

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

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