嵌套的尝试/捕获内块和PMD规则

Question

我试图重构这段代码，导致PMD抛出失败：PreserveStackTrace:3，主要问题是如何将异常e1折叠为e2，现在是反模式吗？我正在考虑第二次尝试或最终投入，但这可能会导致业务逻辑出现问题。

代码：

        try {
            try {
                response = paypal.execute(params);
            } catch (final FailedConnectionAttempException e1) {
                log.warn("The first attempt is failed " + " Tryin again ", e1);

                try {
                    response = paypal.execute(params);
                } catch (final FailedConnectionAttempException e2) {
                    throw new PaymentException("The second attempt failed ", e2);
                }
            }
        } catch (final IntegrationException e) {
            throw new PaymentException("Unable to execute ", e);
        }
    }

Answer 1

我假设您使用的是ApacheHttpClient，因为我识别execute方法。

您可以使用Httpask estRetryHandler来配置重试行为，而不是处理该捕获块。

HttpRequestRetryHandler retryHandler = new HttpRequestRetryHandler() {
    @Override
    public boolean retryRequest(IOException exception, int executionCount, HttpContext context) {
        return executionCount < 5;
    }
};

HttpClient client = HttpClientBuilder.create()
        .setRetryHandler(retryHandler)
        .build();

此外，我还应该提到，HTTPclipse将重试一次失败的请求3次默认，我注意到在你的例外，它说第一次/第二次尝试，在现实中，它是前4次尝试和前7次尝试，所以你可以考虑有多少重试你真正需要的。