При операциях с сетью и IO часто вылазят ограничения, которые легко решить даже пользователю (пусть и опытному, да), но сложно узнать о них заранее. Занятый файл, нестабильное соединение - мне часто нехватает кнопки повторить, особенно когда весь процесс берет и откатывается.
Накидал пока на коленке реализацию более-менее универсального решения этой проблемы:
public enum ExceptionHandle
{
Abort,
Retry,
Ignore
}
public class ExceptionEventArgs
{
public Exception Exception { get; }
public ExceptionHandle? Handled { get; set; }
public ExceptionEventArgs(Exception ex)
{
this.Exception = ex;
}
}
public static class ExceptionHandler
{
public static event EventHandler<ExceptionEventArgs> Handler;
public static void TryExecute(Action action)
{
TryExecute(() => { action(); return true; }, false);
}
public static T TryExecute<T>(Func<T> action, T whenIgnored)
{
ExceptionHandle? handled = ExceptionHandle.Retry;
while (handled == ExceptionHandle.Retry)
{
try
{
return action();
}
catch (Exception ex)
{
handled = OnHandler(new ExceptionEventArgs(ex));
if (handled.HasValue)
{
switch (handled.Value)
{
case ExceptionHandle.Abort:
throw;
break;
case ExceptionHandle.Retry:
break;
case ExceptionHandle.Ignore:
break;
default:
throw new ArgumentOutOfRangeException();
}
}
else
{
throw;
}
}
}
return whenIgnored;
}
private static ExceptionHandle? OnHandler(ExceptionEventArgs e)
{
if (Handler == null || !Handler.GetInvocationList().Any())
{
ExceptionDispatchInfo.Capture(e.Exception).Throw();
}
else
{
Handler.Invoke(null, e);
}
return e.Handled;
}
}
Таким образом, любой подписчик ExceptionHandler.Handler
может либо резолвить проблемы в автоматическом режиме, либо вываливать решение на пользователя.
Любой опасный код теперь можно обернуть:
var tested = ExceptionHandler.TryExecute(() =>
{
using (var destination = new MemoryStream())
{
using (Stream stream = entry.Open())
stream.CopyTo(destination);
return destination.Length == entry.Length;
}
}, false);
В целом, текущая реализация мне кажется уже терпимой и она работает. Но, подозреваю, что такие решения уже где то есть, просто я не смог их найти. Может кто-то посоветует где взять или хотя бы посмотреть готовые решения? Ну и, если в моем коде есть косяки - тоже не отказался бы от помощи.
UPD: да, я понимаю, что даже так остаются проблемные ситуации - экшн может быть одноразовым (закрывать соединение, гробить sql сессию, да что угодно делать). Это уже остается на совести того, кто использует код. Хотя, интересные варианты по этой проблеме я бы тоже глянул, это же фиг ограничишь.
UPD2: пока не смог придумать, можно ли оборачивать один такой блок в другой, а то сейчас в итоге на аборте внутреннего блока внешний снова уходит на обработку.