はじめに
こんにちは、私はエンジニアとしての冒険は3年目！
転職も1度し、これまで2社の会社を経験したよ！
共通指摘事項
さて、現在の仕事では、新人やインターン生のコードレビューも任されてます。
その中で、よくある指摘点をここにまとめてみました。
① コード規約に関する指摘:
JavaDocのコメント、命名規則など、自社ルールに則ったこと。
② ロジックの流れに関するダメ出し:
Nullチェックやエラーハンドリングなど。
※個人的にprintStackTrace()でエラーログ終了だと、保守性悪い気がしてるのですが・・
③ 共通化:
メソッド化、クラス化、リファクタリングの重要性。
④ 外部ライブラリの使用:
動作が保証されているメソッドの使用。
⑤ 例外処理:
後続処理への影響や例外の投げ方。
まとめ
これら全てをカバーできる新人は珍しいですし、私も未だに指摘されます。
ですが、指摘する側も正直めんどくさい・・
最後に
レビュワーの皆さん、優しくしてくださいね。
そして新人の皆さん、この記事の内容を実装時にぜひ参考にしてみてください！