NetBeans / Java / 新提示:Thread.sleep 在循环中调用

2022-08-31 22:14:05

在 NetBeans 中,有一个新的提示:Thread.sleep 调用 in loop。

问题 1:如何/何时会成为一个问题,在循环中睡觉?

问题 2:如果这是一个问题,我该怎么办?

更新:问题 3:下面是一些代码。在这种情况下,告诉我是否应该使用其他东西而不是循环中的Thread.Sleep。简而言之,这是由侦听客户端TCP连接的服务器使用的。此处使用睡眠,以防达到与客户端的最大会话数。在此情况下,我希望应用程序等待,直到空闲会话变为可用。

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();
}
}

答案 1

在循环中调用睡眠通常会导致性能不佳。例如:

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

如果毫秒太大,则此代码将需要很长时间才能意识到输入可用。

如果毫秒太小,则此代码将浪费大量系统资源来检查尚未到达的输入。

in a loop 的其他用法通常也是值得怀疑的。通常有更好的方法。sleep

如果这是一个问题,我该怎么办?

发布代码,也许我们可以给你一个明智的答案。

编辑

IMO,解决这个问题的更好方法是使用ThreadPoolExecutor

像这样:

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();
        }
    }
}

这会将执行程序配置为与代码当前的工作方式相匹配。还有许多其他方法可以做到这一点;请参阅上面的 javadoc 链接。


答案 2

正如其他人所说,这取决于使用情况。合法的用途是设计为每10秒做一些事情的程序(但不是那么重要,以至于需要确切的时间)。我们有很多这样的“实用程序”,它们每隔几分钟就会导入数据和其他此类任务。这是执行这些任务的简单方法,我们通常会将睡眠间隔设置为非常低,并使用计数器,以便程序保持响应并可以轻松退出。

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();
}