如何避免在嵌套if/else语句中重复编写else分支？

首先咱们先梳理下原代码里的两个关键点：一是确实存在重复在else分支赋值defaultPort的冗余问题，二是解析环境变量的错误判断逻辑完全搞反了（原代码里err != nil时才赋值解析后的端口，这会导致解析失败时反而用错误的数值，逻辑是错的）。先把这个逻辑bug修正，再解决重复代码的问题。

方案1：提前设置默认值 #

最直接的优化思路是先把port初始化为defaultPort，之后只有当环境变量存在且解析成功时，才更新port的值。这样所有失败/环境变量不存在的情况都会自动保留默认值，完全不用重复写port = defaultPort。

修改后的代码：

func GetPortFromEnvironment(name string, defaultPort int32) int32 { 
    port := defaultPort // 先给变量赋默认值
    if env := os.Getenv("SERVICE_PORT"); env != "" { 
        if _port, err := strconv.ParseInt(env, 10, 32); err == nil { // 修正err判断逻辑
            port = int32(_port)
        }
        // 解析失败时，port自动保留默认值，无需额外写else分支
    }
    // 环境变量不存在时，port也保持默认值
    return port 
}

方案2：用提前返回（Early Return）减少嵌套 #

如果想要代码的分支逻辑更清晰，可以采用提前返回的方式，彻底消除嵌套的if/else，让每个异常场景直接终结函数，也不会出现重复赋值的情况：

func GetPortFromEnvironment(name string, defaultPort int32) int32 { 
    env := os.Getenv("SERVICE_PORT")
    if env == "" {
        return defaultPort // 环境变量不存在，直接返回默认值
    }
    _port, err := strconv.ParseInt(env, 10, 32)
    if err != nil {
        return defaultPort // 解析失败，直接返回默认值
    }
    return int32(_port) // 成功解析，返回正确端口
}

这个方案里，每个异常分支都直接返回默认值，成功的路径走到最后返回结果，代码的可读性和维护性都更高，完全没有冗余的重复代码。

为什么这两个方案能解决问题？ #

• 方案1利用默认值初始化，把所有“不需要修改变量”的情况统一处理，只在成功的分支里更新变量，从根源上避免了重复赋值。
• 方案2通过提前返回，把每个异常场景直接终结函数，既减少了嵌套层级，也不用在多个else里重复写默认值赋值语句。

这两种方式都能有效消除冗余的else分支，同时修正了原代码中的逻辑错误，让代码更简洁、更健壮。

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