Upgrade to Pro — share decks privately, control downloads, hide ads and more …

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

Sponsored · Ship Features Fearlessly Turn features on and off without deploys. Used by thousands of Ruby developers. →
Avatar for don don
March 31, 2021
Programming
1.5k
1

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

Avatar for don

don

March 31, 2021

More Decks by don

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

Other Decks in Programming

See All in Programming
Agentic UI beyond Chats Architecture Patterns & Open Standards @ngMunich 05/2026
Avatar for Manfred Steyer manfredsteyer
PRO
0
140
AIチームを指揮するOSS「TAKT」活用術 / How to Use “TAKT,” an OSS Tool for Orchestrating AI Teams
Avatar for nrs nrslib
5
630
権限チェックの一貫性を型で守る TypeScript による多層防御
Avatar for aster-mnch (kitagawa) mnch
3
590
今さら聞けないCancellationToken
Avatar for tkym htkym
0
180
CLIであることを活かしたGitHub Copilot CLI活用術 / GitHub Copilot CLI Pro Tips & Tricks
Avatar for Naoya.i nao_mk2
1
980
oxlintはeslint/typescript-eslintを置き換えられるのか
Avatar for shoma-fujita shomafujita
2
210
技術記事、AIに書かせるか、自分で書くか？ 〜それでも私が自分の手で書く理由〜 / #QiitaConference
Avatar for Junichi Ito jnchito
2
360
Kubernetesを使わない環境にもCloud Nativeなデプロイを実現する / Enabling Cloud Native deployments without the complexity of Kubernetes
Avatar for linyows linyows
3
540
柔軟なPDFレイアウトエディタを支える型システム設計 — Discriminated UnionとConditional Typeの実践
Avatar for minako-ph minako__ph
3
580
Skillは並べた。動かなかった。契約で繋いだ。— 65個のSkillから、自走する開発サイクルへ
Avatar for じゅん junholee
0
720
20260514_its_the_context_window_stupid.pdf
Avatar for heita heita
0
1.1k
Java × distroless で 軽量なコンテナイメージを / Java on Distroless
Avatar for Daisuke Garaike contour_gara
0
150

Featured

See All Featured
Context Engineering - Making Every Token Count
Avatar for Addy Osmani addyosmani
9
900
Building Experiences: Design Systems, User Experience, and Full Site Editing
Avatar for Michelle Schulp Hunt marktimemedia
0
510
The Success of Rails: Ensuring Growth for the Next 100 Years
Avatar for Eileen M. Uchitelle eileencodes
47
8.1k
Agile Actions for Facilitating Distributed Teams - ADO2019
Avatar for Mark Kilby mkilby
0
190
Reflections from 52 weeks, 52 projects
Avatar for Jefferson Lam jeffersonlam
356
21k
The Illustrated Children's Guide to Kubernetes
Avatar for Chris Short chrisshort
51
52k
Un-Boring Meetings
Avatar for Sebastian Deterding codingconduct
0
300
The AI Revolution Will Not Be Monopolized: How open-source beats economies of scale, even for LLMs
Avatar for Ines Montani inesmontani
PRO
3
3.5k
Dealing with People You Can't Stand - Big Design 2015
Avatar for Cassini Nazir cassininazir
367
27k
Discover your Explorer Soul
Avatar for Emna emna__ayadi
2
1.1k
More Than Pixels: Becoming A User Experience Designer
Avatar for Michelle Schulp Hunt marktimemedia
3
420
A better future with KSS
Avatar for Kyle Neath kneath
240
18k

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. 自分で解決できないときは 皆さんもペアプロやってみましょう！！