声纳抱怨记录或重新抛出异常

Posted 2023-03-16

【中文标题】声纳抱怨记录或重新抛出异常

【英文标题】：Sonar complaining logging or rethrowing the exception

【发布时间】：2022-01-12 13:00:58

【问题描述】：

当我在与 Maven 集成后运行 SonarQube 进行代码质量检查时，我有以下代码。

但是，Sonar 抱怨我应该记录或重新抛出此异常。

我在这里缺少什么？谁能帮帮我。

代码

public ShippingResponse processShipping(ShippingRequest request) 
        log.debug("Processing Reservation Request ....");
        try
            return helper.processShippingMethod(request);
         catch (ServiceException serviceException) 
            log.error(RESERVATION_EXCE, ExceptionUtils.getStackTrace(serviceException));
            throw serviceException;
         catch (Exception e) 
            throw new ServiceException(ErrorMessages.EPO_SM_ERR_03, e.getMessage());
        
    

【问题讨论】：

这能回答你的问题吗？ Sonar complaining about logging and rethrowing the exception

与上述解决方案相比，没有我的 catch 块差异

如果您阅读答案，您会发现这是完全相同的问题，因为您没有使用异常对象。您只是传递异常消息，导致您丢失所有堆栈跟踪信息

LOG.error("Exception", e);

这条线是不是必须要添加到我的catch块中？

【参考方案1】：

Sonar 试图说明的一点是，您理想地打印或保留异常的根本原因，因此基本上是堆栈。您通过传递异常对象来保留它，因为如果您只保留消息，您将丢失所有这些信息。为了让声纳满意，您要么打印堆栈跟踪 (log.error(ErrorMessages.EPO_SM_ERR_03, e))，要么重新抛出一个新异常，将 Throwable 对象传递给构造函数。

所以理想的解决方案是像这样使用 ServiceException；

public class ServiceException extends Exception 
    public ServiceException(String message, Throwable cause) 
        super(message, cause);
    

throw new ServiceException(ErrorMessages.EPO_SM_ERR_03, e);