Bash条件语句中变量赋值与返回码同步校验相关问题咨询

示例脚本的弊端 #

• 错误无法精准定位：get_username和is_from_earth两个命令通过&&串联执行，只要任意一个失败就会进入else分支，仅从报错信息无法区分到底是用户名获取失败，还是用户身份校验不通过，排查问题效率低。
• 变量值不可控：如果get_username执行失败，username变量会被赋值为空值，若脚本其他地方未做非空判断就直接使用该变量，会引发不可预期的逻辑错误；如果该变量之前已有旧值，甚至会出现复用旧值的问题。
• 调试难度高：无法单独打印两个命令的返回码、执行日志，要定位问题必须拆分条件判断逻辑。

是否建议分离变量赋值/函数调用与返回码校验 #

绝大多数生产环境脚本场景下，都建议做分离。

分离的核心原因 #

• 错误处理更精细：可以单独捕获每个命令的返回码，针对不同错误类型输出对应的报错信息，大幅降低排查成本。
• 变量使用更安全：赋值后先判断命令执行状态，确认成功后再使用变量，从根源上避免空值、旧值带来的逻辑错误。
• 扩展性更强：后续需要新增日志打印、失败重试、监控打点等逻辑时，直接在对应命令后添加即可，不需要重构整个条件判断结构。

该条件判断写法本身的问题 #

语法层面完全符合Bash的规则，执行逻辑没有问题，但是仅适用于不需要区分错误类型、用完即扔的临时脚本。对于需要长期维护、运行在生产环境的脚本来说，可维护性、可观测性都不达标，不建议使用。

优化后写法参考 #

local return_code=0
local username=""

# 单独处理用户名获取逻辑
username=$(get_username "${user}")
if [[ $? -ne 0 ]]; then
    echo "ERROR:  failed to get username for user ${user}"
    return_code=1
else
    # 单独处理地球身份校验逻辑
    if is_from_earth "${user}"; then
        echo "${username} is an earthling"
    else
        echo "ERROR: user ${user} is not from the Earth"
        return_code=1
    fi
fi

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