Discussion :

[Viduc]

Bonjour

j'essaie de faire un script bash avec un test sur une entrée utilisateur. Voici mon code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
echo "Acceptez vous la licence d'utilisation (procéder à l'installation)? (Y,n)"
 
read reponse
 
if ["$reponse" == ""]||["$reponse" == "Y"]||["$reponse" == "y"]
then
	echo "Lancement de l'installation"
else
	echo "Terminé"
fi

quand je test ce script et que je met n'importe quoi (c par exemple) j’obtiens ceci:

Code :

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

1
2
3
4
5
6
7
 
Acceptez vous la licence d'utilisation (procéder à l'installation)? (Y,n)
c
install.sh: 23: install.sh: [c: not found
install.sh: 23: install.sh: [c: not found
install.sh: 23: install.sh: [c: not found
Terminé

si je fais "Entrée" ça fonctionne par contre rien d'autre. Si je met "y" ou "Y" ça ne fonctionne pas...

merci d'avance pour votre aide

[N_BaH]

Bonjour,ilfautmettredesespacesentrelescommandes,sinonc'estincompréhensiblepourl'interpréteurdecommandes.

Code :

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

1
2
3
4
5
6
7
8
9
10
echo "Acceptez vous la licence d'utilisation (procéder à l'installation)? (Y,n)"
 
read reponse
 
if [ "$reponse" == "" ] || [ "$reponse" == "Y" ] || [ "$reponse" == "y" ]
then
   echo "Lancement de l'installation"
else
   echo "Terminé"
fi

[Viduc]

Jaipresquefaillicroirequecétaitçaleproblème

mais non j'ai modifié le script pour rajouter les espaces et toujours le même souci...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
echo "Acceptez vous la licence d'utilisation (procéder à l'installation)? (Y,n)"
 
read reponse
 
if [ "$reponse" == "" ] || [ "$reponse" == "Y" ] || [ "$reponse" == "y" ]
then
	echo "Lancement de l'installation"
else
	echo "Terminé"
fi

Code :

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

1
2
3
4
5
6
7
 
Acceptez vous la licence d'utilisation (procéder à l'installation)? (Y,n)
d
install.sh: 23: [: d: unexpected operator
install.sh: 23: [: d: unexpected operator
install.sh: 23: [: d: unexpected operator
Terminé

[N_BaH]

le problème vient de plus haut dans le script alors.
mais quoi
...
?

[disedorgue]

Bonjour,

Le probblème ne serait pas plutot:

Code :

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

string1 == string2

En principe, le test est plutot:

Code :

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

string1 = string2

Cordialement.

[N_BaH]

en effet, bash y est indifférent, posh sort d'autres erreurs.
par contre, le message d'erreur semble coller avec dash.

[Sve@r]

Bonjour

Et si on essayait la syntaxe classique

Code bash :

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

if test "$reponse" = "" || test "$reponse" = "Y" || test "$reponse" = "y"

Accessoirement moi j'aurais mis les connecteurs test plutôt que les connecteurs shell et utilisé l'option -z dédiée au test chaine vide

Code bash :

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

if test -z "$reponse"  -o "$reponse" = "Y" -o "$reponse" = "y"

[disedorgue]

Personnellement, j'aurai aussi utilisé les connecteurs test, car déjà plus rapide, et surtout plus "compréhensible" si on a par exemple besoin d'utiliser les connecteurs shell sur la même ligne

Cordialement.

[ok.Idriss]

Bonjour.

Toujours indiquer le shell utilisé (copier/coller la ligne du shebang pour être sûr) et l'OS utilisé (le sh sous Ubuntu est un lien symbolique vers dash par exemple).

Sinon les espaces de part et d'autres des crochets comme l'a indique N_Bah restent obligatoires

Cordialement,
Idriss

[Viduc]

whow merci à vous

que de réponses !!!

Code :

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

1
2
 
if test -z "$reponse"  -o "$reponse" = "Y" -o "$reponse" = "y"

fonctionne impec !!!! merci encore

[ok.Idriss]

Oui mais c'est trop facile, il aurait été souhaitable de comprendre pourquoi, ta syntaxe avec les crochets ne fonctionne pas

L'hypothèse du dash qui ne connait pas "==" me parait la plus plausible. Après utiliser test dans ton script final n'est pas un soucis.

[disedorgue]

Pour répondre:

Code :

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

1
2
3
4
5
6
7
8
9
10
$ dash
$ test "tyty" = "tutu"
$ echo $?
1
$ test "tyty" = "tyty"
$ echo $?
0
$ test "tyty" == "tyty"
test: 5: tyty: unexpected operator
$

Cordialement.

[Sve@r]

Oui mais c'est trop facile, il aurait été souhaitable de comprendre pourquoi, ta syntaxe avec les crochets ne fonctionne pas

Ouaip +1. Comprendre l'origine d'un soucis est toujours bénéfique ne serait-ce que pour éviter le même soucis dans un autre script. Et c'est vrai qu'avoir la première ligne du script aurait bien aidé (mais l'hypothèse dash semble la plus plausible)

[Viduc]

en faite je ne connaissais pas le mot clé "test"

j'avais essayé des tests tout simple comme j'aurais fait en perl ou java, j'ai donc testé avec 1 = et puis 2 mais même résultat.

pour info voici le début du script (enfin le début est resté tel qu'il était la suite a changé pour tenir compte de la solution...

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
23
24
25
26
 
# Lecture du fichier d'information et de licence
while read line  
do   
   echo "$line"  
done < lisez-moi.txt
 
# Acceptation de la licence et lancement de l'installation
echo "Acceptez vous la licence d'utilisation (procéder à l'installation)? (Y,n)"
 
read reponse
 
if test -z "$reponse"  -o "$reponse" = "Y" -o "$reponse" = "y"
then
	if test $(whoami) = 'root'
	then 
		installation
	else
		echo "Vous devez être root pour lancer l'installation (sudo sh install.sh)"
		exit
	fi
 
else
	echo "Terminé"
	exit
fi

[N_BaH]

le test sur root devrait être la première chose testée dans le script.
il est inutile d'afficher, ou faire quoi que ce soit avant ce test.

ton script n'a pas de shebang (#!/bin/shell) ?

[Sve@r]

en faite je ne connaissais pas le mot clé "test"

Ce n'est pas un mot clef, c'est une commande shell (au même titre que echo ou grep).

Il faut bien comprendre que dans Unix, chaque commande (grep, echo, ls, mv, etc...) en dehors de ce qu'elle affiche, renvoie un état vrai (0) ou faux (pas 0). Cet état est stocké dans la variable "$?"

Et en shell, la conditionnelle "if" est faite pour vérifier la validité de l'état de la commande citée

Exemple

Code bash :

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

1
2
3
4
5
6
if grep toto /etc/passwd
then
    echo "toto utilisateur possible"
else
    echo "toto utilisateur inconnu"
fi

Si grep ne trouve pas "toto" dans /etc/passwd, il renverra l'état "faux" ce qui permet de vérifier facilement avec un if l'alternative "trouvé/pas trouvé". Il en va de même avec quasiment toute commande (une légère exception avec la commande "expr" qui renvoie 0 si elle affiche qqchose de différent de 0 ; 1 si elle affiche 0 et 2 si elle a un souci).

Ainsi c'est bien pratique en général mais cela impose 2 choses

1. après if, il faut impérativement une commande
2. si on veut vérifier la valeur des variables, il faut aussi passer par une commande dédiée à ce travail

D'où la commande "test" dédiée à la comparaison de valeurs numériques, valeurs textes, ou à examiner la nature des fichiers. La syntaxe [...] n'étant qu'un alias de la commande test. Cette commande n'affiche rien, ne fait que renvoyer un état 0 signifiant "test ok" ou "pas 0" signifiant "test nok"

Donc généralement on verra très souvent if test car généralement les comparaisons les plus usitées se font sur des variables. Mais parfois il ne sera pas impossible de voir if autre_commande.

Exemple: vérifier l'existence d'un fichier en Bourne Shell (dans ce langage, test ne connait pas l'option "-e")
Solution: passer par la commande "ls" qui réussira si le fichier existe et échouera sinon et détecter cette réussite ou échec. L'option "-d" (facultative) permettra, dans le cas d'un dossier, de ne citer que son nom et non son contenu.

Code bash :

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

1
2
3
4
5
6
if ls -d $1 1>/dev/null 2>&1
then
    echo "$1 existe en tant que fichier (on ne connait pas son type mais ce n'est pas demandé)"
else
    echo "le fichier $1 n'existe pas"
fi

PS: même si on redirige l'affichage de la commande (ce qui arrive souvent quand on ne veut pas de retour écran), cela ne change rien à la valeur de son état...

[Viduc]

Ok merci pour ces infos

mais un if finalement est toujours un if et renvoi une valeur booleen (a quelques exceptions près )...

Comme je ne m'étais jamais vraiment penché sur la syntaxe shell je ne savais tout ça en effet...

mon script commence par:

Code :

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

1
2
 
#!/bin/bash

[Sve@r]

mais un if finalement est toujours un if et renvoi une valeur booleen (a quelques exceptions près )...

Attention aux mots.
Ce sont les programmes et les commandes Unix qui renvoient une valeur.
Démo

Code bash :

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

1
2
3
4
5
true
echo $?    => tu devrais voir 0
 
false
echo $?    => tu devrais voir 1

Cette valeur est "considérée" booléenne parce que les programmeurs "considèrent" 0 comme vrai et autre chose comme faux.
if ne renvoie rien, il vérifie juste la véracité de ladite valeur selon la convention ci-dessus et branche dans la partie "then" ou "else" selon la caractéristique booléenne de ladite valeur

mon script commence par:

Code :

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

#!/bin/bash

Vérifie le programme /bin/bash (ls -l /bin/bash). Théoriquement, te devrais voir /bin/bash -> /bin/dash...

[ok.Idriss]

Bonsoir.

Si ton bash est effectivement un lien symbolique vers dash, ce serait étrange.

Habituellement c'est simplement sh qu'on fait pointer vers dash ? Me souviens plus pour Debian mais pour Ubuntu et Mint c'est comme ça il me semble

Peux-tu, en plus de nous donner la sortie de ls -l /bin/bash, nous préciser ton OS comme demandé plus haut ?

[al1_24]

La preuve que if se base sur la valeur retournée par une commande en considérant 0 comme vrai et toute autre valeur comme faux :

Code :

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

1
2
if (( 0 )) ; then ; echo "0 est vrai" ; else ; echo "0 est faux" ; fi
if (( 2 )) ; then ; echo "2 est vrai" ; else ; echo "2 est faux" ; fi

affiche

0 est vrai
2 est faux

___________________________
Edit : Correction

La bonne réponse est

2 est vrai
0 est faux