Discussion :

[jelmouss]

Bonjour,

Merci et petite aide si possible sur test conditionnel avec action.
Voudrais effectuer un test sur existante de repertoire et, si absent, le crée mais aussi retourner un message echo de bonne fin.
# code1 - KO

Code :

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

[ -d "${REP}" ] && echo "$REP ALREADY EXIST" || echo $("$REP WILL BE CREATED" && mkdir "${REP}")

# code2 - KO

Code :

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

[ -d "${REP}" ] && echo "$REP ALREADY EXIST" || echo "$REP WILL BE CREATED && mkdir "${REP}"

# code3 - OK

Code :

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

[ -d "${REP}" ] && echo "$REP ALREADY EXIST" || mkdir "${REP}"

une idée pour combiner mkdir et echo ?
Merci

[disedorgue]

Code :

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

[ -d "$REP" ] && echo "$REP ALREADY EXIST" || { echo "$REP WILL BE CREATED" ; mkdir "$REP" ; }

Ici on test, si vrai (&&) le premier echo sinon (||) on execute le bloc { .... ; }

Détail: cette forme fonctionne bien dans ce cas car un echo en général ne tombe jamais en erreur, car ici sinon on ferait aussi l'execution du bloc si le premier echo générait une erreur...

[N_BaH]

pourquoi ne pas utiliser une structure de test conditionnelle plus explicite :

Code :

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

if COMMANDES; then COMMANDES; [ elif COMMANDES; then COMMANDES; ]... [ else COMMANDES; ] fi

?
quand on débute...

[disedorgue]

Et même quand on ne débute pas ?

[N_BaH]

quand on ne débute pas, on maîtrise l'implicite, et, alors, ça dépend du niveau de complexité du test.

[jack-ft]

pourquoi ne pas utiliser une structure de test conditionnelle plus explicite :

Code :

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

if COMMANDES; then COMMANDES; [ elif COMMANDES; then COMMANDES; ]... [ else COMMANDES; ] fi

?
quand on débute...

Oui !!!

Et quand on débute un peu moins aussi !

Quand j'utilise (très rarement) les "||" et "&&", j'ai très souvent des problèmes pour faire des petites modifs comme ajouter des traces, par exemple.

Je trouve beaucoup plus lisible d'utiliser if...then...else:

Code :

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

1
2
3
4
5
6
7
8
9
set -e
if test -d "$REP"
then echo "$REP ALREADY EXIST"
else echo "$REP WILL BE CREATED"
   if mkdir "$REP"
   then echo "$REP HAS BEEN CREATED"
   else echo "$REP COULD NOT BE CREATED" 1>&2
   fi
fi