problema al hacer varias validaciones en forms

yessi · #1 (**permalink**) 28/02/2005, 13:22

Quiero q al hacer el submit en un formulario se compruebe q los campos obligatorios se hayan rellenado, q el campo mail sea correcto, y q una fecha seleccionada en primer lugar no sea menor q una segunda fecha. El código es el siguiente:

Código PHP:

  <script language="javascript">

function y2k(number) { return (number < 1000) ? number + 1900 : number; }

function esFecha (day,month,year,campo) {

 
    var today = new Date();

    year = ((!year) ? y2k(today.getYear()):year);

    month = ((!month) ? today.getMonth():month-1);

    if (!day) return false

    var test = new Date(year,month,day);

    if ( (y2k(test.getYear()) == year) &&

         (month == test.getMonth()) &&

         (day == test.getDate()) )

        return true;

    else

        alert (campo + ' es incorrecto');

        return false

}

function EsCorreo(cadena) {

    // ¿se soportan expresiones regulares?

    var soportado = false;

    if (window.RegExp) {

          var cadenaTemp = "a";

        var regularTemp = new RegExp(cadenaTemp);

        if (regularTemp.test(cadenaTemp)) soportado = true;

        }

 
    if (!soportado)

           return (cadena.indexOf(".") > 2) && (cadena.indexOf("@") > 0);

 
    var r1 = new RegExp("(@.*@)|(\\.\\.)|(@\\.)|(^\\.)");

    var r2 = new RegExp("^.+\\@(\\[?)[a-zA-Z0-9\\-\\.]+\\.([a-zA-Z]{2,3}|[0-9]{1,3})(\\]?)$");

    return (!r1.test(cadena) && r2.test(cadena));

}

function validaCorreo(correo,campo) {

 
    if (!EsCorreo(correo)) {

        alert(campo + " es incorrecto.");

           return false;

        }

    return true;

}

function esObligatorio(obligatorio,campo) {

 
    if (obligatorio == "") {

        alert(campo + " es obligatorio.");

           return false;

        }

    return true;

}

 
function validar ()

{

  var diahoy;

  var meshoy;

  var anohoy;

  var diarec;

  var mesrec;

  var anorec;

  var diadev;

  var mesdev;

  var anodev;

  var correcto;

 
  correcto = true;

 
  diahoy = document.reserva_vehiculo.diahoy.value;

  meshoy = document.reserva_vehiculo.meshoy.value;

  anohoy = document.reserva_vehiculo.anohoy.value;

  diarec = document.reserva_vehiculo.diarec.value;

  mesrec = document.reserva_vehiculo.mesrec.value;

  anorec = document.reserva_vehiculo.anorec.value;

  diadev = document.reserva_vehiculo.diadev.value;

  mesdev = document.reserva_vehiculo.mesdev.value;

  anodev = document.reserva_vehiculo.anodev.value;

 
  lugrec = document.reserva_vehiculo.lugrec.value;

  mail = document.reserva_vehiculo.mail.value;

 
   //Primer campo obligatorio

   correcto = esObligatorio(lugrec,"El lugar de recogida del vehículo")

    if (!correcto) {document.frmFormulario.oficina.focus();return;}

    //Segundo campo obligatorio

   correcto = esObligatorio(mail,"La dirección de correo electrónico")

    if (!correcto) {document.frmFormulario.oficina.focus();return;}

    

    //Fechas

   correcto = esFecha(diarec, mesrec,anorec,"El día de Recogida");

    if (!correcto) {document.reserva_vehiculo.diarec.focus();return;}

 
    correcto = esFecha(diadev, mesdev,anodev,"El día de Devolución");

    if (!correcto) {document.reserva_vehiculo.diadev.focus();return;}

 
    var today = new Date(anohoy,meshoy-1,diahoy);

 
    var fecharec = new Date(anorec,mesrec-1,diarec);

    if (fecharec < today) {alert('La fecha de recogida no puede ser anterior a la de hoy.');document.reserva_vehiculo.diarec.focus();correcto = false;return;};

 
   var fechadev = new Date(anodev,mesdev-1,diadev);

  if (fechadev < fecharec) {alert('La fecha de devolución no puede ser anterior a la de recogida.');document.reserva_vehiculo.diadev.focus();correcto = false;return;};

 
        //validación de correo

        correcto = validaCorreo(mail,"El formato de correo electrónico")

        if (!correcto) {document.reserva_vehiculo.mail.focus();return;}

 
   if (correcto) {

    document.reserva_vehiculo.submit();}

}

</script>

En el formulario tengo:

Código PHP:

  <input type="submit" value="Reservar" class="botones" onclick="return validar()">

El problema consiste en q me hace la primera comprobación ("El lugar de recogida del vehiculo...") y no me hace el resto, me pasa al action del formulario.

Alguien sabe pq no funciona??

RootK · #2 (**permalink**) 28/02/2005, 13:30

Lo que pasa es que estas haciendo un return inmediatamente que hay un error, y por eso se sale.. si quieres que se ejecuten todos en cada momento no le pongas return hasta el final.. (si es necesario)

Salu2

yessi · #3 (**permalink**) 28/02/2005, 14:00

Hice la siguiente modificación en la función validar(). El código es:

Código PHP:

  function validar ()

{

  var diahoy;

  var meshoy;

  var anohoy;

  var diarec;

  var mesrec;

  var anorec;

  var diadev;

  var mesdev;

  var anodev;

  var correcto;

 
  correcto = true;

 
  diahoy = document.reserva_vehiculo.diahoy.value;

  meshoy = document.reserva_vehiculo.meshoy.value;

  anohoy = document.reserva_vehiculo.anohoy.value;

  diarec = document.reserva_vehiculo.diarec.value;

  mesrec = document.reserva_vehiculo.mesrec.value;

  anorec = document.reserva_vehiculo.anorec.value;

  diadev = document.reserva_vehiculo.diadev.value;

  mesdev = document.reserva_vehiculo.mesdev.value;

  anodev = document.reserva_vehiculo.anodev.value;

 
  lugrec = document.reserva_vehiculo.lugrec.value;

  mail = document.reserva_vehiculo.mail.value;

 
   //Primer campo obligatorio

   correcto = esObligatorio(lugrec,"El lugar de recogida del vehículo")

    if (!correcto) {document.reserva_vehiculo.lugrec.focus();/*return;*/}

    //Segundo campo obligatorio

   correcto = esObligatorio(mail,"La dirección de correo electrónico")

    if (!correcto) {document.reserva_vehiculo.mail.focus();/*return;*/}

    

    //Fechas

   correcto = esFecha(diarec, mesrec,anorec,"El día de Recogida");

    if (!correcto) {document.reserva_vehiculo.diarec.focus();/*return;*/}

 
    correcto = esFecha(diadev, mesdev,anodev,"El día de Devolución");

    if (!correcto) {document.reserva_vehiculo.diadev.focus();/*return;*/}

 
    var today = new Date(anohoy,meshoy-1,diahoy);

 
    var fecharec = new Date(anorec,mesrec-1,diarec);

    if (fecharec < today) {alert('La fecha de recogida no puede ser anterior a la de hoy.');document.reserva_vehiculo.diarec.focus();correcto = false;/*return;*/};

 
         var fechadev = new Date(anodev,mesdev-1,diadev);

        if (fechadev < fecharec) {alert('La fecha de devolución no puede ser anterior a la de recogida.');document.reserva_vehiculo.diadev.focus();correcto = false;/*return;*/};

 
        //validación de correo

        correcto = validaCorreo(mail,"El formato de correo electrónico")

        if (!correcto) {document.reserva_vehiculo.mail.focus();/*return;*/}

 
          if (correcto) {

            document.reserva_vehiculo.submit();}

}

No sé si entendí bien lo q tenía q quitar, pero ahora me salen todos los mensajes de golpe, y me pasa a la siguiente página.

yessi · #4 (**permalink**) 01/03/2005, 04:41

Porfi una ayudita, q tengo mal??.....o alguna alternativa

RootK · #5 (**permalink**) 01/03/2005, 09:05

Ya intentaste colocar un return al final...

algo así:

Cita:

....
....
if (correcto) {
document.reserva_vehiculo.submit();}

return false;

Con ello indicas que cuando acabe toda la validacion si es correcto hará el submit y en caso de que no pase por esa condicion regresará falso y no debería hacer nada.. checalo y nos cuentas que tal ..

yessi · #6 (**permalink**) 01/03/2005, 11:20

sigue sin funcionar. Es decir, q me siguen saliendo todos los mensajes d golpe. Ahora no me pasa a la siguiente página, mientras la dirección d mail sea incorrecta, pero si ésta es correcta sí pasa d página,aunq los otros campos no estén rellenos.