Bash 脚本错误：“语法错误：预期操作数（错误标记为“/backup”）”

Question

我试图制作一个小脚本来检查备份驱动器是否正常及其磁盘使用详细信息，并将其添加到 nagios 作为服务器的主动检查。一旦我写了一个，我有一些未知的错误，我不想更正。

脚本是：

#!/bin/sh

BACKUP_DRIVE='/backup'

if [[ "$( grep $BACKUP_DRIVE /etc/fstab | awk '{print $2}')" -ne "0" ]]; then 

    if [[ "$( df -h | grep $BACKUP_DRIVE | awk '{print $6}')" -ne "0" ]]; then

        if [[ "$( df -h|grep backup|awk '{print $5}' | cut -d "%" -f1)" -ge "95" ]]; then
            echo "CRITICAL: Backup Drive usage exceeded 95%"
        fi
    else
        echo "CRITICAL: Backup Drive not mounted"
    fi
else
    echo "CRITICAL: Backup Drive not added in fstab"
fi

错误是：

# sh /usr/local/src/backupcheck.sh
/usr/local/src/backupcheck.sh: line 6: [[: /backup: syntax error: operand expected (error token is "/backup")
CRITICAL: Backup Drive not added in fstab

它在第一个 if 条件本身时出错，而不是进入内部 if 条件。

在提到变量 $BACKUP_DRIVE 的地方是否需要进行任何更正？请帮忙，因为我是 bash 脚本的初学者。

Answer 1

立即问题 - 可能是触发语法错误的问题 - 是：

您使用的是 shebang #!/bin/sh，它不能保证是 bash（即使是，它的行为也不同），但您使用的是特定于 bash 的语法 ([[ ... ]])。

• 改用#!/bin/bash 作为shebang。
• 当显式启动带有可执行文件的脚本时，使用bash，而不是sh；在你的例子中：bash /usr/local/src/backupcheck.sh

或者，如果您希望能够使用 sh 显式调用您的脚本，则使用 shebang：

• 重写您的脚本以仅使用 POSIX 功能，这需要将 [[ ... ]] 替换为 [ ... ]（但在一般情况下通常需要进行更多更改）。

只有在您确定sh 在您的系统上实际上是bash 并且您不需要脚本是可移植的（在其他平台上运行）时，您才可以不重写。

---

但是，条件句也存在问题：

看起来您的前 2 个条件仅用于测试封闭的命令是否成功。

通常，您根本不需要 [[ ... ]] 进行此类测试，而只需将命令直接与 if 一起使用，可能用 ! 否定，并根据需要使用 >/dev/null 或 @987654337 抑制输出@：

因此，而不是你的命令：

if [[ "$( grep $BACKUP_DRIVE /etc/fstab | awk '{print $2}')" -ne "0" ]]; then 

你应该使用：

if grep $BACKUP_DRIVE /etc/fstab >/dev/null; then 

grep 将表明（至少）找到了与退出代码0 的匹配项，从if 的角度来看，这（也）是成功的。因此，实际上，如果备份驱动器存在，if 命令的主体将被执行。

请注意，我已经删除了 awk 命令，因为它 (a) 不是测试所必需的，并且 (b) 实际上 击败 中的测试即使grep 命令失败，它也会导致整体结果为0（成功）。

同样，您的第二个条件应为：

if df -h | grep $BACKUP_DRIVE >/dev/null; then

最后，您的第三个条件是正确的原则上：它从管道捕获标准输出输出并将其与百分比数字进行比较（尽管双引号数字不是必需的，并且可能会造成混淆）。 但是，您不小心硬编码了驱动器名称，因此它应该是：

if [[ "$(df -h | grep $BACKUP_DRIVE | awk '{print $5}' | cut -d "%" -f1)" -ge 95 ]]; then

---

最后：

• 您应该通过使用2>/dev/null 重定向echo 命令将错误消息输出到stderr；同样，您应该使用exit 1（或任何其他非零退出代码）在出现错误时退出，以便正确地发出错误信号。
• 正如@Charles Duffy 在 OP 上的 cmets 中指出的那样，有可能使您的命令更高效。

Answer 2

在某些时候，它会尝试将 $BACKUP_DRIVE 参数作为数学表达式进行计算。试试这个：

grep $BACKUP_DRIVE /etc/fstab | awk '{print $2}'
if [[ $? -ne 0 ]] ; then
...