AAAAAAAAAAAAAAAAAAAAAA

Signaler
Messages postés
264
Date d'inscription
vendredi 6 mai 2005
Statut
Membre
Dernière intervention
19 septembre 2014
-
Messages postés
152
Date d'inscription
lundi 28 juillet 2003
Statut
Membre
Dernière intervention
29 avril 2009
-
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

Messages postés
152
Date d'inscription
lundi 28 juillet 2003
Statut
Membre
Dernière intervention
29 avril 2009

heu c'est quoi ce troll la .........

si un modo passe qui supprime sa merci ;)
Messages postés
264
Date d'inscription
vendredi 6 mai 2005
Statut
Membre
Dernière intervention
19 septembre 2014

salut !

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++