Discussion :

[N_BaH]

• il manque des espaces autour de =~.
• pourquoi SYNOPSIS (qui devrait être en minuscule) n'est-il pas dans usage() ?
• plutôt que echo -e, pour un affichage formaté, utilise printf
• vérifier le format d'une adresse mail est illusoire. tout ce qu'on peut dire avec certitude c'est qu'il doit y avoir un @. dans ta regex, _ n'est pas pris en compte, ni µ...
• les fonctions ont une commande return pour retourner (!) le code d'erreur indiqué (0=<code=<255; zéro indiquant un succès, sinon cela indique un échec), et c'est ce code d'erreur qui doit être testé.
• personnellement, je préfère test à [...], mais si tu testes les codes d'erreur de tes fonctions tu n'en /.auras plus besoin.
• le printf de bash dispose d'un format horaire, qui rend date quasiment inutile.
• bien sûr, cat | grep, ligne 132, est inacceptable, et il manque un espace avant le crochet fermant.

j'en ai peut-être oublié.

[balkany]

Pourquoi introduire la fonction f_estFichier(), ne fait-elle pas que redéfinir test -f ?
Pourquoi utiliser le fichier temporaire /tmp/tempo.txt ?
Il manque quelques guillemets doubles, ou bien ils sont en trop :

Code :

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

1
2
3
f_estFichier $fichier -> f_estFichier "$fichier"
f_estEmail $email -> f_estEmail "$email"
"bulletinDeSanteLe-"$madate".txt" -> "bulletinDeSanteLe-$madate.txt"

[jack-ft]

• les fonctions ont une commande return pour retourner (!) le code d'erreur indiqué (0=<code=<255; zéro indiquant un succès, sinon cela indique un échec), et c'est ce code d'erreur qui doit être testé.

Pourquoi introduire la fonction f_estFichier(), ne fait-elle pas que redéfinir test -f ?

Outre l'une des remarques de N_BaH et celles de balkany, il semblerait que ta version de la fonction f_estFichier ne teste pas son argument, mais le contenu de la variable $fichier, même si elle devait être appelée avec un autre argument dans le script. C'est un bug sans conséquence dans le cas présent, mais qui pourrait en avoir dans un autre contexte et qui ne serait pas forcément facile à comprendre!

Bref, on pourrait devoir remplacer:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
f_estFichier()
{
	local retour=0
 
	if [[ -f $fichier ]] 
	then
		retour=1
	fi
	echo "$retour"
}
 
elif [ $(f_estFichier $fichier) -eq 0 ]

par

Code :

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

1
2
3
f_estFichier() { test -f "$1"; }
 
elif f_estFichier "$fichier"

Et on voit que le gain de la fonction est passablement minime...