我正在编写一个 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这似乎已经针对几个可能的值进行了修改。我想确保我正在编写一些使用最佳实践的东西,并且可以毫无问题地运行并准确报告。
$?
只有在多次调用其他实用程序时需要它的值时,才需要使用特殊变量。例如,您可能希望在诊断消息中输出它,然后从函数中返回它。输出某些内容printf
将重置$?
为 的退出状态printf
,因此您将无法在之后直接执行return "$?"
(或return
)返回原始退出状态。请注意,即使使用[ ... ]
resets测试某些东西$?
。您可能会改为分配$?
给其他变量并使用它。如果这是项目其余部分使用的样式,您可能需要考虑
$?
在语句中使用。if
在这种情况下,请记住引用扩展,因为从技术上讲,$IFS
可能包含数字,这意味着未引用的扩展$?
可能会完全或部分消失。然而,这会导致大量的输入,并使代码略微混淆并且有点不必要地难以理解。请注意,
utility
上面的调用不以任何直接方式连接到if
它后面的语句。这里还有一些额外的冗余,因为该
if
语句现在正在检查[
实用程序的退出状态以测试变量是否为零。该变量是另一个实用程序的退出状态,我们可以if
直接使用它。很明显,调用是语句
utility
的一个组成部分。if
另请参阅ShellCheck wiki中警告背后的基本原理。
SC2181