1

我不知道这里发生了什么：如果当前时间在给定范围之间，我需要返回 true，例如：现在是 15:45:00 并且在 12:00:00（分钟）和 18:00:00 之间（最大），但这种方法做错了：

private boolean verifyIfInside(String max, String min){
    try {
        DateFormat dateFormat = new SimpleDateFormat ("HH:mm:ss");

        Date date = new Date();

        String hour1 = min;
        String hour2 = max;
        String newHour = dateFormat.format(date);

        Date minHour, maxHour, nowHour;
        minHour = dateFormat.parse(hora1);
        maxHour = dateFormat.parse(hora2);
        nowHour = dateFormat.parse(newHour );

        if ((minHour.compareTo(nowHour) <= 0) && (maxHour.compareTo(nowHour) >= 0)){
            return true;
        } else {
            return false;
        }
    } catch (ParseException parseException){
        parseException.printStackTrace();
        return false;
    }
}
4

3 回答 3

0

我已经用.before()和.after( )对其进行了测试，它工作正常，您还可以减少代码，例如：-

private static boolean verifyIfInside(String min , String max){
    try {
        DateFormat dateFormat = new SimpleDateFormat ("HH:mm:ss");

        String current_time = dateFormat.format(new Date());

        Date minHour = dateFormat.parse(min);
        Date maxHour = dateFormat.parse(max);
        Date curr_time = dateFormat.parse(current_time);

        return minHour.before(curr_time) && maxHour.after(curr_time);

    } catch (ParseException parseException){
        parseException.printStackTrace();
        return false;
    }
}
于 2018-01-12T19:24:16.443 回答
0
  1. 时区至关重要。除非您知道当前时间位于哪个时区，否则您的验证毫无意义。为了读者的利益，也请在您的代码中指定它。
  2. 您使用的日期和时间类Date,DateFormat和SimpleDateFormat, 不仅早已过时，尤其是最后一个以难以使用而闻名。相反java.time，我建议您使用现代 Java 日期和时间 API。这甚至提供了一个LocalTime课程，一个没有日期的时间，它比Date.

因此，您的方法写得很清楚简洁：

private static boolean verifyIfInside(String max, String min) {
    LocalTime now = LocalTime.now(ZoneId.of("Europe/Budapest"));
    LocalTime minHour = LocalTime.parse(min);
    LocalTime maxHour = LocalTime.parse(max);
    return ! (now.isBefore(minHour) || now.isAfter(maxHour));
}

如果它不是欧洲/布达佩斯，请替换您想要的时区。

既然我知道你希望你的范围是包容性的，并且因为isBefore和isAfter意味着前后严格，我用否定来测试时间“不在范围之外”。

我进一步利用了您的时间格式12:00:00与 ISO 8601 一致的事实，现代类将其解析为默认格式，即没有任何显式格式化程序。一个潜在的缺点是时间字符串的验证较弱：parse()也将接受12:00和12:23:29.82376342。如果您需要拒绝这些错误，您确实需要一个明确的格式化程序。

你的代码出了什么问题？

据我所知，您在问题中的代码工作正常（如果我们只是更改hora1为hour1或相反hora2）。您可能对参数的顺序感到困惑。如果在 12 和 18 之间运行，则以下返回 true verifyIfInside("18:00:00", "12:00:00")：。但是，许多人会发现将调用编写为 是很自然的verifyIfInside("12:00:00", "18:00:00")，它总是返回 false。因此，将方法声明为 几乎肯定会更好verifyIfInside(String min, String max)，也就是说，使用minbefore max。

当然，正如我所提到的，另一种可能性是，您的 JVM 使用的时区与您认为它正在使用的时区不同。这很可能会给您带来意想不到的结果。确保不是这种情况的唯一方法是明确指定它应该使用哪个时区。

于 2018-01-12T23:27:32.327 回答
-1

它适用于建议的修改：

  • 反转最小和最大参数。
  • 使用.before()and.after()比较当前小时是否在最小小时之后和最大小时之前。
  • 要包括最小/最大时间界限，您可以使用：!nowHour.after(maxHour) && !nowHour.before(minHour);
  • 要排除最小/最大时间边界，请改用：nowHour.after(minHour) && nowHour.before(maxHour);
  • 如果您受限于较旧版本的 java，请考虑按照@Basil Bourque 的建议使用适当的 java.time ThreeTen-Backport 项目。否则，请使用 Java 8+ 中的最新时间 API。

代码：

import java.text.DateFormat;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.Date;

public class scratch_21 {
    public static void main(String[] args) {
        System.out.println(verifyIfInside("14:00:00", "14:15:00"));
    }

    private static boolean verifyIfInside(String min, String max) {
        try {
            DateFormat dateFormat = new SimpleDateFormat("HH:mm:ss");

            Date date = new Date();

            String hora1 = min;
            String hora2 = max;
            String newHour = dateFormat.format(date);

            Date minHour, maxHour, nowHour;
            minHour = dateFormat.parse(hora1);
            maxHour = dateFormat.parse(hora2);
            nowHour = dateFormat.parse(newHour);

            return  nowHour.after(minHour) && nowHour.before(maxHour);
        } catch (ParseException parseException) {
            parseException.printStackTrace();
            return false;
        }
    }
}
于 2018-01-12T19:09:09.933 回答