若手・未経験エンジニアがコードレビューで指摘されがちなポイント

[@shuikeda5042]

こういうリーダブルコード見たいな内容youtubeでめちゃくちゃ需要あるぞ、、上司忙しいすぎてこういうの少ないんよ

[@暇なカイ]

huskyとかlint-stagedとか使わないと、レビュー大変だからよく指摘する内容はシステムに組み込めないか考えたりして、開発体験を良くするのも大切だよね。

[@プサイ-f1r]

最後の
・コミットログが粗い
・コミットコメントが適当
・プルリクの説明がない
の3つは、むしろベテランエンジニアがやりがちなパターンだと思いますね〜
あとは性格によるところも大きそうな気もします

[@プサイ-f1r]

加えていうならば、
・lint(rubocopなど)を通していない
・テストが書かれていない
この2つは、どちらもCIに組み込むべき内容なので、未経験がやりがちというよりかは会社の開発レベルを問われる観点だと思いますねー
CIはGoogleが発表しているDevOpsの指標でも重要項目として挙げられていますし、、

[@プサイ-f1r]

マジックナンバーやSQLパフォーマンス、ファットコントローラーはまさに未経験がやりがちなポイントだと思います

[@jojo-fz9cb]

ファットコントローラのときこの女の人頷いているけどあの説明でちゃんと理解してるのだろうか？