SlideShare una empresa de Scribd logo
1 de 39
Descargar para leer sin conexión
C O D E R E V I E W 
俺のコードレビュー 
@TAKAKING22 
TAKAO OYOBE 
及部敬雄 
2014.11.5 
俺のコードレビュー勉強会
コードレビューしてますか?
コードレビューの目的は?
品質の担保 
認識合わせ 
教育
Really?
品質の担保 
認識合わせ 
教育 
- CI、静的解析、テスト … 
- コード規約、ペアプロ … 
- ペアプロ、家でやれy …
品質の担保 
認識合わせ 
教育 
- CI、静的解析、テスト … 
コードレビューじゃなくても 
できる方法はある 
- コード規約、ペアプロ … 
- ペアプロ、家でやれy …
まさか… 
品質の担保 
認識合わせ 
教育 
偉い人が見るため 
偉い人が知るため 
偉い人が新人を…(ry
あるいは… 
品質の担保 
認識合わせ 
教育 
した気になってる 
した気になってる 
した気になってる
コードレビューの儀式化
コードレビューは 
問題解決の一手段 
でしかない
隣のコードレビューを 
そのまま真似しても 
効果はあまり出ないかも
なぜなら… 
よそはよそ、うちはうち 
成功した事例には、 
コンテキストや制約が存在し、 
それがあなたのチームに 
当てはまるとは限らないから
うわ… 
うちのコードレビュー 
効果なさすぎ…? 
こうならないために 
どうすればいよいのか? 
http://www.pakutaso.com/20120806242post-1877.html
パターンランゲージ 
http://books.rakuten.co.jp/rb/12432779/
パターン=再現可能な解決策 
http://www.slideshare.net/kawaguti/20111207-u
パターンは組み合わせ可能 
http://www.slideshare.net/kawaguti/20111207-u
組み合わせて意味を持つ 
http://www.slideshare.net/kdmsnr/fearlessjourneypostudy
まとめると、 
•事例だけでは役に立たない •事例からパターンに落とす •パターンを組み合わせて適用する •組み合わせることで効果が出る 
http://www.slideshare.net/kdmsnr/fearlessjourneypostudy
つまり、 
•自分のチームの問題・課題を見える化 •事例からパターンに変換する •パターンを取捨選択する •パターン組み合わせて実施する 
点ではなく、 
線・面で問題解決する 
http://www.slideshare.net/kdmsnr/fearlessjourneypostudy
実はぶっちゃけ… 
パターンってうさんくさくて 
あんまり好きじゃなかった
実はぶっちゃけ… 
でも教科書通りじゃ 
うまくいかねーんだよ 
いろいろ考えてやろうぜ 
ってことを代弁してくれてて 
ちょっと好きになった
持って帰って欲しいもの 
コードレビューパターン
Pull Request 
コンテキスト問題 
レビューのログを残したい 
人によって残し方が 
バラバラでレビューがつらい 
制約解決 
Gitの使い方を覚える必要ありPull Requestを用いる
レビューコンテキスト 
コンテキスト問題 
レビューの目的と観点を 
明確にしたい 
レビューの目的が不明確で 
必要な準備が不足している 
観点がつかみにくい 
制約解決 
時間がかかるレビューの最初に 
レビューコンテキストを 
説明してからレビューを開始
チームレビュー 
コンテキスト問題 
レビューの質をあげたい 
レビュワーを増やしたい 
つまらない指摘が多くて 
レビューに 
改善が盛り込めない 
制約解決 
時間がかかるチームレビューを導入し、 
全員がレビュワーになる 
(レビュワーの観点を知る)
レビューカウント 
コンテキスト問題 
レビュー施策の効果を 
計測したい 
改善策はやっているが 
客観的な効果計測が 
できていない 
制約解決 
レビューカウントが多いから 
といって質が悪いとは 
一概には言えない 
1つのチケットにかかった 
レビュー回数をハンコで 
計測して見える化
レビューの見える化 
コンテキスト問題 
一日のレビュー状況と偏りを 
見える化したい 
自己中心的なレビュー予約が 
増えてレビュワーの作業時間 
が逼迫する 
制約解決 
変更の際は変更コストが発生かんばん&付箋でレビューを 
見える化して朝礼で確認
レビュー時間の固定化 
コンテキスト問題 
レビュー調整のコストを 
下げたい 
レビュワーが忙しく 
レビュー時間の調整に時間が 
かかってしまう 
制約解決 
レビュワーの作業時間が 
不確定 
レビュータイムを朝夕に設けて 
必要な人がその枠を予約する
今紹介したのは、 
自分の現場で実際にやっている 
コードレビューパターンの一例です。 
今日たくさんパターンの原石が 
見つかると思うので、 
それを組み合わせて自分のチームの 
改善に活かしてください。
C O D E R E V I E W 
俺のコードレビュー
今のチームにとって 
必要なコードレビューを 
やっている
ぶっ壊したい!! 
今のチームにとって 
必要なコードレビューを 
やっている
こうじゃなくて、 
チェック 
確認学び
こうなるように 
チェック 
確認 
学び
コードレビューで 
もっとわくわくできるハズ
コードレビューで 
もっと上を目指せるハズ
コードレビューは必要なのか?
お前のコードレビューはどうだ?

Más contenido relacionado

Más de Takao Oyobe

Move into speaker deck
Move into speaker deckMove into speaker deck
Move into speaker deckTakao Oyobe
 
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1Takao Oyobe
 
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamuraiTakao Oyobe
 
俺のインセプションデッキ【Remaster版】
俺のインセプションデッキ【Remaster版】俺のインセプションデッキ【Remaster版】
俺のインセプションデッキ【Remaster版】Takao Oyobe
 
5分でわかった気になるインセプションデッキ
5分でわかった気になるインセプションデッキ5分でわかった気になるインセプションデッキ
5分でわかった気になるインセプションデッキTakao Oyobe
 
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiATakao Oyobe
 
20141213 俺のインセプションデッキ #agilesamurai
20141213 俺のインセプションデッキ #agilesamurai20141213 俺のインセプションデッキ #agilesamurai
20141213 俺のインセプションデッキ #agilesamuraiTakao Oyobe
 
20141028 lightning talk at staff party of rtc48
20141028 lightning talk at staff party of rtc4820141028 lightning talk at staff party of rtc48
20141028 lightning talk at staff party of rtc48Takao Oyobe
 
(Beta)clean release manual
(Beta)clean release manual(Beta)clean release manual
(Beta)clean release manualTakao Oyobe
 
つくる現場 - 変化を支える3つの現場アーキテクト -
つくる現場 - 変化を支える3つの現場アーキテクト -つくる現場 - 変化を支える3つの現場アーキテクト -
つくる現場 - 変化を支える3つの現場アーキテクト -Takao Oyobe
 
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -Takao Oyobe
 
はじめてのふりかえり
はじめてのふりかえりはじめてのふりかえり
はじめてのふりかえりTakao Oyobe
 
えんじにあというおしごと
えんじにあというおしごとえんじにあというおしごと
えんじにあというおしごとTakao Oyobe
 
secret base~jeff pattonがくれたもの~
secret base~jeff pattonがくれたもの~secret base~jeff pattonがくれたもの~
secret base~jeff pattonがくれたもの~Takao Oyobe
 
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~Takao Oyobe
 
Fearless Change - Agile2012 feedback report -
Fearless Change - Agile2012 feedback report -Fearless Change - Agile2012 feedback report -
Fearless Change - Agile2012 feedback report -Takao Oyobe
 
アジャイルペーペーシップとチーム改革
アジャイルペーペーシップとチーム改革アジャイルペーペーシップとチーム改革
アジャイルペーペーシップとチーム改革Takao Oyobe
 
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -Takao Oyobe
 
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -Takao Oyobe
 

Más de Takao Oyobe (19)

Move into speaker deck
Move into speaker deckMove into speaker deck
Move into speaker deck
 
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
チームのチームによるチームのためのアジャイル開発 -誰がやる!?じゃあ俺やるよ!- #devsumi #kansumi #kansumiA1
 
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
僕らのふりかえり戦略 -チームのコンパスをつくる- #agilesamurai
 
俺のインセプションデッキ【Remaster版】
俺のインセプションデッキ【Remaster版】俺のインセプションデッキ【Remaster版】
俺のインセプションデッキ【Remaster版】
 
5分でわかった気になるインセプションデッキ
5分でわかった気になるインセプションデッキ5分でわかった気になるインセプションデッキ
5分でわかった気になるインセプションデッキ
 
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
【20-A-7】アジャイルヒーローは誰だ!? #devsumi #devsumiA
 
20141213 俺のインセプションデッキ #agilesamurai
20141213 俺のインセプションデッキ #agilesamurai20141213 俺のインセプションデッキ #agilesamurai
20141213 俺のインセプションデッキ #agilesamurai
 
20141028 lightning talk at staff party of rtc48
20141028 lightning talk at staff party of rtc4820141028 lightning talk at staff party of rtc48
20141028 lightning talk at staff party of rtc48
 
(Beta)clean release manual
(Beta)clean release manual(Beta)clean release manual
(Beta)clean release manual
 
つくる現場 - 変化を支える3つの現場アーキテクト -
つくる現場 - 変化を支える3つの現場アーキテクト -つくる現場 - 変化を支える3つの現場アーキテクト -
つくる現場 - 変化を支える3つの現場アーキテクト -
 
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
アジャイルサムライ壱の太刀 - 説得しないはじめないアジャイル開発のはじめ形 -
 
はじめてのふりかえり
はじめてのふりかえりはじめてのふりかえり
はじめてのふりかえり
 
えんじにあというおしごと
えんじにあというおしごとえんじにあというおしごと
えんじにあというおしごと
 
secret base~jeff pattonがくれたもの~
secret base~jeff pattonがくれたもの~secret base~jeff pattonがくれたもの~
secret base~jeff pattonがくれたもの~
 
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
20121021 Lean!! Lean!! Lean!! ~DevLOVEと現場と私~
 
Fearless Change - Agile2012 feedback report -
Fearless Change - Agile2012 feedback report -Fearless Change - Agile2012 feedback report -
Fearless Change - Agile2012 feedback report -
 
アジャイルペーペーシップとチーム改革
アジャイルペーペーシップとチーム改革アジャイルペーペーシップとチーム改革
アジャイルペーペーシップとチーム改革
 
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
私がスクラムをやめた理由 - 全員スクラムマスター。@DevLove -
 
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
もうアジャイルなんて言わないよ絶対 - Developers Summit 2012 FB -
 

Último

2024.03.12 Cost drivers of cultivated meat production.pdf
2024.03.12 Cost drivers of cultivated meat production.pdf2024.03.12 Cost drivers of cultivated meat production.pdf
2024.03.12 Cost drivers of cultivated meat production.pdfThe Good Food Institute
 
20140402 - Smart house demo kit
20140402 - Smart house demo kit20140402 - Smart house demo kit
20140402 - Smart house demo kitJamie (Taka) Wang
 
Design and Modeling for MySQL SCALE 21X Pasadena, CA Mar 2024
Design and Modeling for MySQL SCALE 21X Pasadena, CA Mar 2024Design and Modeling for MySQL SCALE 21X Pasadena, CA Mar 2024
Design and Modeling for MySQL SCALE 21X Pasadena, CA Mar 2024Alkin Tezuysal
 
Extra-120324-Visite-Entreprise-icare.pdf
Extra-120324-Visite-Entreprise-icare.pdfExtra-120324-Visite-Entreprise-icare.pdf
Extra-120324-Visite-Entreprise-icare.pdfInfopole1
 
GraphSummit Copenhagen 2024 - Neo4j Vision and Roadmap.pptx
GraphSummit Copenhagen 2024 - Neo4j Vision and Roadmap.pptxGraphSummit Copenhagen 2024 - Neo4j Vision and Roadmap.pptx
GraphSummit Copenhagen 2024 - Neo4j Vision and Roadmap.pptxNeo4j
 
Automation Ops Series: Session 2 - Governance for UiPath projects
Automation Ops Series: Session 2 - Governance for UiPath projectsAutomation Ops Series: Session 2 - Governance for UiPath projects
Automation Ops Series: Session 2 - Governance for UiPath projectsDianaGray10
 
UiPath Studio Web workshop Series - Day 3
UiPath Studio Web workshop Series - Day 3UiPath Studio Web workshop Series - Day 3
UiPath Studio Web workshop Series - Day 3DianaGray10
 
AI Workshops at Computers In Libraries 2024
AI Workshops at Computers In Libraries 2024AI Workshops at Computers In Libraries 2024
AI Workshops at Computers In Libraries 2024Brian Pichman
 
How to become a GDSC Lead GDSC MI AOE.pptx
How to become a GDSC Lead GDSC MI AOE.pptxHow to become a GDSC Lead GDSC MI AOE.pptx
How to become a GDSC Lead GDSC MI AOE.pptxKaustubhBhavsar6
 
Scenario Library et REX Discover industry- and role- based scenarios
Scenario Library et REX Discover industry- and role- based scenariosScenario Library et REX Discover industry- and role- based scenarios
Scenario Library et REX Discover industry- and role- based scenariosErol GIRAUDY
 
TrustArc Webinar - How to Live in a Post Third-Party Cookie World
TrustArc Webinar - How to Live in a Post Third-Party Cookie WorldTrustArc Webinar - How to Live in a Post Third-Party Cookie World
TrustArc Webinar - How to Live in a Post Third-Party Cookie WorldTrustArc
 
The Zero-ETL Approach: Enhancing Data Agility and Insight
The Zero-ETL Approach: Enhancing Data Agility and InsightThe Zero-ETL Approach: Enhancing Data Agility and Insight
The Zero-ETL Approach: Enhancing Data Agility and InsightSafe Software
 
CyberSecurity - Computers In Libraries 2024
CyberSecurity - Computers In Libraries 2024CyberSecurity - Computers In Libraries 2024
CyberSecurity - Computers In Libraries 2024Brian Pichman
 
UiPath Studio Web workshop series - Day 2
UiPath Studio Web workshop series - Day 2UiPath Studio Web workshop series - Day 2
UiPath Studio Web workshop series - Day 2DianaGray10
 
LF Energy Webinar - Unveiling OpenEEMeter 4.0
LF Energy Webinar - Unveiling OpenEEMeter 4.0LF Energy Webinar - Unveiling OpenEEMeter 4.0
LF Energy Webinar - Unveiling OpenEEMeter 4.0DanBrown980551
 
Explore the UiPath Community and ways you can benefit on your journey to auto...
Explore the UiPath Community and ways you can benefit on your journey to auto...Explore the UiPath Community and ways you can benefit on your journey to auto...
Explore the UiPath Community and ways you can benefit on your journey to auto...DianaGray10
 
UiPath Studio Web workshop series - Day 4
UiPath Studio Web workshop series - Day 4UiPath Studio Web workshop series - Day 4
UiPath Studio Web workshop series - Day 4DianaGray10
 
The New Cloud World Order Is FinOps (Slideshow)
The New Cloud World Order Is FinOps (Slideshow)The New Cloud World Order Is FinOps (Slideshow)
The New Cloud World Order Is FinOps (Slideshow)codyslingerland1
 
Outage Analysis: March 5th/6th 2024 Meta, Comcast, and LinkedIn
Outage Analysis: March 5th/6th 2024 Meta, Comcast, and LinkedInOutage Analysis: March 5th/6th 2024 Meta, Comcast, and LinkedIn
Outage Analysis: March 5th/6th 2024 Meta, Comcast, and LinkedInThousandEyes
 

Último (20)

2024.03.12 Cost drivers of cultivated meat production.pdf
2024.03.12 Cost drivers of cultivated meat production.pdf2024.03.12 Cost drivers of cultivated meat production.pdf
2024.03.12 Cost drivers of cultivated meat production.pdf
 
20140402 - Smart house demo kit
20140402 - Smart house demo kit20140402 - Smart house demo kit
20140402 - Smart house demo kit
 
Design and Modeling for MySQL SCALE 21X Pasadena, CA Mar 2024
Design and Modeling for MySQL SCALE 21X Pasadena, CA Mar 2024Design and Modeling for MySQL SCALE 21X Pasadena, CA Mar 2024
Design and Modeling for MySQL SCALE 21X Pasadena, CA Mar 2024
 
Extra-120324-Visite-Entreprise-icare.pdf
Extra-120324-Visite-Entreprise-icare.pdfExtra-120324-Visite-Entreprise-icare.pdf
Extra-120324-Visite-Entreprise-icare.pdf
 
GraphSummit Copenhagen 2024 - Neo4j Vision and Roadmap.pptx
GraphSummit Copenhagen 2024 - Neo4j Vision and Roadmap.pptxGraphSummit Copenhagen 2024 - Neo4j Vision and Roadmap.pptx
GraphSummit Copenhagen 2024 - Neo4j Vision and Roadmap.pptx
 
Automation Ops Series: Session 2 - Governance for UiPath projects
Automation Ops Series: Session 2 - Governance for UiPath projectsAutomation Ops Series: Session 2 - Governance for UiPath projects
Automation Ops Series: Session 2 - Governance for UiPath projects
 
UiPath Studio Web workshop Series - Day 3
UiPath Studio Web workshop Series - Day 3UiPath Studio Web workshop Series - Day 3
UiPath Studio Web workshop Series - Day 3
 
AI Workshops at Computers In Libraries 2024
AI Workshops at Computers In Libraries 2024AI Workshops at Computers In Libraries 2024
AI Workshops at Computers In Libraries 2024
 
How to become a GDSC Lead GDSC MI AOE.pptx
How to become a GDSC Lead GDSC MI AOE.pptxHow to become a GDSC Lead GDSC MI AOE.pptx
How to become a GDSC Lead GDSC MI AOE.pptx
 
Scenario Library et REX Discover industry- and role- based scenarios
Scenario Library et REX Discover industry- and role- based scenariosScenario Library et REX Discover industry- and role- based scenarios
Scenario Library et REX Discover industry- and role- based scenarios
 
TrustArc Webinar - How to Live in a Post Third-Party Cookie World
TrustArc Webinar - How to Live in a Post Third-Party Cookie WorldTrustArc Webinar - How to Live in a Post Third-Party Cookie World
TrustArc Webinar - How to Live in a Post Third-Party Cookie World
 
The Zero-ETL Approach: Enhancing Data Agility and Insight
The Zero-ETL Approach: Enhancing Data Agility and InsightThe Zero-ETL Approach: Enhancing Data Agility and Insight
The Zero-ETL Approach: Enhancing Data Agility and Insight
 
CyberSecurity - Computers In Libraries 2024
CyberSecurity - Computers In Libraries 2024CyberSecurity - Computers In Libraries 2024
CyberSecurity - Computers In Libraries 2024
 
UiPath Studio Web workshop series - Day 2
UiPath Studio Web workshop series - Day 2UiPath Studio Web workshop series - Day 2
UiPath Studio Web workshop series - Day 2
 
SheDev 2024
SheDev 2024SheDev 2024
SheDev 2024
 
LF Energy Webinar - Unveiling OpenEEMeter 4.0
LF Energy Webinar - Unveiling OpenEEMeter 4.0LF Energy Webinar - Unveiling OpenEEMeter 4.0
LF Energy Webinar - Unveiling OpenEEMeter 4.0
 
Explore the UiPath Community and ways you can benefit on your journey to auto...
Explore the UiPath Community and ways you can benefit on your journey to auto...Explore the UiPath Community and ways you can benefit on your journey to auto...
Explore the UiPath Community and ways you can benefit on your journey to auto...
 
UiPath Studio Web workshop series - Day 4
UiPath Studio Web workshop series - Day 4UiPath Studio Web workshop series - Day 4
UiPath Studio Web workshop series - Day 4
 
The New Cloud World Order Is FinOps (Slideshow)
The New Cloud World Order Is FinOps (Slideshow)The New Cloud World Order Is FinOps (Slideshow)
The New Cloud World Order Is FinOps (Slideshow)
 
Outage Analysis: March 5th/6th 2024 Meta, Comcast, and LinkedIn
Outage Analysis: March 5th/6th 2024 Meta, Comcast, and LinkedInOutage Analysis: March 5th/6th 2024 Meta, Comcast, and LinkedIn
Outage Analysis: March 5th/6th 2024 Meta, Comcast, and LinkedIn
 

20141105 俺のコードレビュー(lightning talk) #devraku