5

我正在尝试将现有功能转换为 Polly Retry 策略

public static T Execute<T>(Func<T> getTask) where T : Task
{
    var retryCount = 3;
    while (retryCount-- > 0)
    {
        try
        {
            getTask().Wait();
            return getTask();
        } catch(Exception ex){
            // handle retry
        }
    }
}

转换成这个

public static T Execute<T>(Func<T> func) where T : Task
{
    var task = func();
    Policy.Handle<HttpRequestException>()
        .Or<TimeoutException>()
        .WaitAndRetryAsync(
            3,
            retryAttempt => TimeSpan.FromSeconds(Math.Pow(5, retryAttempt)),
            (exception, timeSpan, retryCount, context) =>
            {
                //do some logging
            })
        .ExecuteAsync(func).Wait();
    return task;
}

测试代码是

var retryCount = 0;
var res = HttpRetryWrapper.Execute(() => Task.Factory.StartNew<string>(function: () =>
{
    if (++retryCount == 3)
    {
        return "fake";
    }
    throw new TimeoutException();
}));

当我断言该res值时,我没有得到正确的结果。调试跟踪我到Execution没有正确等待结果的地方。

调用次数test function正确。但是日志记录混乱,最终结果没有结果fake

4

2 回答 2

4

对于通过硬编码 Polly 策略进行异步执行的辅助方法,其中执行通过 a 异步返回 type TResultTask<TResult>您可以采用:

public static Task<TResult> ExecuteAsync<TResult>(Func<Task<TResult>> func) 
{
    return Policy.Handle<HttpRequestException>()
        .Or<TimeoutException>()
        .WaitAndRetryAsync(
            3,
            retryAttempt => TimeSpan.FromSeconds(Math.Pow(5, retryAttempt)),
            (exception, timeSpan, retryCount, context) =>
            {
                //do some logging
            })
        .ExecuteAsync<TResult>(func); // This is an async-await-eliding contraction of: .ExecuteAsync<TResult>(async () => await func());
    }

(鉴于每次使用的策略都相同,您也可以考虑将策略存储在静态字段中并仅创建一次。)

注意:这(故意)不符合您对原始问题的评论中所述的合同,因为它是牢不可破的:

public static T Execute<T>(Func<T> func) where T : Task

该子句where T : Task最初看起来对设计用于与 or 一起使用的 or async async- like方法很有吸引力。Jon Skeet在这里这里解释了为什么它不适用于异步。您提出的辅助方法签名本身不是异步的:TaskTask<T>

public static T Execute<T>(Func<T> func) where T : Task

但是,.ExecuteAsync(async () => await func());在您的示例代码中引入会导致类似的问题。Polly.ExecuteAsync(...)重载,正是为了与async/很好地配合await,以两种主要形式存在:

(1)Task ExecuteAsync(Func<Task> func)

(2)Task<TResult> ExecuteAsync<TResult>(Func<Task<TResult>> func)

编译器必须在编译时选择一个或另一个:它不能(在您的示例代码中)在运行时将其编译(1) 或 (2) 。因为它只知道它选择 (1),它返回. 因此,您在对@JamesFaix 的回答的评论中看到的错误:。T : TaskTaskCannot implicitly convert type Task to T

如果您想要这种形式的辅助模式,调用者可以将其用于任一TaskTask<TResult>-returning 调用,则必须声明两者:

class HttpRetryWrapper
{
    private static policy = Policy.Handle<HttpRequestException>()
        .Or<TimeoutException>()
        .WaitAndRetryAsync(
            3,
            retryAttempt => TimeSpan.FromSeconds(Math.Pow(5, retryAttempt)),
            (exception, timeSpan, retryCount, context) =>
            {
                //do some logging
            });

    public static Task ExecuteAsync(Func<Task> func) 
    {
        return policy.ExecuteAsync(func);
    }

    public static Task<TResult> ExecuteAsync<TResult>(Func<Task<TResult>> func) 
    {
        return policy.ExecuteAsync<TResult>(func);
    }
}

最后,如果这是为了包装通过 发出的呼叫HttpClient,推荐的模式是将 Polly 策略放置在 aDelegatingHandler中,如@MuhammedRehanSaeed 的答案中所述。ASP.NET Core 2.1 支持使用 IHttpClientFactoryDelegatingHandler创建此类的简洁声明。

于 2018-06-12T22:12:29.120 回答
1

我想你想要的是

public static T Execute<T>(Func<T> func) where T : Task
{
    return Policy.Handle<HttpRequestException>()
        .Or<TimeoutException>()
        .WaitAndRetryAsync(
            3,
            retryAttempt => TimeSpan.FromSeconds(Math.Pow(5, retryAttempt)),
            (exception, timeSpan, retryCount, context) =>
            {
                //do some logging
            })
        .ExecuteAsync(func).Wait();
}

在您的代码示例中,您调用func一次并返回它,然后在定义和调用策略之间,但不返回调用该策略的结果。

如果你想避免Wait,我想你也可以这样做

public static T Execute<T>(Func<T> func) where T : Task
{
    return Policy.Handle<HttpRequestException>()
        .Or<TimeoutException>()
        .WaitAndRetryAsync(
            3,
            retryAttempt => TimeSpan.FromSeconds(Math.Pow(5, retryAttempt)),
            (exception, timeSpan, retryCount, context) =>
            {
                //do some logging
            })
        .ExecuteAsync(async () => await func());
}
于 2018-06-12T13:03:32.290 回答