プログラミングのリファクタリングとは？3つのコード例で解説

Mirai

リファクタリングって、どこから手をつければいいんでしょうか？

Zetto

コードの外から見た動作を変えずに、内部の構造を整理することですね。やり方さえ知れば、今日から少しずつ実践できますよ。

この記事では、リファクタリングの意味・やり方・現場での実践知識までを解説します。

 本記事の専門性
現役エンジニアのZettoです。フリーランスエンジニアとして業務システム開発を複数経験しています。現場でリファクタリングの重要性を身をもって学んできました。

この記事を読めば、リファクタリングの意味・具体的なやり方・現場で使える実践知識が一通りわかります。

ぜひ参考にしてみてください。

リファクタリングとは

リファクタリングとは、「プログラムの動作を変えずに、コードの内部構造を整理すること」です。

この章では、リファクタリングの基礎を押さえていきます。

それぞれ解説します。

リファクタリングの定義と目的

リファクタリングの定義はシンプルです。

「外から見た動作は変えずに、コードの中身を読みやすく・保守しやすく整理すること」

たとえば、動いているコードの変数名が`x`や`tmp`のような意味不明な名前になっていたとします。これを`userName`や`totalPrice`のような意味のある名前に変えるだけで、立派なリファクタリングです。

目的は主に2つです。

「動くコード」と「良いコード」は別物です。リファクタリングは、動くだけのコードを良いコードに育てる作業といえます。

バグ修正・機能追加との違い

リファクタリングと混同しやすいのが、バグ修正と機能追加です。

整理すると次のようになります。

この違いは現場でも重要です。リファクタリング中に「ついでにバグを直そう」とすると、何が原因でテストが落ちたかわからなくなります。

作業を混ぜないことが、安全なリファクタリングの第一歩です。

リファクタリングのメリット

リファクタリングを定期的に行うことで、開発全体がスムーズになります。

主なメリットは次の通りです。

プログラミングの現場では「後で直せばいい」が積み重なり、手がつけられない状態になることがよくあります。リファクタリングはその予防策でもあるんですよね。

Zetto

リファクタリングって、最初は「今すぐ必要？」と感じるかもしれません。でも、半年後・1年後のコードの状態を想像すると、やっておいた方がいいと体感でわかってきますよ。現場で汚いコードに向き合うたびに、「リファクタリングの習慣があれば…」と思うことが正直よくあります。

リファクタリングの具体的なやり方

ここからは、実際にどうやってリファクタリングするかを解説します。

ひとつずつ見ていきましょう。

変数名・関数名を意味のある名前にする

一番取り組みやすいリファクタリングが、命名の改善です。

ブラウザでも動かしやすい JavaScript で解説しますね。

リファクタリング前（悪い例）

function calc(a, b) {
  const x = a * b;
  const y = x * 0.1;
  return x + y;
}

`calc`・`a`・`b`・`x`・`y`…何を計算しているのか、コードを読んでも全くわかりません。

リファクタリング後（良い例）

function calculatePriceWithTax(unitPrice, quantity) {
  const subtotal = unitPrice * quantity;
  const taxAmount = subtotal * 0.1;
  return subtotal + taxAmount;
}

関数名を`calculatePriceWithTax`（税込み価格を計算する）にするだけで、コードの意味が一目でわかります。変数名も`unitPrice`（単価）・`quantity`（数量）・`subtotal`（小計）・`taxAmount`（税額）と整理しました。

命名ルールのポイントをまとめます。

Zetto

命名は地味に見えて、実はコードの品質を大きく左右すると感じています。良い命名ができているコードは、コメントがほぼなくても意図が伝わります。逆に命名が悪いと、コメントで補おうとして余計ごちゃごちゃになるんですよね。

重複コードを関数にまとめる

同じ処理が複数の場所に書かれているコードは、リファクタリングの一番のターゲットです。

リファクタリング前（悪い例）

// 商品Aの税込み価格を表示
const priceA = 1000;
const taxA = priceA * 0.1;
console.log(priceA + taxA);

// 商品Bの税込み価格を表示
const priceB = 2000;
const taxB = priceB * 0.1;
console.log(priceB + taxB);

// 商品Cの税込み価格を表示
const priceC = 3000;
const taxC = priceC * 0.1;
console.log(priceC + taxC);

同じ計算ロジックが3箇所に重複しています。税率が変わったとき、3箇所全部直す必要があります。これが10箇所・100箇所になると修正漏れが起きます。

リファクタリング後（良い例）

function displayPriceWithTax(price) {
  const taxIncludedPrice = price + price * 0.1;
  console.log(taxIncludedPrice);
}

displayPriceWithTax(1000);
displayPriceWithTax(2000);
displayPriceWithTax(3000);

関数にまとめることで、税率が変わっても1箇所の修正で済みます。DRY原則（Don’t Repeat Yourself＝同じことを繰り返すな）と呼ばれる、プログラミングの基本的な考え方です。

プログラミングの関数についての基礎知識は、以下の記事で解説しています。関数がまだよくわからない方はあわせて読んでみてください。

あわせて読みたい

プログラミングの関数とは？引数・戻り値・使い方を基礎から解説 プログラミングで「関数」ってよく出てくるけど、何のことなのかよくわからないんですが……。 関数は「処理をひとまとめにした箱」のようなものですね。一度作っておけば…

複雑な条件分岐を整理する

条件分岐（if文）が深くなりすぎているコードも、よくあるリファクタリング対象です。

リファクタリング前（悪い例）

function getDiscount(user) {
  if (user.isLoggedIn) {
    if (user.isMember) {
      if (user.purchaseCount >= 10) {
        return 0.2;
      } else {
        return 0.1;
      }
    } else {
      return 0.05;
    }
  } else {
    return 0;
  }
}

ネスト（入れ子）が深くなると、どの`if`がどの`else`に対応しているか、追うのが大変になります。

リファクタリング後（良い例）

function getDiscount(user) {
  if (!user.isLoggedIn) return 0;
  if (!user.isMember) return 0.05;
  if (user.purchaseCount >= 10) return 0.2;
  return 0.1;
}

条件を満たさない場合に早期リターン（処理を早めに抜ける）することで、ネストをなくしました。コードの行数も減り、一目でロジックが把握できます。この手法は「早期リターン」または「ガード節」と呼ばれます。

多重ループを回避する

ループの中にループを入れる「多重ループ」は、コードが複雑になるだけでなく処理が遅くなる原因にもなります。

リファクタリング前（悪い例）

const orders = [
  { userId: 1, amount: 1000 },
  { userId: 2, amount: 2000 },
  { userId: 1, amount: 500 },
];

const users = [
  { id: 1, name: "田中" },
  { id: 2, name: "鈴木" },
];

// ユーザーごとの合計金額を計算（二重ループ）
for (const user of users) {
  let total = 0;
  for (const order of orders) {
    if (order.userId === user.id) {
      total += order.amount;
    }
  }
  console.log(`${user.name}: ${total}円`);
}

リファクタリング後（良い例）

const orders = [
  { userId: 1, amount: 1000 },
  { userId: 2, amount: 2000 },
  { userId: 1, amount: 500 },
];

const users = [
  { id: 1, name: "田中" },
  { id: 2, name: "鈴木" },
];

// まず注文をユーザーIDで集計しておく
const totalByUser = {};
for (const order of orders) {
  totalByUser[order.userId] = (totalByUser[order.userId] || 0) + order.amount;
}

// ユーザー一覧と組み合わせて表示
for (const user of users) {
  const total = totalByUser[user.id] || 0;
  console.log(`${user.name}: ${total}円`);
}

先に注文データを集計しておくことで、二重ループをなくしています。データ量が増えるほど、この差は大きくなります。

Zetto

多重ループって、小規模なデータだと全然気にならないんですよね。でも実務でデータ件数が増えてきたとき、突然パフォーマンスが落ちて初めて「あのループが原因か」となることがあります。書くときから意識する習慣が大事だなと感じています。

リファクタリングをするタイミング

リファクタリングをいつやるかは、現場でよく話題になります。

一般的に言われているタイミングは次の3つです。

逆に、リファクタリングに向かないタイミングもあります。

取り組む前の注意点

リファクタリングに入る前に、確認しておくべき点があります。

重要なのは、「テストが通っている状態を必ず確認してから始める」ことです。

テストがあれば、リファクタリング後に同じテストを走らせることで「動作が変わっていないか」を確認できます。テストなしでリファクタリングを進めると、変えたつもりがない動作が変わっていても気づけません。

また、次の点も意識しておくといいです。

Zetto

現場に入りたてのころ、リファクタリングしながらバグ修正もして、テストも通っていない状態で「よし、完了」とコミットしてしまったことがあります。案の定、後からどこで何が壊れたかわからなくなって、かなり苦労しました。小さく・安全にがリファクタリングの鉄則だと、その経験から学びましたね。

現場エンジニアが知っておくべき実践知識

コードの書き方だけでなく、現場での立ち回りにも関わる知識を解説します。

具体的に解説します。

技術的負債を後回しにするとどうなるか

技術的負債（ぎじゅつてきふさい）とは、「いま急いで書いた粗いコード」が将来の開発コストを押し上げる状態を、借金に例えた言葉です。

技術的負債は放置すると雪だるま式に膨らみます。

具体的にどうなるかを挙げると、次のような状態になっていきます。

僕が経験した現場でも、過去に急ぎで作られたコードに後から触れると、変数の意味もわからず、コメントもなく、テストも存在しない状態になっていることがありました。調査と修正で想定の3倍の時間がかかったこともあります。

リファクタリングは「綺麗にしたい」という趣味の話ではなく、開発速度を維持するための投資です。

リファクタリングとテストコードの関係

リファクタリングとテストコードはセットで考えるのが基本です。

テストコードとは、「このコードが正しく動くか自動で確認するプログラム」のことです。たとえば「税込み価格を計算する関数に1000円を渡したら1100円が返ってくること」をコードで検証する仕組みです。

テストがある状態でリファクタリングをすると、次のメリットがあります。

逆にテストがない状態でリファクタリングをすると、「変えた後に正しく動いているかどうか」を手動で全部確認するしかありません。規模が大きくなるほど現実的でなくなります。

テストコードが書けない状況であれば、まずスコープを小さく絞り、「この関数の動作だけ確認できればいい」というレベルから始めるのが現実的です。

チーム開発でリファクタリングを提案する方法

チーム開発では、自分がリファクタリングしたいと思っても、勝手にやるとトラブルになることがあります。

理由は2つあります。

チームでリファクタリングを進めるときは、次の流れが安全です。

コードの話は感情的になりやすいので、「誰のコードが悪い」ではなく「このコードをこう改善するとチームの開発速度が上がる」という前向きな伝え方が大切です。

Zetto

リファクタリングの提案は、言い方一つで受け取られ方が全然違います。「このコードは読みにくい」という言い方より「この部分を整理すると次の機能追加がスムーズになる」という言い方の方が、チームに受け入れられやすいですね。

初学者が今日から始められる練習ステップ

「リファクタリングって、実務じゃないとできないの？」と思う方もいるかもしれません。でも、初学者でも今日から練習できます。

おすすめのステップは次の通りです。

最初から完璧を目指さなくて大丈夫です。命名を1つ変えるだけでもリファクタリングです。スポーツと同じで、毎日の小さな積み重ねが力になります。

Zetto

僕も最初のうちは「リファクタリング」という言葉すら意識せずにコードを書いていました。でも意識し始めてから、過去のコードを見返して「これは命名が雑だった」「ここは関数にできた」と気づく習慣がついてきて、コードの質が上がっていった感覚がありますね。

プログラミング学習をどこから始めればいいか迷っている方は、以下の記事もあわせて読んでみてください。学習の進め方を体系的にまとめています。

あわせて読みたい

【2026年】プログラミング学習の始め方ロードマップ【現役エンジニアが解説】 プログラミング学習って何から始めたらいいかわからない。ロードマップとか言われても、結局どの順番で何を学べばいいの？ 今回は初心者向けに、プログラミング学習の始…

よくある質問

よくある質問と回答をまとめました。

リファクタリングはどのくらいの頻度でやるべきですか？

「都度やる」が基本の答えです。

まとめてやろうとすると、対象が大きくなりすぎてリスクが高くなります。

機能追加・バグ修正のタイミングで「触ったついでに周辺を整理する」という習慣が、一番現実的で効果的です。

プロジェクトによっては、定期的に「リファクタリング週間」を設けているチームもあります。しかしそれが難しい場合は、コードレビューの中でリファクタリングの指摘・改善を行うだけでも積み重ねになります。

「いつかまとめてやろう」は、ほぼやらずに終わります。小さく・こまめにが習慣化のコツです。

リファクタリングしてバグが増えた場合の対処法

まず、落ち着いてテストを走らせることです。

バグが増えた場合は、次の手順で対処します。

「まとめて変えて一気にテスト」というやり方だと、どの変更がバグの原因かわかりにくくなります。1つ変えたら1回テストを走らせる細かいサイクルを守ることで、問題の特定が格段に楽になります。

バグが増えたこと自体は失敗ではなく、テストによって早期発見できたということです。

問題を後工程まで持ち越さずに済んだと前向きに捉えてください。

リファクタリングと「コードの書き直し」の違い

一言で言うと、「動作を変えるかどうか」の違いです。

リファクタリングは動作を変えません。変えるのはコードの中身（構造・命名・重複など）だけです。

一方、コードの書き直しは「今の動作も設計も全部捨てて、ゼロから作る」行為です。書き直しは動作が変わる可能性があり、テストも全部やり直す必要があります。

現場では、「リファクタリングするつもりで始めたのに、いつの間にか書き直しになっていた」というケースが起きやすいです。リファクタリングの範囲はあくまで「動作を変えない整理」に留める意識が大切です。書き直しが必要な場合は、チームで合意を取ってから計画的に進める方が安全です。

小さな改善の積み重ねがコードの質を変える

この記事では、プログラミングのリファクタリングについて解説しました。

ポイントを振り返ります。

リファクタリングは、一気にやるものではありません。コードを触るたびに「この変数名はわかりやすいか？」「同じ処理が重複していないか？」と少しずつ意識していくだけで、コードの質は確実に上がっていきます。

まずは「変数名を1つ直してみる」から始めてみてください。

Mirai

命名を直したり関数にまとめたりするだけで、コードってこんなに読みやすくなるんですね。今すぐ過去のコードを見返してみます。

Zetto

その感覚が大事ですよ。過去の自分のコードを「読み返してみる」ことがリファクタリングの第一歩ですね。続けていくうちに、書くときから意識できるようになっていきます。

プログラミングの考え方全般についてさらに深めたい方は、以下の記事もあわせて読んでみてください。現場で使える思考法をまとめています。

あわせて読みたい

現場で通用するプログラミングの考え方【本質的な思考法】 プログラミングの文法は覚えたけど、いざ自分で何か作ろうとすると手が止まる。どうやって考えればいいんですか？ 今回はプログラミングの考え方を磨く具体的な方法を解…

また、リファクタリングが自然にできるレベルを目指すなら、スキルを体系的に磨く環境も重要です。プログラミングスクールの選び方は以下の記事で詳しく解説しています。

あわせて読みたい

プログラミングスクールの選び方5ステップ【現役エンジニアが解説】 スクールって多すぎてどれを選べばいいかわからない…。高い受講料を払って失敗したくないし、本当に転職できるのか不安で。 今回はプログラミングスクールの選び方を5ス…