为什么 Android Studio 会使用此代码给出空指针异常警告？

Question

我在 Android Studio 中有以下代码，但我收到Method invocation 'toString()' may produce 'NullPointerException' 的警告。

String source_regions_user_id = "";
if (user_ids != null && "".equals(source_region) && user_ids.containsKey(source_region) && user_ids.get(source_region) != null && user_ids.get(source_region) != "") {
    source_regions_user_id = user_ids.get(source_region).toString();
} else {
    return true; // Unable to find a matching user_id for source_region
}

请注意，user_ids 是 Hashmap，source_region 是 String。

我相信我正在检查 toString() 所依赖的所有内容中的空值，那么为什么 Android Studio 仍会发出此警告？

Answer 1

编译器应该能够看到您检查是否为 null，但在 if 语句中有许多条件，因此它可能超出了编译器性能的某个阈值。

由于else 只不过是return，因此您应该翻转语句。这消除了单独声明变量的需要。

我也将条件分开来评论它们。

if (user_ids == null)
    return true;
if (! "".equals(source_region)) // I think you meant the opposite check
    return true;
if (! user_ids.containsKey(source_region)) // Redundant, the next check will cover this
    return true;
if (user_ids.get(source_region) == null)
    return true;
if (user_ids.get(source_region) == "") // Object is not a string, so this will always fail
    return true;
String source_regions_user_id = user_ids.get(source_region).toString();

您还应该使用isEmpty() 或length() 来检查空字符串，而不是与"" 进行比较。

因此，考虑到这些 cmets，我们可以将代码更改为：

if (user_ids == null || source_region.isEmpty())
    return true;
Object obj = user_ids.get(source_region);
if (obj == null)
    return true;
String source_regions_user_id = obj.toString();
if (source_regions_user_id.isEmpty())
    return true;
// use value here

使用此代码，编译器不会感到困惑，因此不会发出警告，并且您只需在地图中查找一次。