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

506 Views

November 11, 22

スライド概要

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

シェア

またはPlayer版

埋め込む »CMSなどでJSが使えない場合

関連スライド

各ページのテキスト
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 その1] ぷるりくがでかいと 全部読む気がしない!!

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 その2] 影響範囲の網羅が むずかしい!!

23.

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

24.

宗教について

25.

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

26.

宗教について

27.

宗教について

28.

宗教について

29.

宗教について

30.

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

31.

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

32.

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

33.

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