LoginSignup
1

More than 3 years have passed since last update.

現場にコードレビューを定着させるためにやるべきこと

Last updated at Posted at 2019-04-06

概要

コードレビューに否定的な現場の話をYYPHPで出したところ、改善策をいただいたので残します。

コードレビューをしない理由

  • レビューは時間がかかるからコストの無駄

時間がかかるからコストの無駄と思われてしまう原因

  • レビューに回す時のコード量が多すぎる

対策

  • PRの粒度を小さくする
  • コミットの粒度を小さくする
  • 誰が何をやってるかタスクを明確化する
  • PRテンプレートを決めてどういうPRなのかをなわかる状態にしておく

そもそもコードレビューは本当に必要なのか

  • やばいコード書く人は普通にいるからコードレビューなりで止めるしか無い

やばいコードとは

  • なんでもセッションにデータを突っ込むとか
  • 1クラス1万行のコードとか

オススメの書籍

レガシーコード改善ガイド

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
1