新卒エンジニアがもらった今週のコードレビュー(4)

先週忘れていました、、

金土とオールでインフラの勉強していたらすっかり忘れていました、、
今週も日曜日の夕方に思い出しました、、今からサッカーなので駆け足で書いています、、

何回も同じことを聞くな！！

何回も同じことを聞いてくる人、ちゃんと聞いておいてよ！！と思いますよね。自分もそれ、してました、、
要件としては、ある条件の時に特定のクエリを追加する、というものでした。そこで自分が書いたコードは、ある条件の時に特定のクエリを追加した、よいうものです。何が悪いのかですが、その「追加したクエリによって他のクエリが必要なくなる」ということを考えていませんでした、、つまりおんなじことをDBに２回来てしまっていました、、
その要件を満たしているからOK‼️ではなくて、それによって影響する範囲を考慮することも同じくらい大事だなと思いました。

明日やろうはばかやろう

後回しにすること、よくないです。今週のレビューで一番多くもらってしまったのが、コメント部分に
//TODO: まるまる
と残していた部分についてでした。残していたのは正解なのですが、ここでいただいたレビューは、
「今は自分の担当だからわかるけど、もし何かあって他の人が担当になった時にいつ、何をすればいいのか伝わらない」
というものでした。確かに、僕の心の中には、TODO:サッカー選手になる、TODO:格闘技でチャンピョンになる、など、その当時は自分がやる気満々でもいつの間にか他の人にその夢を託して（諦めた時のかっこいい言い方）いるな、、
このレビューで、属人性をなできる限り排除する、ということを学びました、、

まとめ

元々フロントしかできなくて、消去法フロントエンジニアと名乗っていたのですが、フロントでレベルの低いレビューをもらうことがだいぶ減ってきました（はず）、、
この試みがいい影響をもたらしていると信じて、バックエンドも頑張っていきたいです。