LE VOYAGEUR

cs_Jack Messages postés 14006 Date d'inscription samedi 29 décembre 2001 Statut Modérateur Dernière intervention 28 août 2015 - 11 nov. 2004 à 17:06
fkx Messages postés 44 Date d'inscription jeudi 29 janvier 2004 Statut Membre Dernière intervention 26 juin 2006 - 17 nov. 2004 à 12:41
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/27493-le-voyageur

fkx Messages postés 44 Date d'inscription jeudi 29 janvier 2004 Statut Membre Dernière intervention 26 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és 2641 Date d'inscription mercredi 2 juin 2004 Statut Membre Dernière intervention 8 janvier 2014 16
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és 15814 Date d'inscription jeudi 8 août 2002 Statut Membre Dernière intervention 4 mars 2013 130
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és 2641 Date d'inscription mercredi 2 juin 2004 Statut Membre Dernière intervention 8 janvier 2014 16
11 nov. 2004 à 19:06
Jack > Ayé c'est corriger
cboulas Messages postés 2641 Date d'inscription mercredi 2 juin 2004 Statut Membre Dernière intervention 8 janvier 2014 16
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és 14006 Date d'inscription samedi 29 décembre 2001 Statut Modérateur Dernière intervention 28 août 2015 79
11 nov. 2004 à 17:06
C'est gentil ...
Quand on refuse l'achat chez le druide, il repose la question à chaque fois.
Rejoignez-nous