Jak mogę skrócić ten skrypt powłoki?

CODE="A"

if test "$CODE" = "A"
then
 PN="com.tencent.ig"
elif test "$CODE" = "a"
 then
 PN="com.tencent.ig"
elif test "$CODE" = "B"
 then
 PN="com.vng.pubgmobile"
elif test "$CODE" = "b"
 then
 PN="com.vng.pubgmobile"
elif test "$CODE" = "C"
 then
 PN="com.pubg.krmobile"
elif test "$CODE" = "c"
 then
 PN="com.pubg.krmobile"
elif test "$CODE" = "D"
 then
 PN="com.rekoo.pubgm"
elif test "$CODE" = "d"
 then
 PN="com.rekoo.pubgm"
else
 echo -e "\a\t ERROR!"
 echo -e "\a\t CODE KOSONG"
 echo -e "\a\t MELAKUKAN EXIT OTOMATIS"
 exit
fi

Użyj caseinstrukcji (przenośna, działa w dowolnej shpowłoce):

case "$CODE" in
    [aA] ) PN="com.tencent.ig" ;;
    [bB] ) PN="com.vng.pubgmobile" ;;
    [cC] ) PN="com.pubg.krmobile" ;;
    [dD] ) PN="com.rekoo.pubgm" ;;
    * ) printf '\a\t%s\n' 'ERROR!' 'CODE KOSONG' 'MELAKUKAN EXIT OTOMATIS' >&2
        exit 1 ;;
esac

Polecam również zmianę nazw zmiennych ze wszystkich wielkich liter (jak CODE) na małe lub małe litery (jak codelub Code). Istnieje wiele nazw wieloznacznych, które mają specjalne znaczenie, a ponowne użycie jednego z nich przez przypadek może spowodować problemy.

Inne uwagi: standardowa konwencja polega na wysyłaniu komunikatów o błędach do „standardowego błędu” zamiast do „standardowego wyjścia”; >&2przekierowanie to robi. Ponadto, jeśli skrypt (lub program) zawiedzie, najlepiej wyjść ze stanu niezerowego ( exit 1), aby każdy kontekst wywołujący mógł stwierdzić, co poszło nie tak. Możliwe jest także użycie różnych statusów w celu wskazania różnych problemów ( dobry przykład znajduje się w sekcji „KODY WYJŚCIA” na curlstronie podręcznika ). (Podziękowania dla Stéphane Chazelas i Monty Harder za sugestie tutaj.)

Polecam printfzamiast echo -e(i echo -n), ponieważ jest bardziej przenośny między systemami operacyjnymi, wersjami, ustawieniami itp. Kiedyś miałem pęk moich skryptów, ponieważ aktualizacja systemu operacyjnego zawierała wersję bash skompilowaną z różnymi opcjami, które zmieniły sposób echozachowania.

Podwójne cudzysłowy $CODEnie są tutaj tak naprawdę potrzebne. Ciąg w a casejest jednym z niewielu kontekstów, w których można je bezpiecznie pominąć. Wolę jednak podwójnie cytować odniesienia do zmiennych, chyba że istnieje konkretny powód, aby tego nie robić, ponieważ trudno jest śledzić, gdzie jest bezpiecznie, a gdzie nie, więc bezpieczniej jest zwyczajnie podwójnie cytować.

Zakładając, że używasz bashwersji 4.0 lub nowszej ...

CODE=A

declare -A domain

domain=(
   [a]=com.tencent.ig
   [b]=com.vng.pubgmobile
   [c]=com.pubg.krmobile
   [d]=com.rekoo.pubgm
)

PN=${domain[${CODE,,}]:?ERROR! CODE KOSONG, MELAKUKAN EXIT OTOMATIS}

W kodzie definiuję tablicę asocjacyjną zawierającą wszystkie nazwy domen, z których każda jest powiązana z jednym klawiszem z małą literą.

$PNZmienna jest przypisana nazwę domeny odpowiadającą dolnym bocznym $CODEwartości ( ${CODE,,}zwraca wartość $CODEzmieniła się małymi literami tylko) z tej tablicy, ale jeśli $CODEnie odpowiada ważnego wpisu w domainwykazie, to wychodzi skrypt ze związkiem błąd.

${variable:?error message}Podstawienie parametr byłoby rozszerzyć do wartości $variable(odpowiedniej domeny w kodzie), ale byłoby wyjść skryptu z komunikatem o błędzie, jeśli wartość nie jest pusta dostępny. Nie dostaniesz dokładnie to samo formatowanie komunikatu o błędzie, jak w kodzie, ale to w zasadzie zachowują się tak samo, jeśli $CODEjest nieprawidłowy:

$ bash script.sh
script.sh: line 12: domain[${CODE,,}]: ERROR! CODE KOSONG, MELAKUKAN EXIT OTOMATIS

Jeśli zależy Ci na liczbie postaci, możemy to jeszcze skrócić:

CODE=A
declare -A domain=( [a]=tencent.ig [b]=vng.pubgmobile [c]=pubg.krmobile [d]=rekoo.pubgm )
PN=com.${domain[${CODE,,}]:?ERROR! CODE KOSONG, MELAKUKAN EXIT OTOMATIS}

Oprócz usuwania niepotrzebnych nowych linii, usunąłem również com.z każdej domeny (jest to zamiast tego dodane w przypisaniu do PN).

Zauważ, że cały powyższy kod działałby nawet dla wartości wieloznakowej w $CODE(jeśli istniałyby dla nich małe litery w domaintablicy).

Jeśli $CODEzamiast tego byłby indeks numeryczny (zero), to nieco uprościłoby kod:

CODE=0

domain=( com.tencent.ig com.vng.pubgmobile com.pubg.krmobile com.rekoo.pubgm )
PN=${domain[CODE]:?ERROR! CODE KOSONG, MELAKUKAN EXIT OTOMATIS}

Ułatwi to dodatkowo odczyt domaintablicy z pliku pomocniczego zawierającego jeden wpis w wierszu:

CODE=0

readarray -t domain <domains.txt
PN=${domain[CODE]:?ERROR! CODE KOSONG, MELAKUKAN EXIT OTOMATIS}

Jeśli twoja powłoka zezwala na tablice, najkrótsza odpowiedź powinna wyglądać następująco:

declare -A site
site=( [a]=com.tencent.ig [b]=com.vng.pubgmobile [c]=com.pubg.krmobile [d]=com.rekoo.pubgm )

pn=${site[${code,}]}

Zakłada się, że $codemoże to być tylko a, b, c lub d.
Jeśli nie, dodaj test:

case ${site,} in
    a|b|c|d)        pn=${site[${code,}]};;
    *)              pn="default site"
                    printf '\a\t %s\n' 'ERROR!' 'CODE KOSONG' 'MELAKUKAN EXIT OTOMATIS'
                    exit 1
                    ;;
esac

Przyjmę tę odpowiedź w innym kierunku. Zamiast kodować dane w skrypcie, umieść te dane w osobnym pliku danych, a następnie użyj kodu, aby wyszukać plik:

$ cat names.cfg 
a com.tencent.ig
b com.vng.pubgmobile
c com.pubg.krmobile
d com.rekoo.pubgm

$ cat lookup.sh
PN=$(awk -v code="${1:-}" 'tolower($1) == tolower(code) { print $2; }' names.cfg)
if [ -z "${PN}" ]; then
  printf '\a\t%s\n' 'ERROR!' 'CODE KOSONG' 'MELAKUKAN EXIT OTOMATIS' >&2
  exit 1
fi
echo "${PN}"

$ bash lookup.sh A
com.tencent.ig
$ bash lookup.sh a
com.tencent.ig
$ bash lookup.sh x
    ERROR!
    CODE KOSONG
    MELAKUKAN EXIT OTOMATIS

Rozdzielenie tych obaw ma kilka zalet:

• Dodawaj i usuwaj dane łatwo i łatwo, bez konieczności obejścia logiki kodu.
• Inne programy mogą ponownie wykorzystywać dane, na przykład licząc, ile dopasowań znajduje się w określonej subdomenie.
• Jeśli masz ogromną listę danych, możesz posortować je na dysku i użyć lookdo wydajnego wyszukiwania binarnego (zamiast linii po linii greplub awk)

Używasz liter do indeksowania wartości, jeśli użyjesz liczb, stanie się to tak proste, jak:

code=1
set -- com.tencent.ig com.vng.pubgmobile com.pubg.krmobile com.rekoo.pubgm

eval pn\=\${"$code"}

To przenośny kod powłoki, będzie działał na większości powłok.
Dla bash można użyć: pn=${!code}albo za bash / ksh / zsh użycia: pn=${@:code:1}.

listy

Jeśli musisz użyć liter użytkownika (od a do z lub od A do Z), muszą one zostać przekonwertowane na indeks:

code=a                              # or A, B, C, ... etc.
set -- com.tencent.ig com.vng.pubgmobile com.pubg.krmobile com.rekoo.pubgm
eval pn\=\"\${$(( ($(printf '%d' "'$code")|32)-96  ))}\"

W dłuższym kodzie, aby wyjaśnić intencję i znaczenie każdej części:

code=A

set -- com.tencent.ig com.vng.pubgmobile com.pubg.krmobile com.rekoo.pubgm

asciival=$(( $(printf '%d' "'$code") ))      # byte value of the ASCII letter.
upperval=$(( asciival |  32 ))               # shift to uppercase.
indexval=$(( upperval -  96 ))               # convert to an index from a=1.
eval arg\=\"\$\{$indexval\}\"                # the argument at such index.

Jeśli chcesz przekonwertować na małe litery, użyj: $(( asciival & ~32 ))(upewnij się, że bit 6 wartości ascii jest wyłączony).

Kod błędu

Dane wyjściowe, które skrypt wyświetla na błędzie, są dość długie (i szczególne).
Najbardziej uniwersalnym sposobem radzenia sobie z tym jest zdefiniowanie funkcji:

errorcode(){ exitcode=$1; shift; printf '\a\t %s\n' "$@"; exit "$exitcode"; }

A następnie wywołaj tę funkcję z konkretnymi potrzebnymi komunikatami.

errorcode 27  "ERROR!" "CODE KOSONG" "MELAKUKAN EXIT OTOMATIS"

Zauważ, że wynikowa wartość wyjścia jest podawana przez exitcode(przykład tutaj to 27).

Pełny skrypt (z kontrolą błędów) staje się:

errorcode(){ exitcode=$1; shift; printf '\a\t %s\n' "$@"; exit "$exitcode"; }

code=${1:-A}

case "$code" in 
    [a-d]|[A-D]) : ;;
    *)           errorcode 27  "ERROR!" "CODE KOSONG" "MELAKUKAN EXIT OTOMATIS" ;;
esac

set -- com.tencent.ig com.vng.pubgmobile com.pubg.krmobile com.rekoo.pubgm
eval pn\=\"\${$(( ($(printf '%d' "'$code") & ~32) - 64  ))}\"

printf 'Code=%s Argument=%s\n' "$code" "$pn"