Verifica password

[Mikele Sagitter]

Ciao ragazzi avevo progettato questo codice per verificare 1= se le password sono uguali 2= se i campi sono vuoti. Però a seconda di quale codice metto prima o dopo mi considera solo il controllo che sta più in basso...

if(document.getElementById("parolachiave").value!= document.getElementById("parolachiave2").value){
evidenzia("parolachiave");
evidenzia("parolachiave2");
window.scroll("0","0");
convalida=false;

} else {
disevidenzia("parolachiave");
disevidenzia("parolachiave2");
}

if(document.getElementById("parolachiave").value== ""){
evidenzia("parolachiave");
window.scroll("0","0");
convalida=false;
} else {
disevidenzia("parolachiave");
}

if(document.getElementById("parolachiave2").value= =""){
evidenzia("parolachiave2");
window.scroll("0","0");
convalida=false;
} else {
disevidenzia("parolachiave2");
}


Le funzioni chiamate sono queste

function evidenzia(e) {
document.getElementById(e).style.border=3;
document.getElementById(e).style.borderColor="#f00 ";
document.getElementById(e).style.borderStyle="soli d";

document.getElementById("boxcontrollo").style.disp lay="block";
document.getElementById(e + "lab").style.color="#f00";
document.getElementById(e + "lab").style.fontWeight="bold";
document.getElementById(e).nextSibling.innerHTML=" *";
}

function disevidenzia(e) {
document.getElementById(e).style.border="";
document.getElementById(e).style.borderColor="";
document.getElementById(e).style.borderStyle="";

document.getElementById("boxcontrollo").style.disp lay="none";
document.getElementById(e + "lab").style.color="";
document.getElementById(e + "lab").style.fontWeight="";
document.getElementById(e).nextSibling.innerHTML=" ";
}

[Xinod]

come richiami queste istruzioni?

questo window.scroll("0","0") a che dovrebbe servire? non ha senso, specie passando stringhe

[Mikele Sagitter]

si hai ragione dovrebbè portare la finestra a "dov'è l'errore" ma non avevo calcolato che non vuole stringhe ma numeri. Beh queste istrozioni sono richiamate nell'if --> evidenzia("parolachiave");
evidenzia("parolachiave2");

[Xinod]

guarda, a occhio non mi sembra che l' una escluda l' altra
sicuro di non sovrapporre nella logica queste evidenzia e disevidenzia?

puoi anche provare a fare tutti i controlli come conseguenze
ribaltando la valorizzazione di convalida, che diventa true solo se tutto va a buon fine

var convalida=false;
if(document.getElementById("parolachiave").value==""){
  evidenzia("parolachiave");
} else {
  disevidenzia("parolachiave");
  if(document.getElementById("parolachiave2").value==""){
    evidenzia("parolachiave2");
  } else {
    disevidenzia("parolachiave2");
    if(document.getElementById("parolachiave").value!=document.getElementById("parolachiave2").value){
      evidenzia("parolachiave");
      evidenzia("parolachiave2");
    } else {
      disevidenzia("parolachiave");
      disevidenzia("parolachiave2");
      convalida=true;
    } 
  }
}
if(convalida) ...blabla...