private void checkRoutePlanItems(List<BNRoutePlanItem> mRoutePlanItems) { if (mRoutePlanItems == null || mRoutePlanItems.size() < 3 || mRoutePlanItems.get(0) == null || mRoutePlanItems.get(1) == null || mRoutePlanItems.get(2) == null) { return; } } 这个方法 提示 'return' is unnecessary as the last statement in a 'void' method

这个提示是因为你的方法返回类型是 `void`，也就是没有返回值，所以 `return` 语句没有实际的作用。你可以直接删除 `return;` 语句，代码也会正常运行。例如：

private void checkRoutePlanItems(List<BNRoutePlanItem> mRoutePlanItems) {
    if (mRoutePlanItems == null || mRoutePlanItems.size() < 3 || mRoutePlanItems.get(0) == null || mRoutePlanItems.get(1) == null || mRoutePlanItems.get(2) == null) {
        // do something
    }
}

这样就可以避免这个编译器提示了。

相关问题

private void checkRoutePlanItems(List<BNRoutePlanItem> mRoutePlanItems) { if (mRoutePlanItems == null || mRoutePlanItems.size() < 3 || mRoutePlanItems.get(0) == null || mRoutePlanItems.get(1) == null || mRoutePlanItems.get(2) == null) { } } 代码提示 'if' statement has empty body 可以改成 private void checkRoutePlanItems(List<BNRoutePlanItem> mRoutePlanItems) { if (mRoutePlanItems != null) { mRoutePlanItems.size(); } } 吗？

不可以。你的代码中if语句的目的是检查mRoutePlanItems是否符合要求，如果不符合则应该执行一些操作。你将if语句改成了一个没有任何实际意义的代码块，不仅没有解决问题，反而会导致代码功能失效。正确的做法是在if语句中加入需要执行的操作，比如抛出异常或者返回错误码等。例如：

private void checkRoutePlanItems(List<BNRoutePlanItem> mRoutePlanItems) {
    if (mRoutePlanItems == null || mRoutePlanItems.size() < 3 || mRoutePlanItems.get(0) == null || mRoutePlanItems.get(1) == null || mRoutePlanItems.get(2) == null) {
        throw new IllegalArgumentException("Invalid mRoutePlanItems");
    }
}

这样，如果mRoutePlanItems不符合要求，就会抛出异常，提醒调用者检查输入参数。

if (mRoutePlanItems == null || mRoutePlanItems.size() < 1 || mRoutePlanItems.get(0) == null) { return; } 请帮我把这段代码封装成一个方法，方法名要见名知义

可以封装成一个名为`checkRoutePlanItems`的方法，如下：

private void checkRoutePlanItems(List<RoutePlanItem> routePlanItems) {
    if (routePlanItems == null || routePlanItems.size() < 1 || routePlanItems.get(0) == null) {
        return;
    }
}

此方法可接收一个`List<RoutePlanItem>`类型的参数`routePlanItems`，用于检查该列表是否为空或者第一个元素是否为空。