即使 mkdir 成功，“mkdir || echo && exit”也会退出

Question

mkdir $2 || echo "I can't create directory $2" && exit 8

大家好，这是我在这里的第一篇文章，请善待。

我现在正在制作一个脚本，这行让我很困扰。

只有在无法创建目录 $2 时才会出现出口 8。 运行脚本并成功创建该目录后，它仍然在 8 退出。

我错过了什么吗？我认为“ || ”之后的命令只有在左侧出现错误时才会发生。

我是 Linux 世界的新手，作为一个有一点到中等 C 经验的人，我很困惑，求助！ （使用 ubuntu，bash，顺便说一句）

Answer 1

正如@fernand0 所建议的，问题在于|| 和&& 运算符的优先级。您希望它运行类似于mkdir || ( echo && exit ) 的东西——也就是说，如果mkdir 失败，则运行echo && exit 部分。但它实际上正在运行类似( mkdir || echo ) && exit 的东西——也就是说，如果mkdir 或echo 命令成功，它将运行exit 部分。 echo 几乎总是会成功，所以它几乎总是会退出。

因此，您需要明确地对命令进行分组，以覆盖此优先级。但是不要使用( )，因为它会在子shell 中运行它的内容，而exit 只会退出子shell，而不是主脚本；您可以改用{ }，或使用显式的if 块。此外，您实际上并不需要echo && exit，因为只有在echo 命令成功时才会运行exit。 echo 几乎总是成功，但在极少数情况下失败，我很确定您希望脚本无论如何都退出。

当我需要在脚本中做这样的事情时，我通常使用这个成语：

mkdir "$2" || {
    echo "I can't create directory $2" >&2
    exit 8
}

（注意：正如@CharlesDuffy 建议的那样，我在$2 周围添加了双引号——双引号变量引用几乎总是一个好主意，以防它们包含任何空格、通配符等。我还发送了错误消息到标准错误 (>&2) 而不是标准输出，这通常也是一种更好的处理方式。）

如果你想更简洁，可以放在一行：

mkdir "$2" || { echo "I can't create directory $2" >&2; exit 8; }

请注意，} 之前需要最后的;（或换行符），否则shell 认为} 只是exit 的一个参数。您也可以采用另一种方式并使用明确的if 块：

if ! mkdir "$2"; then
    echo "I can't create directory $2" >&2
    exit 8
fi

这个选项不太聪明和简洁，但这是一件好事——聪明和简洁正是导致这个问题的首要原因；清晰明确对代码来说更好。

Answer 2

|| 和 && 在其他语言中没有您习惯的优先级。不加括号，它相当于(a || b) && c（严格从左到右），而不是a || (b && c)（其中&& 的优先级高于||）。将命令与|| 和&& 混合在一起并不是一个好主意；相反，请使用 if 语句。

if ! mkdir "$2"; then
  echo "I can't create directory $2" >&2
  exit 8
fi

如果您真的想使用列表运算符，请使用{ ... } 对命令进行适当的分组。

mkdir "$2" || { echo "I can't create directory $2" >&2; exit 8; }