# 2021-07-29 メンターセッション

## 質問

### 特大課題コードレビュー（新井）

#### 気になった点
- QueryServiceに1ページの件数を持つのは良くなさそう
- ユースケースから引数として渡す方が良い
- コントローラーで指定してユースケースに渡すこともあり
- QueryServiceには引数のバリデーションを入れないこともある
- 条件に一致しないパラメーターならそのまま空の配列を結果として返しても良い
- パラメータが正しいかどうかはQSではなくユースケースがした方が自然
- QueryServiceの戻り値を返す時に一度Entityを経由している
- そのままDTOにした方が良い。Entityを経由する必要はなさそう。
- 1ユースケース1ファイル派（好みの問題）
- ユースケースは複数集約をまたぐ場合があるので、集約で1つのファイルにしていると、このユースケースはどっちのファイル？？？となる場合もある

#### 良い点
- Prismaを使うときに必要なカラムだけ取ってきている
- パフォーマンスが悪化しない

###### Tags: `Team-2`