先輩エンジニアになると
『レビュー業務』が発生します💦
普通にゴリゴリOSSとかに参加して
バチバチトークするぜ!!って人以外は
『自分以外のソースに触れ、かつ指摘する』なんてことはしないです。
ってことで
誰かの指針になれば
美しいか 美しくないか ただそれだけだ
汚いソース=だいたいバグってます。。。
間違っているところは
本来試験で確認するところなので
そこまで力を入れてレビューする必要はない。
(※ちゃんと試験をしているのが前提)
ぱっとみなんか汚いと思ったら絶対他にいい方法がある
だいたい先人たちが同じことやろうとしてたはずなので
既に標準関数にあります。探しましょう。
「めちゃくちゃ配列操作してる = ライブラリ探す」
「ifが大量 = クラス構成見直せは改善する」
GitHubはやはり優秀
クローズしたプルリクでたくさんコメントついてそうなのを見つけて
内容を確認し、同じ指摘うけないようにする!!