AFFICHAGE SOUS FORME DE LED DE L'HEURE COMME LES HORLOGES DE RADIOS

Commentaire de Afyn le 23/01/2004 21:57:59

J'ai pas pu charger le Zip ...
Y a ds problèmes sur le serveur ....

A+

Afyn

Commentaire de jack le 24/01/2004 00:13:37 administrateur CS

Juste pour sourire, je n'ai pas encore regardé le code : De zéro à 10, bah non, ça fait pas 10 ! mais 11. En fait, tu as voulu dire de zéro à 9.
Sinon, bravo et merci pour cette présentation en phrases avec des sujets, des verbes : c'est plaisant (et très rare) à lire.
Je vais zyeuter le code.

Commentaire de jack le 24/01/2004 02:00:09 administrateur CS

Bon alors, après la rigolade, les critiques (gentilles et constructives) :
-1- Concept : j'aime bien, j'ai toujours révé d'avoir ce genre d'horloge dans mon salon. Par contre, un détail qui tue : Sur les "vraies", la remise à zéro du cadran se fait à 0, et pas à 1 sec : En fait le programme est bon, mais comme l'index 0 de la première LED est à l'emplacement de la première seconde, ça décale tout.
-2- Une option qui existe sur les "vraies" zhorloges : quand arrive la 60è seconde, les led restent dans l'état, mais les secondes qui défilent éteignent les leds au lieu de les allumer au tour précédant, en alternance. Une idée d'option.
-3- Quand je l'ai lancée, j'ai trouvé que les leds éteintes étaient beaucoup trop voyantes, à cause du bord qui reste rouge --> J'ai donc mis les couleurs à appliquer aux leds éteintes et allumées dans des variables, en créant une variable Fond et une autre Bord. Comme ça, on peut jouer avec les couleurs très facilement. On pourrait donc facilement proposer à l'utilisateur de choisir ses propres couleurs .
-4- Dans le code, l'emploi de If pour chaque valeur de P est un peu lourde. Toi qui a voulu rationaliser le code, il vaudrait mieux utiliser un "Select Case" au lieu de ces If
-5- L'instruction "RAZ ClnShp" est répétée 10 fois --> Je l'ai déplacée au début de la Sub
-6- (et dernier) : Lorsqu'on fait la RAZ, je trouve que les leds qui tombent toutes les 5 secondes devraient ressortir -> j'ai créé une variable couleur de bord pour celles-là aussi.

Si tu veux, je peux te renvoyer le projet modifié -> passe moi ton email en privé (pas dans ces commentaires)

Bref : Principe bien pensé. L'utilisation de collection d'objet pour les passer en argument de Sub est pratique.

Commentaire de Huugooo le 27/01/2004 21:08:54

C'est pas mal j'ai ainsi découvert comment se servir de colections. Merci ! D'ailleurs j'ai ainsi pu découvrir une façon de passer un nom de contrôle en paramêtre (bien qu'il doit y avoir une méthode plus propre non ?).

Juste un gros hic.
A quoi sert ta boucle
For Each SHP In ClnShp  'on boucle sur la collection
Next
?????

Parcequ'en y regardant de plus près elle ne sert à rien, je l'ai enlevée et cela fonctionne toujours et évite ainsi un paquet de calculs.
Hugo

Commentaire de RcrivelliNET le 06/03/2004 00:43:21

Salut,
je veut juste te remercier pour ton code source, qui m'a enfin fait découvrir a quoi servait la proprieté Index lol (et oui, en en apprend tout les jours), merci aussi pour la fabuleuse richesse de tes commantaires... je comprend maintenant pourquoi on m'a temponner aussi souvent en cours pour que je commante mon code !!! :-).

A+

Commentaire de BruNews le 13/03/2004 14:05:14 administrateur CS

Pouvoir > Y a un prob dans ton accusation, cette source est arrivee un mois AVANT la tienne. Sans compter qu'il y a egalement le meme genre de commentaire sur la tienne.

BruNews, Admin CS, MVP Visual C++

Commentaire de muad-dib le 08/02/2006 22:48:20

amusant :)