我使用 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--;
}
}
}
}
答案 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