Как возвращать ошибки из сервисов?

Question

[Дмитрий]

Здравствуйте! Подскажите, кто как возвращает ошибки из Business layer ? К примеру

public User Method(Guid id)
{
 if(id == Guid.Empty) throw new ArgumentException(" some message ");
 return findedUserById;
}

стоит ли так делать, стоит ли использовать exception грубо говоря для валидации business правил? Кто какими методами пользуется, поделить

Answer 1

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

Ещё как вариант - это особый класс возврата, а-ля ASP.NET MVC ActionResut.

Т.е. все методы возвращают некоторый класс Result:

class Result<T> {
   public T Result {get; private set; }}
   public bool Success {get; private set; }}
   public IEnumerable<Exception> Errors { get ... }
   public string ErrorMessage {get ... } // этого можно и не делать

   private Result() {}
   public static Result<T> Success(T result) { return new Result() { Result = result, Success = true }; }
   public static Result<T> Fail(Exception ex) { return new Result() { Success = false, Errors = new List<Exception>() { ex }, ErrorMessage = ex.Message }; }
}

Comments

[Валерий Абакумов]

Александр Евсеев - соглашусь с Вами, недавно тоже стали использовать похожий механизм. Удобен тем, что этот класс (или набор расширяющих классов) очень легко можно покрыть юнит-тестами (и тем самым защитить себя от регресса), и вытащить в отдельный nuget-пакет для повторного использования. Когда изучал исходники ASP.NET Identity (версии 2 для .NET Full, версию 3 для .NET Core еще не смотрел), то обнаружил, что они тоже в очень многих местах используют аналогичный класс - IdentityResult. Плюс этого подхода также в том, что не замедляется выполнение кода, в отличие от выброса исключений (т.к. наличие множества выбросов исключений значительно замедляет выполнение кода - у нас коллега на тестовом наборе данных в цикле с несколькими миллионами итераций пришел к выводу, что код, который в своей логике выбрасывал исключения, выполнялся в несколько раз (или даже десятков раз - уже плохо помню) медленнее, чем тот же самый код, но без генерации исключений).

Answer 2

[Алексей Павлов]

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

Answer 3

[stcmd04236]

Я создаю свое исключение. Если wcf сервис то создаю исключения наследованные от FaultException.