Рефакторинг[В стадии разработки]:

1. Цель: удалить кучу пустых репозиториев. Причина: отсутсвие в их расширении.
2. MailNotificationService

Цель: удалить catch. Причина: одинаковая реализация во всех обработчиках. Цель: использовать BillingType.

3. EmailManagerService

Цель: Добавить к модели валидацию через атрибуты

4. IBaseRepository добавить внедрение :

Цель: Bind(typeof(IBaseRepository<>)).To(typeof(BaseRepository<>));

5. Цель: Заменить реализацию UnitOfWork на Storage. Причина: Unit of Work становится очень большим (fat).
6. Цель: Добавить глобальный фильтр для исключений.
7. FilmDetailedInfoService

Цель: удалить комментарии.

8. PermissionService and RoleService

Цель: добавить интерфейс и сделать binding. Цель: удалить комментарии.

9. Цель: добавить атрибуты авторизации. Причина: ограничить доступ к API.

10. LoggService:

Цель: убрать захордкоженный путь. var path = HostingEnvironment.MapPath("~/App_Data/Logs/2018-06-08.log");

1. Множество моделей дублируется
2. Существует пустые интерфейсы и пустые классы (репозитории, которые не расширены дополнительными методами).

Заметки: пустой интерфейс может быть использован в качестве маркера. (Например в DDD: IAggregateRoot - использовался Microsoft разработчиками в качестве маркера).

3. Существуе классы, которые нарушает S. (в слое сервисов)
4. UnitOfWork очень много репозиториев.