Commande "if" dans un script - OK ou NOK

Bonjour à tous,

Voici mon script:

Code:

---

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 27 28 29 30 31

# Boucle sur base client FILE=$(cat /sauvegardes/client.csv) echo $FILE   for var in $FILE ;do CLIENT=`echo $var | awk -F";" '{ print $1 }'` IP=`echo $var | awk -F";" '{ print $2 }'` LOGIN=`echo $var | awk -F";" '{ print $3 }'` PASS=`echo $var | awk -F";" '{ print $4 }'` echo "${CLIENT}" "${IP}" "${LOGIN}" "${PASS}"   # Creation fichier de LOG touch $log echo ""$date" - Téléchargement de config.conf sur "$IP" en "$jour"_"$CLIENT".conf" >> $log   # Transfert FTP de config.conf ftp -n << EOF open $IP $PORT quote USER $LOGIN quote PASS $PASS bin cd conf get config.conf bye EOF   # Déplacement et Renommage du fichier de configuration mv config.conf /sauvegardes/"$jour"_"$CLIENT".conf   done

---

Mon souhait est d'indiquer dans les logs que:
- Si le fichier de config a été téléchargé -> indiquer dans les logs que c'est OK
- Si le fichier de config n'a pas été téléchargé -> indiquer dans les logs que c'est NOK

Je sais qu'il existe la commande "if" mais je bloque pour l'introduire dans mon script.

Avez-vous svp la possibilité de m'aiguiller ?

Merci d'avance.

Cdt,
Laurent

Salut Laurent,

Regarde sur mon post, tu devrais avoir des réponses à tes questions : http://www.developpez.net/forums/d14...e/#post7835087

Regarde la première réponse de BufferBob, 2ème solution.

Pour mon cas, la condition est que le fichier existe.

En espérant t'avoir aidé !

Bonjour.

Plusieurs remarques :

1) if n'est pas une commande mais une instruction shell, une instruction bash dans votre cas.

2) Pour ce genre de découpage, awk n'est pas utile. Un simple cut pourrait suffire :

Code:

---

1 2 3 4

CLIENT=$(echo $var | cut -d":" -f1) IP=$(echo $var | cut -d":" -f2) LOGIN=$(echo $var | cut -d":" -f3) PASS=$(echo $var | cut -d":" -f4)

---

Toutefois dans votre cas, ce n'est pas non plus la solution, il est préférable de changer de boucle et d'IFS : lire la FAQ à ce sujet.

Du coup ça donnerait :

Code:

---

1 2 3 4 5 6 7

while IFS=':' read -ra array; do CLIENT="${array[0]}" IP="${array[1]}" LOGIN="${array[2]}" PASS="${array[3]}" #... done < /sauvegardes/client.csv

---

Bien sûr, vous n'êtes pas obligé de passer par des variables tampons et manipuler le tableau directement. Vous pouvez également opter pour :

Code:

---

1 2 3

while IFS=':' read -r CLIENT IP LOGIN PASS POUBELLE; do # ... done < /sauvegardes/client.csv

---

La variable "$POUBELLE" contiendra le reste de la ligne parcourue s'il y a plus de 4 champs.

3) Pour répondre à votre question, tester si le fichier existe et s'il n'est pas vide devrait convenir à votre besoin :

Code:

---

[[ -f fichier && "$(cat fichier)" ]] && echo "OK" >> "$log" || echo "NOK" >> "$log"

---

Ici je n'est pas utilisé if mais l'instruction [[ qui est l'équivalent d'un test (ou [) amélioré.

Bien entendu vous pouvez aussi passer par une structure if mais toujours à l'aide de [[ :

Code:

---

1 2 3 4 5

if [[ -f fichier && "$(cat fichier)" ]]; then # ... else # ... fi

---

• -f va tester si le fichier existe et s'il s'agit d'un fichier régulier (capable de stocker de la donnée : txt, csv, xml, tar, zip, etc)
• "$(cat fichier)" équivaut au test -n "$(cat fichier)" qui test que la chaine "$(cat fichier)" est non vide

Au lieu de -n, vous avez également -s fichier qui permet de vérifier qu'un fichier ou répertoire existe et que sa taille est différente de 0. Ce test pourrait également convenir à votre situation (voire est préférable s'il s'agit d'un fichier volumineux).

man bash pour en savoir plus ;)

Et un peu de lecture :

Quelques bonnes pratiques dans l'écriture de scripts en Bash
FAQ : Comment lire/parcourir un fichier ?
FAQ : Pourquoi faut-il éviter d'utiliser la boucle for sur une sortie de commande ?

Et pour approfondir votre connaissance du shell : cours complet sur le shell

Cordialement,
Idriss

Citation:

---

Envoyé par ok.Idriss

tester si le fichier existe et s'il n'est pas vide

---

Code:

---

1 2 3 4

$ help test [...] -s FILE True if file exists and is not empty. [...]

---

Citation:

---

il est préférable de changer de boucle et d'IFS

Code:

---

while IFS=':' read -r

---

---

merci.

:salut:

J'avais également indiqué l'option -s dans ma réponse précédente mais en la définissant plutôt par rapport au man bash :

Citation:

---

-s file
True if file exists and has a size greater than zero.

---

=>

Citation:

---

Envoyé par ok.Idriss

Au lieu de -n, vous avez également -s fichier qui permet de vérifier qu'un fichier ou répertoire existe et que sa taille est différente de 0. Ce test pourrait également convenir à votre situation (voire est préférable s'il s'agit d'un fichier volumineux).

---

Ceci étant, ça revient effectivement à dire que le fichier n'est pas vide :aie:
Et "différente de 0" est inexact, j'aurais du dire "plus grande que 0" :lol:

Idriss

Citation:

---

J'avais également indiqué l'option -s dans ma réponse précédente

---

[mauvaise foi] ah, oui ?! mais, c'était écrit en tout petit. [/mauvaise foi]
:lol:

En tout cas merci à vous !!!!

Avec ceci:

Code:

---

1 2	[[ -f fichier && "$(cat fichier)" ]] && echo "OK" >> "$log" || echo "NOK" >> "$log"

---

Ça marche nikel !!!

Voici le résultat:

Code:

---

1 2	[[ -f /repertoire/"$jour"_"$CLIENT".conf && "$(cat /repertoire/"$jour"_"$CLIENT")" ]] && echo "OK - "$date" - Téléchargement de config.conf sur "$IP" en "$jour"_"$CLIENT".conf" >> $log || echo "NOK - "$date" - Téléchargement échoué sur "$IP" pour le client "$CLIENT"" >> $log

---

c'est confirmé, tu as du goût pour les solutions les plus "complexes". :aie:

Re bonjour.

Citation:

---

Envoyé par lorenzo8769

Ça marche nikel !!!

---

Tant mieux, mais ça marcherait bien plus efficacement si tu suivais l'ensemble de nos conseils notamment :

1) utiliser -f et -s au lieu de -f et "$(cat fichier)", je suis d'accord avec N_Bah c'est plus adapté dans cette situation (je regrette un peu d'avoir proposé l'autre solution aussi mais j'avais d'abord pensé que ce serait utile pour toi de savoir comment tester si une chaine est vide ou non).

2) privilégier la boucle while avec le changement d'IFS plutôt que de faire un for sur le résultat d'un cat avec un tas de awk en prime (ce qui est plus couteux car tu passe par des programmes externes inutilement et plus risqué dans le cas ou tes lignes contiendrait des espaces).

Bon le point 1) c'est pas bien grave. Moi aussi j'ai tendance à faire ça parce que je suis parano (-f pour être sûr qu'il s'agit d'un fichier régulier et me baser sur le contenu du fichier plutôt que sur sa taille) => mais c'est de la paranoïa souvent inutile (comme dans le cas présent) :lol:

Le point 2) en revanche, c'est très important de le mettre en pratique et prendre des bonnes habitudes : éviter de faire inutilement appel à des programmes externes qui peuvent être couteux et surtout maitriser les risques et rendre ses scripts robustes.

Cordialement,
Idriss

Bonjour

Citation:

---

Envoyé par ok.Idriss

(je regrette un peu d'avoir proposé l'autre solution aussi mais j'avais d'abord pensé que ce serait utile pour toi de savoir comment tester si une chaine est vide ou non).

---

Euh... test -n/-z "chaine" (c'est aussi dans le man)...:oops:

Citation:

---

Envoyé par lorenzo8769

Code:

---

[[ -f /repertoire/"$jour"_"$CLIENT".conf && "$(cat /repertoire/"$jour"_"$CLIENT")" ]] && echo "OK - "$date" - Téléchargement de config.conf sur "$IP" en "$jour"_"$CLIENT".conf" >> $log || echo "NOK - "$date" - Téléchargement échoué sur "$IP" pour le client "$CLIENT"" >> $log

---

---

Tu sais que tu peux améliorer la mise en forme de ton script ? Un script efficace c'est bien, un script efficace et agréable à relire (voire à modifier) c'est mieux...

Code:

---

1 2 3

[[ -f /repertoire/"$jour"_"$CLIENT".conf && "$(cat /repertoire/"$jour"_"$CLIENT")" ]]\ && echo "OK - "$date" - Téléchargement de config.conf sur "$IP" en "$jour"_"$CLIENT".conf" >> $log\ || echo "NOK - "$date" - Téléchargement échoué sur "$IP" pour le client "$CLIENT"" >> $log

---

:salut:

Citation:

---

Envoyé par Sve@r

Euh... test -n/-z "chaine" (c'est aussi dans le man)...:oops:

---

J'ai pas dit le contraire :D

Par contre j'ai trouvé utile de préciser que test "chaine" == test -n "chaine". Après comme il s'agit d'un fichier à la base, je suis d'accord avec N_Bah dans la mesure ou -s est un peu plus adapté (ça évite de lire le contenu du fichier en passant par une commande comme cat).

Idriss