Problem mit bash script

[dmant]

Hallo,

Ich habe ein Problem mit folgendem Script

https://nopaste.linux-dev.org/?1172055

Das Script läuft bis Zeile 92 richtig durch.

VPNCONNECTED ist immer 0. Und das obwohl der VPN läuft. Auf der Kommandozeile kann ich den VPN ganz normal anpingen.

Jemand einen Rat?

[owl102]

if [ VPNCONNECTED == 1 ];

Diese Bedingung ist immer falsch, denn die Zeichenkette "VPNCONNECTED" ist nicht gleich der Zeichenkette "1". Du meinst entweder

Code: Alles auswählen

if [ $VPNCONNECTED = 1 ]

oder

Code: Alles auswählen

if (( VPNCONNECTED == 1 ))

Davon ab würde ich

Code: Alles auswählen

xxx
if [ $? -eq 0 ]

durch

Code: Alles auswählen

if xxx

ersetzen. (Siehe auch: http://mywiki.wooledge.org/BashPitfalls ... .7C.7C_die ) Funktional gleich, würde aber zeigen, daß du "if" in Shell-Scripten verstanden hast. (Siehe auch: http://mywiki.wooledge.org/BashPitfalls ... _myfile.5D ) Aber das hatte dir Meillo auch schon geschrieben: viewtopic.php?p=1160131#p1160131

[breakthewall]

Ich kann mich meinem Vorposter nur anschließen. Zusätzlich fiel mir noch auf, dass alle ";" in deinem Shellscript keinem Zweck dienen, und an sich gelöscht werden können. Ebenso nutzt viele groß geschriebene Variablen, was dir Probleme bereiten kann, wenn unbeabsichtigt globale Umgebungsvariablen überschreibst. Daher immer aufpassen und Variablen idealerweise klein schreiben. Würde dir auch empfehlen, Variablenwerte (VAR="WERT") grundsätzlich in Doublequotes zu setzen, was die Sicherheit erhöht bezüglich beliebiger Inhalte. Denn für gewöhnlich ist es nicht wirklich erwünscht, dass eine Kommandozeile falsch interpretiert wird, wenn sich ggf. Werte und Parameter überschneiden. Dann würde das jeweilige Programm letztlich Fehler oder gar ernste Probleme produzieren. Zuzüglich kann es je nach Fall sinnvoll sein, auch ausgeschriebene Variablen ($VAR) der Sicherheit wegen in Doublequotes zu setzen, um den Inhalt einer Variable als Ganzes zu übergeben. Wenn man hier mehrere Werte oder Parameter übergeben will, dann wäre ein Array wieder besser.

Code: Alles auswählen

if PROGRAMM PARAMETER

Wie schon gesagt wurde ist das deutlich empfehlenswerter. Zum einen weil es deutlich Code einspart, und zum anderen können Exitcodes der jeweiligen Bedingungen direkt verarbeitet werden.