병렬로 스레드를 실행하면 무한 루프가 발생하는 것 같습니다.

Question

주어진 스레드가 어떤 이유로 실패하면 무한 루프가 발생하는 것으로 보입니다.

이 나에 의해 작성되지 않은 코드 그래서 심지어 을 편집 할 수는 있지만 여기에 가장 눈에 띄는 문제는, 카운터 변수 totalActions이 같은 휘발성 표시되지 않는다는 생각과 결과 스레드가 최신 값을보고 있지 않습니다.

totalActions의 실제 가치를 얻지 못하는 것처럼 보이므로 계속 기다리고 있을까요?

이로 인해 스레드가 재귀 적으로 실행됩니까? 디버깅하는 동안,

public void PerformActions(List<Action> actions) 
{ 
    object actionLock = new object(); 
    int totalActions = actionts.Count; 

    for(int x = 0; x < accounts.Count; x++) 
    { 
     int y = x; 
     new Thread(delegate() 
     { 
     actions[y].Invoke(); 

     if(Interlocked.Decrement(ref totalActions) == 0) 
     { 
      lock(actionLock) 
      { 
       Monitor.Pulse(actionLock); 
      } 
     } 
    }).Start(); 
    } 

    lock(actionLock) 
    { 
     if(totalActions > 0) 
     { 
      Monitor.Wait(actionLock); 
     }  
    } 
} 

업데이트

사용법이 같다 .... 내가 실행 스레드 (예외가 발생) 실패 통지하고, 그냥 반복해서 불렀다지고 유지하고, 여기서 myService은 API 서비스에서 json 요청을 가져 오기 위해 httpRequest 호출을 수행합니다.

Execute.InParallel(
new Action[] 
    { 
    () => { abc = myService.DoSomething(); }, 
    () => { def = myService.DoSomethingElse(); } 
    }); 

Answer 1

자물쇠는 테스트 if(totalActions > 0) 현재 값을 읽어 보장, 메모리 장벽 역할을합니다. 나는이 규약이 경주에서 자유롭지는 못하지만 적어도 경주는 거의 없을 것이라고 확신한다. 당신은 그것을 재현하는데 어려움을 겪을 것입니다.

그래서 여기에 나와 있지 않은 문제가 있습니다. 디버거를 사용하여 관련된 스레드가 정확히 무엇을하는지 확인할 수 있습니까?

일부 스레드는 처리되지 않은 예외로 인해 사망한다고 말합니다. 일찍 종료 한 스레드가 카운트를 감소시키지 않을 수도 있습니다.

또한 코드를 변경할 수없는 경우 질문의 요점은 무엇입니까? 너에게 무엇을 제안해야할지 모르겠다.

Answer 2

루프가 잘못되었습니다. 변수 x가 잘못 캡처되었습니다. actions[x].Invoke();이 각 스레드에서 실행될 때 항상 x의 마지막 값을가집니다. 따라서 배열에 전달 된 마지막 대리자는 여러 번 호출됩니다. 그것은이

for(int x = 0; x < accounts.Count; x++) 
     { 
      int y = x; // here correct value of y will be captured in delegate 
      new Thread(delegate() 
      { 
      actions[y].Invoke(); 
    ...