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
[PHPerKaigi 2021 (LT)] 新社会人のコード品質カイゼン記録
Search
don
March 31, 2021
Programming
1
1.1k
[PHPerKaigi 2021 (LT)] 新社会人のコード品質カイゼン記録
don
March 31, 2021
Tweet
Share
More Decks by don
See All by don
レガシーシステムへのPHPStan導入から半年での効果と課題
bosshawk
0
1.4k
レガシーシステムへのPHPStan導入から半年での課題と効果
bosshawk
0
1.2k
息の長いサービスの PHP8バージョンアップで見えた 課題と解決法 / Problems and solutions found when upgrading long-term services to PHP8
bosshawk
0
2.2k
レガシーシステムにおけるPHP8バージョンアップのアプリ対応記録
bosshawk
0
1.7k
[おすすめの技術書 LT会 - vol.2] 体系的に学ぶ安全なWebアプリケーションの作り方
bosshawk
0
20k
Other Decks in Programming
See All in Programming
実践 Advanced CallKit 〜快適な通話の実現に向けて〜
mot_techtalk
3
110
(非公開スライド追加)座談会 「Strict ConcurrencyとSwift 6が開く新時代: 私たちはどう生きるか?」
shiz
1
140
ECMAScript仕様を読むのに必要な知識 - ダイジェスト版
syumai
4
2.5k
Prompt Cachingは本当に効果的なのか検証してみた.pdf
ttnyt8701
0
250
快適な開発と高セキュリティを実現するCryptoKitを活用したCoreDataのデータ暗号化術
grandbig
1
300
Swiftコードバトル必勝法
toshi0383
0
140
Appleの新しいプライバシー要件対応: ノーコードアプリ プラットフォームの実践事例
nao_randd
1
450
REXML改善のその後
naitoh
0
150
『ドメイン駆動設計をはじめよう』中核の業務領域
masuda220
PRO
5
840
月間4.5億回再生を超える大規模サービス TVer iOSアプリのリアーキテクチャ戦略 - iOSDC2024
techtver
PRO
1
570
Polars を Kaggle コンペで使ってみた(LMSYS Chatbot Arena)
kohecchi
1
360
connect-go で面倒くささと戦う / 2024-08-27 #newmo_layerx_go
izumin5210
2
590
Featured
See All Featured
Building a Modern Day E-commerce SEO Strategy
aleyda
35
6.8k
The Straight Up "How To Draw Better" Workshop
denniskardys
230
130k
Writing Fast Ruby
sferik
623
60k
The Art of Programming - Codeland 2020
erikaheidi
48
13k
For a Future-Friendly Web
brad_frost
173
9.3k
Practical Orchestrator
shlominoach
185
10k
5 minutes of I Can Smell Your CMS
philhawksworth
201
19k
It's Worth the Effort
3n
182
27k
GraphQLの誤解/rethinking-graphql
sonatard
65
9.7k
Bootstrapping a Software Product
garrettdimon
PRO
304
110k
GraphQLとの向き合い方2022年版
quramy
43
13k
Fight the Zombie Pattern Library - RWD Summit 2016
marcelosomers
230
17k
Transcript
#phperkaigi ©2020 RAKUS Co., Ltd. ©2020 RAKUS Co., Ltd. 新社会人のコード
品質カイゼン記録 頓花貴俊
#phperkaigi ©2020 RAKUS Co., Ltd. 自己紹介 名前:頓花貴俊 (どんが たかとし) 会社:株式会社ラクス
社歴:二年 プログラム遍歴: ・大学時代:C言語、Fortran ・入社後:PHP, Java
#phperkaigi ©2020 RAKUS Co., Ltd. 入社研修終えての最初の数ヶ月 軽微な修正、小規模な実装では 特にコードレビューで大きな問題は なかった
#phperkaigi ©2020 RAKUS Co., Ltd. ですが・・・
#phperkaigi ©2020 RAKUS Co., Ltd. 初めての規模の大きな実装 コードレビューの指摘:約100件 開発期間 全体:3人月 担当:2人月
指摘は 約8割が保守性に関する指摘
#phperkaigi ©2020 RAKUS Co., Ltd. うわっ・・・私のコード品質、 低すぎ・・・?
#phperkaigi ©2020 RAKUS Co., Ltd. 主な指摘 ・不要コード、デバッグコードの削除忘れ ・SQLで必要のないカラムの取得 ・意味が理解しづらいメソッド名、変数名、コメント ・タイポ
その他には...
#phperkaigi ©2020 RAKUS Co., Ltd. 保守性に関する指摘 ー 例1 ー 指摘:
条件が分かりずらい 修正後 修正前
#phperkaigi ©2020 RAKUS Co., Ltd. 保守性に関する指摘 ー 例2 ー 指摘:
・何をしているか読み取れない ・DOM構成が変わったときに 対応しずらい 修正後 修正前
#phperkaigi ©2020 RAKUS Co., Ltd. 指摘が出ないように 対策を考える
#phperkaigi ©2020 RAKUS Co., Ltd. 対策 ・コミット前に実装したコード全体を見直す ・前回の指摘から セルフチェックリストを作り 実装前と後にチェックする
チェックリスト
#phperkaigi ©2020 RAKUS Co., Ltd. セルフチェックを実施した結果 コードレビューの指摘 30件 開発期間 全体:1.5人月
担当:0.5人月 あまり減っていない… → なぜ…? 自分だけではわからない…
#phperkaigi ©2020 RAKUS Co., Ltd. 悩みを相談した結果 先輩とペアプロを実施することに
#phperkaigi ©2020 RAKUS Co., Ltd. ペアプロを実施した結果 コードレビューの指摘 10件 開発期間 全体:3人月
担当:1.5人月 → 大きく減少!!! ぺアプロの良かったところ ・実装しつつ質問できる ・他の人の視点を知ることができた
#phperkaigi ©2020 RAKUS Co., Ltd. 1人では気づけず、ペアプロで気づいたこと ・既存コードやそのコピペなどへのコメントの追加 ・TODOコメントなどを使って気になったところにメモをする ・具体的な処理を書く前にメソッドやコメントのみを書いて、 意図が分かりやすくなっているかチェックする
#phperkaigi ©2020 RAKUS Co., Ltd. 改善を振り返って ・入社するまでコードレビューを受けることがなかったので自身 のコード品質が良くないことに気づいてなかった。 ・自分だけでなんとかしようとしたけど失敗した。 ・ペアプロを行うことで
他の人の視点を取り入れることができ、 視点を広げることができた
#phperkaigi ©2020 RAKUS Co., Ltd. 改善後も品質向上のために実施していること ・実装時には、既存コードにコメントを追加しコード全体を 把握する ・デイリーのミーティング時に実装難易度が高い所を相談し、 モブプロ的に実装を進める
#phperkaigi ©2020 RAKUS Co., Ltd. 自分で解決できないときは 皆さんもペアプロやってみましょう!!