リファクタリングの進め方

リファクタリングとは何か

リファクタリングとは、外部から見た振る舞いを変えずに、 内部の構造だけを改善することです。マーティン・ファウラーの定義では 「観察可能な振る舞いを保ったままソフトウェアの内部構造を改善する規律だった手法」とされます。

安全に進めるための原則

代表的なリファクタリング手法

「関数の抽出(Extract Function)」は最も基本的で頻繁に使う手法の1つです。 長い処理の一部に名前をつけて独立させることで、可読性と再利用性が向上します。

// before: 1つの関数に責任が混在している
function printInvoice(order) {
  let total = 0;
  for (const item of order.items) {
    total += item.price * item.quantity;
  }
  const tax = total * 0.1;
  const finalAmount = total + tax;

  console.log(`小計: ${total}円`);
  console.log(`税額: ${tax}円`);
  console.log(`合計: ${finalAmount}円`);
}

// after: 計算ロジックを関数として抽出する
function calculateSubtotal(items) {
  return items.reduce((sum, item) => sum + item.price * item.quantity, 0);
}

function calculateTax(subtotal, rate = 0.1) {
  return subtotal * rate;
}

function printInvoice(order) {
  const subtotal = calculateSubtotal(order.items);
  const tax = calculateTax(subtotal);
  const total = subtotal + tax;

  console.log(`小計: ${subtotal}円`);
  console.log(`税額: ${tax}円`);
  console.log(`合計: ${total}円`);
}

抽出したcalculateSubtotalやcalculateTaxは 単体テストの対象にしやすく、他の場所でも再利用できます。

いつリファクタリングするか

リファクタリングは単独のタスクとして予定するよりも、「ボーイスカウトルール」のように、 触れたコードを少しだけきれいにしてから離れる習慣として実践するのが効果的です。

• • 新機能を追加する前に、その周辺コードを整理してから着手する
• • コードレビューで指摘された構造上の問題を、次の変更のタイミングで解消する
• • 大規模な書き直しは別途計画を立て、段階的に進める