如果 !(...) 与;如果 [ $? -eq 0] (...)

如果 !(...) 与;如果 [ $? -eq 0] (...)

我正在编写一个 shell 脚本,并决定通过以下方式检查我的工作shellcheck.net。我能够在脚本中获得以下两行的功能相同的行为:

findmnt /dev/sda1 >/dev/null ; if [ $? -eq 0 ]; then echo 1; else echo 0; fi

if ! findmnt /dev/sda1 >/dev/null; then echo 0; else echo 1; fi

然而 shellcheck投掷:

SC2181:直接使用“if mycmd;”检查退出代码,而不是间接使用$?。

我并不清楚该使用哪个。我确实看到:https://github.com/koalaman/shellcheck/issues/1167这似乎已经修正了几个可能的值。我想确保我正在编写的内容使用最佳实践,并且可以毫无问题地运行并准确报告。

答案1

$?仅当您需要在其他实用程序的多次调用中使用特殊变量的值时,才需要使用该特殊变量。例如,您可能希望在诊断消息中输出它,然后从函数返回它。输出 的内容printf将重置$?为 的退出状态printf,因此您无法随后直接执行return "$?"(或return) 来返回原始退出状态。请注意,即使使用[ ... ]重置来测试某些内容$?。您可能会分配$?给其他变量并使用它。

如果这是项目其余部分中使用的样式,您可能需要考虑$?在语句中使用。if在这种情况下,请记住引用扩展,因为从技术上讲,$IFS可能包含数字,这意味着未引用的扩展$?可能会完全或部分消失。

utility

# shellcheck disable=SC2181
if [ "$?" -eq 0 ]; then
    echo ok
else
    echo fail
fi

然而,这会导致大量输入,并使代码有点混乱,而且有点难以理解。请注意,上面的调用utility不会if以任何直接方式连接到其后面的语句。

这里还有一些额外的冗余,因为该if语句现在正在检查实用程序的退出状态[以测试变量是否为零。该变量是另一个实用程序的退出状态,我们可以if直接使用它。

if utility; then
    echo ok
else
    echo fail
fi

这里很明显, 的调用utility是语句的一个组成部分if

也可以看看SC2181警告背后的理由在 ShellCheck wiki 中。

相关内容