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
650
コードレビューで 開発が加速した話
コードレビューでむしろ開発速度が上がった話です!
ヒグ!(樋口修也)
関連情報
KBOYさんに技術顧問をお願いした話
https://dx.sapporo.coop/n/n23376af197fa
higuuu
December 01, 2021
Tweet
Share
More Decks by higuuu
See All by higuuu
副業で入ったけどタスクがないからPMっぽいことをした話
higuuu
0
180
年700万円損するサーバレスの 認可システムをご紹介します!!
higuuu
3
1k
もしも、 上司に鬼退治を命じられたら~プロジェクト計画編~
higuuu
0
590
フロントエンドが知って おきたいセキュリティについて
higuuu
1
910
今年の抱負 81日でやり遂げるぞー
higuuu
1
260
Testing rules for teams that do not write test code
higuuu
1
190
SPAのサイトを アプリのwebviewで利用するときのトークンの渡し方
higuuu
0
1.7k
Other Decks in Programming
See All in Programming
Rustのweb開発を助ける 便利なツール紹介
yuki0418
1
190
CSC307 Lecture 13
javiergs
PRO
0
150
Findy - エンジニア向け会社紹介 / Findy Letter for Engineers
findyinc
2
81k
GraphQL はいいぞ! ~Laravel で学ぶ GraphQL 入門~
azuki
1
160
APIのない大学ログインWebサービスをWKWebViewとJavaScriptでアプリ化した話
akidon0000
1
330
みんなのオブザーバビリティプラットフォームを作ってるんだがパフォーマンスがやばい #mackerelio #srenext
ne_sachirou
0
370
MIERUNE BBQにおけるユーザー中心設計()
mierune
PRO
1
110
I/O Extended Android in Korea 2024 ~ Whats new in Android development tools
pluu
0
250
Webエディタライブラリ 「CodeMirror」から学ぶ Webアプリ開発のテクニック
ryosukeigarashi
0
250
유연한 Composable 설계
l2hyunwoo
0
380
CSC307 Lecture 09
javiergs
PRO
1
500
Terraformテスト入門
msato
0
530
Featured
See All Featured
Art, The Web, and Tiny UX
lynnandtonic
291
20k
The Invisible Customer
myddelton
117
13k
Building Adaptive Systems
keathley
34
2k
How to Ace a Technical Interview
jacobian
274
23k
Robots, Beer and Maslow
schacon
PRO
157
8.1k
Creatively Recalculating Your Daily Design Routine
revolveconf
214
11k
Distributed Sagas: A Protocol for Coordinating Microservices
caitiem20
325
21k
Understanding Cognitive Biases in Performance Measurement
bluesmoon
18
1.2k
RailsConf 2023
tenderlove
16
720
A designer walks into a library…
pauljervisheath
201
24k
Fashionably flexible responsive web design (full day workshop)
malarkey
399
65k
Producing Creativity
orderedlist
PRO
340
39k
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さんに技術顧問をお願いした話