If 语句和返回的编码约定

Question

我不确定我是否想多了，但我应该这样做吗

if (!searchList.isEmpty()) {
    String search = searchList.get(0).getText();
    return List.of(search.split("\n"));
} else {
    return null;
}

或者我应该这样做

if (!searchList.isEmpty()) {
    String search = searchList.get(0).getText();
    return List.of(search.split("\n"));
}
return null;
        

Answer 1

都没有。

if (searchList.isEmpty()) {
    return null;
}
String search = searchList.get(0).getText();
return List.of(search.split("\n"));

我不喜欢实际上不需要的否定条件（“如果不为空”）。

我的想法是“先摆脱边缘情况，然后处理主要逻辑”。

这当然只是个人意见。最终您将培养自己对如何布局代码的品味；好品味是程序员最重要的品质之一。

Answer 2

一些建议：

1. 你永远不应该 return null 值。这是一个不好的做法
2. 您应该测试 true 而不是 false。它使您的代码更具可读性

你应该看起来像这样：

if (searchList.isEmpty()) {
    return Collections.emptyList();
}
String search = searchList.get(0).getText();
return List.of(search.split("\n"));