Что такое хорошая практика с DB Services C#

Question

Я делаю веб-проект в ASP.NET MVC. Каждая служба вызова контроллера выполняет некоторые операции. Давайте посмотрим на примере

public interface IProjectService 
    { 
     bool ActivateProject(int projectId); 
     //others 
    } 

и реализован класс

public class ProjectService : IProjectService 
    { 
     private readonly IContextProvider _contextProvider; 

     public ProjectService(IContextProvider contextProvider) 
     { 
      _contextProvider = contextProvider; 
     } 

     public bool ActivateProject(int projectId) 
     { 
      using (var context = _contextProvider.GetContext()) 
      { 
       var project = context.Projects.Where(p => p.ProjectId == projectId).SingleOrDefault(); 
       project.Status = 1; 
       context.SaveChanges(); 
       return true; 
      } 
     } 
    //others 
    } 

В примере администратор хочет, чтобы активировать проект. У меня есть проверка в jQuery, поэтому администратор не может передавать ничего, кроме projectId.

Итак, что теперь должно вернуть этот метод? Должен ли я оставить его как пустоту? Допустим, есть ошибка с базой данных, поэтому я должен исключить исключение? Какая практика для такого рода операций?

Answer 1

Как правило, вы не должны использовать исключения для обработки стандартного потока выполнения. Исключения используются для ситуаций, когда что-то, что не должно произойти, на самом деле произошло.

С учетом сказанного, существует граница, где исключения и возвращаемые значения подходят хорошо, и это всего лишь вопрос вкуса.

Исключения, как правило, обеспечивают большую гибкость, поскольку вы можете создавать различные типы исключений и обрабатывать их с помощью одной попытки/улова. Они также предоставляют гораздо больше информации. Логическое не говорит о том, что произошло.

Если вы возвращаете логическое значение, вы должны обрабатывать исключения внутри метода и, по крайней мере, регистрировать данные об ошибках где-то.

Я лично брошу исключение на случай, если что-то пойдет не так, и пусть вызывающий его обработает.