[解決済み] 再試行ロジックの最もクリーンな記述方法とは？

質問

時々、操作をあきらめる前に何度も再試行する必要があることがあります。 私のコードは次のようなものです。

int retries = 3;
while(true) {
  try {
    DoSomething();
    break; // success!
  } catch {
    if(--retries == 0) throw;
    else Thread.Sleep(1000);
  }
}

これを一般的なリトライ関数に書き換えると、次のようになります。

TryThreeTimes(DoSomething);

C#で可能でしょうか？ のコードはどうなるのでしょうか？ TryThreeTimes() メソッドを使用できますか？

どのように解決するのですか？

同じ呼び出しを単純に再試行する包括的なcatch文は、一般的な例外処理メカニズムとして使用すると危険な場合があります。とはいえ、ここではどんなメソッドでも使えるラムダベースの再試行ラッパーを紹介します。再試行の回数と再試行のタイムアウトをパラメータにすることで、もう少し柔軟性を持たせることにしました。

public static class Retry
{
    public static void Do(
        Action action,
        TimeSpan retryInterval,
        int maxAttemptCount = 3)
    {
        Do<object>(() =>
        {
            action();
            return null;
        }, retryInterval, maxAttemptCount);
    }

    public static T Do<T>(
        Func<T> action,
        TimeSpan retryInterval,
        int maxAttemptCount = 3)
    {
        var exceptions = new List<Exception>();

        for (int attempted = 0; attempted < maxAttemptCount; attempted++)
        {
            try
            {
                if (attempted > 0)
                {
                    Thread.Sleep(retryInterval);
                }
                return action();
            }
            catch (Exception ex)
            {
                exceptions.Add(ex);
            }
        }
        throw new AggregateException(exceptions);
    }
}

これで、このユーティリティ・メソッドを使って再試行ロジックを実行できるようになりました。

Retry.Do(() => SomeFunctionThatCanFail(), TimeSpan.FromSeconds(1));

または

Retry.Do(SomeFunctionThatCanFail, TimeSpan.FromSeconds(1));

または

int result = Retry.Do(SomeFunctionWhichReturnsInt, TimeSpan.FromSeconds(1), 4);

あるいは async をオーバーロードします。