[Bash] Scripts vsftpd

[Junky]

Bonjour,

Merci du partage.

Sans vouloir aller chercher la p'tit bête, tu devrais imo passer par des variables et éviter de mettre en dur les path des files. Si un jour pour une question ou une autre tu dois changer de rep, tes scripts deviennent pénibles a modifier. Tu peux faire un truc du genre:

Code BASH :


conf_dir="/etc/vsftpd/"
login_file="login.txt"
backup_ext=".bak"

ect....
 

Si un jour, tu changes de repertoire, tu aura besoin de modifier QUE les variables et non tout le programme.

En plus en mettant en place cette dynamique, ca te permet de faire une fonction de check. Plein de fois dans ton code tu vas checker si le backup d'un fichier c'est bien réaliser. Donc a chaque fois tu poses un:

Code BASH :


if [ ! f CONDITION]; then
     blablabla....
fi
 

Dans le premier script tu en as 4 ou 5.. N'oublie pas qu'en dev a partir du moment ou tu fais 2 fois la même chose, il faut faire une fonction.

Donc un truc du genre:

Code BASH :


check() {
if [ ! f $conf_dir$login_file$backup_ext ]; then
        echo "The game"
        exit 1
fi
}
 

Après dans l'ensemble si ca fait le taf j'ai envie de dire GG. Mais regarde un peu plus sur l'optimisation de ton code...

Ah oui dans le 4ème script, mis WTF le if... Il déroule sur 4km le truc... Je l'ai survolé seulement (j'suis au taf pas eu le tps de le lire correctement) mais en revenant sur mon idée précédente de fonction, il y a surement de faire un truc vachement plus propre...

Bon courage.

Junky