test_task,

Да уже можно было написать за это время. Гугл чтоли отключили? Может сам попробуешь, и задашь конкретные вопросы, если что-то не получается?

Откуда вообще такая уверенность или надежда, что каким-то людям в этой жизни больше заняться нечем, кроме как делать чьи-то чужие унылые задания?

Агнец за бортомhVostt кроме как делать чьи-то чужие унылые задания?

А с чего ты решил, что он просит _сделать_ задание? Он просит покритиковать.

Да та же фигня. Критикуешь? Значит объясняй как надо, как было бы лучше, как правильно. А там вообще всё настолько уныло, что не то что на ведущего не тянет, с натяжкой на младшего джуниора самый максимум. Это неплохо, чего-то не знать в начале карьеры. Но по этой теме уже столько информации, что стыдно тупо копировать заезженный шаблон, совершенно не понимая что это и для чего.

Вот на что бы я с удовольствием посмотрел, это на пример SOLID-ного решения с разделением модели доступа к данным на читателей/писателей, с агрегатами доменной модели, с сервисами бизнес-логики, шиной событий и адекватной обвязкой с эффективным выхлопом от применения ООП и статической типизации. Вот такое ещё бы потянуло на ведущего.

А то унылые репозитории, DTO, какие-то дата сервисы, которые к тому же весьма далеки от того, чем они должны по идее являться. Как тут критиковать? Скажешь, это у тебя не репозиторий. А что тогда репозиторий? Давай объясняй, разъясняй, учи... Другое дело конкретные вопросы, целенаправленные. Какие-то попытки разобраться, а не тупо скопировать. Вот это уже интересно.

test_taskок.
а можно, если, всеже, делать нечего, по-конкретней. что, например, не так с репозиторием (skyANA обозначил нарушения, я их устранил)? зачем тут "агрегаты доменной модели" (тут разве они могут быть?)? и шина событий?
и зачем тут "читатели/писатели". это конечно, уже выходит за рамки тестового, в том числе, этого задания. но было бы интересно услышать Вашу критику.

Затем, чтобы продемонстрировать ваши способности в проектировании архитектуры информационной системы с прицелом на развитие и поддержку. Ну это к вопросу, что лично мне было бы интересно увидеть.


test_taskа что имено? Вы, как я понимаю, говорите это с позиции своего опыта. И, наверняка, там есть видение "исправить 1, 2, 3...". можно это "1, 2, 3"? крупными мазками

Как будет реализована фильтрация данных с клиента? По методу на каждый чих в DataService?

Где реализована безопасность доступа к данным? Там же в сервисах на уровне обычных пользовательских фильтров?

Почему Domain-ом обозваны обычные транспортные классы для ORM? В каком же месте это Domain?

Очень странная обработка ошибок.




Почему бы тогда не определить свой тип исключения? Надо всё время помнить вот этот неочевидный, не расширяемый контракт?

Зачем дублировать интерфейсы репозиториев? Почему репозитории возвращают как DTO, так и «domain» классы? Почитайте что такое репозиторий.




ужасно тупо




Возвращать надо или IEnumerable, или IReadOnlyCollection/List, а не изменяемый список.

Create возвращает какой-то decimal. Рыщем в имплементации, выясняем, что это какой-то там баланс. Очуметь.

В то же время GetNew возвращает какой-то объект транзакции... или погодите-ка?




это если нам повезёт, однако. Чё к чему вообще?

Косяков навалом. Вы сами-то их видите? Если нет, то у меня очень печальные новости для вас...

Агнец за бортомВ тестовом задании?

Если у человека есть желание и амбиции занять роль ведущего разработчика в серьёзной компании, то да.

А судя по интервью, которые я провожу, абсолютное большинство кандидатов проходят собеседование на отъе...сь. Больше половины не готовятся вообще. Даже если на 100% очевидно, что будут спрашивать, написано чёрным по белому в вакансии. Даже про это не почитают. Хотя бы поверхностно.

Поэтому можно сказать, накипело.