[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;
}