Tại sao ngắt lời tuyên bố này không hoạt động?

Question

Tôi có đoạn mã sau:

public void post(String message) { 
    final String mess = message; 
    (new Thread() { 
     public void run() { 
      while (true) { 
       try { 
        if (status.equals("serviceResolved")) { 
         output.println(mess); 
         Game.log.fine("The following message was successfully sent: " + mess); 
         break; 
        } else { 
         try {Thread.sleep(1000);} catch (InterruptedException ie) {} 
        } 
       } catch (NullPointerException e) { 
        try {Thread.sleep(1000);} catch (InterruptedException ie) {} 
       } 
      } 
     } 
    }).start(); 
} 

Trong log file của tôi, tôi tìm thấy rất nhiều dòng như thế này:

The following message was successfully sent: blablabla 
The following message was successfully sent: blablabla 
The following message was successfully sent: blablabla 
The following message was successfully sent: blablabla 

Và chương trình của tôi không được đáp ứng.

Dường như với tôi rằng lệnh break không hoạt động. Điều gì có thể là một lý do có thể cho điều đó.

Điều thú vị là nó không phải lúc nào cũng xảy ra. Đôi khi chương trình của tôi hoạt động tốt, đôi khi vấn đề được mô tả ở trên xảy ra.

Answer 1

Bạn đang bắt đầu một chuỗi mới mỗi lần bạn gọi phương thức bài đăng. Tôi điều phương pháp là OK nhưng chương trình người gọi không phải là.

Answer 2

Nó có thể được rằng dòng này thành công:

output.println(mess); 

nhưng dòng này là ném một con trỏ ngoại lệ null:

Game.log.fine(... 

Trong trường hợp này bạn sẽ thấy đầu ra trên giao diện điều khiển, nhưng tuyên bố phá vỡ không bao giờ đạt được. Có phải Game.log có lẽ là không?

Answer 3

Game.log.fine chính xác làm gì? Nó có thể được rằng nó ném một NullPtrException sau khi đầu ra, hoặc nó có thể được rằng bạn gọi phương pháp sau nhiều lần?

Loại bỏ bắt của NullPointerException, đây là kiểu xấu (xuất hiện NullPointerException luôn là lỗi lập trình) và thêm một số thông điệp tường trình khác trong phương thức (hoặc sử dụng trình gỡ lỗi).

Answer 4

Bạn có chắc chắn muốn tiếp tục NullPointerException không? Nếu bạn nhận được một bên trong vòng lặp, bạn có thể chờ đợi mãi mãi.

Nếu bạn chắc chắn rằng status sẽ luôn cuối cùng được "serviceResolved", sau đó đặt một try ... finally bên trong câu lệnh if để nếu một cái gì đó thất bại, vòng lặp vẫn tồn tại:

if (status.equals("serviceResolved")) { 
    // No matter what happens next, we have to bail 
    try { 
     output.println(mess); 
     Game.log.fine("The following message was successfully sent: " + mess); 
    } finally { 
     break; 
    } 
} else { 
    try {Thread.sleep(1000);} catch (InterruptedException ie) {} 
} 

Answer 5

Bạn 'giả định rằng câu lệnh break không hoạt động, nhưng có thể là phương pháp post của bạn đang được gọi nhiều lần. Thử đặt một câu lệnh nhật ký khác ở đầu phương thức để xem tần suất nó được gọi. Ngoài ra, hãy đặt câu lệnh nhật ký sau vòng lặp while nhưng trước khi kết thúc phương thức run để xác minh rằng break thực sự đã thoát ra khỏi vòng lặp.

Tôi cũng đồng ý với các áp phích khác bắt được NullPointerException là một mùi mã. Bạn nên kiểm tra các biến của mình trước tiên cho null.