Question

我目前使用此函数来包装执行命令并记录它们的执行，并返回代码，并在非零返回代码的情况下退出。

然而，这是有问题的，因为显然它会进行双重插值，使得带有单引号或双引号的命令会破坏脚本。

你能推荐一种更好的方法吗？

这是功能：

do_cmd()
{
    eval $*
    if [[ $? -eq 0 ]]
    then
        echo "Successfully ran [ $1 ]"
    else
        echo "Error: Command [ $1 ] returned $?"
        exit $?
    fi
}

Answer 1

"$@"

来自http://www.gnu.org/software/bash/manual/bashref.html#Special-Parameters：

@

从一个开始扩展到位置参数。当。。。的时候   扩展发生在双引号内，每个参数扩展为a   单词。也就是说，“$ @”相当于“$ 1”“$ 2”....如果   双语扩张发生在一个词内，扩展了   第一个参数与原始的开头部分连接在一起   单词，并将最后一个参数的扩展与最后一个参数连接起来   原始单词的一部分。没有位置参数时   “$ @”和$ @扩展为空（即，它们被移除）。

这意味着可以正确地重新引用参数中的空格。

do_cmd()
{
    "$@"
    ret=$?
    if [[ $ret -eq 0 ]]
    then
        echo "Successfully ran [ $@ ]"
    else
        echo "Error: Command [ $@ ] returned $ret"
        exit $ret
    fi
}

Answer 2

除道格拉斯所说的"$@"之外，我会使用

return $?

而不是exit。它会退出你的shell而不是从函数返回。如果在出现问题的情况下你想要退出shell，你可以在调用者中执行此操作：

do_cmd false i will fail executing || exit
# commands in a row. exit as soon as the first fails
do_cmd one && do_cmd && two && do_cmd three || exit

（这样，你可以处理失败，然后优雅地退出）。

编写运行命令并记录退出代码的包装函数的最佳方法是什么

2 个答案: