YYPHPでのコードレビュー会用資料

1. YYPHP #13
コードレビュー初めての
新米ちゃん♬

2. ・福岡生まれの福岡育ち
・IT企業で企画・ディレクター
→自分達で何か作りたくて妻と一緒に退社
→アパートの一室でハウススタジオ運営(今も)
→KaizenPlatformでグロースハッカーとして活動
→プログラミングを学んでサービス開発中(今)
・スキル：PHP/Laravel/Vue.js
・興味：DDD/TDD/OS/Go/Unity/ReactNative..etc
香月 宜浩(かつき よりひろ)
Twitter：ピーターパン@エンジニアン（@hukuzatsu）

3. [前提の共有]
・コードレビューの目的・背景
・とくに見てもらいたいとこ
・レビューしてもらうプロジェクトの概要
・業務の流れ
・業務ルール
[クラス構成]
・クラス図
[コードレビュー]
・GitHub
目次

4. 前提の共有

5. コードレビューの目的・背景
[背景]
・開発中サービスがファットコントローラーになっていた
・設計知識がなかったためちゃんと設計から学ぼう！
・行き着いたのがDDD
・DDDっぽく実践してみたけど、これでええのん？
・いや、良くない気がする...
・そうだ、コードレビューだ！
[目的]
・実際のクラス構成とコードを叩いてもらうことで、
よりDDDの考え方に合った構成やコードの書き方を学ぶ

6. とくに見てもらいたいとこ
①クラス構成
・一つのクラスの責務が結構大きくなっちゃっている感がある
クラスをうまく分割して１クラス１責務みたいな構成にしたい
(業務ルールが複雑なのでどのクラスにどの業務ルールを記述するか悩む)
②依存関係
・ドメインモデル内のクラス間で結構依存し合っているため、
一つのクラスに変更が入った時にちょっと変更が面倒になるかも？
・インスタンス化する時の引数が多くなってしまっている
※その他気づいたことどんどんご指摘下さい！

7. レビューしてもらうプロジェクトの概要
[ドメイン] ハウススタジオ運営
[コンテキスト] 予約受付コンテキスト？
[ニーズ]
・ハウススタジオのお客さんから予約申し込みが入った際に、予約内容が正し
いか + 日程が空いているか自動で判断した上で、予約内容に応じた返信メー
ルを、お客さんと管理者(自分)に自動で送りたい
・予約内容が正しく、日程が空いている場合はGoogleカレンダーに自動で予定 を入
れたい
・予約内容は管理者が一覧で見れるようにしたい
・予約のステータス(問い合わせ、予約確定、入金済..etc)も管理したい

8. ハウススタジオ予約受付業務の流れ
サイトから
予約申し込み
お客さん 管理者 Googleカレンダー メモ帳
予約申し込み
のメール受信
予約内容チェック
空き状況チェック
空き有
無
質問
有無
別日程提示
→終了or再検討
空き無し
Googleカレンダーで確認
空き有り
入金手続き
質問有り
入金確認&アクセス方法連絡
利用
合計金額・日程・名前・利用日
オプションをメモ
質問無し
今日は特に
この部分について
質問に回答
予約内容確認&入金手続き連絡
終了or再検討
ここを自動化したい

9. 業務ルール
[予約内容チェック]でチェックする項目
・GitHub参照
https://github.com/Yorinton/asobiba101

10. クラス構成

11. クラス図