Discussion :

[Aenean]

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 :

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
#!/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 ?

[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...
?

[Aenean]

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 :

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

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"

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

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

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

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

[Sve@r]

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

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

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

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

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

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

Exemple de code de "recup_site_4.sh"

Code bash :

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

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

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

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.

[jack-ft]

Pour info, en voilà un.

Code :

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

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.

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