这个空检查有什么问题吗？

Question

List<Foo> results = null;
results = this.getResults();
if (results == null || results.size() == 0)
{
    LOGGER.warn("results empty");
}
else
{
    LOGGER.warn("results:" + results.toString());
}

当 getResults 返回 null 列表时，上面的代码始终会产生以下输出。

results:[null]

我需要响应这个空场景，但不知道如何捕获它。

Answer 1

我认为结果列表中有一项为空值。

我认为方法是检查列表的第 0 个位置是否包含空值，如果不包含，则列表至少包含一个非空值。

List<Foo> results = null;
results = this.getResults();
if (results == null || results.size() == 0) {
    LOGGER.warn("results empty");
} else if (list.get(0) == null){
    LOGGER.warn("the list has only an null value");
} else {
    LOGGER.warn("results:" + results.toString());
}

或者

List<Foo> results = null;
results = this.getResults();
if (results == null || results.size() == 0 || list.get(0) == null) {
    LOGGER.warn("results empty");
} else {
    LOGGER.warn("results:" + results.toString());
}

Answer 2

根据您的输出，getResults() 返回的 List 看起来有一个 null 元素。

List<Foo> results = null;
results = this.getResults();
if (results == null || results.size() == 0) // you could add the check here
{
    LOGGER.warn("results empty");
}
else if(results.size() == 1 && results.get(0) == null) // Or a new else if here
{
    LOGGER.warn("all I've got is a null in my list :(");
}
else
{
    LOGGER.warn("results:" + results.toString());
}

Answer 3

每个人都正确地认为您的列表包含一个空值。但是，我不明白为什么需要检查第一个元素是否为空。看起来更像是您需要重新考虑 getResults() 中发生的情况，并且不将 null 放入列表中。我的意思是，我们没有理由假设不能有更多的空值，或者空值位于实际对象之间。