引数の数は少ないに越したことはないですが、以下は個人的な基準です。

引数なしが理想

引数がないのが一番シンプルです。

しかし、動的に結果が変わるのがプログラムなので、このケースはあまりない気がします。

関数の引数がフラグ(boolean)だった場合は、その関数をリファクタリングするチャンスかもしれません。

というのも、フラグを引数にとっているケースでは、そのフラグによって処理を分岐している可能性があるからです。

すでにあるデータ(配列)に対して新しいデータを追加する処理があるとします。

あるシステムではaddXXXXというような命名をしていました。

例えば、addCustomer・addHistoryなどです。 ...

ひとつの関数の中で抽象度が異なる処理があるのは誤解を招くもとになります。

例えば、isAdminがtrueかfalseで分岐する処理が関数の中にあるとします。

const hoge = () => { if(i ...

Array(x)をスプレッド演算子を使って配列に変換してから、map を呼び出す必要がある。

.map((_, i) => `tami${i}`)　//

if文の中にさらにif文があり、その中にさらにif文がある、、、というような条件文の地獄はよく発生するコードだと思います。

条件文を書く時には、条件文のブロックを一行にできないかを考えながらやると、簡潔なコードになります。 ...

関数やメソッドに名前をつけるときは、他とは明確に異なることがわかるようにするべきです。

例えば、tamiXXXXXX のように、すべての命名の先頭に同じ文字列をくっつけると、他との区別がしづらくなります。

ta ...

以下のコマンドで可能です。

npm ls --depth=0

forループや短い関数内で1文字の変数が使われることがあります。

これらの変数がどのように可読性に影響するのかを考えてみました。

この変数が非常に狭い範囲ででてくるのであれば、そこまで読む時にストレスは感じない ...

自動入力を使い方たくないけどこの警告が出た場合は、以下を指定して無効化することで警告が解消します。

android:importantForAutofill="no"

参考