難しいよね、コードレビュー

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

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 まとめ

なにすんねん




[その１] でかいぷるりくを読むのが難儀どうすれば全コードを同じテンションで読めるか

[その２] 影響範囲調査が難儀どうすれば網羅できるか

[その３] 戦争が終わらないどうすれば世界から戦争がなくなるのか？

ご意見ちょうだいのコーナー

難しいよね、コードレビュー

Engineering