SonarQube：仅有条件地调用方法

Question

以下代码部分在 SonarQube 上引发了一个主要错误：“仅有条件地调用方法。”

我该如何解决这个问题？

 if(us != null){
    logger.info("Log this: {}", us.toString());
}

原文由 Olezt 发布，翻译遵循 CC BY-SA 4.0 许可协议

Answer 1

对 us.toString() 的调用是多余的， toString() 方法将被调用，无论配置的日志级别如何。您应该只将 us 作为参数传递给 info 而没有 if 语句。

 logger.info("Log this: {}", us);

原文由 Tibor Blenessy 发布，翻译遵循 CC BY-SA 3.0 许可协议

Answer 2

正如问题的评论所述，另一个有效的答案是：

 if(logger.isInfoEnabled() && us != null){
    logger.info("Log this: {}", us.toString());
}

原文由 Olezt 发布，翻译遵循 CC BY-SA 3.0 许可协议