---
# System prepended metadata

title: 2021-07-29 メンターセッション
tags: [Team-2]

---

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

## 質問

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

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

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

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