Có an toàn khi sử dụng cờ boolean để ngăn chặn một luồng chạy trong C#

Question

Mối quan tâm chính của tôi là với cờ boolean ... có an toàn khi sử dụng nó mà không cần đồng bộ hóa không? Tôi đã đọc ở một số nơi mà nó nguyên tử (bao gồm cả tài liệu).

class MyTask 
{ 
    private ManualResetEvent startSignal; 
    private CountDownLatch latch; 
    private bool running; 

    MyTask(CountDownLatch latch) 
    { 
     running = false; 
     this.latch = latch; 
     startSignal = new ManualResetEvent(false); 
    } 

    // A method which runs in a thread 
    public void Run() 
    { 
     startSignal.WaitOne(); 
     while(running) 
     { 
      startSignal.WaitOne(); 
      //... some code 
     } 
     latch.Signal(); 
    } 

    public void Stop() 
    { 
     running = false; 
     startSignal.Set(); 
    } 

    public void Start() 
    { 
     running = true; 
     startSignal.Set(); 
    } 

    public void Pause() 
    { 
     startSignal.Reset(); 
    } 

    public void Resume() 
    { 
     startSignal.Set(); 
    } 
} 

Đây có phải là cách an toàn để thiết kế tác vụ theo cách này không? Bất kỳ đề xuất, cải tiến, ý kiến?

Lưu ý: Tôi đã viết lớp tùy chỉnh CountDownLatch trong trường hợp bạn tự hỏi mình đang lấy nó từ đâu.

Cập nhật:
Đây là CountDownLatch tôi quá:

public class CountDownLatch 
{ 
    private volatile int m_remain; 
    private EventWaitHandle m_event; 

    public CountDownLatch (int count) 
    { 
     if (count < 0) 
      throw new ArgumentOutOfRangeException(); 
     m_remain = count; 
     m_event = new ManualResetEvent(false); 
     if (m_remain == 0) 
     { 
      m_event.Set(); 
     } 
    } 

    public void Signal() 
    { 
     // The last thread to signal also sets the event. 
     if (Interlocked.Decrement(ref m_remain) == 0) 
      m_event.Set(); 
    } 

    public void Wait() 
    { 
     m_event.WaitOne(); 
    } 
} 

Answer 1

Bạn tốt hơn đánh dấu nó volatile mặc dù:

Các từ khóa dễ bay hơi chỉ ra rằng một trường có thể được sửa đổi b y nhiều đồng thời thực hiện chuỗi. Các trường được khai báo dễ bay hơi không phải là tùy thuộc vào tối ưu hóa trình biên dịch mà giả sử truy cập bằng một chuỗi duy nhất. Điều này đảm bảo rằng giá trị cập nhật nhất luôn có mặt tại hiện trường.

Nhưng tôi sẽ thay đổi vòng lặp của bạn:

startSignal.WaitOne(); 
    while(running) 
    { 
     //... some code 
     startSignal.WaitOne(); 
    } 

Vì nó là trong bài viết của bạn 'một số mã' có thể thực hiện khi chủ đề là ngừng (tức là khi dừng được gọi.) Là bất ngờ và thậm chí có thể không chính xác.

Answer 2

Booleans là nguyên tử trong C#: http://msdn.microsoft.com/en-us/library/aa691278(VS.71).aspx

Answer 3

Booleans là nguyên tử trong C#, tuy nhiên, nếu bạn muốn sửa đổi nó trong một chủ đề và đọc nó trong một chủ đề khác, bạn sẽ cần phải đánh dấu nó dễ bay hơi ở rất ít nhất ,. Nếu không thì luồng đọc chỉ có thể đọc nó một lần vào thanh ghi.

Answer 4

BTW, tôi chỉ chú ý phần này của mã:

// A method which runs in a thread 
    public void Run() 
    { 
     startSignal.WaitOne(); 
     while(running) 
     { 
      startSignal.WaitOne(); 
      //... some code 
     } 
     latch.Signal(); 
    } 

Bạn sẽ cần phải mở khóa các sợi nhân hai lần sử dụng "startSignal.Set()" cho mã trong khối thời gian để thực hiện.

Điều này có chủ ý không?