0

我在这个帖子的评论部分提出了一些疑问。

在调试应用程序时，我注意到一些可能的代码错误。它们是错误还是功能？

  1. 在ExpressionEvaluatingRequestHandlerAdvice::evaluateSuccessExp‌​ression中，异常被AdviceMessage发送到后抛出SuccessChannel。这会导致其中的异常有效负载。如果有异常，该方法不应该首先抛出异常吗？
  2. 在同一个类中，propagateOnSuccessEvaluationFailures默认为“false”，这会导致不抛出异常。这反而否定了FailChannel. 我在外部将其设置为“true”。你能解释一下这些背后的思考过程吗？

加里回答说，

我不明白您的第 1 点。对于第 2 点，通常，错误处理程序会处理错误，调用者无需知道存在问题。如果存在，那么通常，错误处理程序流应该抛出一个新异常，以向调用者指示错误处理程序已涉及。布尔值用于您想要处理错误并将原始异常抛出给调用者的罕见实例。

我们可以在这里讨论要点，而不是使该线程混乱。

我认为第 2 点的解释适用于trapException.

为了说明我的观点，我复制了ExpressionEvaluatingRequestHandlerAdvice.

protected Object doInvoke(ExecutionCallback callback, Object target, Message<?> message) throws Exception {
        try {
            Object e = callback.execute();
            if(this.onSuccessExpression != null) {
                this.evaluateSuccessExpression(message);
            }

            return e;
        } catch (Exception var7) {
            Exception actualException = this.unwrapExceptionIfNecessary(var7);
            if(this.onFailureExpression != null) {
                Object evalResult = this.evaluateFailureExpression(message, actualException);
                if(this.returnFailureExpressionResult) {
                    return evalResult;
                }
            }

            if(!this.trapException) {
                throw actualException;
            } else {
                return null;
            }
        }
    }

    private void evaluateSuccessExpression(Message<?> message) throws Exception {
        boolean evaluationFailed = false;

        Object evalResult;
        try {
            evalResult = this.onSuccessExpression.getValue(this.prepareEvaluationContextToUse((Exception)null), message);
        } catch (Exception var5) {
            evalResult = var5;
            evaluationFailed = true;
        }

        if(this.successChannel == null && this.successChannelName != null && this.getChannelResolver() != null) {
            this.successChannel = (MessageChannel)this.getChannelResolver().resolveDestination(this.successChannelName);
        }

        if(evalResult != null && this.successChannel != null) {
            AdviceMessage resultMessage = new AdviceMessage(evalResult, message);
            this.messagingTemplate.send(this.successChannel, resultMessage);
        }

        if(evaluationFailed && this.propagateOnSuccessEvaluationFailures) {
            throw (Exception)evalResult;
        }
    }

    private Object evaluateFailureExpression(Message<?> message, Exception exception) throws Exception {
        Object evalResult;
        try {
            evalResult = this.onFailureExpression.getValue(this.prepareEvaluationContextToUse(exception), message);
        } catch (Exception var6) {
            evalResult = var6;
            this.logger.error("Failure expression evaluation failed for " + message + ": " + var6.getMessage());
        }

        if(this.failureChannel == null && this.failureChannelName != null && this.getChannelResolver() != null) {
            this.failureChannel = (MessageChannel)this.getChannelResolver().resolveDestination(this.failureChannelName);
        }

        if(evalResult != null && this.failureChannel != null) {
            ExpressionEvaluatingRequestHandlerAdvice.MessageHandlingExpressionEvaluatingAdviceException messagingException = new ExpressionEvaluatingRequestHandlerAdvice.MessageHandlingExpressionEvaluatingAdviceException(message, "Handler Failed", this.unwrapThrowableIfNecessary(exception), evalResult);
            ErrorMessage resultMessage = new ErrorMessage(messagingException);
            this.messagingTemplate.send(this.failureChannel, resultMessage);
        }

        return evalResult;
    }

在doInvoke,evaluateFailureExpression中调用 ifcallback.execute()或evaluateSuccessExpressionthrows 异常。

现在evaluateSuccessExpression，如果在评估时出现任何错误SuccessExpression，则将异常存储在evalResult. SuccessChannel如果已配置，则会发送到。如果propagateOnSuccessEvaluationFailures设置为truethen 只会evaluateSuccessExpression抛出错误。由于它默认设置为“false”，evalResult因此不会被抛出，因此不会被evaluateFailureExpression.

即使propagateOnSuccessEvaluationFailures需要根据其他要求设置为“false”，也应该在将结果发送到SuccessChannel.

4

1 回答 1

0

我想你说得有道理。请就此事提出 JIRA，我们将修改该逻辑块。一般来说，我同意最好先检查重新抛出，然后再将该异常发送到successChannel.

不幸的是，没有解决方法（除非您自己的解决方案）并且我们无法立即修复它，因为它将改变行为 - 有人可能真的依赖这种现状。

我什至认为我们可能会修改默认值，propagateOnSuccessEvaluationFailures因为您在这里也有观点。但同样：我们只能在下一个5.1版本中实现。

于 2017-12-12T14:07:01.723 回答