Магазин книг

[alejes]

Магазин книг

[alejes]

@yurii-litvinov

[yurii-litvinov]

• Обычно, если класс связан ассоциацией с другим классом, то поле, соответствующее этой ассоциации, в классе уже не пишется: поле и ассоциация --- это два разных способа сказать одно и то же.
• Непонятно, как реализуется требование 4 ТЗ (работа с wish-листом).
• Вообще, как-то минималистично. То есть это больше похоже на схему БД, чем на модель предметной области. Например, работа с каталогами --- в предметной области есть сущность "Каталог", которая может быть реализована в БД тем, что у каждой книги есть id продавца. Но для модели предметной области не важно, как она может быть реализована, в ТЗ есть понятие "Каталог", его можно сделать сущностью, а то заказчик спросит "а где?" и Вы будете объяснять про inner join-ы.
• То же с поисковым движком и т.д., лучше их нарисовать как отдельные классы, чтобы как можно меньше подразумевалось.

[alejes]

Непонятно, как реализуется требование 4 ТЗ (работа с wish-листом).

Wish-лист является обычным листом. И через API ListProvider пользователь осуществляет с ним взаимодействие

[alejes]

Вообще, как-то минималистично. То есть это больше похоже на схему БД, чем на модель предметной области. Например, работа с каталогами --- в предметной области есть сущность "Каталог", которая может быть реализована в БД тем, что у каждой книги есть id продавца. Но для модели предметной области не важно, как она может быть реализована, в ТЗ есть понятие "Каталог", его можно сделать сущностью, а то заказчик спросит "а где?" и Вы будете объяснять про inner join-ы.

id предоставляется пользователю для удобства идентификации объектов. Потому что все запросы к API идут при помощи id и его просто нужно знать чтобы что-то сделать. Про inner joinы заказчику не нужно объяснять, сейчас каталог есть в виде сущности Section. Которая также говорит о своём идентификаторе. В противном случае мне надо будет объяснять заказчику как через API получить этот Каталог.

[alejes]

@yurii-litvinov Исправил проблемы, ушёл от похожести на базу данных.

[yurii-litvinov]

Теперь мне нравится, зачтена.