今日の開発Tips
Tips

今日の開発Tips（2025-06-17）

複雑なロジックを扱う際に、関数の引数を適切に分割することで、コードの可読性と保守性を向上させることができます。複数の引数を持つ関数は、引数の役割や意味が分かりづらくなり、バグの原因にもなりがちです。関数の引数を適切な数の小さな関数に分割することで、各関数の役割を明確にし、コード全体の見通しが良くなります。これにより、デバッグや修正も容易になります。

関数引数の分割によるコードの整理

多くの引数を持つ関数は、理解しにくく、テストもしにくいです。 引数を小さな関数に分割することで、関数の責務を明確化し、コードの可読性と保守性を向上させましょう。

例：

// 分割前
function calculateTotalPrice(quantity, price, discountRate, taxRate) {
  const discountedPrice = price * (1 - discountRate);
  const taxAmount = discountedPrice * taxRate;
  return quantity * (discountedPrice + taxAmount);
}

// 分割後
function calculateDiscountedPrice(price, discountRate) {
  return price * (1 - discountRate);
}

function calculateTaxAmount(price, taxRate) {
  return price * taxRate;
}

function calculateTotalPrice(quantity, price, discountRate, taxRate) {
  const discountedPrice = calculateDiscountedPrice(price, discountRate);
  const taxAmount = calculateTaxAmount(discountedPrice, taxRate);
  return quantity * (discountedPrice + taxAmount);
}

分割後のコードは、各関数の役割が明確になり、可読性と保守性が向上しています。 calculateTotalPrice 関数は、他の関数の結果を組み合わせて計算する役割に特化し、よりシンプルになっています。

注意点:

  • 分割しすぎると、関数呼び出しのオーバーヘッドが増える可能性があります。適切な粒度を見つけることが重要です。
  • 関数の名前を分かりやすく、役割が明確になるように命名しましょう。

おすすめポイント:

  • 関数の責務を明確化し、コードを理解しやすくする。
  • テストの容易性を向上させる。
  • バグの発生率を減らす。
  • コードの変更や保守を容易にする。
Tips一覧へ戻る