fkx
Messages postés44Date d'inscriptionjeudi 29 janvier 2004StatutMembreDernière intervention26 juin 2006 17 nov. 2004 à 12:41
DarkSidious> NON, "Case 1080840", c'est beaucoup mieux que "Case RGB(24,52,36)" (comparaison directe, sans appel à une fonction).
PAR CONTRE, il faut mettre un petit commentaire à côté pour expliquer d'ou vient le nombre "louche".
On aura donc un truc du genre :
Case 1080840 'RGB(24, 52, 36)
cboulas
Messages postés2641Date d'inscriptionmercredi 2 juin 2004StatutMembreDernière intervention 8 janvier 201416 12 nov. 2004 à 09:20
DarkSidious> oui je sais les variables public faut éviter, mais je ne voulais pas utilise un module juste pour des variables, et vu que j'avais besoin d'altérer les variables depuis une autre form, c'est pour cela.
Pour les remarques je suis totalement d'accord avec toi, ce petit jeu est une nouveauté pour moi je n'avais jamais tenté cela. Merci pour tes remarques.
cs_DARKSIDIOUS
Messages postés15814Date d'inscriptionjeudi 8 août 2002StatutMembreDernière intervention 4 mars 2013130 12 nov. 2004 à 09:08
Je te conseillerais d'utiliser les fonctions de l'API pour récupèrer les couleurs des points plutôt que de passer par les fonctions VB : c'est beaucoup plus rapide ! Voir même ma classe de gestion d'image (environ 30 fois plus rapide). C'est peut-être pas trop perceptible pour le moment, mais si tu veux faire des tests plus aboutis par la suite, tu risque être limité niveau rapidité.
De plus, évite les "Case 1080840" qui ne sont vraiment pas parlant, préfère plutôt des "Case RGB(24,52,36" par exemple.
Beaucoup de duplication de code, tu pourrais faire bien plus compact si tu utilisais des fonctions spécifiques.
Cela fait beaucoup de variables publiques également : il vaux mieux éviter cà !
Bref, beaucoup de remarques, mais c'est pour faire progresser le soft ;)
DarK Sidious
cboulas
Messages postés2641Date d'inscriptionmercredi 2 juin 2004StatutMembreDernière intervention 8 janvier 201416 11 nov. 2004 à 19:06
Jack > Ayé c'est corriger
cboulas
Messages postés2641Date d'inscriptionmercredi 2 juin 2004StatutMembreDernière intervention 8 janvier 201416 11 nov. 2004 à 17:12
Je travail justement dessus car en fait il repose la question on est sur une zone blanche, mais j'y travail je suis en train de rajouter justement des conditions.
cs_Jack
Messages postés14006Date d'inscriptionsamedi 29 décembre 2001StatutModérateurDernière intervention28 août 201579 11 nov. 2004 à 17:06
C'est gentil ...
Quand on refuse l'achat chez le druide, il repose la question à chaque fois.
17 nov. 2004 à 12:41
PAR CONTRE, il faut mettre un petit commentaire à côté pour expliquer d'ou vient le nombre "louche".
On aura donc un truc du genre :
Case 1080840 'RGB(24, 52, 36)
12 nov. 2004 à 09:20
Pour les remarques je suis totalement d'accord avec toi, ce petit jeu est une nouveauté pour moi je n'avais jamais tenté cela. Merci pour tes remarques.
12 nov. 2004 à 09:08
De plus, évite les "Case 1080840" qui ne sont vraiment pas parlant, préfère plutôt des "Case RGB(24,52,36" par exemple.
Beaucoup de duplication de code, tu pourrais faire bien plus compact si tu utilisais des fonctions spécifiques.
Cela fait beaucoup de variables publiques également : il vaux mieux éviter cà !
Bref, beaucoup de remarques, mais c'est pour faire progresser le soft ;)
DarK Sidious
11 nov. 2004 à 19:06
11 nov. 2004 à 17:12
11 nov. 2004 à 17:06
Quand on refuse l'achat chez le druide, il repose la question à chaque fois.