コードレビュー

Qiitaでコードレビューで嫌われる人の特徴、というような記事があがっていたので、それに関連してコードレビューを受ける側のマインドとして重要だと思うことを書きます。

コードレビューを受けると、自分が気がつかなかったミスや、 ...

メソッド、関数が長すぎないか。分割できないか
命名が適切か
変数のスコープが広すぎないか
ファイルの行数が多すぎないか
処理の責務が適切か
実装とコメントに解離がないか
undefinedやnullになる ...

getElementById は null を返すことがあります。

引数で指定したIDを持つ要素が見つからない場合にnullが返ります。

指定したIDが間違っていた場合や、対象の要素がDOMに追加されるタイミン ...

理解できないコードがあるときは素直に聞いてみる

コードレビューをしていて、コードが理解できない時は、ストレートに聞いてみるようにしています。

大抵は、担当者がミスしているか自分が理解していない仕様があるかです。

...

今回もたみぼうずが書いたコードをレビューして悪いところを指摘してあげましょう。

今回のコードは以下です。

const details = getTransportDetails()const orders = get ...

エンジニアになると他の人が書いたコードをレビューすることが多々あります。

いつもこうすれば絶対に間違いない、という書き方はありませんが、ある程度チェックするポイントは決まっているので、今のうちに覚えておきましょう。 ...