Как придерживаться разделения ответственности между репозиториями?

Question

[eugenedrvnk]

В приложении есть необходимость получить список юзеров с постами для каждого из юзеров:

[
  {id: 1, username: 'xyz', age: 22, posts: [{id: 1, title: 'first post'},
  {id: 2, username: 'aaa', age: 41, posts: [{id: 41, title: 'hello'},
] // и так далее...

У меня уже отдельно имплементированы репозитории где можно получить либо только юзеров, либо только посты.
Для юзкейса с получением юзеров включая в них посты, будет ли ок сначала получить юзеров и потом для них фетчить посты?

const users = await this.userRepository.findLatest();
const postsByUserIds = await this.postRepository.findByUserIds(users.map(user => user.id));
return users.map((user) => {
  return {
     ...user,
     posts: postsByUserIds(user.id)  
  }
})

Или же лучше создать в UserReposotory метод findWithPosts в котором просто дополнительно тянуть посты для каждого юзера?
Просто в этом случае тогда вроде как нарушается разделение ответственности. Но такой способ вроде выглядит проще.
А предыдущий - в плане разделения ответственности гуд, но по производительности из-за дополнительных циклов вроде бы хуже.

Comments

[historydev]

Пусть клиент тянет то, что ему нужно. Твоя работа сделать чистое api, будешь пытаться каждый случай покрывать end-поинтом получится примерно так-же, как с 100% покрытием тестами

Answer 1

[Антон Неверов]

1) Используйте ORM, там эти проблемы уже решены.
2) Самый правильный вариант - получать посты по требованию (так называемый lazy loading). Я это говорю, опираясь на то предположение, что пользователь может написать тысячи постов по тысяче символов. Если постов не много и они короткие - можно получать сразу всё через JOIN.
3) Получать список пользователей вместе со список их постов в API звучит как анти-паттерн какой-то. Точно ли такой роут необходим или есть возможность получить посты для выбранного пользователя отдельно?