Submit Search
Upload
20141105 俺のコードレビュー(lightning talk) #devraku
•
14 likes
•
10,835 views
Takao Oyobe
Follow
2014/11/5に開催した「俺のコードレビュー」勉強会のライトニングトーク資料。パターンの文脈でコードレビューについて話してみました。
Read less
Read more
Technology
Report
Share
Report
Share
1 of 39
Download now
Download to read offline
Recommended
Test Driven Development - For Girl Geeks Night Sydney
Test Driven Development - For Girl Geeks Night Sydney
Jo Cranford
高品質軟體的基本動作 101 + 102 for NUU
高品質軟體的基本動作 101 + 102 for NUU
Su Jan
Tooling JavaScript to ensure consistency in coding style
Tooling JavaScript to ensure consistency in coding style
Dmitry Sheiko
振り返ればカンバンがある ~チームとカンバンとProduct Ownership~
振り返ればカンバンがある ~チームとカンバンとProduct Ownership~
Takao Oyobe
20141105 俺のコードレビュー(opening) #devraku
20141105 俺のコードレビュー(opening) #devraku
Takao Oyobe
On what’s attractive in Rakuten Technology Conference 2014, Japanese version
On what’s attractive in Rakuten Technology Conference 2014, Japanese version
Rakuten Group, Inc.
20141108 俺のエンジニアリング #devlove
20141108 俺のエンジニアリング #devlove
Takao Oyobe
日経BPリーン式創業塾 #leanstartup #リーンスタートアップ
日経BPリーン式創業塾 #leanstartup #リーンスタートアップ
Itsuki Kuroda
Recommended
Test Driven Development - For Girl Geeks Night Sydney
Test Driven Development - For Girl Geeks Night Sydney
Jo Cranford
高品質軟體的基本動作 101 + 102 for NUU
高品質軟體的基本動作 101 + 102 for NUU
Su Jan
Tooling JavaScript to ensure consistency in coding style
Tooling JavaScript to ensure consistency in coding style
Dmitry Sheiko
振り返ればカンバンがある ~チームとカンバンとProduct Ownership~
振り返ればカンバンがある ~チームとカンバンとProduct Ownership~
Takao Oyobe
20141105 俺のコードレビュー(opening) #devraku
20141105 俺のコードレビュー(opening) #devraku
Takao Oyobe
On what’s attractive in Rakuten Technology Conference 2014, Japanese version
On what’s attractive in Rakuten Technology Conference 2014, Japanese version
Rakuten Group, Inc.
20141108 俺のエンジニアリング #devlove
20141108 俺のエンジニアリング #devlove
Takao Oyobe
日経BPリーン式創業塾 #leanstartup #リーンスタートアップ
日経BPリーン式創業塾 #leanstartup #リーンスタートアップ
Itsuki Kuroda
Move into speaker deck
Move into speaker deck
Takao Oyobe
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
Takao Oyobe
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
Takao Oyobe
俺のインセプションデッキ【Remaster版】
俺のインセプションデッキ【Remaster版】
Takao Oyobe
5分でわかった気になるインセプションデッキ
5分でわかった気になるインセプションデッキ
Takao Oyobe
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
Takao Oyobe
20141213 俺のインセプションデッキ #agilesamurai
20141213 俺のインセプションデッキ #agilesamurai
Takao Oyobe
20141028 lightning talk at staff party of rtc48
20141028 lightning talk at staff party of rtc48
Takao Oyobe
(Beta)clean release manual
(Beta)clean release manual
Takao Oyobe
つくる現場 - 変化を支える3つの現場アーキテクト -
つくる現場 - 変化を支える3つの現場アーキテクト -
Takao Oyobe
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
Takao Oyobe
はじめてのふりかえり
はじめてのふりかえり
Takao Oyobe
えんじにあというおしごと
えんじにあというおしごと
Takao Oyobe
secret base~jeff pattonがくれたもの~
secret base~jeff pattonがくれたもの~
Takao Oyobe
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
Takao Oyobe
Fearless Change - Agile2012 feedback report -
Fearless Change - Agile2012 feedback report -
Takao Oyobe
アジャイルペーペーシップとチーム改革
アジャイルペーペーシップとチーム改革
Takao Oyobe
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
Takao Oyobe
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
Takao Oyobe
Apidays New York 2024 - Scaling API-first by Ian Reasor and Radu Cotescu, Adobe
Apidays New York 2024 - Scaling API-first by Ian Reasor and Radu Cotescu, Adobe
apidays
A Beginners Guide to Building a RAG App Using Open Source Milvus
A Beginners Guide to Building a RAG App Using Open Source Milvus
Zilliz
Apidays Singapore 2024 - Modernizing Securities Finance by Madhu Subbu
Apidays Singapore 2024 - Modernizing Securities Finance by Madhu Subbu
apidays
More Related Content
More from Takao Oyobe
Move into speaker deck
Move into speaker deck
Takao Oyobe
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
Takao Oyobe
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
Takao Oyobe
俺のインセプションデッキ【Remaster版】
俺のインセプションデッキ【Remaster版】
Takao Oyobe
5分でわかった気になるインセプションデッキ
5分でわかった気になるインセプションデッキ
Takao Oyobe
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
Takao Oyobe
20141213 俺のインセプションデッキ #agilesamurai
20141213 俺のインセプションデッキ #agilesamurai
Takao Oyobe
20141028 lightning talk at staff party of rtc48
20141028 lightning talk at staff party of rtc48
Takao Oyobe
(Beta)clean release manual
(Beta)clean release manual
Takao Oyobe
つくる現場 - 変化を支える3つの現場アーキテクト -
つくる現場 - 変化を支える3つの現場アーキテクト -
Takao Oyobe
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
Takao Oyobe
はじめてのふりかえり
はじめてのふりかえり
Takao Oyobe
えんじにあというおしごと
えんじにあというおしごと
Takao Oyobe
secret base~jeff pattonがくれたもの~
secret base~jeff pattonがくれたもの~
Takao Oyobe
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
Takao Oyobe
Fearless Change - Agile2012 feedback report -
Fearless Change - Agile2012 feedback report -
Takao Oyobe
アジャイルペーペーシップとチーム改革
アジャイルペーペーシップとチーム改革
Takao Oyobe
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
Takao Oyobe
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
Takao Oyobe
More from Takao Oyobe
(19)
Move into speaker deck
Move into speaker deck
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
俺のインセプションデッキ【Remaster版】
俺のインセプションデッキ【Remaster版】
5分でわかった気になるインセプションデッキ
5分でわかった気になるインセプションデッキ
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
20141213 俺のインセプションデッキ #agilesamurai
20141213 俺のインセプションデッキ #agilesamurai
20141028 lightning talk at staff party of rtc48
20141028 lightning talk at staff party of rtc48
(Beta)clean release manual
(Beta)clean release manual
つくる現場 - 変化を支える3つの現場アーキテクト -
つくる現場 - 変化を支える3つの現場アーキテクト -
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
はじめてのふりかえり
はじめてのふりかえり
えんじにあというおしごと
えんじにあというおしごと
secret base~jeff pattonがくれたもの~
secret base~jeff pattonがくれたもの~
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
Fearless Change - Agile2012 feedback report -
Fearless Change - Agile2012 feedback report -
アジャイルペーペーシップとチーム改革
アジャイルペーペーシップとチーム改革
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
Recently uploaded
Apidays New York 2024 - Scaling API-first by Ian Reasor and Radu Cotescu, Adobe
Apidays New York 2024 - Scaling API-first by Ian Reasor and Radu Cotescu, Adobe
apidays
A Beginners Guide to Building a RAG App Using Open Source Milvus
A Beginners Guide to Building a RAG App Using Open Source Milvus
Zilliz
Apidays Singapore 2024 - Modernizing Securities Finance by Madhu Subbu
Apidays Singapore 2024 - Modernizing Securities Finance by Madhu Subbu
apidays
presentation ICT roal in 21st century education
presentation ICT roal in 21st century education
jfdjdjcjdnsjd
+971581248768>> SAFE AND ORIGINAL ABORTION PILLS FOR SALE IN DUBAI AND ABUDHA...
+971581248768>> SAFE AND ORIGINAL ABORTION PILLS FOR SALE IN DUBAI AND ABUDHA...
?#DUbAI#??##{{(☎️+971_581248768%)**%*]'#abortion pills for sale in dubai@
Exploring the Future Potential of AI-Enabled Smartphone Processors
Exploring the Future Potential of AI-Enabled Smartphone Processors
debabhi2
Artificial Intelligence Chap.5 : Uncertainty
Artificial Intelligence Chap.5 : Uncertainty
Khushali Kathiriya
TrustArc Webinar - Stay Ahead of US State Data Privacy Law Developments
TrustArc Webinar - Stay Ahead of US State Data Privacy Law Developments
TrustArc
"I see eyes in my soup": How Delivery Hero implemented the safety system for ...
"I see eyes in my soup": How Delivery Hero implemented the safety system for ...
Zilliz
FWD Group - Insurer Innovation Award 2024
FWD Group - Insurer Innovation Award 2024
The Digital Insurer
Mastering MySQL Database Architecture: Deep Dive into MySQL Shell and MySQL R...
Mastering MySQL Database Architecture: Deep Dive into MySQL Shell and MySQL R...
Miguel Araújo
Corporate and higher education May webinar.pptx
Corporate and higher education May webinar.pptx
Rustici Software
Navi Mumbai Call Girls 🥰 8617370543 Service Offer VIP Hot Model
Navi Mumbai Call Girls 🥰 8617370543 Service Offer VIP Hot Model
Deepika Singh
Apidays Singapore 2024 - Scalable LLM APIs for AI and Generative AI Applicati...
Apidays Singapore 2024 - Scalable LLM APIs for AI and Generative AI Applicati...
apidays
Why Teams call analytics are critical to your entire business
Why Teams call analytics are critical to your entire business
panagenda
EMPOWERMENT TECHNOLOGY GRADE 11 QUARTER 2 REVIEWER
EMPOWERMENT TECHNOLOGY GRADE 11 QUARTER 2 REVIEWER
MadyBayot
DBX First Quarter 2024 Investor Presentation
DBX First Quarter 2024 Investor Presentation
Dropbox
Strategies for Unlocking Knowledge Management in Microsoft 365 in the Copilot...
Strategies for Unlocking Knowledge Management in Microsoft 365 in the Copilot...
Drew Madelung
Apidays Singapore 2024 - Building Digital Trust in a Digital Economy by Veron...
Apidays Singapore 2024 - Building Digital Trust in a Digital Economy by Veron...
apidays
Apidays New York 2024 - Accelerating FinTech Innovation by Vasa Krishnan, Fin...
Apidays New York 2024 - Accelerating FinTech Innovation by Vasa Krishnan, Fin...
apidays
Recently uploaded
(20)
Apidays New York 2024 - Scaling API-first by Ian Reasor and Radu Cotescu, Adobe
Apidays New York 2024 - Scaling API-first by Ian Reasor and Radu Cotescu, Adobe
A Beginners Guide to Building a RAG App Using Open Source Milvus
A Beginners Guide to Building a RAG App Using Open Source Milvus
Apidays Singapore 2024 - Modernizing Securities Finance by Madhu Subbu
Apidays Singapore 2024 - Modernizing Securities Finance by Madhu Subbu
presentation ICT roal in 21st century education
presentation ICT roal in 21st century education
+971581248768>> SAFE AND ORIGINAL ABORTION PILLS FOR SALE IN DUBAI AND ABUDHA...
+971581248768>> SAFE AND ORIGINAL ABORTION PILLS FOR SALE IN DUBAI AND ABUDHA...
Exploring the Future Potential of AI-Enabled Smartphone Processors
Exploring the Future Potential of AI-Enabled Smartphone Processors
Artificial Intelligence Chap.5 : Uncertainty
Artificial Intelligence Chap.5 : Uncertainty
TrustArc Webinar - Stay Ahead of US State Data Privacy Law Developments
TrustArc Webinar - Stay Ahead of US State Data Privacy Law Developments
"I see eyes in my soup": How Delivery Hero implemented the safety system for ...
"I see eyes in my soup": How Delivery Hero implemented the safety system for ...
FWD Group - Insurer Innovation Award 2024
FWD Group - Insurer Innovation Award 2024
Mastering MySQL Database Architecture: Deep Dive into MySQL Shell and MySQL R...
Mastering MySQL Database Architecture: Deep Dive into MySQL Shell and MySQL R...
Corporate and higher education May webinar.pptx
Corporate and higher education May webinar.pptx
Navi Mumbai Call Girls 🥰 8617370543 Service Offer VIP Hot Model
Navi Mumbai Call Girls 🥰 8617370543 Service Offer VIP Hot Model
Apidays Singapore 2024 - Scalable LLM APIs for AI and Generative AI Applicati...
Apidays Singapore 2024 - Scalable LLM APIs for AI and Generative AI Applicati...
Why Teams call analytics are critical to your entire business
Why Teams call analytics are critical to your entire business
EMPOWERMENT TECHNOLOGY GRADE 11 QUARTER 2 REVIEWER
EMPOWERMENT TECHNOLOGY GRADE 11 QUARTER 2 REVIEWER
DBX First Quarter 2024 Investor Presentation
DBX First Quarter 2024 Investor Presentation
Strategies for Unlocking Knowledge Management in Microsoft 365 in the Copilot...
Strategies for Unlocking Knowledge Management in Microsoft 365 in the Copilot...
Apidays Singapore 2024 - Building Digital Trust in a Digital Economy by Veron...
Apidays Singapore 2024 - Building Digital Trust in a Digital Economy by Veron...
Apidays New York 2024 - Accelerating FinTech Innovation by Vasa Krishnan, Fin...
Apidays New York 2024 - Accelerating FinTech Innovation by Vasa Krishnan, Fin...
20141105 俺のコードレビュー(lightning talk) #devraku
1.
C O D
E R E V I E W 俺のコードレビュー @TAKAKING22 TAKAO OYOBE 及部敬雄 2014.11.5 俺のコードレビュー勉強会
2.
コードレビューしてますか?
3.
コードレビューの目的は?
4.
品質の担保 認識合わせ 教育
5.
Really?
6.
品質の担保 認識合わせ 教育
- CI、静的解析、テスト … - コード規約、ペアプロ … - ペアプロ、家でやれy …
7.
品質の担保 認識合わせ 教育
- CI、静的解析、テスト … コードレビューじゃなくても できる方法はある - コード規約、ペアプロ … - ペアプロ、家でやれy …
8.
まさか… 品質の担保 認識合わせ
教育 偉い人が見るため 偉い人が知るため 偉い人が新人を…(ry
9.
あるいは… 品質の担保 認識合わせ
教育 した気になってる した気になってる した気になってる
10.
コードレビューの儀式化
11.
コードレビューは 問題解決の一手段 でしかない
12.
隣のコードレビューを そのまま真似しても 効果はあまり出ないかも
13.
なぜなら… よそはよそ、うちはうち 成功した事例には、
コンテキストや制約が存在し、 それがあなたのチームに 当てはまるとは限らないから
14.
うわ… うちのコードレビュー 効果なさすぎ…?
こうならないために どうすればいよいのか? http://www.pakutaso.com/20120806242post-1877.html
15.
パターンランゲージ http://books.rakuten.co.jp/rb/12432779/
16.
パターン=再現可能な解決策 http://www.slideshare.net/kawaguti/20111207-u
17.
パターンは組み合わせ可能 http://www.slideshare.net/kawaguti/20111207-u
18.
組み合わせて意味を持つ http://www.slideshare.net/kdmsnr/fearlessjourneypostudy
19.
まとめると、 •事例だけでは役に立たない •事例からパターンに落とす
•パターンを組み合わせて適用する •組み合わせることで効果が出る http://www.slideshare.net/kdmsnr/fearlessjourneypostudy
20.
つまり、 •自分のチームの問題・課題を見える化 •事例からパターンに変換する
•パターンを取捨選択する •パターン組み合わせて実施する 点ではなく、 線・面で問題解決する http://www.slideshare.net/kdmsnr/fearlessjourneypostudy
21.
実はぶっちゃけ… パターンってうさんくさくて あんまり好きじゃなかった
22.
実はぶっちゃけ… でも教科書通りじゃ うまくいかねーんだよ
いろいろ考えてやろうぜ ってことを代弁してくれてて ちょっと好きになった
23.
持って帰って欲しいもの コードレビューパターン
24.
Pull Request コンテキスト問題
レビューのログを残したい 人によって残し方が バラバラでレビューがつらい 制約解決 Gitの使い方を覚える必要ありPull Requestを用いる
25.
レビューコンテキスト コンテキスト問題 レビューの目的と観点を
明確にしたい レビューの目的が不明確で 必要な準備が不足している 観点がつかみにくい 制約解決 時間がかかるレビューの最初に レビューコンテキストを 説明してからレビューを開始
26.
チームレビュー コンテキスト問題 レビューの質をあげたい
レビュワーを増やしたい つまらない指摘が多くて レビューに 改善が盛り込めない 制約解決 時間がかかるチームレビューを導入し、 全員がレビュワーになる (レビュワーの観点を知る)
27.
レビューカウント コンテキスト問題 レビュー施策の効果を
計測したい 改善策はやっているが 客観的な効果計測が できていない 制約解決 レビューカウントが多いから といって質が悪いとは 一概には言えない 1つのチケットにかかった レビュー回数をハンコで 計測して見える化
28.
レビューの見える化 コンテキスト問題 一日のレビュー状況と偏りを
見える化したい 自己中心的なレビュー予約が 増えてレビュワーの作業時間 が逼迫する 制約解決 変更の際は変更コストが発生かんばん&付箋でレビューを 見える化して朝礼で確認
29.
レビュー時間の固定化 コンテキスト問題 レビュー調整のコストを
下げたい レビュワーが忙しく レビュー時間の調整に時間が かかってしまう 制約解決 レビュワーの作業時間が 不確定 レビュータイムを朝夕に設けて 必要な人がその枠を予約する
30.
今紹介したのは、 自分の現場で実際にやっている コードレビューパターンの一例です。
今日たくさんパターンの原石が 見つかると思うので、 それを組み合わせて自分のチームの 改善に活かしてください。
31.
C O D
E R E V I E W 俺のコードレビュー
32.
今のチームにとって 必要なコードレビューを やっている
33.
ぶっ壊したい!! 今のチームにとって 必要なコードレビューを
やっている
34.
こうじゃなくて、 チェック 確認学び
35.
こうなるように チェック 確認
学び
36.
コードレビューで もっとわくわくできるハズ
37.
コードレビューで もっと上を目指せるハズ
38.
コードレビューは必要なのか?
39.
お前のコードレビューはどうだ?
Download now