vermutliches Klammern- und Tick-Problem

[MoonKid]

Wahrscheinlich bringe ich hier mal wieder was mit den Anführungszeichen, Ticks usw durcheinander.

An dem grep scheint ihn was zu stören. Hab schon einges probiert, aber das bringt ja auch nix, ohne zu wissen was man tut. Auch backticks funktionieren bei dem grep nicht und machen nach meinem Verständnis auch keinen Sinn.

Code: Alles auswählen

root@pc-kubuntu:/usr/local/bin/buhtzyparanoid# cat test.sh
#!/bin/bash

bp_is_nas_there () {
        if [ $(ping $1 -q -c 1 | grep '"1 received"' | wc -l) -eq 1]; then
                exit 0
        fi

exit 1
}

if [ $(bp_is_nas_there MY_NAS) -eq 1 ]; then
        echo nas_is_there
fi

root@pc-kubuntu:/usr/local/bin/buhtzyparanoid# ./test.sh
ping: unknown host MY_NAS
./test.sh: Zeile 4: [: Fehlende `]'
./test.sh: Zeile 11: [: -eq: Einstelliger (unärer) Operator erwartet.

[4A4B]

Da fehlt noch ein Leerzeichen zwischen "1" und dem schließenden "]":

Code: Alles auswählen

       if [ $(ping $1 -q -c 1 | grep '"1 received"' | wc -l) -eq 1]; then

[Cae]

Das schliessende ] vom test in der ersten Zeile von bp_is_nas_there() hat keinen Whitespace zur 1 davor. test sieht also -eq als zweites und 1] als drittes Argument, anstatt -eq, 1, ].

Uebrigens ist die Abfrage nach Rueckgabewerten kaputt, die werden nicht rausge-echo-t, das geht anders [1]:

Code: Alles auswählen

$ foo() { return 0; }
$ if foo; then echo 0; else echo 1; fi
0
$ foo() { exit 1; }
$ if foo; then echo 0; else echo 1; fi
** Shell beendet mit ret=1 **

Du willst also return anstatt exit verwenden und das zweite test-Konstrukt entfernen.

Gruss Cae

[1] ja, natuerlich kann man die ifs auch weglassen und hat mit foo; echo $? denselben Output. Mir ging's aber gerade um die Demonstration von if mit Funktionen.

[MoonKid]

Mhm... Danke so hauts hin.

Da taucth ein neues Problem auf. test (vermutlich) hat zuviele Argumente. Aber ich verstehe nicht wieso. Für mich sieht der Aufruf ok aus.

Code: Alles auswählen

#!/bin/bash

bp_is_nas_there () {
        if [ $(ping $1 -q -c 1 | grep "1 received" | wc -l) -eq 1 ]; then
                echo 1
        else
                echo 0
        fi

echo 0
}

if [ $(bp_is_nas_there SPARKY) -eq 1 ]; then
        echo nas_is_there
fi

root@pc-kubuntu:/usr/local/bin/buhtzyparanoid# ./test2.sh
./test2.sh: Zeile 13: [: Zu viele Argumente.

[Cae]

Naja, deine Funktion schreibt nun abhaengig vom Ergebnis

Code: Alles auswählen

1|0
0

auf stdout und das sind zwei whitespacegetrennte Werte. Damit hast du zuviele Argumente, was zu besagter Meldung fuehrt.

Wie man Funktionen korrekt verwendet, habe ich ja oben schon geschrieben.

Gruss Cae

[MoonKid]

Ah, er kummuliert, was auf stdout liegt.

Aber irgendwas kommt da immer noch falsch zurück.

Code: Alles auswählen

#!/bin/bash

bp_is_nas_there () {
        if [ 1 -eq 1 ]; then
                return 1
        fi

return 0
}

if [ $(bp_is_nas_there SPARKY) -eq 1 ]; then
        echo nas_is_there
fi

root@pc-kubuntu:/usr/local/bin/buhtzyparanoid# ./test2.sh
./test2.sh: Zeile 11: [: -eq: Einstelliger (unärer) Operator erwartet.

[Cae]

Genau. Da du jetzt nicht mehr ueber magische echos kommunizierst, sondern ueber Rueckgabewerte, solltest du unten auch auf den Rueckgabewert schauen, d.h.:

Code: Alles auswählen

if [ $(bp_is_nas_there SPARKY) -eq 1 ]; then # durch
if bp_is_nas_there SPARKY; then

ersetzen.

Wenn die Funktion mehrere Rueckgabewerte haben kann, ist so ein Konstrukt angebracht:

Code: Alles auswählen

bp_is_nas_there SPARKY
case "$?" in
	0)
		echo success
		;;
	1)
		echo fail
		;;
	2)
		echo almost failed
		;;
	*)
		echo wtf
		;;
esac

Wichtig ist, dass zwischen der Abfrage von $? in case und dem zugehoerigen Befehl (hier bp_is_nas_there()) keine weiterer Befehl liegt, weil sonst dessen Rueckgabewert in $? landet.

Gruss Cae