Variable non reconnue par "case"

Bonjour,

j'écris un script Shell où j'utilise la fonction "case" mais elle ne reconnait pas la variable que je lui indique :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

numPort=${type23: -2}   case $numPort in 10) echo "UTP MAU, half duplex mode" ;; *) echo "Pas de port" ;; esac

---

Et cela me retourne :

Code:

---

1 2	Pas de port

---

Même résultat si je mets :

Code:

---

case ${type23: -2} in

---

*
et j'ai essayé en faisant un read , d'entrer la valeur "10" et cela m'affiche bien UTP MAU, half duplex mode.

Est-ce que quelqu'un aurait une idée de la provenance de mon erreur? Je suis dessus depuis ce matin, et je galère totalement à trouver :?

Merci à vous !

Essaie d'ajouter cette ligne

Code:

---

type23=$(echo $type23)

---

avant:

Code:

---

numPort=${type23: -2}

---

Ça me retourne toujours la même chose : "Pas de port", alors que la variable $numPort vaut bien 16 .

Bonjour,
Normal, 16 n'est pas egal à 10.

Si la variable vaut 16, c'est normal que ton code te retourne "pas de port". Il faut que la variable contienne 10 pour que le premier "case" soit retenu.

Et puis, il faudrait que tu nous expliques ce qui tu veux faire avec l'instruction :

Code:

---

numPort=${type23: -2}

---

Oui désolé, je ne vous ai donné qu'une partie de mon code, donc voilà la partie qui m'intéresse pour que vous situiez mieux le problème :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54

type23=`grep "SNMPv2-SMI::mib-2.26.2.1.1.11.23.1 = OID: SNMPv2-SMI" typePortsRT` || "Pas de port 23"     type23=$(echo $type23) numPort=${type23: -2}   case $numPort in 10) echo "UTP MAU, half duplex mode" ;; 11) echo "UTP MAU, full duplex mode" ;; 12) echo "async fiber MAU, half duplex mode" ;; 13) echo "async fiber MAU, full duplex mode" ;; 15) echo "2 pair category 5 UTP, half duplex mode" ;; 16) echo "2 pair category 5 UTP, full duplex mode" ;; 17) echo "X fiber over PMT, half duplex mode" ;; 18) echo "X fiber over PMT, full duplex mode" ;; 19) echo "2 pair category 3 UTP, half duplex mode" ;; 20) echo "2 pair category 3 UTP, full duplex mode" ;; 22) echo "PCS/PMA, unknown PMD, full duplex mode" ;; 24) echo "Fiber over long-wavelength laser, full duplex mode" ;; 26) echo "Fiber over short-wavelength laser, full duplex mode" ;; 30) echo "Four-pair Category 5 UTP, full duplex mode" ;; *) echo "Pas de port" ;; esac

---

En fait mon but est de sortir les deux derniers caractères de $type23,
lui même contenu dans typePortsRT :

SNMPv2-SMI::mib-2.26.2.1.1.11.23.1 = OID: SNMPv2-SMI::mib-2.26.4.10
SNMPv2-SMI::mib-2.26.2.1.1.11.24.1 = OID: SNMPv2-SMI::mib-2.26.4.10
SNMPv2-SMI::mib-2.26.2.1.1.11.47.1 = OID: SNMPv2-SMI::mib-2.26.4.16
SNMPv2-SMI::mib-2.26.2.1.1.11.48.1 = OID: SNMPv2-SMI::mib-2.26.4.16

pour en déduire le type de port qui me sera donné par les possibilités qui sont dans ma fonction "case".

Est-ce que c'est un peu plus clair comme ça ?

C'est bon enfin !

J'ai trouvé en me basant sur ta commande @jlliagre :

il suffisait que je change la définition de $numPort -->

Code:

---

numPort=$(echo ${type23: -2})

---

Et j'obtiens bien en lançant mon script :

Code:

---

1 2	UTP MAU, half duplex mode

---

Merci de m'avoir aidé, ça fait du bien de plus être bloqué sur ça ! :mrgreen:

Oui, c'est plus clair avec toutes les infos !

J'ai du mal à comprendre pourquoi le echo a un effet là où tu l'a mis car le but est de supprimer les espaces éventuels avant et après la chaîne, mais puisque tu prends les deux derniers et que ça marche, c'est qu'il n'y avait pas d'espace ...

Enfin, si ton code fonctionne comme il faut, c'est l'essentiel ;-)

Bonjour,

si c'est les deux derniers chiffres que tu veux tester dans case/esac, la substitution n'est pas nécessaire :

Code:

---

1 2 3 4 5

case $var in *10) ...;; *16)...;; ... esac

---

Pour juste faire des echo, si on est en bash, autant passer par un tableau:
Exemple (le $ est mon prompt) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

$ declare -A tab $ tab["10"]='UTP MAU, half duplex mode' $ tab["16"]='2 pair category 5 UTP, full duplex mode' $ var=10 $ [ -n "${tab[$var]}" ] && echo ${tab[$var]} UTP MAU, half duplex mode $ var=11 $ [ -n "${tab[$var]}" ] && echo ${tab[$var]} $ var=16 $ [ -n "${tab[$var]}" ] && echo ${tab[$var]} 2 pair category 5 UTP, full duplex mode

---

Ici, il suffit juste d'alimenter le tableau, plus besoin de modifier le code...

pourquoi faire un tableau associatif avec des indices numériques ?

Comme ça, si je veux mettre une chaine de caractère (comme par exemple 2.14), cela fonctionne aussi.

Citation:

---

Envoyé par disedorgue

Ici, il suffit juste d'alimenter le tableau, plus besoin de modifier le code...

---

J'aime beaucoup la séparation code/données (ce qui peut paraître un peu inhabituel pour un vieux lispien).

J'aime beaucoup quand les données sont représentées par... euh... des données...

Il arrive bien souvent (trop, à mon goût) que du code (comme les clauses d'un "case") soit utilisé pour gérer un problème de données, avec, comme conséquence (comme le dit disedorgue) qu'une modification des données entraîne une modification du code.

Je préfère quand les données sont gérées dans des fichiers de configuration.

Ça permet, entre autres, de partager le même code avec des configurations différentes pour des environnements différents.

Bonjour

Citation:

---

Envoyé par WarzAnthem

type23=$(echo $type23)

---

Moi ce genre d'instruction me tourmente au plus haut point. Outre le fait qu'il y a peu d'utilité de remplacer "type23" par "ce que la commande echo $type23 affiche", on peut quand-même simplifier cette instruction en type23=$type23. Ce qui permet alors de mieux exposer son incongruïté..:aie:

Citation:

---

Envoyé par Sve@r

Moi ce genre d'instruction me tourmente au plus haut point. Outre le fait qu'il y a peu d'utilité de remplacer "type23" par "ce que la commande echo $type23 affiche", on peut quand-même simplifier cette instruction en type23=$type23. Ce qui permet alors de mieux exposer son incongruïté..:aie:

---

Non, type23=$(echo $type23), ou mieux type23=$(echo -- $type23), peut modifier le contenu de la variable type23. Voir post #8.

Cette instruction remplace toutes les occurrences successives d'espaces, tabulations et saut de lignes (IFS) par des espaces uniques, et supprime les espaces éventuellement présentes en début et fin de chaîne.

Citation:

---

Envoyé par WarzAnthem

voilà la partie qui m'intéresse pour que vous situiez mieux le problème :

Code:

---

type23=`grep "SNMPv2-SMI::mib-2.26.2.1.1.11.23.1 = OID: SNMPv2-SMI" typePortsRT` || "Pas de port 23"

---

---

J'ai du mal avec cette instruction...

Lorsque je crée un fichier "typePortsRT" sans la ligne indiquée (avec un "bash" sous macos (GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin15)) et non linux), il me répond (ce qui me paraît logique):

Code:

---

-bash: Pas de port 23: command not found

---

J'imagine que tu voudrais utiliser "Pas de port 23" comme valeur par défaut au cas où la chaîne cherchée ne serait pas présente.

Si c'est bien le cas, il vaudrait mieux faire:

Code:

---

1 2	type23=`grep "SNMPv2-SMI::mib-2.26.2.1.1.11.23.1 = OID: SNMPv2-SMI" typePortsRT` type23=${type23:-"Pas de port 23"}

---

Citation:

---

Envoyé par jlliagre

Non, type23=$(echo $type23), ou mieux type23=$(echo -- $type23), peut modifier le contenu de la variable type23. Voir post #8.

Cette instruction remplace toutes les occurrences successives d'espaces, tabulations et saut de lignes (IFS) par des espaces uniques, et supprime les espaces éventuellement présentes en début et fin de chaîne.

---

Pas mal. Effectivement ça le fait.
Mais bon, un petit commentaire explicatif ne serait pas du luxe. Parce que si quelqu'un derrière arrive pour blinder le script et qu'il écrit (ne serait-ce que) type23=$(echo "$type23") ben ça ne le fait plus.

Alors autant je suis généralement partisan du "plus court chemin", autant ici je serais quand-même parti sur une instruction style type23="$(echo "$type23" |tr -d '[:space:]')" (ou une autre peut-être à base de "sed" ou "awk" si le fait que la suppression des espaces internes soit un souci) histoire d'être un peu plus clair...