On 26 May 2008 11:21:44 GMT, abramo wrote:
Quote:
window.onload=function() {
var field=document.getElementsByTagName('form')[0].elements; |
field la dichiari come locale, ma non c'è alcun uso localmente di tale
variabile. Inoltre potresti eliminare getElementsByTagName:
var field = document.forms[0].elements;
Quote:
document.getElementById('cambioPassword').onsubmit =function() {
if(validateForm()){startRequest();}
return false;
} |
Se "startRequest()" fa tutto il lavoro, come sembra, potresti andare a
integrare validateForm() all'interno della funzione. In questo modo si
va a eliminare una closure superflua:
document.getElementById("cambioPassword").onsubmit = startRequest;
Quote:
questo codice puo' andare bene oppure si puo' fare di meglio e con piu'
eleganza? |
Puoi raggiungere una maggiore eleganza se cambi approccio. Ma questo è
più inerente alla metodologia piuttosto che al codice in sé.
Ad esempio, utilizzando qualche Design Pattern; incapsulando il codice
in oggetti.
--
~ Ci si aspetta di tutto, ma non si e' mai preparati a nulla.
(M.Swetchine)