Надлежащее использование Monitor.Exit

Question

void MethodA() 
{ 
    Monitor.Enter(this); 
    if(someCondition) 
    { 
     Monitor.Exit(this); 
     // This point 
     MethodB(); 
    } 
    else 
    { 
     // Set some values only 
Monitor.Exit(this); 
    } 
} 

Если у меня есть вышеупомянутый метод, который может быть вызван в нескольких нитей:

• Предположим, поток 1 находится в //This point
• Другой поток входит Monitor.Enter в то время как поток 1 по-прежнему на //This point
• Будет ли эта остановка MethodB быть выполненной? Если да, есть способ получить MethodB для выполнения.

мне нужно освободить MethodA перед выполнением MethodB(), потому что я не могу ждать, чтобы завершить MethodB перед выпуском MethodA. Кроме того, я не могу запустить MethodB в новом потоке.

Answer 1

Поскольку someCondition не передается в качестве параметра, я могу предположить, что someCondition может быть изменена в любое время (возможно, переменная экземпляра для класса). Таким образом, я бы написать код так:

void MethodA() 
{ 
    if (someCondition) 
    { 
     bool conditionReached = false; 

     Monitor.Enter(this); 
     try 
     { 
      if (someCondition) 
      { 
       conditionReached = true; 
      } 
     } 
     finally 
     { 
      Monitor.Exit(this); 
     } 

     if (conditionReached) 
     { 
      MethodB(); 
     } 
    } 
} 

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

Answer 2

Откладывая вопрос об использовании «это» как замок (плохая практика, так как она подвергает замок публично), эта форма чище и безопаснее (снимает блокировку, если исключение):

void MethodA() 
{ 
    bool condition; 
    lock(this) 
    { 
    condition = someCondition; 
    if(!condition) 
    { 
     // Set some values only 
    } 
    } 
    if (condition) 
     MethodB(); 
} 

Чтобы ответить на ваш конкретный вопрос, и этот код, и ваш исходный код будут выполнять MethodB, даже если что-то еще приобретает блокировку.