给定一个 Calendar 实例，确定它是否已经过一小时的最简单方法是什么？

Question

追踪代码块中一些微妙的、烦人的错误。

此代码给出了一个 Calendar 实例，该实例是在某个事件开始时创建的。我们需要弄清楚该事件是否已经发生了一小时。

我知道有几种方法可以做到这一点：

• 使用 JodaTime 并计算持续时间
• 将给定的日历与一小时前创建的新日历进行
• 比较 将给定的 Calendar.getTime 与一小时前创建的日期进行比较

每个方法都有自己的怪癖并且似乎有代码味道。 Java中有没有一种简单、简洁的方式来表达“从现在到一小时前发生的事情”？

当我对此进行单元测试时，我会检查：

• 45 分钟前（真）
• 61 分钟前（假）
• 0 分钟前（真）
• -1 分钟前（错误还是真？）

Answer 1

您可以创建一个新的日历实例（包含当前日期/时间）并将其与给定的实例进行比较：

public boolean isWithinHour(Calendar given)
{
    long oldTime = given.getTimeInMillis();
    long newTime = Calendar.getInstance().getTimeInMillis();

    long diff = newTime - oldTime;
    return (diff <= 3600 * 1000);    // 3600 seconds in millis
}

Answer 2

作为个人喜好，我会选择 Joda-Time，但我认为实现并不重要。重要的是明确目的并创建一个可以使用的方法。

首先，您最好定义什么意思一小时前发生。

• 是指正好一个小时前吗？
• 是指一个小时前吗？
• 是指45分钟前到1小时15分钟前吗？

之后，您选择您提到的方法之一。然后您编写自己的方法并使用描述性名称，该名称应该在您的代码中自然地读取。

Answer 3

可能与您列出的方法没有太大不同，但我不明白它的气味如何或为何：

if (calendar.getTimeInMillis() + (60L * 60L * 1000L)) <= System.currentTimeMillis()) {
    // created more than one hour ago
}

Answer 4

我使用的方法是与过去 1 小时的时间戳进行比较：

return cal.getTimeInMillis() < (System.currentTimeMillis() - 1000L * 60L * 60L);