我将如何修复以下代码的 NullPointerException？

Question

我有这段代码，当我运行脚本时，我传递了有效的参数，但我不断收到 NPE。 帮助？

代码：

private static Date getNearestDate(List<Date> dates, Date currentDate) {
    long minDiff = -1, currentTime = currentDate.getTime();
    Date minDate = null;
    if (!dates.isEmpty() && currentDate != null) {
        for (Date date : dates) {
            long diff = Math.abs(currentTime - date.getTime());
            if ((minDiff == -1) || (diff < minDiff)) {
                minDiff = diff;
                minDate = date;
            }
        }
    }
    return minDate;
}

我从上面代码的第 2 行得到 NullPointerException，并使用以下代码将 thisDate 作为 currentDate 变量传递。

Date thisDate = null;
try {
    thisDate = (new SimpleDateFormat("MM/dd/yyyy")).parse(Calendar.getInstance().getTime().toString());
} catch (Exception e) {}

Answer 1

由于您已指出在第 2 行引发了 NullPointerException，因此我们可以推断您为 currentDate 参数传递了 null。 currentDate.getTime() 是第 2 行中唯一可能导致 NullPointerException 的部分。

更新：

我刚刚编写了以下 Test.java 代码来真正了解您的问题是什么：

import java.util.*;
import java.text.*;

class Test {
    public static void main(String[] args) {
        Date thisDate = null;
        try {
            thisDate = (new SimpleDateFormat("MM/dd/yyyy")).parse(Calendar.getInstance().getTime().toString());
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}

当我运行它时，我得到：

java.text.ParseException: Unparseable date: "Sat Aug 20 12:42:30 PDT 2011"
    at java.text.DateFormat.parse(DateFormat.java:337)
    at Test.main(Test.java:8)

所以问题是您的 SimpleDateFormat.parse() 需要月/日/年格式，但是 Date 类的 toString() 方法为您提供了不同的内容。

似乎您真正想要的只是当前日期。为什么要麻烦格式化呢？只需将其修剪到此即可完成：

Date thisDate = new Date(); // this gives you the current date

Answer 2

你的行：

thisDate = (new SimpleDateFormat("MM/dd/yyyy")).parse(Calendar.getInstance().getTime().toString());

所做的是

• 创建一个新的 Calender 对象（用当前时间初始化），
• 从中获取一个 Date() 对象，
• 将其转换为字符串，该字符串将使用默认格式，如“EEE MMM d HH:mm:ss z yyyy ”
• 并尝试将结果解析为格式为“MM/dd/yyyy”的日期
• （根据设计将无法解析，导致 null）

如果您解决了这一行中的所有问题，结束结果将是一个包含当前时间的 Date 对象。

获取此类 Date 的更简单方法是：

thisDate = new Date();