while循環和隊列的奇怪java行爲

Question

while（true）loop有奇怪的行爲。下面是代碼：

作爲類的成員我有：

static Queue<Object> response = new LinkedList<Object>(); 

...和的函數：

private void read() { 

    while (true) 
    { 
     System.out.println("foo"); 
     if(!(response.isEmpty())) 
     { 

      if((Boolean)response.peek() == true) 
      { 
       view.dispose(); 
       LogInControler controler= new LogInControler(); 
       disableMasterLogin(); 
       response.poll(); 
       return; 
      } 
      else if((Boolean)response.poll() == false) 
      { 
       JOptionPane.showMessageDialog(view.getRootPane(), 
         "Wrong username or password."); 
       view.tfUsername.requestFocus(); 
       return; 
      } 
     } 
    } 
} 

當從服務器（通過插座）接收到的對象，InputController類將該對象傳遞給適當的控制器，在這種情況下爲MasterLogInController並將其放入Queue響應中。 我正在等待while（true）循環中的響應，但問題是如果我刪除「System.out.printline（」foo「）;」循環只會輸入一次！？有了這個系統行我「強制」while循環做循環，直到收到響應。這裏有什麼問題？

Answer 1

我假設你有幾個線程在運行。

System.out.println創建一個內存屏障，可能會幫助您的代碼看到某些其他變量（因爲缺少同步）而不可見。

特別是，您的隊列不是線程安全的，似乎是安全發佈的。所以這是非常可能的是：

• 您while循環可能會看到response爲空==> NullPointerException異常
• reponse.isEmpty()也可能返回false，但response.peek()可能返回null，然後您可以轉換爲Boolean和拆箱在你的病情if((Boolean)xxx == true) ==> NullPointerException異常
• 等

除了在評論中給出了中肯的意見，幫助理解的原因，你應該使代碼線程安全。例如，您可以使用thread safe BlockingQueue。但這可能還不夠（因爲你的各種if/if/else if語句的佈局以及這些語句之間的另一個線程可能會改變隊列的事實）。