Go中自定义IO操作辅助函数是否属于不良实践？

首先需要说明的是：Go 1.16及以上版本标准库已经内置了和C# File.ReadAllLines/File.WriteAllBytes对等的便捷方法 #

• 读整个文件内容：os.ReadFile，一次性返回文件所有字节
• 写整个文件内容：os.WriteFile，功能和你实现的WriteBytes完全一致，会自动处理文件创建、覆盖、关闭全流程

你现有代码的不妥之处 #

• 重复造轮子：WriteBytes、WriteString的功能完全可以用os.WriteFile一行实现，不需要自己手动处理文件打开、关闭逻辑，标准库的实现已经做了足够多的边界处理，稳定性比自己实现的更高
• WriteLines存在设计缺陷：
  1. 逐行调用WriteString会触发多次系统调用，写入性能差，尤其是行数量多的时候开销非常明显
  2. 换行符默认用fmt.Sprintln生成的\n，和Windows平台的文本换行规范\r\n不兼容，如果你的代码需要跨平台处理文本文件会出现格式问题
  3. 没有暴露文件权限参数，os.Create默认的文件权限是0666（受系统umask影响），无法满足需要自定义权限的场景，比如写敏感配置文件时需要设置0600仅允许所有者读写
• 代码冗余度高：三个写入函数的文件打开、关闭逻辑完全重复，没有复用基础实现

优化建议 #

如果确实需要WriteString、WriteLines的便捷封装，完全可以基于标准库的os.WriteFile实现，代码量大幅减少，可靠性也更高：

import (
    "bytes"
    "os"
    "runtime"
)

// WriteString 写入字符串到指定文件，自动覆盖原有内容
func WriteString(filepath string, text string, perm os.FileMode) error {
    return os.WriteFile(filepath, []byte(text), perm)
}

// WriteLines 写入行列表到指定文件，自动覆盖原有内容，自动适配平台换行符
func WriteLines(filepath string, lines []string, perm os.FileMode) error {
    var buf bytes.Buffer
    lineBreak := "\n"
    if runtime.GOOS == "windows" {
        lineBreak = "\r\n"
    }
    for _, line := range lines {
        buf.WriteString(line)
        buf.WriteString(lineBreak)
    }
    return os.WriteFile(filepath, buf.Bytes(), perm)
}

内容的提问来源于stack exchange，提问作者K. Claesson