Pull to refresh

Comments 6

Несколько вопросов:

1. Объясните смысл конструкции:
            catch (BusinessException be)
            {
                return JsonError(form, be, logger);
            }
            catch (FormHandlerException fhe)
            {
                return JsonError(form, fhe, logger);
            }
            catch (Exception e)
            {
                return JsonError(form, e, logger);
            }

Все три кэтча ведут в одно место, причем последний обобщающий, зачем?

2. Зачем оборачивать DbContext в IUnitOfWork, он ведь и так им является, и он не привязан к конкретной реализации БД, абстракция ради абстракции?

3. (это уже моя вкусовщина) Не думали про логирование в аспектом стиле, чтобы не загружать код логированием?

4. Зачем использовать шаблон MVC, если приложение WebAPI?

5. Почему не используете для клиентского кода grunt/gulp/webpack?
1. Проект не идеален, но все еще в процессе модернизации, но пока не могу вспомнить, почему именно так написал.
2. Да, действительно, абстракция ради абстракции =) Честно, пока еще разбираюсь в этом и сам иногда задавался вопросом «Почему?».
3. Спасибо за Вашу вкусовщину. Пойду по читаю про «логгирование в аспектном стиле». Звучит интересно.
4. Шаблон, правильнее будет, не я выбирал. Я просто чуточку модернизировал то, что автор этой архитектуры создал.
5. Про клиентскую часть вообще пока не задумывался, так как этим постом я хотел показать интересную реализацию архитектуры, которая меня удивила.

Спасибо большое за замечания. В свободное время буду дополнять, допиливать и может и получится «Идеальная архитекура» для проектов средней сложности
UFO just landed and posted this here
Это не только доменные ексепшны, а общие, Business Exceptions, которая пригодится еще в дальнейшем, к примеру, в других сервисах.
UFO just landed and posted this here
Sign up to leave a comment.

Articles