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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

宗教について

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

宗教について

宗教について

宗教について

宗教について

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

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

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

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