AAAAAAAAAAAAAAAAAAAAAA
cs_Kimjoa
Messages postés
262
Date d'inscription
vendredi 6 mai 2005
Statut
Membre
Dernière intervention
19 septembre 2014
-
18 déc. 2009 à 16:03
djmmix Messages postés 152 Date d'inscription lundi 28 juillet 2003 Statut Membre Dernière intervention 29 avril 2009 - 19 déc. 2009 à 12:48
djmmix Messages postés 152 Date d'inscription lundi 28 juillet 2003 Statut Membre Dernière intervention 29 avril 2009 - 19 déc. 2009 à 12:48
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/50987-aaaaaaaaaaaaaaaaaaaaaa
https://codes-sources.commentcamarche.net/source/50987-aaaaaaaaaaaaaaaaaaaaaa
19 déc. 2009 à 12:48
si un modo passe qui supprime sa merci ;)
18 déc. 2009 à 16:03
J'ai quelques remarques.
D'abord tu devrais utiliser le prototype (en js les attribut et méthode sont écrite dedans)
Là tout passe par des closures, donc elle sont créer pour chaque instance, ce qui consomme bcp de mémoire.
Ton menu n'est pas du tout référençable, il aurait mieux fallut écrire ton menu en html , via des ul/li par exemple, et de parser la structure pour instancier les items.
Ton modèle ajax devrait être mit à part, pour être réutilisable.
T'as oublié pas mal de var, le mot clé est très important sinon la déclaration est évaluer au niveau globale, et non pas au niveau de la fonction, ca peux engendré des conflits.
La variable AjaxFlagMenuItems, référence tout les items crée et non pas uniquement ceux d'une instance AjaxFlagMenu, c'est pas logique, ou tu aurais du l'appeler autrement.
Y'a encore d'autre chose à revoir, notamment ton modelé évènementielle, l'utilisation des balises table ...
sinon c'est pas mal codé pour un débutant !!
a++