Discussion :

[mederik]

Bonjour,

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

[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.

[Invité]

En effet,

man bash

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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

[jack-ft]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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