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不过,外壳检查抛出:
SC2181:直接检查退出代码,例如“if;”,而不是使用$间接检查。
我想确定的是,我写的东西使用了最佳实践,并将运行没有问题和准确的报告。
发布于 2021-09-28 19:12:01
您只需要在其他实用程序的几个调用中需要它的值时才使用特殊变量$?。例如,您可能希望将其输出到诊断消息中,然后从函数中返回它。用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。
https://unix.stackexchange.com/questions/671019
复制相似问题