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「服務器負載警報」保持在同一行。