Kohler
Messages postés21Date d'inscriptionmardi 17 février 2004StatutMembreDernière intervention 1 juin 2004
-
1 juin 2004 à 11:20
ManDeq59
Messages postés55Date d'inscriptiondimanche 12 mai 2002StatutMembreDernière intervention 6 juillet 2004
-
1 juin 2004 à 19:50
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
ManDeq59
Messages postés55Date d'inscriptiondimanche 12 mai 2002StatutMembreDernière intervention 6 juillet 2004 1 juin 2004 à 19:50
Merci pour tous es commentaires Xya et merci aussi Kohler pour ton appréciation.
Je vais faire les modifs
Xya
Messages postés103Date d'inscriptionlundi 8 juillet 2002StatutMembreDernière intervention24 novembre 2005 1 juin 2004 à 18:42
Plusieurs petits conseils en vrac:
Tu pourrais enlever un tas de références qui servent pas (ComCtl3, MSChart20Lib, MSForms et VBRUN) qui rendent ton zip inutilement gros (1,7 Mo sur 1,8 Mo du contenu, en les supprimant tu passe à un zip de 28 Ko...)
Ensuite pour activer les thèmes XP pas besoin de manifeste, il suffit de placer "Application.EnableVisualStyles()" dans le Sub New de la form (Dans la région "Code généré par le Concepteur"), juste avant InitializeComponent() et bien sur de mettre pour tous les contrôles qui en ont besoin FlatStyle = FlatStyle.System comme tu le fais.
Pour déterminer le type d'un objet, utilise "Is" plutôt que "X.GetType.Name.IndexOf("CheckBox") >= 0" (Dans EnableXP)
--> "X Is CheckBox"
Déclare un sub plutôt qu'une fonction si tu ne retournes pas de valeur (EnableXP)
Autre chose évite l'utiliser On Error Goto xxx, utilise pluôt un Try/Catch avec un bloc Catch vide (dans NbSSReseau_MouseDown)
Voilà, j'espère que ça te sera utile, le code est bien mais tu pourrais encore le rendre plus clair et facile à lire.
Xya
Kohler
Messages postés21Date d'inscriptionmardi 17 février 2004StatutMembreDernière intervention 1 juin 2004 1 juin 2004 à 11:20
1 juin 2004 à 19:50
Je vais faire les modifs
1 juin 2004 à 18:42
Tu pourrais enlever un tas de références qui servent pas (ComCtl3, MSChart20Lib, MSForms et VBRUN) qui rendent ton zip inutilement gros (1,7 Mo sur 1,8 Mo du contenu, en les supprimant tu passe à un zip de 28 Ko...)
Ensuite pour activer les thèmes XP pas besoin de manifeste, il suffit de placer "Application.EnableVisualStyles()" dans le Sub New de la form (Dans la région "Code généré par le Concepteur"), juste avant InitializeComponent() et bien sur de mettre pour tous les contrôles qui en ont besoin FlatStyle = FlatStyle.System comme tu le fais.
Pour déterminer le type d'un objet, utilise "Is" plutôt que "X.GetType.Name.IndexOf("CheckBox") >= 0" (Dans EnableXP)
--> "X Is CheckBox"
Déclare un sub plutôt qu'une fonction si tu ne retournes pas de valeur (EnableXP)
Autre chose évite l'utiliser On Error Goto xxx, utilise pluôt un Try/Catch avec un bloc Catch vide (dans NbSSReseau_MouseDown)
Voilà, j'espère que ça te sera utile, le code est bien mais tu pourrais encore le rendre plus clair et facile à lire.
Xya
1 juin 2004 à 11:20