【リーダブルコード】その条件式、冗長では？

久しぶりにブログを書く時間が取れました、岩崎真也です。

true と比較する条件式のケース

let is_initialized = false

...

if (is_initialized === true) {
  ...
}

プロジェクトによっては「わかりにくいので true と比較する式を書くように」という方針もあるかもしれません。

でもこれは冗長です。以下でいいでしょう。

let is_initialized = false

...

if (is_initialized) {
  ...
}

let count = 0

...

const exists = (count > 0) ? true : false

これも意味がありません。以下でいいでしょう。

let count = 0

...

const exists = (count > 0)

こんなことはないと思いたいですが、コードレビューをすると結構しっかり見かけます。無駄を削ぎ落としましょう。

弊社では、「さらに高みを目指したいエンジニア」を育てる環境を用意しています。

株式会社 #sinProject CEO/CTO

Twitterアプリ1位 #Tweecha2 企画制作

サービス・システム設計開発、コンサルタント

#kimurakan ピアノ弾き語り