重複する処理

説明（どんな問題か）

どんな状態か？

• 同じような処理が複数の場所でコピペされている
• 「ちょっとだけ違う処理」が、ベースから複製されて手動で書き換えられている
• 読みやすく見えても、保守性・一貫性が失われている

なぜ問題か？

• 修正があったとき、すべてのコピーを見つけて直す必要がある
• 一部に修正漏れが発生し、不具合の原因になる
• テスト・レビューの手間が増え、コードの信頼性が低下

アンチパターンのコード例

TypeScript

class PdfExporter {
  export(data: string) {
    console.log("開始ログ");
    console.log(`PDF出力: ${data}`);
    console.log("完了ログ");
  }
}

class CsvExporter {
  export(data: string) {
    console.log("開始ログ");
    console.log(`CSV出力: ${data}`);
    console.log("完了ログ");
  }
}

class XmlExporter {
  export(data: string) {
    console.log("開始ログ");
    console.log(`XML出力: ${data}`);
    console.log("完了ログ");
  }
}

PHP

<?php

class PdfExporter {
  public function export(string $data): void {
    echo "開始ログ\n";
    echo "PDF出力: {$data}\n";
    echo "完了ログ\n";
  }
}

class CsvExporter {
  public function export(string $data): void {
    echo "開始ログ\n";
    echo "CSV出力: {$data}\n";
    echo "完了ログ\n";
  }
}

class XmlExporter {
  public function export(string $data): void {
    echo "開始ログ\n";
    echo "XML出力: {$data}\n";
    echo "完了ログ\n";
  }
}

Python

class PdfExporter:
    def export(self, data: str):
        print("開始ログ")
        print(f"PDF出力: {data}")
        print("完了ログ")

class CsvExporter:
    def export(self, data: str):
        print("開始ログ")
        print(f"CSV出力: {data}")
        print("完了ログ")

class XmlExporter:
    def export(self, data: str):
        print("開始ログ")
        print(f"XML出力: {data}")
        print("完了ログ")

問題点：

• 各 Exporter が開始・終了ログを共通して持っている
• 出力部分だけが違うのに、同じ処理が何度も書かれている
• ログの変更やフォーマット修正などがすべての Exporter に波及

パターン別のリファクタリング

対応可能なデザインパターン例

パターン	概要	主な解決アプローチ
Template Method	処理の流れを共通化し、差分だけをサブクラスで定義	「処理の型」を 1 ヶ所にまとめて再利用
Strategy	差分処理を外から注入	動的に処理を切り替え可能な構造に
Decorator	機能をラップして追加・差し替え	コア機能に装飾的な処理を重ねて再利用
Visitor	処理とデータ構造を分離	処理ごとに振る舞いを外に出して複数形式に対応
Prototype	インスタンスをテンプレートから複製	毎回ベタ書きする代わりに、複製して差分だけ編集