cs_Urgo
Messages postés780Date d'inscriptionlundi 16 décembre 2002StatutMembreDernière intervention16 avril 2009
-
19 mai 2003 à 14:33
Saros
Messages postés921Date d'inscriptionvendredi 20 décembre 2002StatutMembreDernière intervention23 septembre 2010
-
19 mai 2003 à 18:33
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
Saros
Messages postés921Date d'inscriptionvendredi 20 décembre 2002StatutMembreDernière intervention23 septembre 2010 19 mai 2003 à 18:33
Bijour ;
Houlà c'est vrai que le code est vraiment pas optimisé... Le nombre de timer est hallucinant (t'as mis combien te temps à tous les renommer ?), il aurait fallu en créer dynamiquement, en utiliser les tableaux de contrôle : tu crées 1 seul timer que tu appelles tmrMatrix (par exemple), tu lui mets la propriété Index à 0, puis :
for i = 1 to nbrTimers
load tmrMatrix(i)
next i
t'en crée nbrTimers autres. Tu peux faire ça avec tous les contrôles que tu veux.
Sinon pour les variables même chose, utilise des tableaux :
Dim testl(0 to n) as boolean
tu verras le code sera beaucoup plus court et plus simple à lire...
Et bien à part ça tout va bien (pas de bugs notables)
Bon je mettrais pas de notes pour l'instant, avant de voir l'évolution...
Bonne prog
Saros
lumesh
Messages postés564Date d'inscriptionjeudi 21 février 2002StatutMembreDernière intervention 7 novembre 2008 19 mai 2003 à 18:20
Private ou Dim peut importe cela ne change strictement rien !
Par contre il est vrai que tu accelerais grandement laccces aux variables si tu les mettaient toutes en boolean au lieu de variant (les temps d'acces sont differents).
Sinon ouai quand tu fais un code plutot graphique, essaye de merttre une capture, c'est toujours sympa de voir de quoi ca retourne surtout que les titres ne sont pas toujours tres explicite !)
Ne pas mettre de capture, forcer les gens a DL ta source n'est pas ce que l'on fait de plus apprécié et au contraire rebute le plus (perso je DL pas n'importe koi ! pas par parano mais pour eviter le bordel sur mon DD).
Voila voilou :)
facdaar
Messages postés64Date d'inscriptionlundi 24 mars 2003StatutMembreDernière intervention23 février 2009 19 mai 2003 à 17:43
Juste une remarque :
Quand tu déclares la ligne suivante dans Form1:
Dim testl1, testl2, testl3, testl4, testl5, testl6, testl7, testl8, testl9, testl10 As Boolean
D'une part, tu utilises Dim, alors que Private serait plus approprié.
D'autre part, dans cette déclaration, seul testl10 est un boolean, les autres sont des Variants car tu n'as pas précisé...
Sinon RAS
ZedMaTrix
Messages postés178Date d'inscriptionvendredi 25 octobre 2002StatutMembreDernière intervention14 août 2008 19 mai 2003 à 17:09
ben franchement pour mon niveau en regardant vite fais, je trouve que c'est pas mal du tout du tout :o) (vous comprendrez pourquoi !)
Moi je croix que ce qui claquerai plus c'est si il y en avait plus de lettres.Ca fait un peu vide sinon j'aime BCP
Améliore c'est un trés trés bon début !
kilomaster
Messages postés130Date d'inscriptionmercredi 2 janvier 2002StatutMembreDernière intervention 6 décembre 2006 19 mai 2003 à 14:51
le résultat est moyen, mais t'es sure que t'as besoin d'autant de timers?
sinon il reste à l'améliorer un peu, et c pas trop mal
moi je mets 5/10
@++
sebleboss2002
Messages postés189Date d'inscriptionlundi 30 décembre 2002StatutMembreDernière intervention 3 avril 2005 19 mai 2003 à 14:38
Je suis pas un pro en VB (désolé), mais je peux quand-même te dire que ton code est vachement pas optimisé ;)
cs_Urgo
Messages postés780Date d'inscriptionlundi 16 décembre 2002StatutMembreDernière intervention16 avril 20091 19 mai 2003 à 14:33
T'aurait pu mettre une capture ou décrire de quoi ça parle car là on n'en sait rien a part un truc sur matrix pfff
19 mai 2003 à 18:33
Houlà c'est vrai que le code est vraiment pas optimisé... Le nombre de timer est hallucinant (t'as mis combien te temps à tous les renommer ?), il aurait fallu en créer dynamiquement, en utiliser les tableaux de contrôle : tu crées 1 seul timer que tu appelles tmrMatrix (par exemple), tu lui mets la propriété Index à 0, puis :
for i = 1 to nbrTimers
load tmrMatrix(i)
next i
t'en crée nbrTimers autres. Tu peux faire ça avec tous les contrôles que tu veux.
Sinon pour les variables même chose, utilise des tableaux :
Dim testl(0 to n) as boolean
tu verras le code sera beaucoup plus court et plus simple à lire...
Et bien à part ça tout va bien (pas de bugs notables)
Bon je mettrais pas de notes pour l'instant, avant de voir l'évolution...
Bonne prog
Saros
19 mai 2003 à 18:20
Par contre il est vrai que tu accelerais grandement laccces aux variables si tu les mettaient toutes en boolean au lieu de variant (les temps d'acces sont differents).
Sinon ouai quand tu fais un code plutot graphique, essaye de merttre une capture, c'est toujours sympa de voir de quoi ca retourne surtout que les titres ne sont pas toujours tres explicite !)
Ne pas mettre de capture, forcer les gens a DL ta source n'est pas ce que l'on fait de plus apprécié et au contraire rebute le plus (perso je DL pas n'importe koi ! pas par parano mais pour eviter le bordel sur mon DD).
Voila voilou :)
19 mai 2003 à 17:43
Quand tu déclares la ligne suivante dans Form1:
Dim testl1, testl2, testl3, testl4, testl5, testl6, testl7, testl8, testl9, testl10 As Boolean
D'une part, tu utilises Dim, alors que Private serait plus approprié.
D'autre part, dans cette déclaration, seul testl10 est un boolean, les autres sont des Variants car tu n'as pas précisé...
Sinon RAS
19 mai 2003 à 17:09
Moi je croix que ce qui claquerai plus c'est si il y en avait plus de lettres.Ca fait un peu vide sinon j'aime BCP
Améliore c'est un trés trés bon début !
19 mai 2003 à 14:51
sinon il reste à l'améliorer un peu, et c pas trop mal
moi je mets 5/10
@++
19 mai 2003 à 14:38
19 mai 2003 à 14:33