VINCEBULLES
killerkurgan
Messages postés
9
Date d'inscription
mercredi 28 juillet 2010
Statut
Membre
Dernière intervention
19 août 2010
-
18 août 2010 à 09:34
killerkurgan Messages postés 9 Date d'inscription mercredi 28 juillet 2010 Statut Membre Dernière intervention 19 août 2010 - 19 août 2010 à 10:34
killerkurgan Messages postés 9 Date d'inscription mercredi 28 juillet 2010 Statut Membre Dernière intervention 19 août 2010 - 19 août 2010 à 10:34
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/52191-vincebulles
https://codes-sources.commentcamarche.net/source/52191-vincebulles
19 août 2010 à 10:34
C'est vrai que ce code n'est pas très optimisé, mais à ma décharge c'était pour me remettre à VB6 que je n'avais plus pratiqué depuis 3 ou 4 ans...
19 août 2010 à 07:45
2-Optimiser un code aussi simple est necessaire pour montrer que sur ce site on se fou pas de la tronche des autres ! Ecrire 25 fois les 3 mêmes lignes dasn des subs de bouton_click n'est vraiment pas follichont. Si tu considére que c'est une prise de tête attend le jour ou tu devra rajouter une action dans toutes ces lignes !
3-C'est pas mon code c'est le TIENS ! Et ma question et pourquoi tester le Down, puis le Press et enfin le Up, Trop n'a jamais manqué, mais sur un site ou l'on poste pour montrer des exemples UTILES il est préférable de savoir qu'elle événement est le bon, plutôt que de tous les utiliser en ce disant, là au moins cela marchera. Tu me comprend ?
Conclusion : Les débutants qui vont lire ta source et essayer de la comprendre vont vite se perdre avec trois évenements qui font la même chose, pire il vont prendre de mauvaise habitude en te lisant. Grouper les boutons en VB6 est super simple, en VB.NET c'est un poil plus chaud, mais cela reste à mon sens une OBLIGATION ! Tout comme déclarer ses variables (test)
Cordialement, bonne prog.
18 août 2010 à 20:56
2/ Pour la factorisation, je le laisse a ceux qui en auront le courage, optimiser un code si simple n'apportera rien si ce n'est de la prise de tête.
3/ Ton code pour décharger la form est insuffisant pour un screensaver, il ne prend pas en compte la souris.
Enfin je te remercie quand même, tu a été le seul a avoir pris le temps de me laisser un commentaire, et j'apprécie :D
18 août 2010 à 18:13
pourquoi :
422.Dim Test
423.Test = ModifINI("couleurbulles", "4")
1-dim test c'est quoi ?
2-pourquoi pas dim une bonne fois pour toute ?
3-pourquoi faire retourner un "truc" à ModifINI s'il n'y a pas de test derriére ?
4-pourquoi ne pas factoriser un peu tout ce code ?
52.Private Sub Form_KeyDown(KeyCode As Integer, Shift As Integer)
53.Unload Me
54.End Sub
55.
56.Private Sub Form_KeyPress(KeyAscii As Integer)
57.Unload Me
58.End Sub
59.
60.Private Sub Form_KeyUp(KeyCode As Integer, Shift As Integer)
61.Unload Me
62.End Sub
ah ben là tu es sûr de décharger ta form ^__^, pourquoi autant de test ?
@++
18 août 2010 à 10:55
Il y a donc juste le .scr à mettre dans System32...
18 août 2010 à 10:42
18 août 2010 à 09:34