コードレビュー

1. なぜ「ネスト」は読み手を疲れさせるのか？

コードのネストが深くなると、私たちの脳内では「スタック（記憶の積み上げ）」が発生します。

「このif文が成立したってことは、さっきの条件Aは真で、その前の条件Bも真だったな……」

AIにロジックを書かせると、以下のようなコードがよく生成されます。

//😱 何を判定しているのか、一瞬で理解できますか？if (user.age >= 18 && user.hasAgreed && !us ...

クラス設計において、ある値を「プロパティ（フィールド）」として保持すべきか、それとも「メソッドの引数」として渡すべきか。この選択に、明確な正解がないように思えることがあります。

しかし、ひとつの強力な判断材料があります。そ ...

AIエージェントが数秒でプルリクエスト（PR）を作成する時代。私たちの仕事は「コードを書くこと」から、AIが生成した膨大なアウトプットを「精査し、責任を持つこと」へと変わりつつあります。

非エンジニアにはできないがエンジニ ...

テクノロジーの歴史は、人間の「苦労」を「外部化」してきた歴史でもあります。

インターネットは『記憶』をクラウドへ開放し、AIは今、私たちの『思考と実装』を自動化しようとしています。ツールが変われば、そこで戦うエンジニアに求 ...

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

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

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

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

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

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

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

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

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

...

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

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

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