此線程安全嗎？

Question

此線程安全嗎？

private static bool close_thread_running = false; 
public static void StartBrowserCleaning() 
{ 
    lock (close_thread_running) 
    { 
     if (close_thread_running) 
      return; 

     close_thread_running = true; 
    } 

    Thread thread = new Thread(new ThreadStart(delegate() 
    { 
     while (true) 
     { 
      lock (close_thread_running) 
      { 
       if (!close_thread_running) 
        break; 
      } 

      CleanBrowsers(); 

      Thread.Sleep(5000); 
     } 
    })); 

    thread.Start(); 
} 

public static void StopBrowserCleaning() 
{ 
    lock (close_thread_running) 
    { 
     close_thread_running = false; 
    } 
} 

Answer 1

那麼，它甚至不會編譯，因爲你試圖鎖定一個值類型。

引入引用類型的單獨鎖定變量，例如，

private static readonly object padlock = new object(); 

從

旁白：

如果StopBrowserCleaning()被稱爲雖然是清洗線（雖然它睡覺），但隨後StartBrowserCleaning()的是，它的意思是關閉第一個線程通知之前再次呼籲，你最終會得到兩個線程。

您可能需要考慮有兩個變量 - 一個用於「是否意味着是清理線程」，另一個是「實際上是否有清理線程」。

另外，如果你使用一個監視器Wait/Pulse，或EventHandle（例如ManualResetEvent），你可以讓你的睡眠更活潑的等待時間，在哪裏停止的請求將被更迅速地處理。