Problema con mi funcion checkForm()

Enishy · #1 (**permalink**) 30/12/2008, 11:59

Hola foreros desde ya las gracias por tomarce el tiempo en leer este tema y darme unas sugerencias

estoy utilizando Appserv y Win XP

Esta es mi Funcion que utilizo para registrar Usuarios probandola cumple con las restricciones que tiene que realizar como:

1>>No enviar si la clave y su confirmacion son diferentes
2>>No enviar si el usuario y la clave son iguales
3>>No enviar si el usuario o clave estan en blanco

Si todo va bien en tonces los datos ingresados se guardan en la BD por ejemplo si el usuario coloca :

Usuario : marco
Contraseña : qwerty
Repite Contraseña : qwerty

El problema es que no puedo enviar mi datos a la BD y me manda a la restriccion 2 lo que no entiendo pues esta restriccion se cumple si el usuario y la clave son iguales lo que no es asi pues en el ejemplo como se observa son diferentes eh tratado y tratado pero no doy con la solucion y recurro a ustedes

Código:

<script language="JavaScript" type="text/javascript">
function checkForm() {
usuario = document.getElementById("usuario").value;
clave = document.getElementById("clave").value;
reclave = document.getElementById("reclave").value;
	if(reclave != clave) {
	hideAllErrors(); 
	document.getElementById("reclaveError").style.display = "inline";
	document.getElementById("reclave").select();
	document.getElementById("reclave").focus();
  	return false;
  	} else if (clave = usuario) {
	hideAllErrors(); 
	document.getElementById("claveError").style.display = "inline";
	document.getElementById("clave").select();
	document.getElementById("clave").focus();
  	return false;
  	} else if (usuario == "" || clave == "") {
	hideAllErrors(); 
	document.getElementById("usuarioError").style.display = "inline";
	document.getElementById("usuario").select();
	document.getElementById("usuario").focus();
  	return false;
  	} else {
	form.submit();
	return true;
	}
}
function hideAllErrors() {
	document.getElementById("claveError").style.display = "none"
	document.getElementById("usuarioError").style.display = "none"
	document.getElementById("reclaveError").style.display = "none"
}
</script>

Código HTML:

<body>
<form onSubmit="return checkForm();" action="saveUsers.php" method="post">
<table cellpadding="3" cellspacing="1" border="0" align="center">
<tr>
<td align="right"><strong>Usuario:</strong></td>
<td><input type=text size="30" name="register_username" id="usuario" style="border: 1px solid #7F9DB7;">&nbsp;<font color="#FF0000">*</font></td>
<div class=error id=usuarioError>Alerta...Falta introducir usuario o contraseña <br></div> <br>
</tr>
<tr>
<td align="right"><strong>Tipo Usuario:</strong></td>
<td><input disabled value="User" type=text size="30" name="type_user" style="border: 1px solid #7F9DB7;">&nbsp;<font color="#FF0000">*</font></td>
<td><input value="User" type="hidden" name="type_user"></td>
</tr>	
<tr>
<td align="right"><strong>Contraseña:</strong></td>
<td><input type=password size="30" name="register_password" id="clave" style="border: 1px solid #7F9DB7;">&nbsp;<font color="#FF0000">*</font></td>
<div class=error id=claveError>Alerta...La contraseña no puede ser igual al nombre de usuario.<br></div> <br>
</tr>
<tr>
<td align="right"><strong>Repite Contraseña:</strong></td>
<td><input type=password size="30" name="register_password1" id="reclave" style="border: 1px solid #7F9DB7;">&nbsp;<font color="#FF0000">*</font></td>
<div class=error id=reclaveError>Alerta...La repetición de la contraseña no coincide.<br></div> <br>
</tr>
<tr>
<td><input value="Registrarse" type="submit"></td>
<td><a href='Maintenance.php'><img src='../../images/app_utilities.png' width='32' height='32' border="0" align='absmiddle'>Retornar Mantenimiento</a></td>
</tr>
</table><br />
</form>
</body>

GatorV · #2 (**permalink**) 30/12/2008, 12:26

Tema trasladado a Javascript.

caricatos · #3 (**permalink**) 30/12/2008, 16:04

Hola:

Según parece no haces una comparación sino una asignación...

Código:

...else if (clave == usuario) {

Cuando pones form.submit() haces un envío forzado, que en este caso se puede obviar, porque el return true que pones luego (y que también podrías obviar) confirma ese envío.

Podrías simplificar bastante la validación si pasas el propio formulario como parámetro en la función validadora... te paso un artículo que puede ayudarte: El abc de los formularios

Saludos

Enishy · #4 (**permalink**) 01/01/2009, 16:27

hola caricatos...probe lo que me sugieres y si guarda los datos en la BD pero sin respetar la restriccion 2 pues cuando voy a checar la tabla el usuario y la clave son iguales y no puede ser asi.... entonces sigo con lo mismo como te cuento

caricatos · #5 (**permalink**) 01/01/2009, 16:39

Hola:

Es que pensaba que esa asignación era una comparación de igualdad "=="... para desigualdad veo que ya lo sabes: "!="

Saludos