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

Question

在将SonarQube与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());
        }
    }

Answer 1

Sonar试图指出的一点是，理想情况下，打印或保留异常的根本原因，因此基本上是堆栈。通过传递exception对象来保留它，因为如果只保留消息，则会丢失所有信息。为了让sonar满意，您可以打印堆栈跟踪（log.error（ErrorMessages.EPO\u SM\u ERR\u 03，e）），或者重新抛出一个新的异常，将Throwable对象传递给构造函数。

因此，理想的解决方案是像这样使用服务异常；

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

throw new ServiceException(ErrorMessages.EPO_SM_ERR_03, e);