Upgrade to PRO for Only $50/Year—Limited-Time Offer! 🔥

[PHPerKaigi 2021 (LT)] 新社会人のコード品質カイゼン記録

Avatar for don don
March 31, 2021
Programming
1
1.4k

[PHPerKaigi 2021 (LT)] 新社会人のコード品質カイゼン記録

Avatar for don

don

March 31, 2021
Tweet

More Decks by don

See All by don
レガシーシステムへのPHPStan導入から半年での効果と課題
Avatar for don bosshawk
0
2.2k
レガシーシステムへのPHPStan導入から半年での課題と効果
Avatar for don bosshawk
1
1.9k
息の長いサービスの PHP8バージョンアップで見えた 課題と解決法 / Problems and solutions found when upgrading long-term services to PHP8
Avatar for don bosshawk
0
2.7k
レガシーシステムにおけるPHP8バージョンアップのアプリ対応記録
Avatar for don bosshawk
0
2.1k
[おすすめの技術書 LT会 - vol.2] 体系的に学ぶ安全なWebアプリケーションの作り方
Avatar for don bosshawk
0
21k

Other Decks in Programming

See All in Programming
DevFest Android in Korea 2025 - 개발자 커뮤니티를 통해 얻는 가치
Avatar for Suhyeon Kim wisemuji
0
170
AIコーディングエージェント（Manus）
Avatar for kondai kondai24
0
210
Pythonではじめるオープンデータ分析〜書籍の紹介と書籍で紹介しきれなかった事例の紹介〜
Avatar for Ryo YOSHI welliving
2
540
マスタデータ問題、マイクロサービスでどう解くか
Avatar for Kato Keisuke kts
0
110
Canon EOS R50 V と R5 Mark II 購入でみえてきた最近のデジイチ VR180 事情、そして VR180 静止画に活路を見出すまで
Avatar for kazuhiro hara karad
0
130
The Art of Re-Architecture - Droidcon India 2025
Avatar for Sid Patil siddroid
0
120
ELYZA_Findy AI Engineering Summit登壇資料_AIコーディング時代に「ちゃんと」やること_toB LLMプロダクト開発舞台裏_20251216
Avatar for 株式会社ELYZA elyza
2
580
re:Invent 2025 のイケてるサービスを紹介する
Avatar for maroon1st maroon1st
0
150
新卒エンジニアのプルリクエスト with AI駆動
Avatar for Yuta Fukunaga fukunaga2025
0
230
モデル駆動設計をやってみよう ワークショップ開催報告(Modeling Forum2025) / model driven design workshop report
Avatar for haru860 haru860
0
280
Cell-Based Architecture
Avatar for Luram Archanjo larchanjo
0
140
ローカルLLMを⽤いてコード補完を⾏う VSCode拡張機能を作ってみた
Avatar for NearMeの技術発表資料です nearme_tech
PRO
0
150

Featured

See All Featured
Information Architects: The Missing Link in Design Systems
Avatar for Michelle Chin soysaucechin
0
710
Claude Code どこまでも/ Claude Code Everywhere
Avatar for nwiizo nwiizo
61
47k
Building Flexible Design Systems
Avatar for Yesenia Perez-Cruz yeseniaperezcruz
330
39k
Marketing Yourself as an Engineer | Alaka | Gurzu
Avatar for Gurzu gurzu
0
89
StorybookのUI Testing Handbookを読んだ
Avatar for Shingo Yamazaki zakiyama
31
6.5k
Making the Leap to Tech Lead
Avatar for Ryan Cromwell cromwellryan
135
9.7k
ピンチをチャンスに：未来をつくるプロダクトロードマップ #pmconf2020
Avatar for Aki / @LoveIdahoBurger aki_iinuma
128
55k
AI Search: Implications for SEO and How to Move Forward - #ShenzhenSEOConference
Avatar for Aleyda Solis aleyda
1
1k
Into the Great Unknown - MozCon
Avatar for Noah Learner thekraken
40
2.2k
Building an army of robots
Avatar for Kyle Neath kneath
306
46k
How To Speak Unicorn (iThemes Webinar)
Avatar for Michelle Schulp Hunt marktimemedia
1
340
Efficient Content Optimization with Google Search Console & Apps Script
Avatar for Katarina Dahlin katarinadahlin
PRO
0
250

Transcript

  1. #phperkaigi ©2020 RAKUS Co., Ltd. ©2020 RAKUS Co., Ltd. 新社会人のコード

    品質カイゼン記録 頓花貴俊

  2. #phperkaigi ©2020 RAKUS Co., Ltd. 自己紹介 名前：頓花貴俊 (どんが たかとし) 会社：株式会社ラクス

    社歴：二年 プログラム遍歴： ・大学時代：C言語、Fortran ・入社後：PHP, Java

  3. #phperkaigi ©2020 RAKUS Co., Ltd. 入社研修終えての最初の数ヶ月 軽微な修正、小規模な実装では 特にコードレビューで大きな問題は なかった

  4. #phperkaigi ©2020 RAKUS Co., Ltd. ですが・・・

  5. #phperkaigi ©2020 RAKUS Co., Ltd. 初めての規模の大きな実装 コードレビューの指摘：約100件 開発期間 全体：3人月 担当：2人月

    指摘は 約8割が保守性に関する指摘

  6. #phperkaigi ©2020 RAKUS Co., Ltd. うわっ・・・私のコード品質、 低すぎ・・・？

  7. #phperkaigi ©2020 RAKUS Co., Ltd. 主な指摘 ・不要コード、デバッグコードの削除忘れ ・SQLで必要のないカラムの取得 ・意味が理解しづらいメソッド名、変数名、コメント ・タイポ

    その他には...

  8. #phperkaigi ©2020 RAKUS Co., Ltd. 保守性に関する指摘 ー 例１ ー 指摘：

    条件が分かりずらい 修正後 修正前

  9. #phperkaigi ©2020 RAKUS Co., Ltd. 保守性に関する指摘 ー 例２ ー 指摘：

    ・何をしているか読み取れない ・DOM構成が変わったときに 対応しずらい 修正後 修正前

  10. #phperkaigi ©2020 RAKUS Co., Ltd. 指摘が出ないように 対策を考える

  11. #phperkaigi ©2020 RAKUS Co., Ltd. 対策 ・コミット前に実装したコード全体を見直す ・前回の指摘から セルフチェックリストを作り 実装前と後にチェックする

    チェックリスト

  12. #phperkaigi ©2020 RAKUS Co., Ltd. セルフチェックを実施した結果 コードレビューの指摘 30件 開発期間 全体：1.5人月

    担当：0.5人月 あまり減っていない… → なぜ…？ 自分だけではわからない…

  13. #phperkaigi ©2020 RAKUS Co., Ltd. 悩みを相談した結果 先輩とペアプロを実施することに

  14. #phperkaigi ©2020 RAKUS Co., Ltd. ペアプロを実施した結果 コードレビューの指摘 10件 開発期間 全体：3人月

    担当：1.5人月 → 大きく減少！！！ ぺアプロの良かったところ ・実装しつつ質問できる ・他の人の視点を知ることができた

  15. #phperkaigi ©2020 RAKUS Co., Ltd. 1人では気づけず、ペアプロで気づいたこと ・既存コードやそのコピペなどへのコメントの追加 ・TODOコメントなどを使って気になったところにメモをする ・具体的な処理を書く前にメソッドやコメントのみを書いて、 意図が分かりやすくなっているかチェックする

  16. #phperkaigi ©2020 RAKUS Co., Ltd. 改善を振り返って ・入社するまでコードレビューを受けることがなかったので自身 のコード品質が良くないことに気づいてなかった。 ・自分だけでなんとかしようとしたけど失敗した。 ・ペアプロを行うことで

    他の人の視点を取り入れることができ、 視点を広げることができた

  17. #phperkaigi ©2020 RAKUS Co., Ltd. 改善後も品質向上のために実施していること ・実装時には、既存コードにコメントを追加しコード全体を 把握する ・デイリーのミーティング時に実装難易度が高い所を相談し、 モブプロ的に実装を進める

  18. #phperkaigi ©2020 RAKUS Co., Ltd. 自分で解決できないときは 皆さんもペアプロやってみましょう！！