---
marp: true
paginate: true
backgroundColor: #000000
color: #FFFFFF
---

# ここ1年のコードレビュー

- Hamamarsu.rb #170 - yowasou

---

# コードレビューやってる？
## 対面？リモート？プルリク読む？
- 基本的には対面だったが添付して送ってもらうことが増えた
- 添付して送るんじゃなくてコミットしてくれ・・・
- 昔はビルドできないソースコミットする人がよく居たがなくなった。いいことだ
## 1vs1？1vsN？
- 昔は1vsNだったが時間的余裕がなくなって1vs1基本になった
- 1vsNは新人教育で割と有効。議事録皆読まない

---

# 誰とやってる？
- レビュアーは開発3年以上
## AI？もレビューできるか
- やってる人の意見を聞いてみたい
- 自社はルール上まだAIを使ってはいけないことになっている

---

# いつやってる？
## 単体テストの前
## 単体テストの後
## リリースの直前(死)
- このタイミングだと指摘しても直してもらえないことが多い

---

# なぜコードレビューするのか
## 品質
- たしかにちょっと良くなる気がする
- コードレビュー後に不具合が見つかるとちょっと怒られる
## 保守性を高める
## コードの書き方を統一
- コーディングルール作ったけど皆読まない
## 仕様不具合がよく見つかる…

---

# どうやってやってる？
## コーディングルール帳とかありますか
- いちおう自社はコーディングルールがある
- 英訳されてない
- 海外チームのコードは割と書き方が統一できている。1人でコードチェックしてるかも？
## 参考になる本とかありますか
- リーダブルコードは読んだ

---

# 何をコードレビューしている？
- 同時に単体テスト計画書も見てます
- コードではないもの(画像とか帳票レイアウトのバイナリファイル)は基本見てない…
- データ変更用のSQL数行でもレビューしている
- コードのちょっとした変更(1行でも)レビューしている。1行の変更でもバグ見つかることも
- 別のブランチから全部上書きするだけ（いわゆるマージ）といったものはレビューしていない
- 部分的な変更になる場合はマージでもレビューしている

---

# コードレビューしたほうがいい？
- 絶対した方がいい
- 欲を言えばペアプロ。今はAIがあるからいいか？
- バグがあるとレビュアーに文句が来ることがある

- 言われると悲しいことある？
- 強く言うとパワハラになるかも
- 指摘しても直してもらえないことがある