bash-skipt Hilfe benötigt

[stschulze]

Hallo,

ich versuche eine Code-Schnipsel für ein bash-Skript zu bauen, welches die im System verfügbaren Netzwerk-Devices ausfindig macht und anschließend über einen Dialog-Befehl zur Auswahl anbietet. Anschließend wird nach dem Dialog-Befehl eine Funktion aufgerufen, welche den Wert der Dialog-Auswahl übergeben bekommt.

Da dieNetzwerkkarten von System zu System variieren können, sollte es möglichst dynamisch arbeiten. Wo liegt mein Fehler?

Code: Alles auswählen

#!/bin/sh

    netdevsum="$(ls /sys/class/net -l | awk '{print $9}'| wc -l)"
    echo "Anzahl der ermittelten Netzwerk-Devices: $netdevsum"	
    i="1"
    while [ $i -lt $netdevsum ]; 
    do 
		#echo "Ermittlung steht an für: $netdev$i"
		#sleep 2
		netdev$i="$(ls /sys/class/net -l | awk '{print $9}'| head -n2| tail -n1)"
                i=$[$i++] 
    done

#Auch das dynamische Erzeugen der Einträge in der Dialog-Box wäre super ;-)
    netdev=`dialog --radiolist "Netzwerkkarte mit LEERTASTE auswaehlen" 0 0 7 \
     $netdev1 "" off\
     $netdev2 "" off\
     $netdev3 "" off\
     $netdev4 "" off\
     $netdev5 "" off 3>&1 1>&2 2>&3`
    dialog --clear
    clear
    echo "Ihre Auswahl: $netdev"

[whisper]

Ich beziehe mich nur auf die unnötige Leerzeile
Man kann z.B.

Code: Alles auswählen

ls -l /sys/class/net  | awk '{print $9}'| grep -v "^$"|wc -l

verwenden

[DeletedUserReAsG]

Wo liegt mein Fehler?

Du schreibst nicht, was nicht funktioniert.

[JTH]

Dein Vorgehen, um die Netzwerkdevices aufzulisten, ist unnötig kompliziert.

Code: Alles auswählen

ls -l /sys/class/net

gibt viel mehr aus, als du willst. Passender wäre

Code: Alles auswählen

ls -1 /sys/class/net/

(eine Eins, statt einem kleinen L). Aber auch das geht noch einfacher. Pfade mit Wildcard vervollständigen kann die Shell auch selbst und das sollte man in den aller-allermeisten Fällen ausnutzen. Es erlaubt nämlich – oder erleichtert zumindest wesentlich – den Umgang mit Dateipfaden mit z.B. Leerzeichen.

Du könntest dir dann z.B. so das notwendige Argument für dialog konstruieren:

Code: Alles auswählen

netdevs=
for netdev_path in /sys/class/net/*; do
        netdev=${netdev_path##*/}    # Das entfernt alles bis zu und inkl. des letzten / im Pfad
        netdevs="$netdevs $netdev '' off"
done

Der Aufruf wird dann kürzer:

Code: Alles auswählen

netdev=$(dialog --radiolist "Netzwerkkarte mit LEERTASTE auswaehlen" 0 0 7 $netdevs 3>&1 1>&2 2>&3)

Code: Alles auswählen

netdev$i=

Das dürfte neben

Du schreibst nicht, was nicht funktioniert.

dein Fehler sein. So eine Variablenzuweisung ist syntaktisch falsch. Eine Zuweisung mit „dynamischem“ Namen müsste man anders lösen.

[Meillo]

Ich beziehe mich nur auf die unnötige Leerzeile
Man kann z.B.

Code: Alles auswählen

ls -l /sys/class/net  | awk '{print $9}'| grep -v "^$"|wc -l

verwenden

Nur kurz nebenbei: `grep -v "^$"' (was ``keine Zeile die leer ist'' bedeutet) ist identisch zu `grep .' (was ``jede Zeile, die zumindest ein Zeichen enthaelt'' bedeutet). Zweiteres ist einfacher und kuerzer.

[MSfree]

Ich beziehe mich nur auf die unnötige Leerzeile
Man kann z.B.

Code: Alles auswählen

ls -l /sys/class/net  | awk '{print $9}'| grep -v "^$"|wc -l

verwenden

Nur kurz nebenbei: `grep -v "^$"' (was ``keine Zeile die leer ist'' bedeutet) ist identisch zu `grep .' (was ``jede Zeile, die zumindest ein Zeichen enthaelt'' bedeutet). Zweiteres ist einfacher und kuerzer.

Naja, und dieses ganze umständliche awk|grep|wc Kontrukt liefert doch auch nichts anderes als

Code: Alles auswählen

ls /sys/class/net | wc -l

[smutbert]

Bei ähnlichen Dingen verwende ich unheimlich gerne fzf. Das erlaubt die Auswahl aus einer Liste mit den Pfeiltasten, der Eingabe von Textbruchstücken und in den meisten Terminals auch mit der Maus. Aussehen könnte ein Skript etwa so

Code: Alles auswählen

Interface="$(ls /sys/class/net/ | fzf)"

# abbrechen falls kein Interface ausgewaehlt wurde
test -z "${Interface}" && exit 0

# irgendetwas mit dem Interface machen
echo "Ausgewaehltes Netzwerkinterface: ${Interface}"