基本情報技術者 2014年 春期 午前（科目A） 問50

リファクタリング【午前解説】

正解の理由

解法ステップ

1. 問題文のキーワードを探す：「リファクタリング」「外部仕様を変更することなく」「内部構造を変更」など。
2. 定義と照合する：リファクタリングの定義は「外部仕様を保ったまま内部の構造改善」であると覚える。
3. 選択肢を消去する：ペアプログラミング（ア）、テスト駆動開発（ウ）、プロトタイプ作成（エ）はそれぞれ目的や手法が異なるため除外。
4. 正答を確定する：定義に一致する選択肢を選ぶ。

選択肢別の誤答解説

• ア：2人のプログラマが協力して一つのプログラムをコーディングする手法はペアプログラミング（エクストリームプログラミングの技法）であり、リファクタリングそのものではありません。
• イ：正解。内部構造改善で外部振る舞いを変えないという点がリファクタリングの本質です。
• ウ：テストケースを先に設定してからコーディングする手法はテスト駆動開発（TDD）で、開発プロセスの流れを示すものでリファクタリングとは異なる目的と手順です。ただしTDDのサイクルの中で「リファクタリング」は明確に位置づけられています（リファクタリングはTDDのステップの一つ）。
• エ：試作品を早期に作成して利用者のフィードバックを得るのはプロトタイプやMVP（最小実用製品）の考え方であり、リファクタリングの定義には該当しません。

よくある誤解

• 「コードを変えれば全てリファクタリング」：バグ修正や機能追加のための改変はリファクタリングではなく、外部仕様を変える作業は該当しません。
• 「リファクタリングはテスト不要」：むしろリファクタリングは既存の振る舞いを壊さないために単体テストや自動化テストが必須とされます。
• 「早く動くものを作る=リファクタリング」：プロトタイプや短期開発での試作は目的が異なり、振る舞いを保持して内部を改善するという点で区別されます。

補足コラム

# リファクタ前：長い関数で複数責務を持つ
def process(items):
    total = 0
    for i in items:
        if i > 0:
            total += i * 0.1
        else:
            total += i * 0.2
    print("合計:", total)
    return total

# リファクタ後：計算部分を抽出して責務を分割
def calc_value(i):
    return i * 0.1 if i > 0 else i * 0.2

def process(items):
    total = sum(calc_value(i) for i in items)
    print("合計:", total)
    return total

FAQ