Bash脚本局部变量声明位置最佳实践相关咨询

你提到的两种写法在简单场景下运行结果确实一致，但将所有局部变量统一声明在函数顶部是更值得遵循的最佳实践。
先直观对比两种写法的代码：

写法1：函数顶部统一声明（推荐） #

show_animal() {
  # 所有局部变量在函数入口集中声明
  local animal
  if [ "$1" = "dog" ]; then
    animal="小狗"
  else
    animal="小猫"
  fi
  echo "$animal"
}

写法2：分支内声明赋值 #

show_animal() {
  if [ "$1" = "dog" ]; then
    local animal="小狗"
  else
    local animal="小猫"
  fi
  echo "$animal"
}

推荐顶部声明的核心原因 #

• 可读性更强：所有函数内部使用的局部变量集中在入口处，维护者不需要遍历所有分支逻辑就能快速梳理清楚函数的变量依赖，大幅降低理解和修改成本。
• 完全规避全局变量污染：Bash 中local关键字的作用域是整个函数，并不会被限制在声明位置所在的代码块（比如if分支、循环块）。如果在分支内才声明变量，很容易出现声明前就调用变量的情况，此时Bash会自动将该变量创建为全局变量，隐蔽地污染全局命名空间，参考反面示例：

bad_demo() {
  # 声明前就调用animal变量，会自动创建全局变量animal
  echo "提前调用结果：$animal"
  if [ "$1" = "dog" ]; then
    local animal="小狗"
  fi
  echo "最终结果：$animal"
}
bad_demo "dog"
# 外部可以直接访问到全局的animal变量
echo "全局作用域的animal值：$animal"

• 避免分支遗漏错误：当函数逻辑变复杂后，很容易出现某分支忘记加local关键字直接赋值的情况，此时也会意外创建全局变量。统一在顶部声明后，后续所有赋值操作都只会修改局部变量，完全规避这类低级错误。

如果是只有十余行、逻辑极其简单的小函数，分支内声明变量不会造成明显问题，但只要函数复杂度稍高，统一在顶部声明是Bash脚本开发领域普遍认可的规范。

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