Problema simple con comparacion de contraseñas

[manumaf]

Hola a todos, estoy validando un formulario y tengo un problema al tratar de ver si las contraseñas son iguales.

Código PHP:

  function validar(formulario) {
if (formulario.nombreyapellido.value.length < 4) {
alert("Nombre y apellido demaciado corto");
formulario.nombreyapellido.focus();
return (false);
}
var checkOK = "ABCDEFGHIJKLMNÑOPQRSTUVWXYZÁÉÍÓÚ" + "abcdefghijklmnñopqrstuvwxyzáéíóú ";
var checkStr = formulario.nombreyapellido.value;
var allValid = true;
for (i = 0; i < checkStr.length; i++) {
ch = checkStr.charAt(i);
for (j = 0; j < checkOK.length; j++)
if (ch == checkOK.charAt(j))
break;
if (j == checkOK.length) {
allValid = false;
break;
}
}
if (!allValid) {
alert("Tu nombre y apellido solo pueden llevar letras");
formulario.nombreyapellido.focus();
return (false);
}
if (formulario.nick.value.length < 4) {
alert("Nick demaciado corto");
formulario.nick.focus();
return (false);
}
 
if (formulario.pass.value.length != formulario.pass2.value.length) {
alert("Nick demaciado corto");
formulario.pass.focus();
return (false);
}
 
return (true);
} 
   

Todo funciona bien menos:

Código PHP:

  if (formulario.pass.value.length != formulario.pass2.value.length) {
alert("Nick demaciado corto");
formulario.pass.focus();
return (false);
} 
   

Que es lo que hago mal?

[zerokilled]

¿cual es el problema que te produce y cual es el resultado que esperas? por otro lado, estoy viendo que estas comparando las longitudes de ambos campos. es decir, en un campo puedo entrar manu y en otro maaf y el script entendera que esta todo bien pese a ser distintos valores. para comparar ambas cadenas debes comparar por valor y no por longitud. simplemente quita length.

[manumaf]

Ya le quite los length, el problema es sean o no iguales me devuelve true.

[osvier]

Creo que te equivocaste de mensaje de error:

estas poniendo "Nick demaciado corto" cuando en realidad estas validando que sean iguales...

Y otro detallito... demasiado

[Tecna]

Buenas,

para ser exactos en este trozo de código:

Código Javascript:

if (formulario.pass.value.length != formulario.pass2.value.length)
{
          alert("Nick demaciado corto"); 
          // mas código
}
 

lo que está comparando es la longitud de las dos cadenas contenidas en la propiedad value de los campos pass, así que lo de nick tampoco parece tener mucho sentido. Y como el alert solo se ejecutará cuando sean de distinta longitud está claro que una será mas corta que la otra siempre. Poner nombres de variable y mensajes coherentes ayuda bastante, y conviene incluir comentarios. Además es la única comparación que haces con esos campos que no servirá de mucho como te ha dicho zerokilled.

De lo que comentas de que siempre te devuelve true, te comento un par de errores en el código:

Código Javascript:

for (j = 0; j < checkOK.length; j++)
{      // te falta cerrar esta llave
 
       if (ch == checkOK.charAt( j ) ) break;
 
       if (j == checkOK.length)      // este if nunca se ejecuta
       {
               allValid = false;            // allValid sigue siendo true
               break;
       }
} 
 
if (!allValid) {          // este tampoco
 

y no se ejecuta por dos razones: por la condición del for, j nunca llegaría a ser igual a length y además porque el último caracter de una cadena es length -1 no el que usas en la comparación del if del for .

Como ya te ha dicho zerokilled las cadenas por defecto se comparan por valor y caracter a caracter y solo son iguales si tienen el mismo contenido y la misma longitud, así que no te hacen falta los for aninados, simplemente compara las cadenas

[manumaf]

No pude arreglar lo de las contraseñas no entiendo por que, pero ahora otra duda ¿Como colo php dentro del javascript? Por que quiero un control para ver si el usuario que se esta registrando ya esta dentro de la base de datos. Muchas gracias