AFFICHE DE 1 A PLEIN DE MESSAGES DANS LA BARRE DE STATUT DU VISITEUR
pysco68
Messages postés
681
Date d'inscription
samedi 26 février 2005
Statut
Membre
Dernière intervention
21 août 2014
-
2 janv. 2009 à 11:46
Optitech Messages postés 134 Date d'inscription samedi 19 octobre 2002 Statut Membre Dernière intervention 3 janvier 2009 - 3 janv. 2009 à 12:05
Optitech Messages postés 134 Date d'inscription samedi 19 octobre 2002 Statut Membre Dernière intervention 3 janvier 2009 - 3 janv. 2009 à 12:05
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/48861-affiche-de-1-a-plein-de-messages-dans-la-barre-de-statut-du-visiteur
https://codes-sources.commentcamarche.net/source/48861-affiche-de-1-a-plein-de-messages-dans-la-barre-de-statut-du-visiteur
3 janv. 2009 à 12:05
Voilà quelque remarque pour améliorer ton script :
1. Chose qui te sera très utlsie dans le futur : INDENTE ton code ! (http://fr.wikipedia.org/wiki/Indentation#Informatique)
2. Au lieu d'utliser plei nde variable pour tes message (msg_A, msg_B, ... msg_Z, msg_AA, ...) utlise un tableau c'est plus rapide et plus efficace a gérer.
3. Pour tes if même s'il y a qu'une action mes les { } comme cela si tu rajoute une action tu n'aura pas à les rajouter et ca rend ton code plus clair.
4. Tu as : un 'if' un 'if else' ... mais il est ou ton else ! Heureseument que JavaScript est un langage très gentil.
5. Ta varaible B n'est pas décalré. (OCamL t'aurais dit : Unbound value B <= il fallait que je place le Unboun value)
6. temp est un paramètre local pourquoi le copier dans une varaible global ??? Si tu ne fait pas ca ta ligne 'setTimeout("msg(delai)",temp);' deveitn : setTimeout('msg(' + temp + ')',temp); (oui je suis un fan des simple quote)
7. Donne un nom explicite à tes varaibles. Le paramètre local temp c'est quoi et à sert à quoi ???
8. Nom n'utlise plus language="JavaScript" ! Maintenamt c'est type="text/javascript"
@++
Optitech
2 janv. 2009 à 19:27
2 janv. 2009 à 19:19
about:config -> dom.disable_window_status_change (à changer pour false)
2 janv. 2009 à 12:18
mais bon je tenais par là aussi à dire qu'une présentation correcte sans trop d'immondités n'est pas en option... surtout que c'est pas la première fois...
Bonne journée
2 janv. 2009 à 12:12
D'accord avec toi, mais relis bien ton message : le correcteur orthographique ne corrige pas tout.
2 janv. 2009 à 11:46
s'il-te-plaît Kangourouxxx... ai l'amabilité de ne pas bombarder codes-Sources de sources qui y sont déjà plusieurs fois et que l'on a aucun mal a trouver 10000x dans des collections de scripts....
pour ta prochaine source informe toi par rapport aux évènements JS... choses très utiles...
et puis le correcteur orthographique existe.... (sous Firefox par défaut!!!)