HORLOGE TYPE DELS

Commentaire de SuperDevy le 26/06/2004 21:09:09

L'idée est plutôt bonne (certainement inspirée).
Les événements programmés

J'aime beaucoups le système de points pour les secondes. C'est du déjà vu, mais ça m'amuse toujours :)

J'ai quelques remarques (ne le prend pas mal, mais je trouve que c'est un peut du bricolage ton truc) :

- Il y a beaucoups plus simple pour la fonction 'Mois()' : le tableau.
nomsMois = new Array("Janvier","Février","Mars",...,"Décembre");
txt_mois = nomsMois[myDate.getMonth()];
- Pareil pour la fonction 'Jour()'

- Tu n'as pas besoin de déclarer les variables avant de les utiliser !
fonction 'createBallon()' : "var i;" est inutile.

- C'est lourd de taper '_root' devant tout tes noms de variable ! Tu est déjà sur la racine, donc c'est inutile.

- Evite de mettre de 'Mouse.hide()' à tire la rigot ! OK, ça montre que tu sais utiliser Flash, mais c totalement inutile et destabillisant pour ton utilisateur.

- Au lieu de mettre une boucle entre deux frames (2 et 3), tu devrais utiliser l'événement _root.onEnterFrame.

Peut-être que ça fait beaucoups de choses alors je te met une version à MA manière de ta source : http://j.tamarelle.free.fr/Flashkod/faves - horloge type dels (d'ici quelques minutes). Tu pourras comparer.

PS: si tu n'es pas d'accord avec moi certains points, n'ésite pas à me le dire. (Je ne m'en suis pas privé !)

SuperDevy

Commentaire de faves le 27/06/2004 23:22:34

Merci pour les commentaire SuperDevy
il y a certain poin sur lesquels javais déjà plaché, mais cété pour une utilisation perso :
- Pour Mouse.hide() -> je n'avais pas besaoin de la souris vu que je m'en sert comme écran de veille.
- createBallon() c'était une fonction où je voullais modifier le nbre de ballons qui entrait; mais c'est devenu inutil effectivement!

De plus, comme je l'ai mis sur flashkod un peu vite, il est vrai que certaine modif serai la bien venue.
Donc je vais voir ce que deviens ma source après ces modifs ...

Commentaire de faves le 27/06/2004 23:53:35

bon, je te laisse lire et je met la sourse modifiée à jours (avec ton logo ????)

Commentaire de faves le 28/06/2004 13:05:39

c'est bon, j'ai compris tt seul :
le // Clignotement
event_midi = !event_midi;
event_noel = !event_noel;
event_nouvAn = !event_nouvAn;
event_paques = !event_paques;

est à peu près équivalent aux else if(event_truc && machin != 0 && ...)
c'est peu être plus long mon truc, mais je comprend moin bien quand il est sous la forme
// Clignotement
event_midi = !event_midi;
event_noel = !event_noel;
event_nouvAn = !event_nouvAn;
event_paques = !event_paques;

donc j'ai juste remis cela tel que je l'avais mis au débu

J'ai modifier // Minuit : il attend de se remettre en place pour s'arrêter. Ca fait plus 'jolie'.

------------------------------------------

Au fait, merci pour le 8/10, et surtout merci pour tes commentaire.
Ils m'ont permis de comprendre certains truc : moi je n'ai acheté aucun livre, jai tt apris en regardant les sources et les commentaires !

Commentaire de faves le 21/12/2004 19:39:28

Bon, j'ai mis à jour ma source
Si quelqu'un pouvai m'aider à résoudre le bug:
"Bug connu :
A cause de la mise à jour du 20/12/2004, un bug est apparu:
si l'on créer un évènement avec un message contenant des caractères spéciaux (éèà¨^), seul les évènements inscrits avant celui-ci seront pris en compte."

voila
++