我正在编写一个 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 中。