Go遗留代码重构：从A到Z完整指南

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

Go (Golang) 以其简洁性、高性能和内置的并发工具而闻名。然而，任何项目都会随着时间的推移积累“技术债务”：全局变量、缺乏接口、魔法数字、不可读的函数以及缺少测试。这样的代码被称为遗留代码。重构不是从头重写所有内容，而是在不改变外部行为的前提下逐步改进结构。在本文中，我们将详细讲解Go遗留代码重构的逐步计划，从分析到实施。

步骤1：诊断遗留代码

在进行任何更改之前，需要了解我们面对的是什么。Go项目中的主要“危险信号”：

\n
• 全局变量 (var db *sql.DB) — 使测试和并行工作变得复杂。
\n
• 巨型函数 (超过50-100行) — 使代码难以阅读。
\n
• 缺乏接口 — 与具体实现紧密耦合。
\n
• 魔法字符串/数字 — 使查找和修改变得困难。
\n
• 缺少测试 — 没有测试的重构如同在薄冰上行走。
\n

分析工具：使用 go vet 和 staticcheck 查找可疑结构。gocyclo 对于计算圈复杂度也很有用。

问题代码示例：

package main\n\nimport "fmt"\n\nvar config map[string]string // 全局变量\n\nfunc main() {\n    config = map[string]string{"port": "8080"}\n    startServer()\n}\n\nfunc startServer() {\n    // 魔法字符串 "port"\n    fmt.Println("Starting on port:", config["port"])\n    // ... 100行代码\n}

步骤2：创建测试保护网

没有测试，重构就变成了猜测。你的任务是在更改之前用测试覆盖关键逻辑。对于复杂输出或集成测试，使用Golden Files（基准文件）方法。

为遗留代码中的函数添加测试的示例：

// legacy.go (原始代码)\nfunc CalculateDiscount(price float64, isPremium bool) float64 {\n    if isPremium {\n        return price * 0.9\n    }\n    return price * 0.95\n}\n\n// legacy_test.go\npackage main\n\nimport "testing"\n\nfunc TestCalculateDiscount_Premium(t *testing.T) {\n    result := CalculateDiscount(100.0, true)\n    expected := 90.0\n    if result != expected {\n        t.Errorf("Expected %f, got %f", expected, result)\n    }\n}\n\nfunc TestCalculateDiscount_Regular(t *testing.T) {\n    result := CalculateDiscount(100.0, false)\n    expected := 95.0\n    if result != expected {\n        t.Errorf("Expected %f, got %f", expected, result)\n    }\n}

提示：使用 go test -cover 评估覆盖率。在开始重构之前，尽量覆盖至少70%的业务逻辑。

步骤3：逐步分解并提取接口

重构的主要目标是使代码可测试且灵活。从为外部依赖项（数据库、HTTP客户端、文件系统）提取接口开始。

3.1 提取接口替代直接使用数据库

假设我们有一个直接操作 *sql.DB 的函数：

// 遗留代码\nfunc GetUser(db *sql.DB, id int) (*User, error) {\n    row := db.QueryRow("SELECT id, name FROM users WHERE id = ?", id)\n    // ...\n}

重构：创建 UserRepository 接口：

type UserRepository interface {\n    GetByID(id int) (*User, error)\n}\n\ntype userRepo struct {\n    db *sql.DB\n}\n\nfunc (r *userRepo) GetByID(id int) (*User, error) {\n    row := r.db.QueryRow("SELECT id, name FROM users WHERE id = ?", id)\n