关于golang:代码重构实战

背景

用户头像上传性能中，服务端上传流程如下：

1. base64解码成字符串
2. 图片写入服务器本地长期目录
3. 上传到 阿里云 OSS/ 七牛云存储
4. 图片审核

代码如下

type Parms struct {Head string // 客户端发送 base64 字符串}
func Upload(p Params){
    if p.Head == ""{return errors.New("img is empty")
    }
    
    str,err := Base64Decode(p.Head)
    if err != nil{return err}
    
    filename := makeUniqueName()
    if err := FileWrite(filename,str);err != nil{return err}
    
    err := UploadAliOss(filename)
    if err != nil{return err}
    
    err := VerifyImg(filename)
    if err != nil{return err}

}

问题

以前多个我的项目通过 jenkins 公布不同的上线指令，每新建一个我的项目，运维须要独自写一套 shell 命令，无奈复用之前的，保护老本比拟高。起初咱们降级部署形式，只须要批改我的项目部署目录配置文件，具体后续再介绍。

始终以来稳固运行，周末收到反馈呈现问题，用户无奈上传头像。

• 查看代码的 git 历史提交记录，近期没有批改代码，排除代码bug
• 报错提醒只在调用 FileWrite 和UploadAliOss两个办法时呈现
• 在打包镜像的时候没有在容器中创立长期目录，所以无奈写入文件，用户头像无奈上传

长期解决

手动在容器中创立一个长期目录，保留图片文件。

齐全解决

我认为，图片文件全副保留在第三方对象存储中，服务器保留一份是齐全没必要的，只会占用磁盘空间。

咱们能够创立 Reader 对象把图片文件读取到缓冲区，再上传到第三方对象存储。

代码如下

func Upload(p Params){

    if p.Head == ""{return errors.New("img is empty")
    }
    
    str,err := Base64Decode(p.Head)
    if err != nil{return err}
    
    filename := makeUniqueName()
    buf := bytes.NewReader([]byte(str))
    
    err := UploadAliOssBuff(filename,buf)
    if err != nil{return err}
    
    err := VerifyImg(filename)
    if err != nil{return err}

}

总结

在接手保护老我的项目时候，防止不了遇到坑。除了吐槽前人写的蹩脚外，咱们能够尽我所能晋升代码保护品质，让后来者更容易保护。

一片空地，可能开始只有几片垃圾。如果始终不清理，你来丢，他也丢，很快就会变成垃圾堆。

微信公众号