新聞のようにコードを書く
新聞のようにコードを書くことで秩序のある書式を保つことができるという論があります。
以下がポイントです。
クラス名や関数名をみればそのファイルが何をするコードが書かれているかがわかる(見出し)ファイルの上部に ...
コードにコメントをつけることがダメなケース 更新が追いついていない
例えば以下のようなコメントがあったとします。
//ここでのurlは、テスト環境では 本番環境では です。補足のため非常に親切なように見えますが、このような情報をコードに埋め込んでしまうと、いざurlが変わったときにコメン ...
関数の引数の数について
引数の数は少ないに越したことはないですが、以下は個人的な基準です。
引数なしが理想引数がないのが一番シンプルです。
しかし、動的に結果が変わるのがプログラムなので、このケースはあまりない気がします。
関数の引数がbooleanだった場合は気をつけよう
関数の引数がフラグ(boolean)だった場合は、その関数をリファクタリングするチャンスかもしれません。
というのも、フラグを引数にとっているケースでは、そのフラグによって処理を分岐している可能性があるからです。
1文字の変数がもたらすストレス
forループや短い関数内で1文字の変数が使われることがあります。
これらの変数がどのように可読性に影響するのかを考えてみました。
この変数が非常に狭い範囲ででてくるのであれば、そこまで読む時にストレスは感じない ...
論理的凝集のサンプルを考えてみた
以下の getProductDetail は商品の名前と価格を返す関数です。
apple: 100
const getProductDetail = (product) => { cons ...
My Strange Code#4 関数のパラメータの改善と関数の抽出(JavaScript)
自分が過去に書いたコードを見返してみておかしなコードを直してみます。
今回のコードは以下です。
const setSearchParam = (current, object) => { let result ...
My Strange Code#2 一時変数を使わずにすっきり書いてみる(JavaScript)
需要がないとは思いますが、自分が過去に書いたコードを見返してみておかしなコードを直してみます。
一時変数をなくしてみる以下のコードはそこまでおかしくありません。
const getSelectedOrders = l ...
My Strange Code#1 map の使い方がおかしい(JavaScript)
需要がないとは思いますが、自分が過去に書いたコードを見返してみておかしなコードを直してみます。
商品の数量算出処理以下は商品の処理数を算出する関数です。
const getProcessedNum = list =& ...
コードを関数やメソッドに切り出す時の基準について
長くて理解が難しいコードを関数やメソッドに切り出すことはとても重要なことです。
しかし、どのような場合に切り出すかは人それぞれで、判断が難しいこともあります。
私なりの基準ですが、一行一行読まないと目的が分から ...