Задать вопрос

Ревью ТЗ, по которому отказ. Какие грубые ошибки совершил?

Добрый день. Откликнулся на вакансию, выполнив тестовое задание. (получил отказ)

Старался отделить домен от инфраструктуры\фреймворка, чтобы уменьшить сцепление и в случае расширения было легко поддерживать. Взамен пришлось писать кастомные решения, потеряв в легкочитаемости \ понимании кода, в силу того, что решения не стандартны

Стоит ли убрать разделение между доменом и инфраструктурой? Можете пожалуйста подсказать, какие грубые ошибки совершил и что можно исправить, чтобы улучшить качество кода?

Репозиторий (клик)
Тестовое задание (клик)
  • Вопрос задан
  • 2874 просмотра
Подписаться 13 Простой 11 комментариев
Решение пользователя xfg К ответам на вопрос (2)
@xfg
subsetofmultiset Бегло посмотрел код. Видно сразу что имеется определенное представление о слоистой/чистой/луковой архитектуре, возможно даже читал что-то из книг Эванса/Вернона по DDD и PoEAA от Мартина Фаулера, ну или как минимум какие-то статьи базирующиеся на информации из этих книг.

Довольно толково сделано. Единственное это то, что доменные сущности без поведения получились, состояние - есть, поведения - нет. Например следовало пету добавить метод любить человека, туда поместить бизнес-правила и если они все соблюдаются то создавать и возвращать из этого метода объект LovesHumanRelation и тому подобное. А так получилась что называется анемичная модель.

Но это всё можно быстро рассказать такому кандидату, дать почитать соответствующие главы из книг и будет реально толковый специалист. С такими знаниями не то что джуны, даже те кто сеньорами себя мнят ничерта подобного не знают. Одни ифоделы приходят (if-else-if-else).

Я так понял, для техлида то что ты написал получилось слишком сложно. Он походу хотел чтобы ты написал всё используя подход transaction script и готово, как это делают кнопкодавы.

Для меня, человек который вас не нанял - кретин. Показать бы ему что к нам приходит на собеседования за 100к.
Ответ написан