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.

難しいよね、コードレビュー

1,265 views

Published on

コードレビューって、コード見るだけじゃだめなのよねっていうお話。

Published in: Engineering
  • Be the first to comment

難しいよね、コードレビュー

  1. 1. 難しいよね、 コードレビュー 2015/05/20@関西PHPUG勉強会 @takapyyy
  2. 2. なにすんねん ● あんただれ? ● むずかしいはなし ● ご意見ちょうだいのコーナー
  3. 3. なにすんねん ● あんただれ? ● むずかしいはなし ● ご意見ちょうだいのコーナー
  4. 4. あんただれ? ● 高橋 慎一 / たかぴー / 26歳 / ♂ TwitterID: @takapyyy GitHub:@shinichi-takahashi ● 所属:株式会社ロックオン / EC-CUBE事業部 ● PHPカンファレンス関西2015実行委員長 ● 関西フロントエンドUG立ち上げの一人 ● すごくカノジョがほしい ● すごくカノジョがほしい
  5. 5. あんただれ? オープンソースプロダクトを つくったり、運営したりしてます。
  6. 6. なにすんねん ● あんただれ? ● むずかしいはなし ● ご意見ちょうだいのコーナー
  7. 7. むずかしいはなし コードレビューの観点 ● コーディングスタイルの確認 ● ビジネスロジックの妥当性 ● 宗教について
  8. 8. むずかしいはなし コードレビューの観点 ● コーディングスタイルの確認 ● ビジネスロジックの妥当性 ● 宗教について
  9. 9. コーディングスタイルの確認 ● PSR-2に準拠 ● EC-CUBE独自のコーディング規約を定義
  10. 10. コーディングスタイルの確認 \Scrurinizer超がんばれ/
  11. 11. むずかしいはなし コードレビューの観点 ● コーディングスタイルの確認 ● ビジネスロジックの妥当性 ● 宗教について
  12. 12. ビジネスロジックの妥当性のちょっと前 ぷるりく
  13. 13. ビジネスロジックの妥当性のちょっと前 ぷるりく
  14. 14. ビジネスロジックの妥当性のちょっと前 ぷるりく
  15. 15. ビジネスロジックの妥当性のちょっと前 ぷるりく
  16. 16. ビジネスロジックの妥当性のちょっと前 ぷるりく
  17. 17. ビジネスロジックの妥当性のちょっと前 [ I am 困ってるよ point その1] ぷるりくがでかいと 全部読む気がしない!!
  18. 18. ビジネスロジックの妥当性 確認方法 1. GitHub上でソースじろじろ 1. ぷるりくのブランチをローカルにポンッ 1. モンキーっぽいWEBテスト 1. 裏でCIが ‹‹¥(´ω` )/››‹‹¥( ´)/››‹‹¥( ´ω`)/›› マワルー!
  19. 19. ビジネスロジックの妥当性 <ポチッ
  20. 20. ビジネスロジックの妥当性
  21. 21. ビジネスロジックの妥当性 ___ o ( ;;;;;_;;;;; )/''' 丶.-'~;;;;;;;;;;;;;;~'-、 /~;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;,ヽ、 ,;';;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;ヽ、 ./;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;ヘ l;;;;;;;;;' ~''''''''''~ ';;;;;;;;l .l;;;;;;;;;; ;;;;;;;l l;;;;;;;;;;' .;;;;;;l k;;;;;;;' '''''' ~''''''.;. ;;;;;;.l ヘ ヽ;; (●), 、(●)、.::;; ノ ∧. ,,ノ(、_, )ヽ、,, l / キ `-=ニ=- ' /ヽ、 ./ ヽ 、 `ニニ´ / ヽ / ヽヽ、_ ___ / | `\\//
  22. 22. ビジネスロジックの妥当性のちょっと前 [ I am 困ってるよ point その2] 影響範囲の網羅が むずかしい!!
  23. 23. むずかしいはなし コードレビューの観点 ● コーディングスタイルの確認 ● ビジネスロジックの妥当性 ● 宗教について
  24. 24. 宗教について
  25. 25. 宗教について 戦争は終わりません
  26. 26. 宗教について
  27. 27. 宗教について
  28. 28. 宗教について
  29. 29. 宗教について
  30. 30. ビジネスロジックの妥当性 [ I am 困ってるよ point その3] どちらかが納得いくまで 戦争が終わらない!! しかも多くの場合、納得いかないので 決めることができない...(・v・`)
  31. 31. [その1] でかいぷるりくを読むのが難儀 どうすれば全コードを同じテンションで読めるか [その2] 影響範囲調査が難儀 どうすれば網羅できるか [その3] 戦争が終わらない どうすれば世界から戦争がなくなるのか? I am 困ってるよ point まとめ
  32. 32. なにすんねん ● あんただれ? ● むずかしいはなし ● ご意見ちょうだいのコーナー
  33. 33. [その1] でかいぷるりくを読むのが難儀 どうすれば全コードを同じテンションで読めるか [その2] 影響範囲調査が難儀 どうすれば網羅できるか [その3] 戦争が終わらない どうすれば世界から戦争がなくなるのか? ご意見ちょうだいのコーナー

×