une optimisation de code

bonjour.

je viens ici pour vous demander de m'aider à faire une optimisation de mon code
il transforme un formulaire en objet.

FormToJson.js :
Pièce jointe 62679

si c'est possible.

vue que je suis dedans je vois pas ce qui est optimisable je suis donc en quête d'un point de vue exterieur car si c'est pas optimisé ça risque de bourrer le navigateur du client.

alors au premier coup d'oeil :
- Eviter les eval !!
- stocker la taille des tableaux dans les boucles :

Code:

---

1 2 3 4

var taille = montab.length ; for(var i = 0 ; i < taille ; i++){ ... }

---

créer des méthodes pour les sous-traiter plutôt que de faire des boucles imbriquées :)

voila mes premières impressions

en fait là j'ai 2 dimention à parcourir.

les forms qui ont le même nom et les éléments de chaque form.

que j'explique comme on ne peut pas faire de sous formulaires (sauf si j'ai loupé quelque chose 8-)) je suis obligé dans ma situation de faire des forms fragmentés.

j'ai une question ...

comment je peut contourner les evals de ce genre ?

Code:

---

1 2 3 4 5 6 7 8 9

// code épuré pas de contrôles ici var temp = document.getElementsByName(nom);   for(var k = 0; k < temp.length; k++){ for(var i = 0; i < temp[k].elements.length; i++){ eval("obj."+temp[k].elements[i].type.replace(/\-/, '_').toLowerCase()+"."+temp[k].elements[i].name+"."+temp[k].elements[i].id+" = \""+temp[k].elements[i].value+"\""); } }

---

tu peux contourner les éval en récupérant tous les éléments de ton formulaire , puis en vérifiant s'il appartiennent a ton formulaire :)

je pige pas bien là.

moi ce que je veux c'est un truc comme ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

<form name="toto"> <input type="text" name="nom" value="dragonfel"/> <input type="text" name="prenom" value="hiul" /> <input type="text" name="adresse" value="chez moi" /> <select name="genre"> <otpion value="homme" selected="selected" >homme</option> <otpion value="femme">femme</option> </select> <input type="checkbox" name="show[]" id="FA022215" value="true" checked="checked"/> <input type="checkbox" name="show[]" id="FA022225" value="true"/> <input type="checkbox" name="show[]" id="FA022268" value="true" checked="checked"/> </form>

---

j'obtienne ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

commande lien : toto result : {"text" :{"nom" : "dragonfel", "prenom" : "hiul", "adresse" : "chez moi"}, "hidden" :{}, "checkbox" :{"show" : {"FA022215" : "true", "FA022225" : "false", "FA022268" : "true"}}, "radio" :{}, "textarea" :{}, "select_one" :{"genre" : "homme"}, "select_multiple" :{}, "password" :{} }

---

donc j'ai besoin d'insérer le nom et l'id (quand il existe) de chaque élément.
comment faire pour éviter le eval dans ce cas ?

à moins que je créé une chaine Json que j'évalue à la fin :roll:

tiens un très bon exemple sur les objets :)

http://t-templier.developpez.com/tut...cript-poo1/#L1

si tu n'y arrives pas je t'aiderais a créer la méthode

j'avais peur que en utilisant les "[]" ie6 ne comprenne pas.

'avais créé ces fonctions à la base pour pouvoir passer des tableaux associatifs à AJAX sous ie :( ducoup je pensais pas que ça marcherai.

;-)

sinon au pireeee du pire !
tu crée une string , pas besoins de l'utiliser au format objet lors de l'envoie , a la récupération coté client oui mais coté serveur tu l'évalues ...

lors de l'envois je la transform en string json.

l'avantage de l'objet c'est que je peux faire un traitement éventuel dessus avant l'envois au serveur.

Pièce jointe 62752

le fichier mis à jours.

je ne vois pas trop quoi d'autre à changer.
getElement() : est un alias de document.getElementById();
et inArray() donne ça.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

function inArray(text) { for (a=0;a<this.length;a++) { if(this[a] == text) { return true; } } return false; } Array.prototype.inArray = inArray;

---