NetBeans/Java/Gợi ý mới: Thread.sleep được gọi trong vòng lặp

Question

Trong NetBeans, có một gợi ý mới cho biết: Thread.sleep được gọi trong vòng lặp.

Câu hỏi 1: Làm thế nào khi nào có thể xảy ra sự cố khi ngủ trong vòng lặp?

Câu hỏi 2: Nếu đó là vấn đề, tôi nên làm gì?

CẬP NHẬT: Câu hỏi 3: Dưới đây là một số mã. Nói với tôi trong trường hợp này nếu tôi nên sử dụng cái gì khác thay vì Thread.Sleep trong một vòng lặp. Trong ngắn hạn, điều này được sử dụng bởi một máy chủ mà lắng nghe các kết nối TCP khách hàng. Giấc ngủ được sử dụng ở đây trong trường hợp đạt đến số lượng phiên tối đa với khách hàng. Trong tình huống này, tôi muốn ứng dụng đợi cho đến khi phiên miễn phí khả dụng.

public class SessionManager { 
    private static final int DEFAULT_PORT = 7500; 
    private static final int SLEEP_TIME = 200; 
    private final DatabaseManager database = new DatabaseManager(); 
    private final ServerSocket serverSocket = new ServerSocket(DEFAULT_PORT); 

public SessionManager() throws IOException, SQLException 
{ 
} 

public void listen() 
{ 
while (true) 
    if (Session.getSessionCount() < Session.getMaxSessionCount()) 
     try 
     { 
      new Thread(new Session(database, serverSocket.accept())).start(); 
     } 
     catch (IOException ex) { ex.printStackTrace(); } 
    else 
     try 
     { 
      Thread.sleep(SLEEP_TIME); 
     } 
     catch (InterruptedException ex) { ex.printStackTrace(); } 
} 

public static void main(String[] args) throws IOException, SQLException 
{ 
new SessionManager().listen(); 
} 
} 

Answer 1

Gọi ngủ trong vòng lặp thường dẫn đến hiệu suất kém. Ví dụ:

while (true) { 
    if (stream.available() > 0) { 
     // read input 
    } 
    sleep(MILLISECONDS); 
} 

Nếu MILLISECONDS quá lớn, thì mã này sẽ mất nhiều thời gian để nhận ra rằng đầu vào có sẵn.

Nếu MILLISECONDS quá nhỏ, thì mã này sẽ lãng phí rất nhiều tài nguyên hệ thống kiểm tra đầu vào chưa đến.

Các ứng dụng khác của sleep trong vòng lặp thường có vấn đề. Thường có cách tốt hơn.

Nếu đó là vấn đề, tôi nên làm gì?

Đăng mã và có thể chúng tôi có thể cung cấp cho bạn câu trả lời hợp lý.

EDIT

IMO, một cách tốt hơn để giải quyết vấn đề là sử dụng một ThreadPoolExecutor.

Something như thế này:

public void listen() { 
    BlockingQueue queue = new SynchronousQueue(); 
    ThreadPoolExecutor executor = new ThreadPoolExecutor(
      1, Session.getMaxSessionCount(), 100, TimeUnit.SECONDS, queue); 
    while (true) { 
     try { 
      queue.submit(new Session(database, serverSocket.accept())); 
     } catch (IOException ex) { 
      ex.printStackTrace(); 
     } 
    } 
} 

này cấu hình người thi hành để phù hợp với cách mã của bạn hiện đang hoạt động. Có một số cách khác bạn có thể làm; xem liên kết javadoc ở trên.

Answer 2

Làm thế nào khi nào có thể xảy ra sự cố khi ngủ trong vòng lặp?
Mọi người đôi khi sử dụng nó thay cho các phương pháp đồng bộ hóa thích hợp (như chờ/thông báo).

Nếu đó là vấn đề, tôi nên làm gì?
Phụ thuộc vào những gì bạn đang làm. Mặc dù điều đó khiến tôi tưởng tượng ra tình huống khi làm điều này là cách tiếp cận tốt nhất, tôi đoán điều đó cũng có thể xảy ra.

Bạn có thể kiểm tra về chủ đề này.

Answer 3

Như những người khác đã nói nó phụ thuộc vào cách sử dụng.Sử dụng hợp pháp sẽ là một chương trình được thiết kế để thực hiện điều gì đó sau mỗi 10 giây (nhưng không quá quan trọng đến mức cần có thời gian chính xác). Chúng tôi có rất nhiều "ứng dụng tiện ích" này nhập dữ liệu và các tác vụ khác cứ sau vài phút. Đây là một cách dễ dàng để thực hiện các tác vụ này và chúng tôi thường sẽ đặt khoảng thời gian ngủ là rất thấp và sử dụng bộ đếm để chương trình luôn đáp ứng và có thể thoát dễ dàng.

int count = 0; 
while (true) { 

    try { 
     // Wait for 1 second. 
     Thread.sleep(1000); 
    } 
    catch (InterruptedException ex) {} 

    // Check to see if the program should exit due to other conditions. 
    if (shouldExit()) 
     break; 

    // Is 10 seconds up yet? If not, just loop back around. 
    count++; 
    if (count < 10) continue; 

    // 10 seconds is up. Reset the counter and do something important. 
    count = 0; 
    this.doSomething(); 
} 

Answer 4

Tôi nghĩ rằng tôi bắt gặp một cách sử dụng hoàn toàn hợp pháp phương pháp sleep() trong vòng lặp.

Chúng tôi có kết nối một chiều giữa máy chủ và ứng dụng khách. Vì vậy, khi khách hàng muốn đạt được truyền thông không đồng bộ với máy chủ, anh ta gửi tin nhắn đến máy chủ và hơn bình chọn định kỳ cho một số phản hồi từ máy chủ. Cần có khoảng thời gian chờ.

Response resp = null; 
for (int i = 0; i < POLL_REPEAT && resp == null; i++) { 
    try { 
     Thread.sleep(POLL_INTERVAL); 
    } catch (InterruptedException ie) { 
    } 
    resp = server.getResponse(workflowId); 
} 

POLL_REPEAT * POLL_INTERVAL ~ khoảng TIMEOUT