Test d'une variable

Version imprimable

Test d'une variable

Bonjour,

un truc qui est en train de me rendre dingue. Voici un test que je veux faire

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
echo "#############################################################################"
echo "#         Script global de chargement des cubes en DEVELOPPEMENT            #"
echo "#         ======================================================            #"
echo "#                                                                           #"
echo "# Option 1  - Génération et mise à disposition des fichiers plats sans      #"
echo "#             sans lancer la reconstructions des cubes                      #"
echo "# Option 2  - Cycle complet : génération, tranfert et alim                  #"
echo "#                                                                           #"
echo "#############################################################################"
echo "\n"
echo "Quelle option, 1 ou 2 ? :"
read RESP1
if ( [[ -n $RESP1 || $RESP1 = "1" || $RESP1 = "2" ]] ); then
  option=OK
  echo "1er test option : >$option< pour le choix $RESP1"
else
 echo "******************************************************"
  echo "*  !!!     Erreur dans le choix des options     !!!  *"
  echo "*  !!! Relancez le script et choisissez 1 ou 2  !!!  *"
  echo "******************************************************"
  echo "\n"
fi

en ligne 13 je vérifie que mon user a fait les bons choix. Pour une valeur nulle, non saisie, je file bien dans le "else". Mais que je tape 1, 2 ou 3, il file dans le "then" alors que seul 1 et 2 devraient y mener.

Je ne comprends pas pourquoi d'autant que je n'ai pas de message d'erreur...

Pouvez vous m'aider ?

13/06/2013, 17h01
al1_24

Si RESP1 vaut 3, alors -n $RESP1 est vrai.
C'est tout à fait normal que tu "rentres dans le then".

La ligne if [[ "${RESP1}" = [12] ]]; then est largement suffisante pour répondre à ton besoin.
13/06/2013, 21h12
Invité
En effet,

man bash
Code:

1 2 3 -n string True if the length of string is non-zero
Donc si tu introduis n'importe quelle chaine de caractère dont la longueur est supérieure à 0(1,5, aaaa, b, ...), tu rentreras dans le bloc then.
14/06/2013, 09h05
mederik

Je ne veux rentrer dans le "then" que pour 1 ou 2. Tout autre saisie de valeur ou chaine vide aille dans le "else".

Merci.

Bon WE
14/06/2013, 10h26
jack-ft
Citation:

Envoyé par mederik

Je ne veux rentrer dans le "then" que pour 1 ou 2. Tout autre saisie de valeur ou chaine vide aille dans le "else".

Bonjour

Pas grave: ce n'est qu'un problème de pure logique!

Actuellement, ton test dit de faire le "then" si la réponse est non vide ou bien égale à 1 ou bien égale à 2.
Donc, pour toute réponse non vide, la condition est réalisée et tu passes dans le "then".

Sauf raison valable, il est recommandé de mettre les variables entre guillemets.
Outre la proposition de al1_24 que je plussoie, pour rester dans ton style, tu peux écrire (en supprimant les parenthèses inutiles et en ajoutant les guillemets (recommandés) (et les accolades (tant qu'on y est)))):

Code:

if [[ "${RESP1}" = "1" || "${RESP1}" = "2" ]] ; then

PS: ton script produira une erreur si l'utilisateur est en 'nounset' (s'il a tapé set -u avant de sourcer le script).
Dans ce cas, il vaut mieux faire:
Code:

1 2 3 if [[ -z "${RESP1-}" ]] ; then : # traitement réponse vide elif [[ "${RESP1}" = "1" || "${RESP1}" = "2" ]] ; then