Go语言遗留代码重构：从混乱到有序

引言：为什么遗留Go代码需要重构？

Go（Golang）以其简洁性和高性能而闻名。然而，即使在Go项目中，遗留代码也会随着时间的推移而积累——过时的结构、难以阅读的函数、"魔法"数字以及缺乏测试。处理此类代码变成一场噩梦：每次更改都有风险，添加新功能需要数周时间。

重构并非从头重写项目。它是在不改变外部行为的前提下对内部代码结构进行安全重组。在本文中，我们将分解一个从分析到实施的遗留Go代码重构逐步计划。您将学习如何将混乱的"意大利面条式代码"转变为干净、可维护且可测试的项目。

第一阶段：遗留代码审计与映射

在更改代码之前，您需要了解自己正在处理什么。跳过此阶段——您就有破坏无人记得的逻辑的风险。

1.1. 发现"危险信号"

首先寻找Go中典型的反模式：

• 上帝函数——超过100行的函数。
• 全局变量——高度耦合且测试复杂。
• 魔法数字/字符串——散布在代码中的常量。
• 缺乏错误处理——忽略返回的error。
• 使用Panic代替错误——在业务逻辑中使用panic/recover。
• 层次混合——在HTTP处理器中编写SQL查询。

1.2. 创建依赖关系图

使用go callgraph或guru等工具可视化函数调用。这将有助于识别：

• 包之间的循环依赖。
• 从未被调用的"死"代码。
• 在没有事先测试覆盖的情况下不应触及的关键路径。

第二阶段：创建测试"安全网"

没有测试的重构就像没有安全网的跳跃。您的任务是编写特征测试，捕获系统的当前行为，即使它看似不正确。

2.1. 函数级测试

从代码中最关键和最隔离的部分开始。以下是一个计划重构函数的示例：

// Legacy codefunc CalculatePrice(base float64, discountCode string) float64 {    if discountCode == "SALE10" {        return base * 0.9    }    if discountCode == "VIP20" {        return base * 0.8    }    return base}

// Characterization testfunc TestCalculatePrice(t *testing.T) {    tests := []struct {        name string        base float64        code string        want float64    }{        {"no discount", 100.0, "", 100.0},        {"10% discount", 100.0, "SALE10", 90.0},        {"20% discount", 100.0, "VIP20", 80.0},    }    for _, tt := range tests {        t.Run(tt.name, func(t *testing.T) {            got := CalculatePrice(tt.base, tt.code)            if got != tt.want {                t.Errorf("CalculatePrice(%v, %v) = %v; want %v", tt.base, tt.code, got, tt.want)            }        })    }}

2.2. 通过Golden文件进行集成测试

对于复杂结构（JSON响应、报告生成），使用"golden文件"技术。将当前输出记录到文件中，并在重构期间与其进行比较。

func TestHandler(t *testing.T) {    // Execute request    req := httptest.NewRequest("GET", "/api/data", nil)    rec := httptest.NewRecorder()    myHandler(rec, req)

    // Compare with golden file    expected, _ := os.ReadFile("testdata/golden.json")    if !bytes.Equal(rec.Body.Bytes(), expected) {        t.Errorf("Output mismatch")    }