Come posso abbreviare questo script di shell?

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

Usa caseun'istruzione (portatile, funziona con qualsiasi shshell simile):

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

Consiglierei anche di cambiare i nomi delle variabili da tutte le lettere maiuscole (come CODE) in lettere minuscole o miste (come codeo Code). Esistono molti nomi di maiuscole che hanno significati speciali e il riutilizzo accidentale di uno di essi può causare problemi.

Altre note: la convenzione standard prevede l'invio di messaggi di errore a "errore standard" anziché "output standard"; il >&2reindirizzamento fa questo. Inoltre, se uno script (o un programma) fallisce, è meglio uscire con uno stato diverso da zero ( exit 1), quindi qualsiasi contesto chiamante può dire cosa è andato storto. E 'anche possibile utilizzare diversi status per indicare diversi problemi (vedere la sezione "codici di uscita" della della curlpagina man per un buon esempio). (Ringraziamo Stéphane Chazelas e Monty Harder per suggerimenti qui.)

Consiglio printfinvece di echo -e(e echo -n), perché è più portabile tra sistemi operativi, versioni, impostazioni, ecc. Una volta ho avuto una serie di interruzioni dei miei script perché un aggiornamento del sistema operativo includeva una versione di bash compilata con diverse opzioni, che ha cambiato il echocomportamento.

Le doppie virgolette $CODEnon sono davvero necessarie qui. La stringa in a caseè uno dei pochi contesti in cui è sicuro lasciarli fuori. Tuttavia, preferisco citare due volte i riferimenti alle variabili a meno che non ci sia un motivo specifico per non farlo, perché è difficile tenere traccia di dove è sicuro e dove non lo è, quindi è più sicuro fare solo una doppia citazione abitualmente.

Supponendo che tu stia utilizzando la bashversione 4.0 o successiva ...

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}

Nel codice, definisco un array associativo che contiene tutti i nomi di dominio, ciascuno associato a una singola lettera minuscola.

Alla $PNvariabile viene assegnato il nome di dominio corrispondente al $CODEvalore minuscolo ( ${CODE,,}restituisce il valore di $CODEsolo lettere minuscole trasformate) da questo array, ma se $CODEnon corrisponde a una voce valida domainnell'elenco, esce dallo script con un errore.

La ${variable:?error message}sostituzione dei parametri si espanderebbe al valore di $variable(il dominio appropriato nel codice) ma uscirebbe dallo script con il messaggio di errore se il valore è vuoto non disponibile. Non ottieni esattamente la stessa formattazione del messaggio di errore del codice, ma sostanzialmente si comporterebbe lo stesso se $CODEnon è valido:

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

Se ti interessa il conteggio dei personaggi, possiamo accorciare ulteriormente questo:

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}

Oltre all'eliminazione di newline non necessarie, ho anche rimosso com.da ciascun dominio (questo viene invece aggiunto nell'assegnazione a PN).

Si noti che tutto il codice sopra funzionerebbe anche per un valore composto da più caratteri $CODE(se esistevano chiavi con lettere minuscole per questi domainnell'array).

Se invece $CODEfosse un indice numerico (a base zero), questo semplificherebbe un po 'il codice:

CODE=0

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

Ciò renderebbe inoltre più semplice la lettura domaindell'array da un file ausiliario contenente una voce per riga:

CODE=0

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

Se la tua shell consente array, la risposta più breve dovrebbe essere come questo esempio in bash:

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

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

Ciò presuppone che $codepotrebbe essere solo a, b, c o d.
In caso contrario, aggiungi un test come:

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

Prenderò questa risposta in un'altra direzione. Invece di codificare i tuoi dati nello script, inseriscili in un file di dati separato, quindi usa il codice per cercare il file:

$ 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

Separare queste preoccupazioni ha alcuni vantaggi:

• Aggiungi e rimuovi i dati facilmente e semplicemente, senza dover aggirare la logica del codice.
• Altri programmi possono riutilizzare i dati, come contare quante corrispondenze si trovano in un determinato sottodominio.
• Se disponi di un vasto elenco di dati, puoi ordinarli su disco e utilizzarli lookper effettuare ricerche binarie in modo efficiente (anziché riga per riga grepo awk)

Stai usando le lettere per indicizzare i valori, se dovessi usare i numeri, diventa semplice come:

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

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

Questo è un codice shell portatile, funzionerà sulla maggior parte delle shell.
Per bash è possibile utilizzare: pn=${!code}o per bash / ksh / zsh uso: pn=${@:code:1}.

lettere

Se è necessario che le lettere dell'utente (dalla a alla z o dalla A alla Z) siano convertite in un indice:

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  ))}\"

In un codice più lungo per chiarire l'intento e il significato di ogni parte:

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.

Se è necessario convertire in valori minuscoli, utilizzare: $(( asciival & ~32 ))(accertarsi che il bit 6 del valore ascii non sia impostato).

codice di errore

L'output che lo script stampa su un errore è piuttosto lungo (e particolare).
Il modo più versatile per gestirlo è definire una funzione:

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

Quindi chiama quella funzione con i messaggi specifici di cui hai bisogno.

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

Si noti che il valore di uscita risultante è dato da exitcode(l'esempio qui è 27).

Uno script completo (con controllo degli errori) diventa quindi:

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"