prefer-specific-method

より特化した関数を優先する

同じ結果が得られる方法が複数ある場合は、 やりたいことに最も特化した関数を選んでください。

特化した関数は名前そのものが意図を表すので、 読み手が「この比較は何のためか」を推測せずに済みます。 汎用的な関数を条件式と組み合わせて目的を表現すると、 意図が式の形に埋もれて読みにくくなります。

これはavoid-forでforを高抽象度の関数に置き換えるのと同じ発想を、 関数同士の選択にも広げたものです。

存在するかどうかを調べる

要素が含まれるかを知りたいだけなら、 位置を返すindexOfではなく、 真偽値を返すincludesを使います。

// 避ける
if (items.indexOf(target) !== -1) {
}

// 良い
if (items.includes(target)) {
}

includesは意図が明確なだけでなく、 NaNを正しく見つけられるという正確性の利点もあります。 indexOfは厳密等価(===)で比較するためNaNを見つけられませんが、 includesはSameValueZeroで比較するためNaNも見つけられます。

[NaN].indexOf(NaN); // -1 (見つからない)
[NaN].includes(NaN); // true

条件を満たす要素の有無を調べる

絞り込んだ結果の件数を見るのではなく、 someやeveryで真偽を直接求めます。

// 避ける
if (0 < users.filter((user) => user.active).length) {
}

// 良い
if (users.some((user) => user.active)) {
}

someは最初に条件を満たした時点で打ち切るので、 全件を絞り込むfilterより無駄がありません。 「すべてが条件を満たすか」はeveryを使います。

条件を満たす最初の要素を取り出す

絞り込んでから先頭を取るのではなく、 findで直接取り出します。

// 避ける
const found = users.filter((user) => user.id === targetId)[0];

// 良い
const found = users.find((user) => user.id === targetId);

最後の要素が欲しい場合はfindLast、 位置が欲しい場合はfindIndexやfindLastIndexを使います。

変換してから平坦化する

mapしてからflatするのではなく、 flatMapを使います。

// 避ける
const tags = posts.map((post) => post.tags).flat();

// 良い
const tags = posts.flatMap((post) => post.tags);

末尾や負の位置の要素にアクセスする

lengthを使った添字計算ではなく、 atを使います。

// 避ける
const last = items[items.length - 1];

// 良い
const last = items.at(-1);

atは文字列にもあります。

文字列の前方一致と後方一致を調べる

位置を計算するのではなく、 専用のメソッドを使います。

// 避ける
if (path.indexOf("/etc/") === 0) {
}
if (name.lastIndexOf("hosts") === name.length - "hosts".length) {
}

// 良い
if (path.startsWith("/etc/")) {
}
if (name.endsWith("hosts")) {
}

文字列に部分文字列が含まれるかも、 indexOfではなくincludesで調べます。

文字列をすべて置換する

グローバルフラグ付きの正規表現ではなく、 replaceAllで意図を示せます。

// 避ける
const escaped = text.replace(/&/g, "&");

// 良い
const escaped = text.replaceAll("&", "&");