Appel de sous-programmes dans un script Bash

Version imprimable

Bonsoir à tous et à toutes,

J'ai une petit script qui me permet de récupérer des sites Internet, en lftp, pour les sauvegarder.
Pour en faciliter la lisibilité ainsi que la modularité, je voudrais le scinder en un programme principal et en autant de sous-programmes que de sites.

J'ai donc fait ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 #!/bin/bash #========================================================================================= # Récupérer les sites Internet via LFTP #========================================================================================= #utilisateur=abach DATE=`date +%Y%m%d_%H%M%S` destracine=/home/abach/En-cours/lftp fichierlog=recup_sites.log cd /home/$utilisateur/scripts clear # ==================================== $home/scripts/recup_site_1.sh $home/scripts/recup_site_2.sh $home/scripts/recup_site_3.sh $home/scripts/recup_site_4.sh clear cat $fichierlog
La première partie étant commune, je voudrais passer les variables 'DATE, destracines, fichierlog' aux sous-progammes mais manifestement, ça ne fonctionne pas aussi facilement que je ne le pensais...

Auriez-vous une solution ?

31/08/2015, 01h28
N_BaH

Bonjour,

qu'est-ce qui différencie les scripts recup_site_[1-4].sh ?

j'écrirais plutôt une fonction unique, qui accepte des paramètres, appelées dans une boucle pour traiter chaque site...
?

Pas grand chose ne différencie les différents "sous-scripts".
Pour info, en voilà un. J'ai mis en gras les particularités

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
# ========================================================================================
# Récupération de site_1.fr
# ========================================================================================
site=site_1.fr
DATE_SITE=`date +%Y%m%d_%H%M%S`
echo "========================================================="
echo "Récupération de $site"
echo "========================================================="
echo "Récupération de $site commencée à $DATE_SITE" > $fichierlog
mkdir $destracine/$DATE
dest=$destracine/$DATE/$site
echo $dest
mkdir $dest
lftp ftp://identifiant:motdepasse@serveur_ftp -e "mirror -e /www/ $dest/ ; quit"

31/08/2015, 06h52
N_BaH
ça confirme ce que je pensais.

mets toutes les données de chaque site dans un seul ficher CSV (cf. Comment lire un fichier en shell).
parcours ce fichier pour récupérer les informations de chaque site; le traitement se fera par une fonction unique dans le seul script principal.
Code:

1 2 3 4 5 6 7 maFonctionAmoi() { echo "premier argument : $1" echo "deuxième argument : $2" echo "troisième argument : $3" } maFonctionAmoi un "deux trois" quatre cinq
31/08/2015, 18h04
Aenean

Merci pour le tuyau !
J'ai effectivement fait une fonction et je passe les info en paramètres.

Pour faire simple, je n'ai pas fait de "CSV" mais directement dans le script.

Citation:

Envoyé par Aenean

Pour en faciliter la lisibilité ainsi que la modularité, je voudrais le scinder en un programme principal et en autant de sous-programmes que de sites.

Bonjour

L'utilité d'un sous-script est de pouvoir être aussi appelé de façon externe par un utilisateur si celui-ci a besoin d'effectuer juste le travail du sous-script. En revanche, si ton sous-script est juste dédié à être exécuté depuis celui-là, il devient alors inutile de le mettre en tant que "script externe". Il vaut mieux le mettre en tant que "fonction". Au résultat il n'y a aucune différence (ni dans la façon de l'appeler) mais ça évite de démultiplier inutilement les petits scripts.
Code:

1 2 3 4 5 6 7 8 9 #!bin/bash toto() # Définition de ma fonction { ... ... ... } toto # Appel de ma fonction
Citation:

Envoyé par Aenean

La première partie étant commune, je voudrais passer les variables 'DATE, destracines, fichierlog' aux sous-progammes mais manifestement, ça ne fonctionne pas aussi facilement que je ne le pensais...
Auriez-vous une solution ?

Suffit de passer tes variables (ou même des valeurs) en tant qu'argument. Ceux-ci seront reçus dans le sous-script (ou dans la fonction) dans les paramètres positionnels $1, $2, $3,.... C'est expliqué dans tous les tutos de shell. Ce n'est pas parce qu'il y a des forums qu'il faut se dispenser de commencer par étudier par soi-même...

Code:

$home/scripts/recup_site_4.sh valeur1 valeur2 $(pwd) $HOME $LOGNAME

Exemple de code de "recup_site_4.sh"
Code:

1 2 3 4 5 6 7 8 #!/bin/bash echo "Le premier paramètre reçu est $1" echo "Le second paramètre reçu est $2" echo "Je me trouve dans $3" echo "Mon home directory est $4 (suis-je bête, je l'avais déjà dans $HOME !!!)" echo "Je suis l'utilisateur $5 (ou $LOGNAME)" echo "J'ai reçu $# paramètres qui sont (en vrac) $*" ...
Et la même chose avec une fonction
Code:

1 2 3 4 5 6 7 8 9 10 11 12 #!/bin/bash recup_4_site() { echo "Le premier paramètre reçu est $1" echo "Le second paramètre reçu est $2" echo "Je me trouve dans $3" echo "Mon home directory est $4 (suis-je bête, je l'avais déjà dans $HOME !!!)" echo "Je suis l'utilisateur $5 (ou $LOGNAME)" echo "J'ai reçu $# paramètres qui sont (en vrac) $*" } recup_4_site valeur1 valeur2 $(pwd) $HOME $LOGNAME
PS: $HOME et $LOGNAME sont des variables prédéfinies. Ca t'évitera de recréer les tiennes contenant la même chose...
PS2: tu as (ou t'as le droit de créer) un dossier nommé "bin" dans ton $HOME dédié à recevoir tes scripts persos. C'est plus conventionnel que "scripts" (même si ce n'est pas interdit) et ton environnement est (sauf cas particulier) déjà prévu pour taper dedans (variable "$PATH") quand tu appelles un programme
PS3: c'est très rarement utile de faire du "cd" dans un script.

Citation:
Envoyé par Aenean
Pour info, en voilà un.
Code:

1 2 3 4 5 # ======================================================================================== # Récupération de site_1.fr # ======================================================================================== [snip] lftp ftp://identifiant:motdepasse@serveur_ftp -e "mirror -e /www/ $dest/ ; quit"
Hors-sujet, mais attention!

Pour des raisons de sécurité, il n'est pas recommandé de mettre des mots de passe en clair dans un script. :rouleau:

Il vaut mieux utiliser un mécanisme de clés publiques/privées.