SonarLint 警告，此代码将始终返回相同的值

Question

SonarLint 告诉我：

"Refactor this code to not always return the same value."

但我似乎不明白为什么

private List<InvoiceRejectionMessage> invoiceList = new ArrayList<>();

public String getMessage() {
    if(!invoiceList.isEmpty()) {
        return invoiceList
                .stream()
                .max(comparing(InvoiceRejectionMessage::getCreatedAt))
                .get()
                .getMessage();
    }
    return null;
}

InvoiceList 已定义并且将始终被初始化，因此它不能为空，只能为空。如果为空，则返回 null。如果不是，我们确定其中有一个元素可以由 .max() 和 .get() 返回

我不愿意仅仅因为 Sonarlint 告诉我重构这个方法，我宁愿知道为什么我会收到这个警告

Answer 1

与此提示关联的规则是

squid:S3516 - 方法返回不应该是不变的

此规则的 SonarQube 实现在 GitHub 上可用。

如果没有看到您的代码整体，我不能 100% 确定为什么会触发此规则。但是，我怀疑声纳能够弄清楚

1. invoiceList 无条件为空并且；因此
2. if-then-else 语句的if 分支永远不会被执行；因此，
3. the getMessage 方法无条件返回null。

---

无论如何，没有必要将空列表视为特殊情况；您可以通过以下方式简化代码，这可能会安抚 Sonar：

private List<InvoiceRejectionMessage> invoiceList = new ArrayList<>();

public String getMessage() {
    return invoiceList
            .stream()
            .max(comparing(InvoiceRejectionMessage::getCreatedAt))
            .map(InvoiceRejectionMessage::getMessage)
            .orElse(null);
}

顺便说一句，如果您可以更改类的 API，它的客户端将受益于将 getMessage 的返回类型更改为 Optional<String>（只需删除最后一个 orElse(null) 调用）：

public Optional<String> getMessage() {
    return invoiceList
            .stream()
            .max(comparing(InvoiceRejectionMessage::getCreatedAt))
            .map(InvoiceRejectionMessage::getMessage)
}