声纳违规：仅有条件地调用方法

Question

我们公司刚刚建立了一个声纳农场。我对我们的代码质量很好奇，想改进。

我有一个代码包含这样的记录器调用：

LOGGER.error(String.format("Cannot load object in status %s (%s)", status, statusDescription), e);

LOGGER.info(String.format("%s object(s) loaded in status %s (%s)", objects.size(), status, statusDescription));

Sonar触发规则squid:S262，仅有条件地调用方法“前提条件”和日志参数不应要求求值。由这两行触发的规则。

关于这件事，我不太清楚到底发生了什么。这个解释不太适合我的用例。Sonar doc提供了以下示例：

logger.log(Level.DEBUG, "Something went wrong: " + message);  // Noncompliant; string concatenation performed even when log level too high to show DEBUG messages

我完全理解（调试不会记录在生产中，因此会发生不必要的操作）。但对于信息和错误级别，我假设您无论如何都要记录它。此外，在我的例子中，我希望两者都被记录。

哪种方法好？重写不同不使用String.format？调整声纳以不在信息/错误级别开火？忽略这个的声纳？别的东西？

Answer 1

您的连接将在记录器中的条件检查之前完成。因此，如果您调用记录器10次，并且计算结果返回false，那么您的字符串也会无缘无故地连接10次。记录器将在通过评估后处理所有连接和格式化，并需要打印一些内容，这样可以节省无用操作的时间。

LOGGER.error("Cannot load object in status {} ({})", status, statusDescription, e);