在我的Java代码中，关键的SonarLint问题S1166是否是误报？

Question

SonarLint 1.0.0 for Eclipse在我的代码中标记了一个关键问题，我无法理解为什么以及如何解决它。 这对我来说真的是假阳性 - 或者我错过了什么？

import org.apache.log4j.Logger;

[...]

public final class Foo {

    private static final Logger logger = Logger.getLogger(Foo.class);

    [...]

    public static void foo() {

        MyCommand command = new MyCommand(foo, bar);
        try {
            commandService.executeCommand(command);
        } catch (CommandException e) {
            logger.error("My command execution failed", e);
        }
    }

    [...]

以下是匹配的SonarLint规则说明的摘录：

处理捕获的异常时，应记录或传递原始异常的消息和堆栈跟踪。

不合规的代码示例

\n //不合规 - 异常丢失\n try {/ * ... * /} catch（Exception e）{LOGGER.info（“context”）;  }   \n\n //不合规 - 异常丢失（仅保留消息）       \n try {/ * ... * /} catch（Exception e）{LOGGER.info（e.getMessage（））;  }\n\n //不合规 - 异常丢失\n try {/ * ... * /} catch（Exception e）{throw new RuntimeException（“context”）;  }\n

合规解决方案

\n try {/ * ... * /} catch（例外e）{LOGGER.info（e）;  }   \n\n try {/ * ... * /} catch（Exception e）{throw new RuntimeException（e）;  }\n\n try {/ * ... * /} catch（RuntimeException e）{\n     做一点事（）;  \n     扔掉;\n } catch（例外e）{\n     //也允许转换为未经检查的异常\n     抛出新的RuntimeException（e）;\n }\n

在我看来，我的代码符合给定的兼容解决方案的第一个变体，但SonarLint不接受它。

不久之前还有另一个关于Sonar规则S1166的讨论 ，但它并没有真正与我有同样的问题。

编辑：回答以下问题：我使用log4j进行日志记录。 我扩展了代码以反映这一点。

Answer 1

实际上，您正在记录原始异常的消息和堆栈跟踪; 这是一个错误的发现。

可能是规则没有Log4j的特定知识，但缺乏所有日志库的全知性，异常作为参数传递的事实就足够了。