0

tengo un formulario en el que solo me falta de señalizar la validacion en el que al realizar la validación quiero que muestren los campos vacios (en este caso los datos de un cliente), de tal manera que los bordes de los campos sean de color rojo, la validación la hice con javascript, espero que me puedan ayudar.

javascript:

 function validarclientes(elemento)
  {
   nombre = document.getElementById("txtnombre").value;
   telefono = document.getElementById("txttelefono").value;
   email = document.getElementById("txtemail").value;
   estado = document.getElementById("txtestado").value;
   numero = document.getElementById("txtnumero").value;
   direccion = document.getElementById("txtdireccion").value;
   tipo = document.getElementById("cmbtipo").value;

   var dalecolor = elemento.style.borderColor = "red";
   dalecolor = false;
    var datosCorrectos=true;
    var error="";
    var exp = /^(([^<>()\[\]\\.,;:\s@"]+(\.[^<>()\[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/;


    if(nombre =="" && telefono == "" && numero =="" && direccion == "" &&tipo == "" && estado == "" && !exp.test(email))
    {
      dalecolor = true;
      datosCorrectos=false;
      error="\n Primero llena el formulario";

    }


    else(!datosCorrectos)
    {
      alert('Error del formulario:' + error);
    }

    return datosCorrectos;

  }

parte de html:

<form id="frmclientes" name="frmclientes" method="post" action="controlador.php" onsubmit="return validarclientes(this)">

1 respuesta 1

0

Tienes algunos errores y te sobra un poco de código. Tal cual tienes el formulario puedes simplificarlo de la siguiente manera:

function validarclientes(elemento) {

    /* recuerda declarar las variables con la sentencia `var` a menos que quieras
       utilizar más tarde estas variables a través del objeto window, lo que se
       conoce como ámbito global o "global scope" */

    var nombre = document.getElementById("txtnombre").value;
    var telefono = document.getElementById("txttelefono").value;
    var email = document.getElementById("txtemail").value;
    var estado = document.getElementById("txtestado").value;
    var numero = document.getElementById("txtnumero").value;
    var direccion = document.getElementById("txtdireccion").value;
    var tipo = document.getElementById("cmbtipo").value;

    /* las variables que tenías aquí no son necesarias, ya que podemos
       manejar la respuesta y los errores dentro del `if` */

    var exp = /^(([^<>()\[\]\\.,;:\s@"]+(\.[^<>()\[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/;

    if(nombre =="" && telefono == "" && numero =="" && direccion == "" &&tipo == "" && estado == "" && !exp.test(email)) {

        /* si no pasa el filtro, aplicamos el borde al elemento. si sólo
           le aplicamos el color de borde no se mostrará el borde, ya que además
           del color, debemos decirle que debe tener un borde como tal. en un
           caso de uso real, lo ideal sería añadir una clase al elemento y, con
           css, aplicar los estilos necesarios */

        elemento.style.border = "solid 1px red";

        /* como a estas alturas ya sabemos que no se pasa el filtro podemos mostrar
           el mensaje de error directamente, sin necesidad de variables de control */

        alert('Error del formulario:\n Primero llena el formulario');

        // y finalmente detener el envío del formulario
        return false;
    }

    // pasa el filtro, se envía el formulario
    return true;
}

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.