Bash脚本错误。意外令牌附近的语法错误

Question

我试图通过一个脚本来向我发送通知，如果我们的服务器上的负载太高。我找到了一个好的，但是当我运行它时它给了我和错误，我看不出为什么。

运行下面的代码给出了错误：

line 13: syntax error near unexpected token `fi'

我想我不得不制定了正确不过。谢谢！

#!/bin/bash 

THR=10 
MAIL="address@domain.com" 

VAR=`uptime|awk -F, '{print $4}'|awk '{print $3}'` 
OUT=`echo "$VAR $THR" | awk '{if ($1 > $2) print "yes"; else print "no"}'` 
if [ "$VAR" == "" ] 
then 
    # it's within the first 24 hours of uptime 
    VAR=`uptime|awk -F, '{print $3}'|awk '{print $3}'` 
    OUT=`echo "$VAR $THR" | awk '{if ($1 > $2) print "yes"; else print "no"}'` 
fi 
if [ "$OUT" == "yes" ] 
then 
    echo "The current load $VAR is greater than the threshold $THR" | mail $MAIL 
    -s "Server Load Alert" 
    echo "Alert generated because $VAR is greater than $THR" 
else 
    echo "No alert as $VAR > $THR" 
fi 
echo "load = $VAR" 

Answer 1

对不起，没有冒犯，但你的bash风格是可怕的！

这里有一个更好的版本：

#!/bin/bash 

thr=10 
mail="address@domain.com" 

read var _ < /proc/loadavg 

if (($(bc -l <<< "$var>$thr"))); then 
    echo "The current load $var is greater than the threshold $thr" | mail "$mail" -s "Server Load Alert" 
    echo "Alert generated because $var is greater than $thr" 
else 
    echo "No alert as $var <= $thr" 
fi 
echo "load = $var" 

的变化如下：

• 使用小写的变量名，如大写变量名被认为是不好的bash的做法。
• 不解析使用数以百万计的管道，次外层和awk是因为它是低效的命令uptime的输出，是直接从文件/proc/loadavg获得相同的信息，具有read内置。
• 请勿使用awk来测试不等式，使用bc，它更有效率（并且根本不需要变量$OUT）。
• 没有反引号！改用$(...)构造（更易于阅读，嵌套和更好的bash练习）。

我还没有测试脚本，只是纠正了你的阅读。请告诉我它是否适合你。

Answer 2

#!/bin/bash 

THR=10 
MAIL="address@domain.com" 

VAR=`uptime|awk -F, '{print $4}'|awk '{print $3}'` 
OUT=`echo "$VAR $THR" | awk '{if ($1 > $2) print "yes"; else print "no"}'` 
if [ "$VAR" == "" ] 
then 
# it's within the first 24 hours of uptime 
VAR=`uptime|awk -F, '{print $3}'|awk '{print $3}'` 
OUT=`echo "$VAR $THR" | awk '{if ($1 > $2) print "yes"; else print "no"}'` 
fi 
if [ "$OUT" == "yes" ] 
then 
echo "The current load $VAR is greater than the threshold $THR" | mail $MAIL -s "Server Load Alert" 
echo "Alert generated because $VAR is greater than $THR" 
else 
echo "No alert as $VAR > $THR" 
fi 
echo "load = $VAR" 

这对我的作品。我改变了，以便“mail $ MAIL”和-s“服务器负载警报”保持在同一行。