Golang如何正确返回error？解决viper+gomail邮件代码的两类报错

• 错误1 cannot use "" (untyped string constant) as [4]string value in return statement：
  Email函数声明的第一个返回值是固定长度为4的字符串数组[4]string，但错误处理时返回了字符串类型的空值""，类型不匹配导致编译失败。
• 错误2 code can not execute unreachable return statement：
  执行完可能报错的逻辑后没有做错误判断就直接return，导致return之后的代码永远无法被执行，属于逻辑错误。比如LoadEnvVariable中调用viper.ReadInConfig()后直接返回，后续读取配置项的逻辑完全不会运行；Email函数中每次调用LoadEnvVariable后直接返回，后续发邮件的逻辑也无法执行。

1. 所有返回错误的逻辑前必须加if err != nil的判断，只有真的出现错误时才提前返回
2. Email函数报错返回时，第一个返回值要和声明类型一致，使用空的[4]string{}代替""
3. 修正LoadEnvVariable中的逻辑，配置读取成功后才执行类型断言，类型断言失败时主动返回自定义错误
4. 可优化viper初始化逻辑，把配置加载操作提到程序启动阶段全局执行一次，避免每次读取环境变量都重复加载.env文件减少不必要IO

import "fmt"

func LoadEnvVariable(key string) (string, error) {
    // 若要优化可将下面两行viper初始化代码提到main函数启动阶段执行
    viper.SetConfigFile(".env")
    err := viper.ReadInConfig()
    if err != nil {
        return "", err
    }

    value, ok := viper.Get(key).(string)
    if !ok {
        return "", fmt.Errorf("配置项%s类型错误，预期为字符串", key)
    }
    return value, nil
}

func Email(value [4]string) ([4]string, error) {
    mail := gomail.NewMessage()

    myEmail, err := LoadEnvVariable("EMAIL")
    if err != nil {
        return [4]string{}, err
    }
    appPassword, err := LoadEnvVariable("APP_PASSWORD")
    if err != nil {
        return [4]string{}, err
    }

    mail.SetHeader("From", myEmail)
    mail.SetHeader("To", myEmail)
    mail.SetHeader("Reply-To", value[1])
    mail.SetHeader("subject", value[2])
    mail.SetBody("text/plain", value[3])

    a := gomail.NewDialer("smtp.gmail.com", 587, myEmail, appPassword)
    err = a.DialAndSend(mail)
    if err != nil {
        return [4]string{}, err
    }

    return value, nil
}

内容的提问来源于stack exchange，提问作者user17198758