Discussion :

[lorenzo8769]

Bonjour à tous,

Voici mon script:

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

[nanooby]

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é !

[ok.Idriss]

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 :

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

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 :

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

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 :

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

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 :

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

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

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

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

[N_BaH]

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

Code :

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

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

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

Code :

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

while IFS=':' read -r

merci.

[ok.Idriss]

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 :

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

=>

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
Et "différente de 0" est inexact, j'aurais du dire "plus grande que 0"

Idriss

[N_BaH]

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]

[lorenzo8769]

En tout cas merci à vous !!!!

Avec ceci:

Code :

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

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

Ça marche nikel !!!

Voici le résultat:

Code :

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

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

[N_BaH]

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

[ok.Idriss]

Re bonjour.

Ç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)

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

[Sve@r]

Bonjour

(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)...

Code :

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

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

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

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

[ok.Idriss]

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

J'ai pas dit le contraire

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