Observable.Retry没有按预期工作

时间:2016-05-06 18:25:52

标签: c# .net system.reactive reactive-programming

我使用 async 方法处理数字序列。我正在模拟可能失败的远程服务电话。如果失败,我想重试,直到通话成功。

问题在于,对于我尝试的代码,每次在异步方法中抛出异常时,序列似乎永远挂起

您可以使用这个简单的代码段进行测试(它在LINQPad中测试过)

Random rnd = new Random();

void Main()
{
    var numbers = Enumerable.Range(1, 10).ToObservable();
    var processed = numbers.SelectMany(n => Process(n).ToObservable().Retry());
    processed.Subscribe( f => Console.WriteLine(f));
}

public async Task<int> Process(int n)
{
    if (rnd.Next(2) == 1)
    {
        throw new InvalidOperationException();
    }

    await Task.Delay(2000);
    return n*10;    
}

它应该处理每个元素,重试那些失败的元素。相反,它永远不会结束,我也不知道为什么。

我怎样才能做到我想要的呢?

编辑:(感谢@CharlesNRice和@JonSkeet提供的线索!):

这个有效!

Random rnd = new Random();

void Main()
{
    var numbers = Enumerable.Range(1, 10).ToObservable();
    var processed = numbers.SelectMany(n => RetryTask(() => MyTask(n)).ToObservable());
    processed.Subscribe(f => Console.WriteLine(f));
}

private async Task<int> MyTask(int n)
{
    if (rnd.Next(2) == 1)
    {
        throw new InvalidOperationException();
    }

    await System.Threading.Tasks.Task.Delay(2000);
    return n * 10;
}

async Task<T> RetryTask<T>(Func<Task<T>> myTask, int? retryCount = null)
{
    while (true)
    {
        try
        {
            return await myTask();
        }
        catch (Exception)
        {
            Debug.WriteLine("Retrying...");


            if (retryCount.HasValue)
            {
                if (retryCount == 0)
                {
                    throw;
                }

                retryCount--;
            }
        }
    }
}

2 个答案:

答案 0 :(得分:3)

在这种情况下,滚动自己的Retry是过度的。只需将方法调用包装在Defer块中即可实现相同的功能,并在重试时重新执行。

var numbers = Enumerable.Range(1, 10).ToObservable();

var processed = numbers.SelectMany(n => 
  //Defer call passed method every time it is subscribed to,
  //Allowing the Retry to work correctly.
  Observable.Defer(() => 
    Process(n).ToObservable()).Retry()
);

processed.Subscribe( f => Console.WriteLine(f));

答案 1 :(得分:1)

您正在重试处于故障状态的同一任务。重试将重新订阅可观察的来源。重试的来源是ToObservable()。它不会像任务工厂那样行动并创建一个新的任务,并且由于任务出现故障,它将继续重试故障任务并永远不会成功。

您可以查看此答案,了解如何制作自己的重试包装器 https://stackoverflow.com/a/6090049/1798889