コードレビュー, プログラミング入門

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

JavaScript, コードレビュー

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

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

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

コードレビュー

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

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

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

...

JavaScript, コードレビュー, プログラミング入門

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

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

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

JavaScript, コードレビュー, プログラミング入門

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

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