Validación de formulario ya no funciona

[ivechile]

Saludos! Tengo un problema con la validación javascript de un formulario. Un asunto sencillo, lo extraño es que de repente ha dejado de funcionar, el código no ha cambiado, y no entiendo qué sucede.

Mis funciones las defino así:

<script language="javascript" type="text/javascript">
function validarEntero(valor){
//intento convertir a entero.
//si era un entero no le afecta, si no lo era lo intenta convertir
valor = parseInt(valor)

//Compruebo si es un valor numérico
if (isNaN(valor)) {
//entonces (no es numero) devuelvo el valor cadena vacia
return “”
}else{
//En caso contrario (Si era un número) devuelvo el valor
return valor
}
}
function Validar(f)
{
if (f.nombre.value == "")
{ alert("Por favor ingrese su nombre"); f.nombre.focus(); return false; }

if (f.mail.value == "")
{ alert("Por favor ingrese su dirección de e-mail"); f.mail.focus(); return false; }

if (f.edad.value != ""){
edad = f.edad.value
edad = validarEntero(edad)
f.edad.value=edad;
if (edad == "")
{ alert("Tiene que introducir un número entero en su edad, o dejar el campo en blanco"); f.edad.focus(); return false; }
}
}

</script>

Y luego, el formulario se define así (pongo solo el encabezado, ya que es muy largo):

<form id="registrarse" action='bajarmeditaciones.asp' method='post' name="registrarse" onSubmit="return Validar(this)">

Cómo contaba, andubo un par de veces de lo más bien, pero de un momento a otro dejó de validar. Alguna idea, por favor? Gracias!

[lair]

intenta asignando a cada elemento del formulario un id y desde javascript lo buscas como:
document.getElementById('id').value y eso lo colocas en tu if

suerte

[ivechile]

Muchas gracias Lair, he probado con tu propuesta, pero sigue sin funcionar... Cambiará en algo si le pongo id a todos los campos, incluso los que no me interesa validar? En la modificación que me propones, ya no es necesario pasar el mismo formulario como parámetro a la función, usando "this", no es cierto? De todas formas he probado de las dos maneras, y nada. Gracias igual

[Sergestux]

Por que no intentas cambiar el codigo por algo como esto:

Código javascript:

if (f.edad.value != ""){
    edad = validarEntero(f.edad.value) 
    if (edad == f.edad.value)
        return true
    else
        { alert("Tiene que introducir un número entero en su edad, o dejar el campo en blanco"); f.edad.focus(); return false; }
}
 

[raul_romero]

Hola amigo, estuve probando tu codigo. Solo encontre una falla lo cambie y funciono la validación, te explico:

Código:

function validarEntero(valor){
//intento convertir a entero.
//si era un entero no le afecta, si no lo era lo intenta convertir
valor = parseInt(valor)

//Compruebo si es un valor numérico
if (isNaN(valor)) {
//entonces (no es numero) devuelvo el valor cadena vacia
return ""
}else{
//En caso contrario (Si era un número) devuelvo el valor
return valor
}

Haz como en el texto marcado de rojo, utiliza las comillas dobles que normalmente se utilizan para cadenas en cualquier lenguaje de programación (" ") sin espacios, al momento de retornar el valor.

Yo lo hice asi y funciono, espero que sea la solución.

Saludos...

[ivechile]

Gracias Raul. Me parece que esas comillas ya estaban, aunque en una tipografía rara. También probé tus cambios, Sergestux, pero nada aún. Es como si no existiera el "onSubmit", no se llama a la función, y envia el formulario directamente ¡Todo muy extraño!!

[Sergestux]

Te recomiendo que te instales algun depurador de javascript en tu navegador, por que al parecer tu error puede estar en otro lado. Para comprobar que si entra a la funcion validar pon un alert de prueba al inicio de la funcion

[ivechile]

Gracias Sergestux! Con el depurador he descubierto un error de sintaxis demasiado común... un cierre "}" de más! El código ya anda a la perfección, gracias