LoginSignup
0
0

More than 1 year has passed since last update.

【LevelUp】先輩から学ぶレビュー術

Posted at

先輩エンジニアになると
『レビュー業務』が発生します💦

普通にゴリゴリOSSとかに参加して
バチバチトークするぜ!!って人以外は
『自分以外のソースに触れ、かつ指摘する』なんてことはしないです。

ってことで
誰かの指針になれば

美しいか 美しくないか ただそれだけだ

汚いソース=だいたいバグってます。。。

間違っているところは
本来試験で確認するところなので
そこまで力を入れてレビューする必要はない。
(※ちゃんと試験をしているのが前提)

ぱっとみなんか汚いと思ったら絶対他にいい方法がある

だいたい先人たちが同じことやろうとしてたはずなので
既に標準関数にあります。探しましょう。

「めちゃくちゃ配列操作してる = ライブラリ探す」
「ifが大量 = クラス構成見直せは改善する」

GitHubはやはり優秀

クローズしたプルリクでたくさんコメントついてそうなのを見つけて
内容を確認し、同じ指摘うけないようにする!!

0
0
0

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
0
0