一个动作中的C＃异步

Question

我想写一个接受几个参数的方法，包括一个动作和一个重试量并调用它。

所以我有这个代码：

public static IEnumerable<Task> RunWithRetries<T>(List<T> source, int threads, Func<T, Task<bool>> action, int retries, string method)
    {
        object lockObj = new object();
        int index = 0;

        return new Action(async () =>
        {
            while (true)
            {
                T item;
                lock (lockObj)
                {
                    if (index < source.Count)
                    {
                        item = source[index];
                        index++;
                    }
                    else
                        break;
                }

                int retry = retries;
                while (retry > 0)
                {
                    try
                    {
                        bool res = await action(item);
                        if (res)
                            retry = -1;
                        else
                            //sleep if not success..
                            Thread.Sleep(200);

                    }
                    catch (Exception e)
                    {
                        LoggerAgent.LogException(e, method);
                    }
                    finally
                    {
                        retry--;
                    }
                }
            }
        }).RunParallel(threads);
    }

RunParallel是Action的扩展方法，它看起来像这样：

public static IEnumerable<Task> RunParallel(this Action action, int amount)
    {
        List<Task> tasks = new List<Task>();
        for (int i = 0; i < amount; i++)
        {
            Task task = Task.Factory.StartNew(action);
            tasks.Add(task);
        }
        return tasks;
    }

现在，问题是：线程正在消失或崩溃而不等待操作完成。

我写了这个示例代码：

private static async Task ex()
    {
        List<int> ints = new List<int>();
        for (int i = 0; i < 1000; i++)
        {
            ints.Add(i);
        }

        var tasks = RetryComponent.RunWithRetries(ints, 100, async (num) =>
        {
            try
            {
                List<string> test = await fetchSmthFromDb();
                Console.WriteLine("#" + num + "  " + test[0]);
                return test[0] == "test";
            }
            catch (Exception e)
            {
                Console.WriteLine(e.StackTrace);
                return false;
            }

        }, 5, "test");

        await Task.WhenAll(tasks);
    }

fetchSmthFromDb是一个简单的Task>，它从db中获取一些东西，并且在这个例子之外调用时工作得很好。

只要List<string> test = await fetchSmthFromDb(); 调用行，线程似乎正在关闭和Console.WriteLine("#" + num + " " + test[0]); 甚至没有被触发，也是在调试断点时从未命中。

最终工作守则

private static async Task DoWithRetries(Func<Task> action, int retryCount, string method)
    {
        while (true)
        {
            try
            {
                await action();
                break;
            }
            catch (Exception e)
            {
                LoggerAgent.LogException(e, method);
            }

            if (retryCount <= 0)
                break;

            retryCount--;
            await Task.Delay(200);
        };
    }

    public static async Task RunWithRetries<T>(List<T> source, int threads, Func<T, Task<bool>> action, int retries, string method)
    {
        Func<T, Task> newAction = async (item) =>
        {
            await DoWithRetries(async ()=>
            {
                await action(item);
            }, retries, method);
        };
        await source.ParallelForEachAsync(newAction, threads);
    }

Answer 1

问题出在这一行：

return new Action(async () => ...

您使用异步lambda启动异步操作，但不要返回任务以等待。 即它在工作线程上运行，但你永远不会知道它何时完成。 并且您的程序在异步操作完成之前终止 - 这就是您没有看到任何输出的原因。

它需要是：

return new Func<Task>(async () => ...

---

UPDATE

首先，您需要分割方法的职责，因此不要将重试策略（不应该硬编码为检查布尔结果）与并行运行的任务混合使用。

然后，如前所述，您运行while (true)循环100次而不是并行执行。

正如@MachineLearning所指出的，使用Task.Delay而不是Thread.Sleep 。

总的来说，您的解决方案如下所示：

using System.Collections.Async;

static async Task DoWithRetries(Func<Task> action, int retryCount, string method)
{
    while (true)
    {
        try
        {
            await action();
            break;
        }
        catch (Exception e)
        {
            LoggerAgent.LogException(e, method);
        }

        if (retryCount <= 0)
            break;

        retryCount--;
        await Task.Delay(millisecondsDelay: 200);
    };
}

static async Task Example()
{
    List<int> ints = new List<int>();
    for (int i = 0; i < 1000; i++)
        ints.Add(i);

    Func<int, Task> actionOnItem =
        async item =>
        {
            await DoWithRetries(async () =>
            {
                List<string> test = await fetchSmthFromDb();
                Console.WriteLine("#" + item + "  " + test[0]);
                if (test[0] != "test")
                    throw new InvalidOperationException("unexpected result"); // will be re-tried
            },
            retryCount: 5,
            method: "test");
        };

    await ints.ParallelForEachAsync(actionOnItem, maxDegreeOfParalellism: 100);
}

您需要使用AsyncEnumerator NuGet包才能使用System.Collections.Async命名空间中的ParallelForEachAsync扩展方法。

Answer 2

除了最后的完整再造之外，我认为强调原始代码的真正错误是非常重要的。

0）首先，正如@Serge Semenov立即指出的那样，Action必须被替换为

Func<Task>

但仍有其他两个重要变化。

1）使用异步委托作为参数，必须使用更新的Task.Run而不是旧的模式new TaskFactory.StartNew（否则你必须显式添加Unwrap（））

2）此外ex（）方法不能是异步的，因为必须使用Wait（）并且没有await等待Task.WhenAll。

那时，即使存在需要重新设计的逻辑错误，从纯粹的技术角度来看，它确实有效并且产生了输出。

可在线获得测试： http ： //rextester.com/HMMI93124