Validation d'une valeur saisie.

Version imprimable

Bonjour tout le monde.

J'ai un petit soucis dans un script, dans lequel je fais saisir une valeur.

En gros, je recherche un fichier suivant un schéma et si j'en ai plusieurs, j'affiche la liste des fichiers :

Code:

1
2
3
4
0) fichier0
1) fichier1
2) fichier2
faites votre choix[defaut:2]

Le problème viens que si on fait Entrée directement sans saisir de valeur, j'obtiens un null!.

En gros, voila mon code (plus simple :D)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
      echo "# Attention, il y a plus d'un fichier M696M00.xxxxxxxx.ibm" >> $Rep_Log$Fic_Log
      echo "Plusieurs fichiers M696M00.xxxxxxxx.ibm sont disponibles"
      j=0
      for i in $(ls -tr $Rep_Trv | grep "M696M00.[A-L][0-9]\{7\}.ibm")
      do
        echo "$(ls -l | awk '{print $9", date du "$7" "$6", a "$8}'| grep $i) > $(cat $Rep_Trvx$i)" >> $Rep_Log$Fic_Log
        # Affichage de la liste des fichiers, avec leur date de modification et le contenu
        Tab_Fic[$j]=$i
        echo "$j) $(ls -l | awk '{print $9", date du "$7" "$6", a "$8}'| grep $i) > $(cat $Rep_Trvx$i)"
        j=$(($j+1))
      done
      j=$(($j-1))
      echo "Merci de rentrer votre choix [defaut:$j]"
      choix=$j
      read choix
      if [ $choix -ge 0 ] && [ $choix -le $j ]
      then
        echo "valide"
        j=$choix
      fi

Si je rentre pas de valeur j'ai :

Code:

1
2
3
4
5
6
7
Plusieurs fichiers M696M00.xxxxxxxx.ibm sont disponibles
0) M696M00.B0000000.ibm, date du 19 Feb, a 15:26 > 21
1) M696M00.B0000001.ibm, date du 25 Feb, a 12:34 > 23
Merci de rentrer votre choix [defaut:1]
 
./Manuel.sh[36]: test: argument expected
Fic_NbT=M696M00.B0000001.ibm

J'ai bien le fichier pris par défaut mais j'ai une erreur.

Si vous avez des idées :king:

21/03/2008, 15h41
MarcG

normal :
read réinitialise de toute façon la variable donc si rien n'est saisie, ta variable choix et mise à rien ...... et ton test plante !
21/03/2008, 15h55
iuchiban
Ca je sais bien, et si je rajoute un test :
Code:

1 2 3 4 if [ ! $choix ] then Justement, que faire?? fi
Et un case ne fonctionne pas.

Alors comment faire pour gérer l'erreur et qu'elle ne s'affiche pas, puisque j'ai la bonne valeur de toute facon??

Merci à vous.

Un tuyau, insére la commande set -x pour voir comment le shell interprète tes instructions :

Pas bon :aie:

Code:

1
2
3
4
5
6
7
+ echo 'Merci de rentrer votre choix [defaut:2]'
Merci de rentrer votre choix [defaut:2]
+ choix=2
+ read choix

+ '[' -ge 0 ']'
./xscript02.sh: line 8: [: -ge: unary operator expected

Presque bon :mrgreen: :

Code:

1
2
3
4
5
6
7
8
9
 
read choix
if [[ ! -n $choix ]] ; then
	echo "Vide !!!"
	choix=$j
elif [[ $choix -ge 0  &&  $choix -le $j ]] ; then
	echo "valide"
	j=$choix
fi

Code:

1
2
3
4
5
6
7
8
9
10
 
+ echo 'Merci de rentrer votre choix [defaut:2]'
Merci de rentrer votre choix [defaut:2]
+ choix=2
+ read choix
 
+ [[ ! -n '' ]]
+ echo 'Vide !!!'
Vide !!!
+ choix=2

Code:

1
2
3
4
5
6
7
8
9
10
11
 
Merci de rentrer votre choix [defaut:2]
+ choix=2
+ read choix
1
+ [[ ! -n 1 ]]
+ [[ 1 -ge 0 ]]
+ [[ 1 -le 2 ]]
+ echo valide
valide
+ j=1

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
+ echo 'Merci de rentrer votre choix [defaut:2]'
Merci de rentrer votre choix [defaut:2]
+ choix=2
+ read choix
xyz
+ [[ ! -n xyz ]]
+ [[ xyz -ge 0 ]]
+ [[ xyz -le 2 ]]
+ echo valide
valide
+ j=xyz

24/03/2008, 14h31
Sve@r
Citation:

Envoyé par iuchiban

Si vous avez des idées :king:

C'est un problème classique dans la gestion d'une saisie et dans ce que "comprend" le shell
Exemple
Code:

1 2 3 echo "On continue ?" read rep test $rep = oui && echo "ok" || echo "bye"
Le problème arrive quand tu ne saisis rien => ta variable "rep" est vide et le shell voit

Citation:

test = oui

ce qui est totalement bancal vu que l'opérateur "=" exige deux opérandes (et là il n'en voit qu'un seul)

Pour résoudre le problème, il suffit de mettre des guillemets chaque fois que tu veux traiter la variable saisie
Code:

1 2 3 echo "On continue ?" read rep test "$rep" = "oui" && echo "ok" || echo "bye"
Là, même si la variable est vide, le shell verra

Citation:

test "" = "oui"

et saura gérer vu que la syntaxe est respectée

Au delà de cette astuce qui te supprimera un tas de problèmes dans la gestion des valeurs saisies, tu peux utiliser l'opérateur ${var:-valeur} qui te renvoie "valeur" si "var" est vide ou inexistante