BunoCS
Messages postés15476Date d'inscriptionlundi 11 juillet 2005StatutModérateurDernière intervention 3 mai 2024103 16 sept. 2011 à 10:51
Hello,
Je n'ai pas générer d'exe, j'ai juste regardé le code, mais j'ai quelques remarques:
- attention aux chemins en durs et en absolus: celui(celle) qui va vouloir tester ton source va être surpris
- la fonction GenerateKeyWPA semble avoir été intégrée: elle ne garde pas la même homogénéité que le reste du code et elle est avare en commentaire ;)
- dans cette même fonction, je te conseille de renommer tes variables locales: b1, b2, b3,...ça ne parle pas beaucoup
- les couleurs des boutons sont, comment dire...enfin, voilà quoi. Etait-ce vraiment nécessaire ;)
Lors de ta prochaine mise à jour, ne zippe pas bêtement ton répertoire de dev! Il y a des choses inutiles:
- répertoires bin, obj, _Upgrade..
- pourquoi 2 fichiers sln? Un seul suffit, non?
- fichiers .suo, .user : qui contiennent des infos "personnelles"
16 sept. 2011 à 10:51
Je n'ai pas générer d'exe, j'ai juste regardé le code, mais j'ai quelques remarques:
- attention aux chemins en durs et en absolus: celui(celle) qui va vouloir tester ton source va être surpris
- la fonction GenerateKeyWPA semble avoir été intégrée: elle ne garde pas la même homogénéité que le reste du code et elle est avare en commentaire ;)
- dans cette même fonction, je te conseille de renommer tes variables locales: b1, b2, b3,...ça ne parle pas beaucoup
- les couleurs des boutons sont, comment dire...enfin, voilà quoi. Etait-ce vraiment nécessaire ;)
Lors de ta prochaine mise à jour, ne zippe pas bêtement ton répertoire de dev! Il y a des choses inutiles:
- répertoires bin, obj, _Upgrade..
- pourquoi 2 fichiers sln? Un seul suffit, non?
- fichiers .suo, .user : qui contiennent des infos "personnelles"