Shell Script - błąd składni w pobliżu nieoczekiwanego tokena `else '

Z poniższego skryptu powłoki, dlaczego otrzymuję błędy

syntax error near unexpected token `else'

Skrypt powłoki

echo "please enter username"
read user_name
echo "please enter password"
read -s pass
echo ${ORACLE_SID}
SID=${ORACLE_SID}
if ["${ORACLE_SID}" != 'Test'] then
sqlplus -s -l $USER_NAME/$PASS@$SID <<EOF
copy from scott/tiger@orcl insert EMP using select * from EMP
exit
EOF
else
echo "Cannot copy"
fi

Musisz zakończyć warunek w iften sposób:

if [ "${ORACLE_SID}" != 'Test' ]; then

lub tak:

if [ "${ORACLE_SID}" != 'Test' ]
then

Uwaga: musisz także wstawiać spacje [przed i przed ].

Powodem ;lub podziałem ifwiersza jest to, że część warunku instrukcji jest tylko poleceniem. Dokładnie każde polecenie dowolnej długości. Powłoka wykonuje to polecenie, sprawdza status wyjścia polecenia, a następnie decyduje, czy wykonać thenczęść, czy elseczęść.

Ponieważ polecenie może mieć dowolną długość, musi istnieć znacznik do oznaczenia końca części warunku. To jest ;lub nowa linia, po której następuje then.

Powodem spacji [jest to, że [jest to polecenie. Zwykle wbudowane w powłokę. Powłoka wykonuje polecenie [z resztą jako parametrami, w tym ]jako obowiązkowy ostatni parametr. Jeśli nie umieścisz spacji po [tym, jak powłoka spróbuje wykonać [whateverpolecenie i zakończy się niepowodzeniem.

Powód miejsca przed tym ]jest podobny. Ponieważ w przeciwnym razie nie zostanie rozpoznany jako parametr sam w sobie.

Możesz łatwo sprawdzić skrypty powłoki za pomocą ShellCheck online (dostępny również jako samodzielne narzędzie).

W takim przypadku będzie wskazywać, że instrukcja if wymaga spacji, [przed i przed ], oraz że potrzebujesz ;(lub nowego wiersza) przed thentym samym wierszem.

Kiedy to naprawisz, wyświetli się informacja, że USER_NAMEjest używany bez inicjowania czegokolwiek. Jest tak, ponieważ masz również user_namezmienną (wielkość liter ma znaczenie). To samo dotyczy PASSi pass.

Mówi ci również, abyś używał, read -raby powstrzymać się readod zniekształcania \(może to być ważne na przykład w przypadku haseł) oraz że powinieneś podwójnie cytować zmienne podczas wywoływania, sqlplusaby zapobiec przypadkowemu globalizacji nazw plików i dzieleniu słów przez powłokę (ponownie jest to ważne, jeśli hasło zawiera na przykład znaki globowania plików, takie jak *spacje).

Wcięcie kodu sprawi, że będzie on również bardziej czytelny:

#!/bin/bash

read -r -p 'please enter username: ' user_name
IFS= read -rs -p 'please enter password: ' pass

printf 'ORACLE_SID = %s\n' "$ORACLE_SID"
sid=$ORACLE_SID

if [ "$sid" = 'Test' ]; then
    echo 'Cannot copy' >&2
    exit 1
fi

sqlplus -s -l "$user_name/$pass@$sid" <<'SQL_END'
copy from scott/tiger@orcl insert EMP using select * from EMP
exit
SQL_END

Tutaj umożliwiłem także używanie haseł z wiodącymi lub końcowymi znakami spacji, tymczasowo ustawiając IFSpusty ciąg znaków do odczytu hasła read.

Zmieniono również logikę, aby ratować, jeśli $ORACLE_SID/ $sidis Test. Pozwala to uniknąć posiadania głównej części operacyjnej skryptu w ifgałęzi.

Podczas pisania shchcesz

if [ "$ORACLE_SID" != "Test" ]
then
  ...
fi

Podczas pisania bash

if [[ "$ORACLE_SID" != "Test" ]]
then
  ...
fi

Uważaj na przestrzenie, proszę. Musi być odstęp między [[pierwszym operatorem.