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

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

Avatar for don don
March 31, 2021
Programming
1.6k
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.5k
レガシーシステムへのPHPStan導入から半年での課題と効果
Avatar for don bosshawk
1
2.3k
息の長いサービスの 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.3k
[おすすめの技術書 LT会 - vol.2] 体系的に学ぶ安全なWebアプリケーションの作り方
Avatar for don bosshawk
0
21k

Other Decks in Programming

See All in Programming
琵琶湖の水は止められてもNet--HTTPのリトライは止められない / You might be able to stop the water flow of Lake Biwa but you can't stop Net::HTTP retries
Avatar for luccafort luccafort
PRO
0
650
仕様駆動開発の消費期限
Avatar for watany watany
20
7.9k
型も通る、synthも通る、それでも危ない 〜AIのCDKの権限とコストを機械で検証する〜 / It Passes Type Checks, It Passes Synth Checks, but It’s Still Risky — Automatically Verifying Permissions and Costs in AI’s CDK —
Avatar for shiro seike seike460
PRO
1
550
メールのエイリアス機能を履き違えない
Avatar for 船長（せんちょう） isshinfunada
0
230
源内ハンズオン概要編
Avatar for Hideyuki Nagata hideg
0
160
これって Effect でできたのでは? / TSKaigi Mashup Kansai #2
Avatar for Susisu susisu
0
150
今さら聞けない .NET CLI
Avatar for tkym htkym
0
190
VibeCodingからAgenticWorkflowへ
Avatar for starfish719 starfish719
0
380
自動化したのに回らない テスト運用の壁 ―AI時代の品質責任と生産性
Avatar for Masahiko Funaki(舟木 将彦) mfunaki
0
120
React本体のコードリーディング
Avatar for high-g high_g_engineer
1
140
使いながら育てる Claude Code — 開発フローの1コマンド化 × 繰り返し指摘の自動仕組み化
Avatar for endo shiki shiki_kakaku
0
1.7k
そこに3びきプロダクトがいるじゃろう——生成AI時代における“価値が届かない理由”の構造
Avatar for kosuke T kosuket
0
460

Featured

See All Featured
How to audit for AI Accessibility on your Front & Back End
Avatar for davetheseo davetheseo
0
490
Large-scale JavaScript Application Architecture
Avatar for Addy Osmani addyosmani
515
110k
I Don’t Have Time: Getting Over the Fear to Launch Your Podcast
Avatar for Joe Casabona jcasabona
34
2.8k
How to Talk to Developers About Accessibility
Avatar for Jason CranfordTeague jct
2
500
Testing 201, or: Great Expectations
Avatar for Joseph Mastey jmmastey
46
8.2k
Being A Developer After 40
Avatar for Adrian Kosmaczewski akosma
91
590k
Why Mistakes Are the Best Teachers: Turning Failure into a Pathway for Growth
Avatar for Umar Saidu Auna auna
0
200
[RailsConf 2023 Opening Keynote] The Magic of Rails
Avatar for Eileen M. Uchitelle eileencodes
31
10k
Leo the Paperboy
Avatar for Maya Tellez mayatellez
8
2.1k
Amusing Abliteration
Avatar for ianozsvald ianozsvald
1
240
Bioeconomy Workshop: Dr. Julius Ecuru, Opportunities for a Bioeconomy in West Africa
Avatar for AKADEMIYA2063 akademiya2063
PRO
1
210
Stewardship and Sustainability of Urban and Community Forests
Avatar for Eric Wiseman pwiseman
0
450

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