プログラミング入門, リファクタリング

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

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

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

JavaScript, システム開発, プログラミング入門, リファクタリング

凝集度の論理的凝集の例

以下の getProductDetail は商品の名前と価格を返す関数です。

apple: 100

const getProductDetail = (product) => { cons ...

JavaScript, プログラミング入門, リファクタリング

自分が過去に書いたコードを見返してみておかしなコードを直してみます。

今回のコードは以下です。

const setSearchParam = (current, object) => { let result ...

JavaScript, プログラミング入門, リファクタリング

需要がないとは思いますが、自分が過去に書いたコードを見返してみておかしなコードを直してみます。

一時変数をなくしてみる

以下のコードはそこまでおかしくありません。

const getSelectedOrders = l ...

JavaScript, プログラミング入門, リファクタリング

需要がないとは思いますが、自分が過去に書いたコードを見返してみておかしなコードを直してみます。

商品の数量算出処理

以下は商品の処理数を算出する関数です。

const getProcessedNum = list =& ...

プログラミング入門, リファクタリング

長くて理解が難しいコードを関数やメソッドに切り出すことはとても重要なことです。

しかし、どのような場合に切り出すかは人それぞれで、判断が難しいこともあります。

私なりの基準ですが、一行一行読まないと目的が分から ...

プログラミング入門, リファクタリング

複雑なプログラムを書いていると、ついコメントで内容を補足したくなります。

コメントを書くこと自体が悪いわけではないですが、コメントは実装と乖離してしまうと逆に誤解を招いてしまうというデメリットがあります。

でき ...

プログラミング入門, リファクタリング

巨大なコードを一気にやろうとしない

リファクタリング対象が巨大であればあるほど、想定通りに動かない可能性が高くなります。

そして、動かなくなったときにどの辺りが原因なのかを特定することが難しくなります。

結果とし ...

JavaScript, プログラミング入門, リファクタリング

前回の課題の続きです。

課題

今回はクライアントからの依頼ではなく、これまでのコードを少しリファクタリングしてみます。

今のコードはこんな感じです。

const getRandom = (min, max) ...

プログラミング入門, リファクタリング

変数はプログラムを書く上で必要ですが、何も考えずに使っているとわかりにくいコードが生まれます。

以下、変数を使う上で気をつけるポイントです。

不要な一時変数は削除する
変数のスコープは狭くする
同じ変数を ...