[laurent45]

Bonsoir Jam, le fil, le Forum,

Jam écrit:

J'espère que tu ne prendras pas mal ces quelques réflexions.

Je te remercie beaucoup pour tes critiques. C'est tout ce que j'attendais en ouvrant ce post.

Je vais maintenant répondre aux modifications que tu me propose.

- Install: Si tu pouvais laisser le choix de l'emplacement aux utilisateurs ce sera parfait (moins il y a de chose à la racine de C: et mieux c'est en général).

Ce que je voulais protéger, c'est pour plus tard. Je souhaite faire des sauvegardes dans un sous-dossier ('Sauvegardes') du dossier 'CaissesV2.00'.

Pour cela tu peux utiliser Thisworkbook.Path afin d'obtenir le chemin de ton fichier XL et gérer le reste à partir de là.

Je ne l'avais pas vu comme cela. Pourtant j'ai recherché un moyen de faire ça, mais sans succès.
Je n'ai pas due mettre les bon mots clés. :unsure:

Bravo pour l'utilisation d'un installeur (l'initiative est assez rare pour être soulignée). Bien joué.

C'était le meilleur moyen pour avoir mon dossier où je le souhaitais. Et puis l'ouverture d'un 'Lisez Moi', j'aime bien (ça fait pro B) )

- Les barres d'outils: je partage l'avis de Pascal concernant la 'disparition' momentanée des barres d'outils. Même si ce n'est pas très grave, il arrive souvent que des utilisateurs crée une multitude de barre, et il faudrait
1: que tu les retires toutes,
2: si on bascule vers une autre feuille les rétablir
Et comme l'a souligné Hervé en cas de plantage tu met le bazar dans un truc qui rend souvent nerveux l'utilisateur...et il risque de pas aimé ton appli à cause de ce détail

J'ai rétablis les barres d'outils. Il n'y a que les onglets et les entêtes de colonnes et lignes, que je retire. Ils se rétablissent dès que l'on change de feuille (les entêtes).
C'est uniquement, pour que l'utilisateur ne puisse pas faire de modif sur les autres feuilles (c'était le souhait du demandeur, je peux presque dire client)

- Le démarrage est assez lent: En fait j'ai remarqué que tu jouais beaucoup avec la sélection des cellules...c'est lent en général. Essaie d'utiliser des noms de cellules pour accéder directement à la ligne ou carrément via un fichier INI. De même tu pourrais avant toute chose dans Workbook_Open mettre

With Application
    .ScreenUpdating = False
    .Calculation = xlCalculationManual
End With

histoire d'accélerer un peu et le rétablir une fois tes manip effectuées.

Pour ce qui est des sélections, je ne vois pas trop ce que tu veux dire.
En fait, j'avais commencé par faire un fichier INI, mais j'ai trop galéré avec les INPUT et OUTPUT (ou WRITE). J'ai une Aide (Excel97), qui est plus que limitée sur ce genre de sujet.
Pour le ScreenUpdating, il est vrai que j'aurais pu l'utiliser ici, alors qu'il est présent dans le reste du code...

- Concernant: la gestion des objets dans tes pages...please, nomme-les et utilises des boucles pour assigner leur valeur. Ton code y gagnera en lisibilité et en rapidité.

Que veux tu dire par nommes-les ?
Si ça concerne les TextBox de la page Chiffre, je trouvais plus simple de les gérer comme ça.

- Les variables: Tu as de nombreuses variables non utilisées dans ton code (cf fichier joint). Supprime les ou à défaut REM-les. De même tu as de nombreuses variables en Variant (beurk, beurk). Evite si possible d'utiliser ce type. C'est consommateur de mémoire et ça pénalise la vitesse.

Après utilisation de ton fichier .txt, j'ai fait le ménage. Mais pas tout :

La variable 'CvtMidi, CvtSoir, CARestMidi, CARestSoir, CAVAE5_5, CAVAE19_6'

est utilisée dans ModuleGeneral.Enregitrer.
Pour les Variant, il faut que je teste mes tableaux en Double.

- J'ai remarqué (je ne sais pas si c'est forcément justifié) qu'il y avait beaucoup de répétitions de code...en générale cela signifie qu'il doit y avoir un moyen de transférer cela dans une procédure à part (les événements AfterUpdate dans le userform Accueil).

Oui, j'avais au départ une fonction que Didier MyDearFriend m'a fait partager, mais avec un plantage de PC, je n'ai pu récupérer qu'une version97 d'Excel, et la fonction Replace ne fonctionnait plus. Comme certaines autres, que j'ai due modifier. d'où ces répétitions de code.
Je n'ai pas encore réessayé avec le Select dans les With proposé par notre Maître _Thierry.

Edition: Après réflexion (au fil du post, je réfléchi sur ce que j'ai écris avant), pour les évènements AfterUpdate, c'est pour avoir un séparateur décimal 'Français', plutôt qu'un point.

ReEdition: Replace ne fonctionne pas sous Excel97. J'ai refait des essais

- Remarque générale: La cosmétique c'est super dans un programme et c'est aussi important. Cependant elle ne doit intervenir qu'en fin de développement une fois qu'on est sur (on l'est jamais vraiment hein :lol: ) que le code est au point. (Ex. j'ai pas la zoulie icone qui apparaît dans XL).

Pour la fin du développement : il fallait que j'accélère pour mon client.
Tu n'as pas l'icône dans Excel ?
C'est pas cool !!
Vu le nombre de versions que j'en ai créées !!

Bon, à part cela, je trouve ton programme très bien présenté avec de bonne idées. L'utilisation des pages est assez chouette (moi qui les ai souvent détestée ). Ton code est assez riche et cela montre le temps que tu y as passé.

Un grand merci pour le temps que tu as consacré à cette critique très constructive.

Une dernière chose (pour ce soir). J'ai créé un fichier d'aide, mais je n'arrive pas à le faire prendre en compte par mon fichier.

Quelqu'un pourraît-il m'éclairer sur ce problème ?

Merci

Bonne nuit !

Edition : Toutes mes excuses Didier MDF

Message édité par: laurent45, à: 24/11/2005 15:14