Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.
YYPHP #13
    コードレビュー初めての
 新米ちゃん♬
・福岡生まれの福岡育ち
・IT企業で企画・ディレクター
 →自分達で何か作りたくて妻と一緒に退社
 →アパートの一室でハウススタジオ運営(今も)
 →KaizenPlatformでグロースハッカーとして活動
 →プログラミングを学んでサービス開...
[前提の共有]
 ・コードレビューの目的・背景
 ・とくに見てもらいたいとこ
 ・レビューしてもらうプロジェクトの概要
 ・業務の流れ
 ・業務ルール
[クラス構成]
 ・クラス図
[コードレビュー]
 ・GitHub
目次
前提の共有
コードレビューの目的・背景
[背景]
 ・開発中サービスがファットコントローラーになっていた
 ・設計知識がなかったためちゃんと設計から学ぼう!
 ・行き着いたのがDDD
 ・DDDっぽく実践してみたけど、これでええのん?
 ・いや、良くない気...
とくに見てもらいたいとこ
①クラス構成
 ・一つのクラスの責務が結構大きくなっちゃっている感がある
  クラスをうまく分割して1クラス1責務みたいな構成にしたい
  (業務ルールが複雑なのでどのクラスにどの業務ルールを記述するか悩む)
②依存関...
レビューしてもらうプロジェクトの概要
[ドメイン] ハウススタジオ運営
[コンテキスト] 予約受付コンテキスト?
[ニーズ]
 ・ハウススタジオのお客さんから予約申し込みが入った際に、予約内容が正し
  いか + 日程が空いているか自動で判断し...
ハウススタジオ予約受付業務の流れ
サイトから
予約申し込み
お客さん 管理者 Googleカレンダー メモ帳
予約申し込み
のメール受信
予約内容チェック
空き状況チェック
空き有
無
質問
有無
別日程提示
→終了or再検討
空き無し
Goo...
業務ルール
[予約内容チェック]でチェックする項目
・GitHub参照
https://github.com/Yorinton/asobiba101
クラス構成
クラス図
Upcoming SlideShare
Loading in …5
×

YYPHP #13 初めてのコードレビュー

429 views

Published on

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

Published in: Engineering
  • Be the first to comment

YYPHP #13 初めてのコードレビュー

  1. 1. YYPHP #13     コードレビュー初めての  新米ちゃん♬
  2. 2. ・福岡生まれの福岡育ち ・IT企業で企画・ディレクター  →自分達で何か作りたくて妻と一緒に退社  →アパートの一室でハウススタジオ運営(今も)  →KaizenPlatformでグロースハッカーとして活動  →プログラミングを学んでサービス開発中(今) ・スキル:PHP/Laravel/Vue.js ・興味:DDD/TDD/OS/Go/Unity/ReactNative..etc 香月 宜浩(かつき よりひろ) Twitter:ピーターパン@エンジニアン(@hukuzatsu)
  3. 3. [前提の共有]  ・コードレビューの目的・背景  ・とくに見てもらいたいとこ  ・レビューしてもらうプロジェクトの概要  ・業務の流れ  ・業務ルール [クラス構成]  ・クラス図 [コードレビュー]  ・GitHub 目次
  4. 4. 前提の共有
  5. 5. コードレビューの目的・背景 [背景]  ・開発中サービスがファットコントローラーになっていた  ・設計知識がなかったためちゃんと設計から学ぼう!  ・行き着いたのがDDD  ・DDDっぽく実践してみたけど、これでええのん?  ・いや、良くない気がする...  ・そうだ、コードレビューだ! [目的]  ・実際のクラス構成とコードを叩いてもらうことで、   よりDDDの考え方に合った構成やコードの書き方を学ぶ
  6. 6. とくに見てもらいたいとこ ①クラス構成  ・一つのクラスの責務が結構大きくなっちゃっている感がある   クラスをうまく分割して1クラス1責務みたいな構成にしたい   (業務ルールが複雑なのでどのクラスにどの業務ルールを記述するか悩む) ②依存関係  ・ドメインモデル内のクラス間で結構依存し合っているため、   一つのクラスに変更が入った時にちょっと変更が面倒になるかも?  ・インスタンス化する時の引数が多くなってしまっている ※その他気づいたことどんどんご指摘下さい!
  7. 7. レビューしてもらうプロジェクトの概要 [ドメイン] ハウススタジオ運営 [コンテキスト] 予約受付コンテキスト? [ニーズ]  ・ハウススタジオのお客さんから予約申し込みが入った際に、予約内容が正し   いか + 日程が空いているか自動で判断した上で、予約内容に応じた返信メー   ルを、お客さんと管理者(自分)に自動で送りたい  ・予約内容が正しく、日程が空いている場合はGoogleカレンダーに自動で予定  を入 れたい  ・予約内容は管理者が一覧で見れるようにしたい  ・予約のステータス(問い合わせ、予約確定、入金済..etc)も管理したい
  8. 8. ハウススタジオ予約受付業務の流れ サイトから 予約申し込み お客さん 管理者 Googleカレンダー メモ帳 予約申し込み のメール受信 予約内容チェック 空き状況チェック 空き有 無 質問 有無 別日程提示 →終了or再検討 空き無し Googleカレンダーで確認 空き有り 入金手続き 質問有り 入金確認&アクセス方法連絡 利用 合計金額・日程・名前・利用日 オプションをメモ 質問無し 今日は特に この部分について 質問に回答 予約内容確認&入金手続き連絡 終了or再検討 ここを自動化したい
  9. 9. 業務ルール [予約内容チェック]でチェックする項目 ・GitHub参照 https://github.com/Yorinton/asobiba101
  10. 10. クラス構成
  11. 11. クラス図

×