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

1. 難しいよね、
コードレビュー
2015/05/20@関西PHPUG勉強会
@takapyyy

2. なにすんねん
● あんただれ？
● むずかしいはなし
● ご意見ちょうだいのコーナー

3. なにすんねん
● あんただれ？
● むずかしいはなし
● ご意見ちょうだいのコーナー

4. あんただれ？
● 高橋 慎一 / たかぴー / 26歳 / ♂
TwitterID: @takapyyy
GitHub:@shinichi-takahashi
● 所属：株式会社ロックオン / EC-CUBE事業部
● PHPカンファレンス関西2015実行委員長
● 関西フロントエンドUG立ち上げの一人
● すごくカノジョがほしい
● すごくカノジョがほしい

5. あんただれ？
オープンソースプロダクトを
つくったり、運営したりしてます。

6. なにすんねん
● あんただれ？
● むずかしいはなし
● ご意見ちょうだいのコーナー

7. むずかしいはなし
コードレビューの観点
● コーディングスタイルの確認
● ビジネスロジックの妥当性
● 宗教について

8. むずかしいはなし
コードレビューの観点
● コーディングスタイルの確認
● ビジネスロジックの妥当性
● 宗教について

9. コーディングスタイルの確認
● PSR-2に準拠
● EC-CUBE独自のコーディング規約を定義

10. コーディングスタイルの確認
＼Scrurinizer超がんばれ／

11. むずかしいはなし
コードレビューの観点
● コーディングスタイルの確認
● ビジネスロジックの妥当性
● 宗教について

12. ビジネスロジックの妥当性のちょっと前
ぷるりく

13. ビジネスロジックの妥当性のちょっと前
ぷるりく

14. ビジネスロジックの妥当性のちょっと前
ぷるりく

15. ビジネスロジックの妥当性のちょっと前
ぷるりく

16. ビジネスロジックの妥当性のちょっと前
ぷるりく

17. ビジネスロジックの妥当性のちょっと前
[ I am 困ってるよ point その１]
ぷるりくがでかいと
全部読む気がしない!!

18. ビジネスロジックの妥当性
確認方法
1. GitHub上でソースじろじろ
1. ぷるりくのブランチをローカルにポンッ
1. モンキーっぽいWEBテスト
1. 裏でCIが ‹‹¥(´ω` )/››‹‹¥( ´)/››‹‹¥( ´ω`)/›› ﾏﾜﾙｰ!

19. ビジネスロジックの妥当性
＜ﾎﾟﾁｯ

20. ビジネスロジックの妥当性

21. ビジネスロジックの妥当性
＿＿＿ o
( ;;;;;＿;;;;; )／'''
丶.-'~;;;;;;;;;;;;;;~'-､
／~;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;,ヽ、
,;';;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;ヽ、
./;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;ﾍ
l;;;;;;;;;' ~''''''''''~ ';;;;;;;;l
.l;;;;;;;;;; ;;;;;;;l
l;;;;;;;;;;' .;;;;;;l
k;;;;;;;' '''''' ~''''''.;. ;;;;;;.l
ﾍ ヽ;; （●）, ､（●）､.::;; ﾉ
∧. ,,ﾉ(､_, )ヽ､,, l
／ ｷ ｀-=ﾆ=- ' /ヽ、
./ ヽ 、 ｀ﾆﾆ´ ／ ヽ
/ ヽヽ､_ ___ ／ |
｀＼＼／／

22. ビジネスロジックの妥当性のちょっと前
[ I am 困ってるよ point その２]
影響範囲の網羅が
むずかしい!!

23. むずかしいはなし
コードレビューの観点
● コーディングスタイルの確認
● ビジネスロジックの妥当性
● 宗教について

24. 宗教について

25. 宗教について
戦争は終わりません

26. 宗教について

27. 宗教について

28. 宗教について

29. 宗教について

30. ビジネスロジックの妥当性
[ I am 困ってるよ point その３]
どちらかが納得いくまで
戦争が終わらない!!
しかも多くの場合、納得いかないので
決めることができない...(･v･`)

31. [その１] でかいぷるりくを読むのが難儀
どうすれば全コードを同じテンションで読めるか
[その２] 影響範囲調査が難儀
どうすれば網羅できるか
[その３] 戦争が終わらない
どうすれば世界から戦争がなくなるのか？
I am 困ってるよ point まとめ

32. なにすんねん
● あんただれ？
● むずかしいはなし
● ご意見ちょうだいのコーナー

33. [その１] でかいぷるりくを読むのが難儀
どうすれば全コードを同じテンションで読めるか
[その２] 影響範囲調査が難儀
どうすれば網羅できるか
[その３] 戦争が終わらない
どうすれば世界から戦争がなくなるのか？
ご意見ちょうだいのコーナー