Symfonyでコントローラのコードをリファクタリングする方法をやさしく解説！初心者向けガイド

1. リファクタリングとは？コードの「整理整頓」で効率アップ！

1. リファクタリングとは？コードの「整理整頓」で効率アップ！

リファクタリングとは、プログラムが提供する「動作」や「機能」はそのままで、内部のソースコードを読みやすく、管理しやすい構造に書き換える作業のことです。

例えば、日々の生活で「机の上が散らかっていると、探し物に時間がかかる」のと同じように、プログラムも放置すると複雑になり、修正や機能追加に時間がかかるようになります。これを「コードの腐敗」と呼びますが、リファクタリングを行うことで、バグの発見が早まり、将来のメンテナンスが格段に楽になります。

具体的なイメージを持つために、簡単なPHPの例を見てみましょう。まずは「リファクタリング前」の少し読みづらいコードです。

// リファクタリング前：何回も同じ計算が出てきて読みづらい
echo "リンゴの価格は " . (100 * 1.1) . " 円です。";
echo "バナナの価格は " . (150 * 1.1) . " 円です。";
echo "ミカンの価格は " . (80 * 1.1) . " 円です。";

次に、リファクタリングをして「税込み計算」を共通化したコードです。

// リファクタリング後：計算ルールを1箇所にまとめてスッキリ！
function addTax($price) {
    return $price * 1.1;
}

echo "リンゴの価格は " . addTax(100) . " 円です。";
echo "バナナの価格は " . addTax(150) . " 円です。";
echo "ミカンの価格は " . addTax(80) . " 円です。";

このように、Symfonyのコントローラにおいても、複雑になった「条件分岐」や「データの計算処理」を整理することで、誰が見ても理解しやすい「美しい設計」へと進化させることができます。

2. リファクタリング前の例（ファットコントローラの問題点）

2. リファクタリング前の例（ファットコントローラの問題点）

まずは、初心者の方が書いてしまいがちな「少しごちゃごちゃした状態」のプログラム（コントローラ）を見てみましょう。PHPのフレームワーク「Symfony」を使った例ですが、他の言語でも考え方は同じです。

このコードには、「データの取得」「エラーチェック」「消費税の計算（ビジネスロジック）」という3つの異なる役割が1ヶ所に詰め込まれています。

#[Route('/product/{id}', name: 'product_detail')]
public function detail(int $id): Response
{
    // 1. データベースから商品情報を探してくる
    $product = $this->getDoctrine()->getRepository(Product::class)->find($id);

    // 2. もし商品がなかったらエラー画面を出す（例外処理）
    if (!$product) {
        throw $this->createNotFoundException('商品が見つかりません');
    }

    // 3. 商品の価格に消費税（10%）を上乗せする計算を行う
    $priceWithTax = $product->getPrice() * 1.1;

    // 4. 最終的な結果を画面（HTML）に表示する
    return $this->render('product/detail.html.twig', [
        'product' => $product,
        'priceWithTax' => $priceWithTax,
    ]);
}

このプログラムは、実際に動かしてみるとブラウザには以下のように表示されます。

商品名：ノートパソコン
販売価格（税込）：110,000円

一見すると問題なく動いているように見えます。しかし、プログラミングの世界ではこのように「1つの場所になんでも書きすぎてしまうこと」を、太った（詰め込みすぎな）状態に例えて「ファットコントローラ」と呼びます。

今は数行なので読みやすいですが、もし「会員ランク別に割引をする」「キャンペーン期間中だけポイント2倍にする」といったルールが増えていくと、あっという間にコードが複雑になり、どこを直せばいいのか分からなくなってしまいます。これが、バグ（不具合）が生まれる大きな原因になります。

3. リファクタリングのポイント

3. リファクタリングのポイント

プログラムが動くようになったら、次は「読みやすさ」を意識してコードを整理（リファクタリング）しましょう。初心者の方が特に意識すべきなのは、「1つの関数に役割を詰め込みすぎない」ことです。

例えば、以下の2つのポイントを整理するだけで、コードのメンテナンス性は劇的に向上します：

• 共通処理の集約： データベースからの「商品の取得」など、何度も使う処理を専用の関数にまとめます。
• ロジックの分離： 「税込み金額の計算」といった計算処理を独立させることで、修正が必要な際に見つけやすくなります。

具体的に、リファクタリング前と後のイメージを比較してみましょう。まずは、すべてを1箇所に詰め込んだ「読みにくい例」です。

// リファクタリング前：処理が混ざっていて読みづらい
function checkout($itemId, $quantity) {
    // 1. 商品取得
    $item = $db->query("SELECT * FROM products WHERE id = $itemId");
    
    // 2. 金額計算（計算式が直接書かれている）
    $total = ($item['price'] * $quantity) * 1.1; 
    
    echo "合計金額は " . $total . " 円です。";
}

次に、役割ごとに処理を切り出した「きれいな例」です。プログラミング未経験の方でも、各行が「何をしているか」が言葉として入ってきやすくなります。

// リファクタリング後：役割が分かれていてスッキリ！

// 計算だけを担当する関数
function calculateTax($price, $quantity) {
    $taxRate = 1.1;
    return ($price * $quantity) * $taxRate;
}

// メインの処理（流れだけを記述）
function checkout($itemId, $quantity) {
    $item = getProduct($itemId); // 商品取得は別で定義
    $total = calculateTax($item['price'], $quantity); // 計算も外にお任せ
    
    echo "合計金額は " . $total . " 円です。";
}

このように整理することで、コントローラ本体は「全体の流れ」を把握するだけの中身になり、後から見返したときや、他の人がコードを読んだときの理解スピードが格段に早くなります。これが、バグを防ぎ、拡張性の高いサイトを作るための第一歩です。

4. リファクタリング後のコード：読みやすく保守しやすい設計

4. リファクタリング後のコード：読みやすく保守しやすい設計

リファクタリングを適用した後のプログラムを見てみましょう。初心者の方でも、コードをパズルを組み立てるように分割することで、驚くほど整理された印象を受けるはずです。

// 1. 商品を検索し、なければ404エラーを投げる専用の部品（メソッド）
private function findProductOr404(int $id): Product
{
    $product = $this->getDoctrine()->getRepository(Product::class)->find($id);

    if (!$product) {
        throw $this->createNotFoundException('ご指定の商品は見つかりませんでした。IDを確認してください。');
    }

    return $product;
}

// 2. 複雑な計算（消費税計算など）を切り出した部品
private function calculateTaxPrice(float $price): float
{
    $taxRate = 1.1; // 消費税率10%
    return $price * $taxRate;
}

// 3. 実際にページを表示するメインの処理
#[Route('/product/{id}', name: 'product_detail')]
public function detail(int $id): Response
{
    // 「何をするか」が英語の動詞で明確になり、まるで文章を読んでいるようです
    $product = $this->findProductOr404($id);
    $priceWithTax = $this->calculateTaxPrice($product->getPrice());

    return $this->render('product/detail.html.twig', [
        'product' => $product,
        'priceWithTax' => $priceWithTax,
    ]);
}

リファクタリング後のコードは、一箇所に全ての命令を詰め込むのではなく、「検索する」「計算する」「表示する」という役割ごとに処理を分割しました。

このように整理することで、プログラミング未経験者が見ても「この関数（部品）の名前からして、ここで税金の計算をしているんだな」と直感的に理解できるようになります。また、後から「消費税が8%の商品も作りたい」といった変更が必要になった際も、修正箇所が明確なのでミスを防ぐことができます。

5. Symfonyでよくあるリファクタリングのアイデア

5. Symfonyでよくあるリファクタリングのアイデア

• 長すぎる関数 → 小さく分割
• 同じコードが何度も登場 → 関数にまとめる
• HTMLテンプレートへの渡し方 → 配列名を整理する
• ベースコントローラを使う → 共通処理の集約

「読みやすいコード」は、後から自分やチームの仲間が見たときに理解しやすくなります。

6. 初心者でもできる！リファクタリングのコツ

6. 初心者でもできる！リファクタリングのコツ

リファクタリングは、小さなところから少しずつ始めるのがポイントです。

最初は「同じコードが2回以上出てきたら関数にまとめる」だけでも十分です。

慣れてきたら、ベースコントローラにまとめたり、サービスクラスに分ける方法もあります（今回は紹介しません）。

「キレイにすることは悪ではない」どころか、長く使うコードほど大事な考え方です。

まとめ

まとめ

Symfonyを用いたWebアプリケーション開発において、コントローラーの肥大化は避けて通れない課題の一つです。初心者のうちは、一つのアクションメソッド内にデータベースの取得処理、バリデーション、計算ロジック、レスポンスの生成などを全て詰め込んでしまいがちです。しかし、今回の解説で学んだように、「リファクタリング」という工程を挟むことで、コードの可読性とメンテナンス性は劇的に向上します。

リファクタリングの第一歩は、「役割を分担させること」です。例えば、特定のIDからエンティティを取得して、存在しない場合に404エラーを投げる処理などは、複数のメソッドで使い回される可能性が高い共通処理です。これをプライベートメソッドに切り出すだけで、メインとなるアクションメソッドの見通しが良くなります。

また、Symfony独自の機能を活用したリファクタリングも非常に効果的です。例えば、引数にエンティティを指定して自動的にデータを取得する「ParamConverter」のような仕組みを使えば、さらにコードを短縮できます。

さらなるステップアップ：サービスへの切り出し

今回のリファクタリングでは、コントローラー内のプライベートメソッドへ処理を分割しました。しかし、さらにプロジェクトが大きくなると、コントローラー自体が長くなってしまいます。その場合は、「サービスクラス（Service Class）」を作成し、ビジネスロジックをコントローラーの外へ追い出すのがSymfonyの王道的な設計手法です。

以下のサンプルコードは、計算ロジックをコントローラーに残さず、依存注入（Dependency Injection）を意識した構成の一歩手前のイメージです。

// 商品関連の計算ロジックをまとめるイメージ
private function formatProductData(Product $product): array
{
    return [
        'id' => $product->getId(),
        'name' => $product->getName(),
        'price_tax' => $this->calculateTaxPrice($product->getPrice()),
        'stock_status' => $product->getStock() > 0 ? '在庫あり' : '品切れ中',
    ];
}

#[Route('/api/product/{id}', name: 'api_product_detail')]
public function apiDetail(int $id): Response
{
    // 共通化されたメソッドで取得
    $product = $this->findProductOr404($id);
    
    // 表示用データの整形を分離
    $data = $this->formatProductData($product);

    return $this->json($data);
}

このように、出力結果を整形する処理を分けるだけでも、修正が必要になった際に「どこを直せばいいか」が明確になります。例えば「消費税率が10%から変わった」という時も、calculateTaxPriceメソッド一箇所を直すだけで、アプリ全体の計算結果を修正できるのです。

リファクタリングで意識すべき3つのポイント

1. DRY原則（Don't Repeat Yourself）：同じことを二度書かない。重複したコードは一つにまとめましょう。
2. 単一責任の原則：一つのメソッドには、一つのことだけをさせる。例えば「データ取得」と「メール送信」を一つのメソッドでやらないようにします。
3. 直感的な命名：$aや$tempといった変数名ではなく、$priceWithTaxのように、中身がすぐにわかる名前を付けましょう。

Symfonyの開発では、フレームワークが用意してくれている便利なコマンドやディレクトリ構造を最大限に活かすことが、美しいコードへの近道です。最初は少し面倒に感じるかもしれませんが、後からバグを探す時間や、新機能を追加する際の苦労を考えれば、リファクタリングに投資する時間は決して無駄にはなりません。

まずは今日書いた自分のコードを眺めてみて、「ここ、もっと短く書けないかな？」と自問自答するところから始めてみましょう。それがエンジニアとしての成長への大きな一歩になります。