Salut à tous,
J'ai survolé l'article, je n'ai même pas testé le code... mais j'ai des critiques
: (j'ai lu succinctement le code quand même
)
- Le module M_Replace ne sert pas. Il serait préférable de ne pas le mettre.
- nomDossier = CurrentProject.Path :
il serait préférable :
nomDossier = environ("temp"
- il serait préférable d'avoir une variable que de mettre en dur "Réabonnements" (si c'est dans le temp ce n'est plus utile)
- le pdf généré n'est pas détruit alors qu'il ne sera jamais réutilisé. (le temp ferait dans ce cas encore très bien l'affaire surtout lors d'un arrêt intempestif)
- il n'y a pas de gestion d'erreur pour EnvoiDocuments dans TestReabonnements alors qu'il est bien traité dans CmdEnvoyerDocuments_Click
- mais ce que je déplore c'est qu'il n'y pas de gestion sur les erreurs d'envoi. Sans entrer dans quelque chose d'élaboré, je pense qu'à minima il serait bon d'avoir une table ou fichier log.
... je regarde pas plus en détail, je voudrai pas te rendre dépressif User
histoire de pas me faire descendre, un peu de flatterie :
J'ai apprécié la forme, le codage structuré, clair et bien commenté.
Au passage Je n'ai pas testé la base, je me suis arrêté avant l'envoi. J'ai eu un doute :
J'espère que les emails sont factices parce que sinon j'en connais qui vont te maudire.
Merci pour ta (tes) contribution(s).
1 |
0 |
Bonjour à tous,
Envoyé par tee_grandbois
et 
