在Shell函数中执行命令时出现Command not found错误求助

问题描述 #

我是Shell脚本新手，希望编写一个简单的脚本将我的脚本安装到系统中，为此我编写了如下代码：

#! /bin/bash
#Purpose: It is the install script for Template.sh
#Version:1.0
#Created Date: Thu Mar 16 04:10:20 PM IST 2023
#Modified Date:
#Author: Sayantan Sinharay
# START #

installPackage() {
  local PATH=$1
  if [[ -f PATH/template ]]; then
    cat template.sh >template
  else
    touch template
    cat template.sh >template
    chmod 777 template
    sudo mv template PATH
  fi
}
INSTALL_PATH="/bin/"

if [[ $# -eq 1 ]]; then
  INSTALL_PATH=$1
elif [[ $# -gt 1 ]]; then
  echo "ERROR: Expected 1 argument. Got $#"
  exit 1
fi

installPackage INSTALL_PATH
# touch hello.sh --> These are working
# chmod +x hello.sh --> These are working
# END #

运行该脚本时，我遇到了Command not found错误，终端报错信息如下：

./installTemplate.sh: line 15: touch: command not found
./installTemplate.sh: line 16: cat: command not found
./installTemplate.sh: line 17: chmod: command not found
./installTemplate.sh: line 18: sudo: command not found

问题根源 #

• 你在函数内定义了local PATH=$1，而PATH是Shell核心环境变量，用于指定系统查找命令的路径。覆盖该变量后，系统无法定位touch、cat等命令的位置，直接触发报错。
• 函数中使用PATH/template和mv template PATH时，未添加变量引用符$，导致代码直接把PATH当作字符串处理，逻辑完全错误。
• 调用installPackage函数时，传递的是字符串INSTALL_PATH而非变量值$INSTALL_PATH，函数无法获取正确的安装路径。

修复后的脚本 #

#! /bin/bash
#Purpose: It is the install script for Template.sh
#Version:1.0
#Created Date: Thu Mar 16 04:10:20 PM IST 2023
#Modified Date:
#Author: Sayantan Sinharay
# START #

installPackage() {
  # 避免使用PATH作为自定义变量名，改用install_path
  local install_path=$1
  # 拼接完整目标文件路径，用双引号包裹防止空格问题
  local target_file="${install_path}/template"
  
  if [[ -f "$target_file" ]]; then
    cat template.sh > "$target_file"
  else
    # 无需提前touch，直接写入内容到目标文件
    cat template.sh > "$target_file"
    # 使用更安全的755权限（而非777），避免任意用户修改
    chmod 755 "$target_file"
    # 移动文件到目标路径，sudo仅在系统目录安装时需要
    sudo mv "$target_file" "$install_path" || exit 1
  fi
}
INSTALL_PATH="/bin/"

if [[ $# -eq 1 ]]; then
  INSTALL_PATH=$1
elif [[ $# -gt 1 ]]; then
  echo "ERROR: Expected 1 argument. Got $#"
  exit 1
fi

# 传递变量的实际值而非字符串
installPackage "$INSTALL_PATH"
# END #

关键修复点说明 #

• 保护系统环境变量：将函数内的变量名从PATH改为install_path，避免破坏系统命令查找机制。
• 正确引用变量：所有变量使用$符号调用，并用双引号包裹，防止路径包含空格时出现解析错误。
• 优化文件操作：删除冗余的touch步骤，直接通过cat写入文件；将权限从风险极高的777改为更安全的755。
• 修正参数传递：调用函数时使用installPackage "$INSTALL_PATH"，传递变量的实际值而非字符串字面量。

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