[SOLUCIONADO] Pasar validación a una manera generica con funciones

[dlmv]

Hola tengo que pasar esta validaciones del formulario a una manera generica con funciones y eso... no se hacerlo necesito ayuda

El codigo es el siguiente:

Código PHP:

  function validar(){ 
  
//RECOGEMOS LOS CAMPOS EN VARIABLES 
var nombre = document.getElementById("nombre").value; 
var dni = document.getElementById("dni").value; 
var email  = document.getElementById("email").value; 
var telefono = document.getElementById("tfn").value; 
var errorNombre = document.getElementById("erno"); 
var errorDni = document.getElementById("erdn"); 
var errorEmail = document.getElementById("erem");  
  
//LAS LETRAS SON PARA COMPROBAR EL DNI 
var letras = ['T', 'R', 'W', 'A', 'G', 'M', 'Y', 'F', 'P', 'D', 'X', 'B', 'N','J', 'Z', 'S', 'Q', 'V', 'H', 'L', 'C', 'K', 'E', 'T']; 
  
//COMPROBAR QUE LOS CAMPOS OBLIGATORIOS NO ESTAN VACIOS 
if( nombre == null || nombre.length == 0 || /^s+$/.test(nombre) ) { 
errorNombre.innerHTML = "<font color='red'>Debes introducir tu nombre</font>"; 
return false; 
}else if( dni == null || dni.length == 0 || /^s+$/.test(dni) ) { 
errorNombre.innerHTML = ""; 
errorDni.innerHTML = "<font color='red'>Debes introducir tu DNI</font>"; 
return false; 
//COMPROBAR  QUE EL DNI ES CORRECTO 
}else if(dni.charAt(8).toUpperCase()  != letras[(dni.substring(0, 8))%23]) { 
errorDni.innerHTML = "<font color='red'>El DNI no es correcto</font>" ; 
return false; 
}else if( email == null || email.length == 0 || /^s+$/.test(email) ) { 
errorDni.innerHTML = ""; 
errorEmail.innerHTML = "<font color='red'>Debes introducir tu E-mail</font>"; 
return false; 
//COMPROBAR QUE EL FORMATO DEL E-MAIL ES CORRECTO 
}else if(!(/[w-.]{3,}@([w-]{2,}.)*([w-]{2,}.)[w-]{2,4}/.test(email)) ){ 
errorDni.innerHTML = ""; 
errorEmail.innerHTML = "<font color='red'>El E-mail no es correcto</font>"; 
return false 
} 
return true; 
}  
   

[emprear]

suponiendo que las condiciones estén correctas, tu función de por si tiene identificados todos los elementos que se requieren, los campos con sus values y aquellos donde aparecerán los mensajes de error, bastaría llamarla sin parámetros desde el evento onsubmit del form

<form action="#" onsubmit="return validar()">

hacerla más genérica implicaria pasar parámetros a la función. Además tus validaciones son diferentes. si solo fuera verificar que no estén vaciós se podría hacer, pero como está, no lo veo tan práctico

SAludos

[dlmv]

Hola gracias por contestar
Si tengo que ver que el campo no esté vacío , que en el nombre sea letras y todo lo que pueda añadir pero de forma generica por funciones

[emprear]

Iniciado por dlmv

Hola gracias por contestar
Si tengo que ver que el campo no esté vacío , que en el nombre sea letras y todo lo que pueda añadir pero de forma generica por funciones

Ya tenés una función, Crear algo genérico para algo tan poco generíco, dejaría de ser genérico

esto
//COMPROBAR QUE EL DNI ES CORRECTO
y esto
//COMPROBAR QUE EL FORMATO DEL E-MAIL ES CORRECTO
requieren una función personalizada

[dlmv]

GRacias queria una cosa asi:

Código PHP:

  /* 
*     Comprueba que un campo de texto no vacio 
*/ 
function estaVacio(texto){ 
    if (texto == null || texto.length==0 || /^s+$/.test(texto)) 
        return true; 
    else  
        return false; 
} 
 
/* 
 * Comprobamos que es un numero 
 */ 
function esUnNum (valor) { 
    if( isNaN(valor))  
          return false; 
} 
 
 
/* 
* Comprobamos que se ha selecionado un campo de radio B. 
*/ 
 
function estaSeleccionadoRadio(opcion){ 
 
    for (var i=0; i<opcion.length; i++){ 
        if(opcion[i].checked){ 
            return true; 
        } 
    } 
    return false; 
} 
 
/* 
* Comprobamos que se ha seleccionado un campo checkbox 
*/ 
 
function estaSeleccionadoCheck(opcion){ 
    if( !opcion.checked ){ 
        return false; 
    }  
    return true; 
} 
/* 
 * Todos los checkbox han sido seleccionados 
 */ 
 
function checkSeleccionados(formulario){ 
    for(var i=0; i<formulario.elements.length; i++) { 
        var elemento = formulario.elements[i]; 
        if(elemento.type == "checkbox") { 
            if(!elemento.checked) { 
                  return false; 
              } 
          } 
    } 
     
} 
 
/* 
* Comprobamos que se ha seleccionado un elemento de una lista desplegable 
* Comparamos el valor del elemento que tenemos no tenga valor 0, ya que no habremos seleccionado nada. 
*/ 
function estaSeleccionadoLista(opcion){ 
    if (opcion == null ||opcion == 0) 
        return false; 
    return true; 
}  
   

[dlmv]

un ejemplo de formulario normal de contacto para esta validacion:

Código PHP:

  /* 
*     Comprueba que un campo de texto no vacio 
*/ 
function estaVacio(texto){ 
    if (texto == null || texto.length==0 || /^s+$/.test(texto)) 
        return true; 
    else  
        return false; 
} 
 
/* 
 * Comprobamos que es un numero 
 */ 
function esUnNum (valor) { 
    if( isNaN(valor))  
          return false; 
} 
 
 
/* 
* Comprobamos que se ha selecionado un campo de radio B. 
*/ 
 
function estaSeleccionadoRadio(opcion){ 
 
    for (var i=0; i<opcion.length; i++){ 
        if(opcion[i].checked){ 
            return true; 
        } 
    } 
    return false; 
} 
 
/* 
* Comprobamos que se ha seleccionado un campo checkbox 
*/ 
 
function estaSeleccionadoCheck(opcion){ 
    if( !opcion.checked ){ 
        return false; 
    }  
    return true; 
} 
/* 
 * Todos los checkbox han sido seleccionados 
 */ 
 
function checkSeleccionados(formulario){ 
    for(var i=0; i<formulario.elements.length; i++) { 
        var elemento = formulario.elements[i]; 
        if(elemento.type == "checkbox") { 
            if(!elemento.checked) { 
                  return false; 
              } 
          } 
    } 
     
} 
 
/* 
* Comprobamos que se ha seleccionado un elemento de una lista desplegable 
* Comparamos el valor del elemento que tenemos no tenga valor 0, ya que no habremos seleccionado nada. 
*/ 
function estaSeleccionadoLista(opcion){ 
    if (opcion == null ||opcion == 0) 
        return false; 
    return true; 
}