Есть более простой сниппет «повторять до успеха»?

Question

[martensit]

while (true)
   {
      try
      {
         File.WriteAllText(@"C:\\serv1.txt", "text");
         break;
      }
      catch (Exception)
      {
         Thread.Sleep(300);
      }
   }

Answer 1

[Александр Павлов]

Попробуйте такой метод (нет возможности проверить). На вход поддается делегат, который необходимо выполнить.

void UntilSuccess(Action action) {
  if (action == null)
    throw new ArgumentNullException();

  while (true) {
      try
      {
         action.Invoke();
         break;
      }
      catch { Thread.Sleep(300); }
  }
}

// Использовать так:
UntilSuccess(() => File.WriteAllText(@"C:\\serv1.txt", "text"));

Comments

[martensit]

.Invoke() для консольного приложения необязателен как понимаю?

[Александр Павлов]

martensit, верно, можно написать action().

[mletov]

martensit, Александр Павлов, А насколько вообще верна такая постановка задачи? А если файл держит какой-то процесс и успеха никогда не будет и получится бесконечный цикл? Может лучше ограничить код количеством неудачных попыток или задать макс время, за которое код должен отработать?

[Александр Павлов]

mletov, не поддерживаю использование такой функции, но возможно у автора вопроса необычная задача, для которой требуется именно такая функциональность.

[martensit]

mletov, Софт для себя пишу. Процесс иногда файл и держит. Но такое бывает редко и очень кратковременно.

Answer 2

[Дмитрий Еремин]

а если в рекурсию?
типа

private void writeText(){
  try
      {
         File.WriteAllText(@"C:\\serv1.txt", "text");
      }
      catch (Exception)
      {
         Thread.Sleep(300);
         writeText();
      }
}

И просто в коде вызвать writeText();

Comments

[martensit]

Уже сделал такое,
Но хотелось бы как нибудь поуверсальней вариант,
чтоб любая функция работала, например:
WriteAllText
ReadAllText
...

[freeExec]

martensit, Передавайте в функцию Action, будет вам универсально.
Но сам подход конечно ужасный, что толку тыкаться если допустим нет места на диске или прав на запись, а то и вовсе флешку уже давно вынули.

[Игорь]

martensit, а если сделать рекурсивную функцию, и как аргумент передавать ей нужную вам функцию ( WriteAllText, ReadAllText ), и вызывать её ?

[Griboks]

Игорь, рекурсия - не выход. Это как бы ещё больше трата ресурсов, хотя они и так ужасная. Тем более, существует ограничение на количество рекурсивных вызовов.

[Игорь]

Griboks, просто выше в комментах предлагали рекурсию, вот и меня туды понесло:)
Конечно лучше через цикл. Но основная моя идея была написать функцию, которой как аргумент передаётся функция, которую нужно повторять до успеха ( да, на словах это выглядит неочень... )