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

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