Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
コードレビューで 開発が加速した話
Search
higuuu
December 01, 2021
Programming
0
730
コードレビューで 開発が加速した話
コードレビューでむしろ開発速度が上がった話です!
ヒグ!(樋口修也)
関連情報
KBOYさんに技術顧問をお願いした話
https://dx.sapporo.coop/n/n23376af197fa
higuuu
December 01, 2021
Tweet
Share
More Decks by higuuu
See All by higuuu
副業で入ったけどタスクがないからPMっぽいことをした話
higuuu
0
240
年700万円損するサーバレスの 認可システムをご紹介します!!
higuuu
3
1.2k
もしも、 上司に鬼退治を命じられたら~プロジェクト計画編~
higuuu
0
660
フロントエンドが知って おきたいセキュリティについて
higuuu
1
1.1k
今年の抱負 81日でやり遂げるぞー
higuuu
1
300
Testing rules for teams that do not write test code
higuuu
1
230
SPAのサイトを アプリのwebviewで利用するときのトークンの渡し方
higuuu
0
2.3k
Other Decks in Programming
See All in Programming
なぜ適用するか、移行して理解するClean Architecture 〜構造を超えて設計を継承する〜 / Why Apply, Migrate and Understand Clean Architecture - Inherit Design Beyond Structure
seike460
PRO
3
770
PHP 8.4の新機能「プロパティフック」から学ぶオブジェクト指向設計とリスコフの置換原則
kentaroutakeda
2
900
PHPで始める振る舞い駆動開発(Behaviour-Driven Development)
ohmori_yusuke
2
390
dbt民主化とLLMによる開発ブースト ~ AI Readyな分析サイクルを目指して ~
yoshyum
3
1k
The Modern View Layer Rails Deserves: A Vision For 2025 And Beyond @ RailsConf 2025, Philadelphia, PA
marcoroth
1
150
git worktree × Claude Code × MCP ~生成AI時代の並列開発フロー~
hisuzuya
1
570
AI コーディングエージェントの時代へ:JetBrains が描く開発の未来
masaruhr
1
150
Deep Dive into ~/.claude/projects
hiragram
14
2.5k
チームで開発し事業を加速するための"良い"設計の考え方 @ サポーターズCoLab 2025-07-08
agatan
1
420
Team operations that are not burdened by SRE
kazatohiei
1
310
Composerが「依存解決」のためにどんな工夫をしているか #phpcon
o0h
PRO
1
260
猫と暮らす Google Nest Cam生活🐈 / WebRTC with Google Nest Cam
yutailang0119
0
120
Featured
See All Featured
[Rails World 2023 - Day 1 Closing Keynote] - The Magic of Rails
eileencodes
35
2.4k
jQuery: Nuts, Bolts and Bling
dougneiner
63
7.8k
Put a Button on it: Removing Barriers to Going Fast.
kastner
60
3.9k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
kevinliebholz
34
5.9k
Learning to Love Humans: Emotional Interface Design
aarron
273
40k
Code Reviewing Like a Champion
maltzj
524
40k
The Power of CSS Pseudo Elements
geoffreycrofte
77
5.9k
Intergalactic Javascript Robots from Outer Space
tanoku
271
27k
Java REST API Framework Comparison - PWX 2021
mraible
31
8.7k
Measuring & Analyzing Core Web Vitals
bluesmoon
7
510
Connecting the Dots Between Site Speed, User Experience & Your Business [WebExpo 2025]
tammyeverts
6
300
Build your cross-platform service in a week with App Engine
jlugia
231
18k
Transcript
コードレビューで 開発が加速した話 2021.12.1 生活協同組合コープさっぽろ 樋口修也 コープさっぽろDX
発表者 フロントエンド,認証認可 2015年 旭川→岩手大学へ入学 2019年 東京のIT企業に新卒入社 2020年 コープさっぽろへ転職し札幌へ ダブルダッチ、筋トレ、ダンス AWS
CDK, Vue.js Youtubeはじめました! 樋口修也(24) 担当: 経歴: 趣味: 好み: 最近:
コードレビューのたま〜にあるある 時間かかるから省こうぜ!
コードレビューのイメージ コードレビュー 品質 向上↑ 速度 低下↓
コードレビューのイメージ コードレビュー 品質 向上↑ 速度 低下↓ 非エンジニアには支持されない
コードレビューにお金をかけたら コードレビュー 品質 向上↑ 速度 向上↑
実際にあった話 コードレビュー 品質 向上↑ 速度 向上↑ 非エンジニアも支援してくれる
むか〜し、むかし(現在進行中) アプリリニューアルPJ開始!
アプリリニューアルプロジェクト 業務委託3名 Flutter: 未経験 実装 内製エンジニア 0.2~0.5名 Flutter経験 アリ 4月〜
アプリリニューアルプロジェクト 業務委託3名 Flutter: 未経験 実装 内製エンジニア 0.2~0.5名 Flutter経験 アリ 4月〜6月
他のプロジェクトが 炎上
メンバーチェンジ 実装 業務委託3.5 名 Flutter経験 なし 実装 伊藤さん(正社員) Flutter経験 アリ
サポート・実装 自分 0.3人月 Flutter経験 ナシ 6月中旬〜
6月中旬にジョインした時のアプリの状態 • アーキテクチャが不鮮明 • 状態管理ライブラリ riverpod と provider が共存していた •
それぞれの役割分担も特になし • コード規約なし • キャメルケース、パスカルケースなども特に指定なし • 画像などのデータの配置場所も複数存在 • その他 • 各フォルダの役割を正直みんなよくわかってない • 大量のグローバル変数 • webviewライブラリの混在 • etc… 統制が取れていない開発状態
アプリリニューアルプロジェクト 実装 業務委託3.5 名 Flutter経験 なし メイン実装 伊藤さん(正社員) Flutter経験 アリ
アーキテクチャ サポート・実装 自分 0.3人月 Flutter経験 ナシ リポ ジトリ管 理 コード規約
アプリリニューアルプロジェクト 実装 業務委託3.5 名 Flutter経験 なし メイン実装 伊藤さん(正社員) Flutter経験 アリ
アーキテクチャ サポート・実装 自分 0.3人月 Flutter経験 ナシ リポ ジトリ管 理 コード規約 Help!
アプリリニューアルプロジェクト 実装 業務委託3.5 名 Flutter経験 なし メイン実装 伊藤さん(正社員) Flutter経験 アリ
アーキテクチャ サポート・実装 自分 0.3人月 Flutter経験 ナシ リポ ジトリ管 理 コード規約 無理っす! Help!
当時の問題(再掲) • アーキテクチャが不鮮明 • 状態管理ライブラリ riverpod と provider が共存していた •
それぞれの役割分担も特になし • コード規約なし • キャメルケース、パスカルケースなども特に指定なし • 画像などのデータの配置場所も複数存在 • その他 • 各フォルダの役割を正直みんなよくわかってない • 大量のグローバル変数 • webviewライブラリの混在 • etc…
当時の問題(再掲) • アーキテクチャが不鮮明 • 状態管理ライブラリ riverpod と provider が共存していた •
それぞれの役割分担も特になし • コード規約なし • キャメルケース、パスカルケースなども特に指定なし • 画像などのデータの配置場所も複数存在 • その他 • 各フォルダの役割を正直みんなよくわかってない • 大量のグローバル変数 • webviewライブラリの混在 • etc… ルールの策定と監視が必要 コード規約と レビュー
そんなリソースもスキルもない 実装 業務委託3.5 名 Flutter経験 なし メイン実装 伊藤さん(正社員) Flutter経験 半年
アーキテクチャ サポート・実装 自分 0.3人月 Flutter経験 ナシ リポ ジトリ管 理 コード規約
コード規約の作成補助とレビューをKBOYさんに依頼 おなしゃす! いいよ! Twitter の DM で詳細を説明して依頼
以前 実装 設計 PR • ライブラリ調査 • 選定に悩む時間 • 複雑なアーキ
• わからないことを相 談できる相手がいな い • 実装したら既にあっ たUtilなど • 誰にレビュー依頼し たらいいかわからな い • 自信がないので放置 されたPR 負のスパイラル
改善結果 実装 設計 PR • よく使うライブラリを 規約に記載 • MVCでシンプルな アーキ
• 調べてもわからない ことがあればKBOY さんに相談 • フォルダ整理によっ てUtilを使いまわせ る • KBOYさんにレ ビューしてもらったの で自信を持ってマー ジできる • 随時規約の更新 開発効率10%改善(概算)
さらに 自信がついてリファクタリングが加速した
最後に いくら確認を省いても エンジニアは気になるものは気になります 目に見えない エンジニア心理も工数に影響します
DXコープさっぽろ KBOYさんに技術顧問をお願いした話