Day 24｜為了「以後可能會畫」先買好的十種畫框：猜測性通用 (Speculative Generality)

2026 iThome 鐵人賽

DAY 24

Software Development

文藝復興：這段程式碼，好像有點味道系列第 24 篇

Day 24｜為了「以後可能會畫」先買好的十種畫框：猜測性通用 (Speculative Generality)

18th鐵人賽 code smell speculative generality

Henry Chiang

2026-09-15 17:22:18

39 瀏覽

分享至

有位畫室主人，開店前想得很遠

他心想：「以後客人可能會要油畫框、水彩框、粉彩框……」於是一口氣訂了十種畫框，擺滿整面牆

半年後盤點，九種畫框，一次都沒被用過，畫室目前，只接油畫委託

那九種畫框，不是「準備好了、還沒用到」，是佔用了牆面、綁住了一筆錢
只為了一個從未發生的「以後」

一個只有單一實作的抽象類別

Day 15 之後，訂單報表功能持續在長大
開發者想到「以後可能要支援匯出成 CSV、Excel、甚至 PDF」，於是先把架構「做好」：

public abstract class OrderExportGenerator
{
    protected readonly IReadOnlyList<Order> Orders;

    protected OrderExportGenerator(IReadOnlyList<Order> orders)
    {
        Orders = orders;
    }

    public abstract void Export(string path, string reservedForFutureLocale);
}

目前系統裡，唯一的子類別實作：

public class CsvOrderExportGenerator : OrderExportGenerator
{
    public CsvOrderExportGenerator(IReadOnlyList<Order> orders) : base(orders) { }

    public override void Export(string path, string reservedForFutureLocale)
    {
        // ...寫出 CSV 的邏輯...
    }
}

呼叫端長這樣：

public class OrderExportService
{
    public void ExportMonthlyOrders(IReadOnlyList<Order> orders)
    {
        OrderExportGenerator generator = new CsvOrderExportGenerator(orders);
        generator.Export("monthly-orders.csv", null);
    }
}

十種畫框，只有一種真的被用到

打開這段程式碼，會看到三個「準備好但沒派上用場」的訊號：

抽象類別 OrderExportGenerator，只有一個子類別 CsvOrderExportGenerator
- 這個抽象層，沒有帶來任何實際的彈性，因為根本沒有第二種型態需要被抽換
參數 reservedForFutureLocale，從建立以來，一直只被傳入 null
- 它是為了「以後可能要支援多語系檔名」預留的，但那個「以後」還沒有到來
變數宣告成父類別型別 OrderExportGenerator generator，但實際上只可能被指派成 CsvOrderExportGenerator，多型帶來的彈性，在這裡是虛的

這跟今天開頭那十個畫框是同一種處境，它們看起來像是「未雨綢繆」，實際上只是把牆面（程式碼的複雜度）先佔住，換來一個可能永遠不會兌現的方便

更諷刺的是：如果半年後真的要支援 Excel 匯出，工程師打開這個抽象類別，大概率會發現
「當初猜的介面設計，跟 Excel 匯出真正需要的介面，對不上」

得先拆掉這層猜測性的架構，才能開始做真正的需求

摺疊繼承體系，回到當下真正需要的樣子

解法是摺疊繼承體系 (Collapse Hierarchy)：把子類別的實作，直接併回一個具體類別，砍掉那層沒有實際作用的抽象

public class CsvOrderExportGenerator
{
    private readonly IReadOnlyList<Order> _orders;

    public CsvOrderExportGenerator(IReadOnlyList<Order> orders)
    {
        _orders = orders;
    }

    public void Export(string path)
    {
        // ...寫出 CSV 的邏輯...
    }
}

沒用過的參數 reservedForFutureLocale，用移除參數 (Remove Parameter) 直接拿掉

呼叫端也跟著變得誠實：

public class OrderExportService
{
    public void ExportMonthlyOrders(IReadOnlyList<Order> orders)
    {
        var generator = new CsvOrderExportGenerator(orders);
        generator.Export("monthly-orders.csv");
    }
}

少了一個抽象類別、少了一個永遠是 null 的參數，程式碼變得更簡單，卻沒有損失任何目前真正用得到的能力