本文介绍如何将多段重复的日志文件初始化逻辑重构为简洁、可扩展的循环结构，通过映射（map）关联日志路径与 logger 实例，避免硬编码和冗余判断，显著提升代码可读性与可维护性。
本文介绍如何将多段重复的日志文件初始化逻辑重构为简洁、可扩展的循环结构，通过映射（map）关联日志路径与 logger 实例，避免硬编码和冗余判断，显著提升代码可读性与可维护性。
在 Go 应用中，为不同日志级别（如 Trace、Info、Warning、Error）分别配置独立文件输出是常见需求。但原始实现存在明显问题：四段高度相似的 if 块造成严重代码重复，不仅增加维护成本，还容易因疏忽导致行为不一致（例如权限掩码写错、错误处理逻辑差异等）。
最直接且推荐的优化方式是提取共性逻辑，使用 map 驱动循环初始化。核心思路是：将日志路径字符串作为键，对应 *log.Logger 实例作为值，构建映射关系，再遍历该映射完成统一打开与绑定操作。
以下是优化后的完整示例：
// 构建日志路径 → logger 的映射（注意：必须传入 *log.Logger 指针）
logFiles := map[string]*log.Logger{
config.Process.TraceLog: Trace,
config.Process.InfoLog: Info,
config.Process.WarningLog: Warning,
config.Process.ErrorLog: Error,
}
// 统一处理每个日志配置
for filePath, logger := range logFiles {
if filePath == "" {
continue // 跳过未配置的路径
}
file, err := os.OpenFile(filePath, os.O_RDWR|os.O_CREATE|os.O_APPEND, 0644) // ⚠️ 推荐改为 0644
if err != nil {
log.Fatalf("failed to open log file %q: %v", filePath, err) // 使用 log.Fatalf 替代 fmt.Println + os.Exit
}
logger.SetOutput(file)
}
✅ 关键优化点说明：
- 权限安全升级：将 0666 改为 0644，避免组/其他用户意外获得写权限，符合最小权限原则；
- 错误处理更规范：使用 log.Fatalf 替代 fmt.Println + os.Exit(1)，自动打印时间戳并确保程序终止；
- 空路径健壮性：显式 continue 处理空字符串，语义清晰且避免潜在 panic；
- 变量命名更语义化：logFiles 比 m 更具可读性，利于团队协作。
⚠️ 注意事项：
- 确保 Trace、Info 等 logger 变量已正确定义为 *log.Logger 类型（而非 log.Logger 值类型），否则 SetOutput 将无法修改原实例；
- 若需支持动态增删日志通道，可将映射定义为切片 []struct{ Path string; Logger *log.Logger }，进一步提升灵活性；
- 生产环境建议配合 defer file.Close()（需额外管理生命周期）或使用 lumberjack 等轮转日志库替代基础 os.OpenFile。
通过这一重构，代码行数减少约 60%，逻辑集中、意图明确，后续新增日志类型（如 DebugLog）仅需在 map 中追加一行，真正实现“开闭原则”。
