錯誤的腳本

Question

WBINFO="/usr/bin/wbinfo -t" 
TMP="/tmp/winbind" 
RESTART="/sbin/service winbind restart" 
TXT="failed" 


$WBINFO > $TMP 
TARGET='cat $TMP |grep $TXT | wc -l' 

if [ "$TARGET" -eq "1" ]; 
then 
$RESTART 
else 
echo good 

fi 

「整數表達式預期」我得到這個錯誤：

line 10: [: cat $TMP |grep $TXT | wc -l: integer expression expected 

Answer 1

單引號字符串不擴大$FOO到變量FOO的內容。使用雙引號（"）。

此外，它看起來像你想要的內容TARGET是cat命令的輸出。如果是這樣，你可能想：

TARGET=$(cat "$TMP" | grep "$TXT" | wc -l) 

即使再進一步，cat file | grep pattern是次優的 - grep知道如何把文件作爲參數來分析，而不是調用cat，這是一個整體的其他過程中產卵。你可能真的想：

if [[ $(grep -c "$TXT" "$TMP") -eq 1 ]]; then 

Answer 2

改變這一行

TARGET='cat $TMP |grep $TXT | wc -l' 

到

TARGET=$(cat $TMP |grep $TXT | wc -l) 

或

TARGET=`cat $TMP |grep $TXT | wc -l` 

Answer 3

TARGET='cat $TMP |grep $TXT | wc -l' 

這assig ns字符串'cat $TMP |grep $TXT | wc -l'到變量$TARGET。

它看起來像你想要的是命令的輸出，這需要反引號：

TARGET=`cat $TMP |grep $TXT | wc -l` 

，或者，如果你有一個比較現代的外殼，$(...)語法：

TARGET=$(cat $TMP |grep $TXT | wc -l) 

而且，該命令可以大大簡化，從上面到這個：

TARGET=$(grep $TXT $TMP | wc -l) 

這樣：

if [ "$TARGET" -eq "1" ]; 

這樣：：

TARGET=$(grep -c $TXT $TMP) 

最後，$TARGET變量可以完全如果您更改這個if聲明消除

if [ $(grep -c "$TXT" "$TMP") = 1 ]; 

或者你可以使用[[ ... ]]而不是[ ... ]（它是bash的首選）。

或者，如果你只關心是否在所有發生的模式（而不是要求它恰好出現一次）：

if grep -q "$TXT" "$TMP"; 

對於這個問題，可以消除$TMP文件爲好;我將把它作爲一個練習。 8）}

請查閱有關grep的文檔以瞭解選件的功能。 （-c選項告訴它打印匹配數，-q什麼都不打印，但仍然設置狀態以指示是否找到該模式）。

請注意，我還在變量引用周圍添加了引號，如果它們的值可能包含任何特殊字符，那麼這是很好的做法。