为什么Thread.sleep使用不好

时间:2013-07-24 06:25:20

标签: java multithreading java-threads

对于这个重复的问题道歉,但我还没有找到任何令人满意的答案。大多数问题都有自己的具体用例:
Java - alternative to thread.sleep
Is there any better or alternative way to skip/avoid using Thread.sleep(1000) in Java?

我的问题是非常通用的用例。等待条件完成。做一些操作。检查条件。如果条件不成立,请等待一段时间再做同样的操作。

例如考虑一种通过调用其createAPI表来创建DynamoDB表的方法。 DynamoDB表需要一些时间才能变为活动状态,因此该方法会调用其DescribeTable API以定期轮询状态,直到某个时间(假设5分钟 - 由于线程调度而导致的偏差是可接受的)。如果表在5分钟内变为活动状态,则返回true,否则抛出异常。

这是伪代码:

public void createDynamoDBTable(String name) {
  //call create table API to initiate table creation

  //wait for table to become active
  long endTime = System.currentTimeMillis() + MAX_WAIT_TIME_FOR_TABLE_CREATE;

  while(System.currentTimeMillis() < endTime) {
    boolean status =  //call DescribeTable API to get status;
    if(status) {
         //status is now true, return
         return
    } else {
        try {
            Thread.sleep(10*1000);
        } catch(InterruptedException e) {
        }
    }
  }

  throw new RuntimeException("Table still not created");
}

我理解通过使用Thread.sleep阻止当前线程,从而消耗资源。但是在一个相当中等规模的应用程序中,一个主题是一个大问题吗? 我在某处使用ScheduledThreadPoolExecutor读取并在此处进行此状态轮询。但同样,我们必须使用至少一个线程初始化此池,其中运行的runnable方法将运行。

有关使用Thread.sleep的原因的任何建议都被认为是一个糟糕的主意,以及实现上述目标的替代方案有哪些。

http://msmvps.com/blogs/peterritchie/archive/2007/04/26/thread-sleep-is-a-sign-of-a-poorly-designed-program.aspx

2 个答案:

答案 0 :(得分:31)

在这种情况下使用Thread.sleep很好。人们不鼓励Thread.sleep的原因是因为它常常用于修复竞争条件,用于基于通知的同步是更好的选择等。

在这种情况下,AFAIK您没有选项但是轮询,因为API没有为您提供通知。我也可以看到这是一个不经常的操作,因为大概你不会创建千表。

因此,我觉得在这里使用Thread.sleep很好。如你所说,当你要阻止当前线程时产生一个单独的线程似乎使事情变得复杂而没有任何价值。

答案 1 :(得分:6)

是的,应该尽量避免使用Thread.sleep(x),但不应该完全忘记:

为什么要避免

  • 它不释放锁
  • 它不保证执行将在睡眠时间后开始(所以它可能会一直等待 - 显然是罕见的情况
  • 如果我们错误地将前台处理线程置于睡眠状态,那么我们将无法关闭该应用程序直到x毫秒。
  • 我们现在已经完全加载了针对特定问题的新并发包(比如设计模式( ofcourse不完全),为什么要使用Thread.sleep(x)呢。

在哪里使用Thread.sleep(x):

  • 用于在后台运行线程中提供延迟
  • 还有其他人。