たみぼうず

以前種から撒いたひまわりは今は２メートルを超えるほど立派になりました。

しかし、この前強風が吹いた時に、ものすごく勢いでひまわりがしなって折れそうになっていたので、ずっとそばで寄り添っていました。

今後も雨の中 ...

例えば以下のようなコメントがあったとします。

//ここでのurlは、テスト環境では 本番環境では です。

補足のため非常に親切なように見えますが、このような情報をコードに埋め込んでしまうと、いざurlが変わったときにコメン ...

重複がなく、命名も明確でわかりやすい コードを書くことは非常に重要ですが、どのタイミングでそのコードをかけばいいのでしょうか。

できれば、はじめから書けると良いですが、まだシステム全体が把握できていない初期の段階ではこれら ...

try/catchブロック内の処理は、それぞれ関数として抽出すると良いとされています。

try{//　ここに細かい処理を書かかない。関数に抽出してそれを呼び出す calcPrice()}catch(e){//　上記と同様 rep ...

以下のような関数があるとします。

const set = (obj, property, value) => {//objにpropertyがあればvalueをセットしてtrueを返す。なければfalseを返す}

この ...

関数の引数がどうしても3つや4つになる場合は、ひとつのオブジェクトして渡すことを考えてみます。

const myFunc = (hoge, foo, bar) => {//do something}const data = ...

関数の引数が2つ以上だと順序を気にしないといけない、というデメリットがあります。

以下の関数は name と file という二つの引数をとります。

const getHtml = (name, file) => ...

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

引数なしが理想

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

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

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

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

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

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

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