为什么这个bash函数不能像直接运行那样工作？

Question

我写了一个函数,/etc/bashrc因为我经常使用它,但现在它似乎不起作用:

function replaceall() {
  find "$1" -type f -exec sed -i "s|$2|$3|g" {} \;
}

直接运行时,我会做类似的事情:

find ./*/conf/production/*.conf -type f -exec sed -i "s|/home/user/sites|/var/www/vhosts|g" {} \;

哪个工作正常.但是在使用该replaceall函数调用时它不起作用:

replaceall ./*/conf/production/*.conf "/home/user/sites" "/var/www/vhosts"

注意:它在我使用时确实有效,replaceall . [...]这让我想知道,我是否遗漏了一些重要的语法？

Answer 1

看起来你根本不依赖于遍历功能find,所以我建议你只使用循环的glob扩展:

replaceall() {
    for file in $1; do
        sed -i.bak "s|$2|$3|g" "$file"
    done
}

然后调用脚本,引用每个参数:

replaceall "./*/conf/production/*.conf" "/home/user/sites" "/var/www/vhosts"

引用每个参数可确保路径的扩展发生在函数内.我还为-i交换机添加了一个后缀,以便对每个受影响的文件进行备份,否则你的脚本非常危险!

正如评论中所提到的,这种方法仍然存在潜在问题.如果要传递包含空格的路径,则必须对其进行转义以防止分词.例如,类似的路径"./*/conf/production files/*.conf"需要在空间前面使用反斜杠"./*/conf/production\ files/*.conf".