ayuda validando NIE

[ferranWEB]

Buenas a todos,

con la ayuda del foro, hice la validacion de mi campo NIF de la siguiente manera:

function nif() {
var dni = frm.dni.value;
numero = dni.substr(0,dni.length-1);
let = dni.substr(dni.length-1,1);
numero = numero % 23;
letra='TRWAGMYFPDXBNJZSQVHLCKET';
letra=letra.substring(numero,numero+1);
if (letra!=let)
{
alert('NIF erróneo');
frm.dni.focus();
return false;
}
return true;

el problema que me encuentro es que tambien se registra gente extranjera, y claro, el NIE no es lo mismo que el NIF! (perdon por mi ignorancia pero pensaba que solo cambiaba el nombre).

podria validar al mismo tiempo el campo? hay algun codigo ya programado para ello??

saludos y gracias

ferranWEB

[mgusdl]

Buenas
hace tiempo publiqué esa funcion

Código javascript:

function checkNIF(nif)
    {
    if (!nif) return false;
    var letras = 'TRWAGMYFPDXBNJZSQVHLCKE';
    if (nif.length != 9) return false;
    else {
        letra = nif.substr(8, 1); letra = letra.toUpperCase();
        dni = nif.substr(0, 8);
        dni = dni.toUpperCase(); dni = dni.replace('X','0');
        dni -= parseInt(dni/23) * 23;
        if (letras.charAt(dni) != letra) return false;
        else return true;
        }
    }
 

[ferranWEB]

buenas mgus,

gracias por tu respuesta. estoy intentando mandar la variable a la funcion pero no me hace nada. que hago mal?

onBlur="javascript:checkNIF(this.value)"

la idea es que al perder el focus, es decir, cambiar de campo me de el error. que hago mal?

saludos,

ferran

[mgusdl]

En principio esta funcion sólo dice si es bueno o no, necesitarias una funcion adicional o modificar esa

Código javascript:

function checkNIF(nif)
  {
  if (!nif) return false;
  var letras = 'TRWAGMYFPDXBNJZSQVHLCKE';
  if (nif.length != 9) return false;
  else
    {
    letra = nif.substr(8, 1); letra = letra.toUpperCase();
    dni = nif.substr(0, 8);
    dni = dni.toUpperCase(); dni = dni.replace('X','0');
    dni -= parseInt(dni/23) * 23;
    if (letras.charAt(dni) != letra) return false;
    else return true;
    }
  }
function validaNif(campo)
  {
  if (campo.length == 9 && !checkNIF(campo)) alert('Ese NIF no vale ni para abrir puertas');
  }
 

Y el campo quedaría así:

Código HTML:

<input type="text" id="nif" maxlength="9" onBlur="validaNif(this.value)"> 

[ferranWEB]

gracias mgusdl,

he hecho lo que comentabas, y no me sale. te pongo el codigo:

Código:

function checkNIF(nif)
{
if (!nif) return false;

var letras = 'TRWAGMYFPDXBNJZSQVHLCKE';

if (nif.length != 9) return false;
else
{
letra = nif.substr(8, 1); letra = letra.toUpperCase();
dni = nif.substr(0, 8);
dni = dni.toUpperCase(); dni = dni.replace('X','0');
dni -= parseInt(dni/23) * 23;
if (letras.charAt(dni) != letra) return false;
else return true;
}
}

function validaNif(campo)  {  

if (campo.length == 9 && !checkNIF(campo)) 
alert('Ese NIF no vale ni para abrir puertas');  }
}

y el campo nif

Código:

<input name="nif" type="text" id="nif" value="" size=9 maxlength=9 onBlur="validaNif(this.value)">

que hago mal?

[David]

El if que tienes en la función validaNIF tiene llave de cierre } pero falta una llave de apertura {. O quita la llave de cierre o agrega la llave de apertura.

[mgusdl]

Pues posiblemente sea por el evento onBlur.
Podrias hacerte una funcion que pida el dato con prompt, pero esto da una alerta de seguridad en IE7. Yo en la aplicación que estoy haciendo los campos NIF, NSS y CIF los solicito mediante un panel flotante donde se valida al pulsar un boton, como puedes ver en la imagen:


Te incluyo también las funciones para hacer el panel movible (drag).

Código javascript:

function getNIF(field, field2, chk) {
    var panelid = 'panelNIF';
    if (!field2 && !chk)
        {
        if (document.getElementById(panelid)) return;
        var dv = crearPanel(50, 300, panelid, 'NIF', field);
        dv.appendChild(document.createElement('br'));
        dv.style.textAlign = 'center';
        var lbl = document.createElement('label');
        lbl.setAttribute('for', 'txtNIF');
        lbl.appendChild(document.createTextNode('NIF: '));
        dv.appendChild(lbl);
        var txtfld = document.createElement('input');
        txtfld.type = 'text';
        txtfld.id = 'txtNIF'
        txtfld.size = 12;
        txtfld.maxLength = 9;
        txtfld.value = field.value;
        txtfld.onkeyup = function(e) { if(window.event) key = window.event.keyCode; else if(e.which) key = e.which; if (key == 13) getNIF(document.getElementById(field.id), document.getElementById('txtNIF'), 1); } 
        dv.appendChild(txtfld);
        var btn = document.createElement('input');
        btn.type = 'button';
        btn.value = 'comprueba';
        btn.onclick = function() { getNIF(document.getElementById(field.id), document.getElementById('txtNIF'), 1); }
        dv.appendChild(btn);
        var dv2 = document.createElement('div');
        dv2.id = 'msg'+panelid;
        dv2.className = 'warn2';
        dv.appendChild(dv2);
        document.body.appendChild(dv);
        txtfld.focus();
        }
    else
        {
        if (checkNIF(field2.value))
            {
            field.value = field2.value.toUpperCase();
            cerrar(panelid);
            }
        else
            {
            document.getElementById(panelid).style.height = '60px';
            document.getElementById('msg'+panelid).innerHTML = 'Numero incorrecto!';
            }
        }
    }
 
function cerrar(elm) { document.body.removeChild(document.getElementById(elm)); }
 
function crearPanel(alto, ancho, ident, titulo, elmOrig)
    {
    if (!alto) alto = 100;
    if (!ancho) ancho = 200;
    if (!ident) ident = 'popwindow';
    if (!titulo) titulo = ' ';
    elmOrig = document.getElementById('meca');
    var tbl = document.getElementById('tablaPpal');
    var eDiv = document.createElement('div');
    eDiv.id = ident;
    eDiv.className = 'ventana';
    eDiv.style.width = ancho + 'px';
    eDiv.style.height = alto + 'px';
    eDiv.style.left = (elmOrig.offsetLeft + tbl.offsetLeft  + 100) + 'px';
    eDiv.style.top = (elmOrig.offsetTop + tbl.offsetTop + 100) + 'px';
    sp = document.createElement('span');
    sp.onclick = function() { cerrar(ident); }
    sp.className = 'cerrar';
    sp.appendChild(document.createTextNode("X"));
    eDiv.appendChild(sp);
            
    var cabeza = document.createElement('div');
    cabeza.className = 'cabeza';
    cabeza.onmousedown = function (evt) { dragStart(evt || window.event, ident); }
    cabeza.appendChild(document.createTextNode(titulo));
    eDiv.appendChild(cabeza);
    return eDiv;
    }
 
/*************************************************************************/
/* Copyright 2001 by Mike Hall */
/* Please see [url]http://www.brainjar.com[/url] for terms of use. */
/*************************************************************************/
// Determine browser and version.
 
function Browser() {
    var ua, s, i;
    this.isIE = false;
    this.isNS = false;
    this.version = null;
    ua = navigator.userAgent;
    s = "MSIE";
    if ((i = ua.indexOf(s)) >= 0) { this.isIE = true; this.version = parseFloat(ua.substr(i + s.length)); return; }
    s = "Netscape6/";
    if ((i = ua.indexOf(s)) >= 0) { this.isNS = true; this.version = parseFloat(ua.substr(i + s.length)); return;   }
    // Treat any other "Gecko" browser as NS 6.1.
    s = "Gecko";
    if ((i = ua.indexOf(s)) >= 0) { this.isNS = true; this.version = 6.1; return; }
    }
 
var browser = new Browser();
 
var dragObj = new Object();
dragObj.zIndex = 0;
 
function dragStart(event, id) {
    var el;
    var x, y;
    // If an element id was given, find it. Otherwise use the element being
    // clicked on.
    if (id) dragObj.elNode = document.getElementById(id);
    else {
        if (browser.isIE) dragObj.elNode = window.event.srcElement;
        if (browser.isNS) dragObj.elNode = event.target;
    // If this is a text node, use its parent element.
        if (dragObj.elNode.nodeType == 3) dragObj.elNode = dragObj.elNode.parentNode;
        }
    // Get cursor position with respect to the page.
    if (browser.isIE) { x = window.event.clientX + document.documentElement.scrollLeft + document.body.scrollLeft; y = window.event.clientY + document.documentElement.scrollTop + document.body.scrollTop; }
    if (browser.isNS) { x = event.clientX + window.scrollX; y = event.clientY + window.scrollY; }
    // Save starting positions of cursor and element.
 
    dragObj.cursorStartX = x;
    dragObj.cursorStartY = y;
    dragObj.elStartLeft = parseInt(dragObj.elNode.style.left, 10);
    dragObj.elStartTop = parseInt(dragObj.elNode.style.top, 10);
 
    if (isNaN(dragObj.elStartLeft)) dragObj.elStartLeft = 0;
    if (isNaN(dragObj.elStartTop)) dragObj.elStartTop = 0;
 
    // Update element's z-index.
    dragObj.elNode.style.zIndex = ++dragObj.zIndex;
 
    // Capture mousemove and mouseup events on the page.
    if (browser.isIE) { document.attachEvent("onmousemove", dragGo); document.attachEvent("onmouseup", dragStop); window.event.cancelBubble = true; window.event.returnValue = false; }
    if (browser.isNS) { document.addEventListener("mousemove", dragGo, true); document.addEventListener("mouseup", dragStop, true); event.preventDefault(); }
    }
 
function dragGo(event) {
    var x, y;
    // Get cursor position with respect to the page.
    if (browser.isIE) { x = window.event.clientX + document.documentElement.scrollLeft + document.body.scrollLeft; y = window.event.clientY + document.documentElement.scrollTop + document.body.scrollTop; }
    if (browser.isNS) { x = event.clientX + window.scrollX; y = event.clientY + window.scrollY; }
    // Move drag element by the same amount the cursor has moved.
    dragObj.elNode.style.left = (dragObj.elStartLeft + x - dragObj.cursorStartX) + "px";
    dragObj.elNode.style.top = (dragObj.elStartTop + y - dragObj.cursorStartY) + "px";
    if (browser.isIE) { window.event.cancelBubble = true; window.event.returnValue = false; }
    if (browser.isNS) event.preventDefault();
    }
 
function dragStop(event) {
    // Clear the drag element global.
    dragObj.elNode = null;
    // Stop capturing mousemove and mouseup events.
    if (browser.isIE) { document.detachEvent("onmousemove", dragGo); document.detachEvent("onmouseup", dragStop); }
    if (browser.isNS) { document.removeEventListener("mousemove", dragGo, true); document.removeEventListener("mouseup", dragStop, true); }
    }
 

Y los estilos del panel

Código css:

.ventana { position:absolute; z-index:999px; border-top:1px solid #000; border-left:1px solid #000; border-bottom:3px solid #000; border-right:3px solid #000; width:500px; top:300px; left:300px; padding:2px; padding-top:0px; background:#FFF; }
.cerrar { float:right; overflow:auto; background:#F60; color:#FFF; border:1px solid #000; font-weight:bold; cursor:pointer; }
.cabeza { background:#336; color:#FFF; text-align:center; font-weight:bold; }
 

Y el campo:

Código HTML:

<input type="text" size="12" id="nif" onBlur="getNif(this)" readonly="readonly"> 

[mgusdl]

Iniciado por David el Grande

El if que tienes en la función validaNIF tiene llave de cierre } pero falta una llave de apertura {. O quita la llave de cierre o agrega la llave de apertura.

Al ser una única función la que se ejecuta, no necesita que el if lleve las llaves {} de apertura y cierre, en cuyo caso soba una }

[ferranWEB]

buenas david,

se me habia colado ese {, aunque ese no es el problema. tengo una funcion validar que se limita a validar los otros campos del formulario. y se cerraba al final. ahora la he cerrado (la llave) antes del codigo, y tampoco funciona...

[ferranWEB]

mgusdl...

me he perdido totalmente!!! jajaja. yo es que soy programador de php y bases de datos, y esto del javascript lo toco muy poquito... no tengo ni pajotera de lo que me apuntas!

[David]

Iniciado por mgusdl

Al ser una única función la que se ejecuta, no necesita que el if lleve las llaves {} de apertura y cierre, en cuyo caso soba una }

Exacto, por eso le dije que debería o quitar definitvamente la llave de cierre o agregar la llave de apertura.

Iniciado por ferranWEB

tengo una funcion validar que se limita a validar los otros campos del formulario. y se cerraba al final. ahora la he cerrado (la llave) antes del codigo, y tampoco funciona...

No entiendo muy bien, ¿no muestra directamente el alert o muestra pero no hace lo que deseas? . Si no muestra el alert deberías mirar la Consola de Errores.

[mgusdl]

Iniciado por ferranWEB

mgusdl...

me he perdido totalmente!!! jajaja. yo es que soy programador de php y bases de datos, y esto del javascript lo toco muy poquito... no tengo ni pajotera de lo que me apuntas!

Bueno, quizás me he pasado de avanzado :P
El evento onBlur no es el más indicado para validar el campo, ya que actúa cuando este es seleccionado.
Yo creo que puedes añadir la validación del NIF/NIE a la función que valida todo el formulario y sacar el error junto al resto. Como la validación devuelve un valor booleano no es nada complicado, un simple IF y tira millas.

[ferranWEB]

david no me hace ni el alert. por eso te digo.. una cosa, en la funcion:

Código:

function validaNif(campo)  {  

if (campo.length == 9 && !checkNIF(campo)) 
alert('Ese NIF no vale ni para abrir puertas');  }
}

tengo que substituir el valor de campo por nif??

saludos

[mgusdl]

Iniciado por David el Grande

Exacto, por eso le dije que debería o quitar definitvamente la llave de cierre o agregar la llave de apertura.

Perdón, me di cuenta tarde

[David]

Sigue sobrando la llave de cierre .

[mgusdl]

Iniciado por ferranWEB

david no me hace ni el alert. por eso te digo.. una cosa, en la funcion:

Código:

function validaNif(campo)  {  

if (campo.length == 9 && !checkNIF(campo)) 
alert('Ese NIF no vale ni para abrir puertas');  }
}

tengo que substituir el valor de campo por nif??

saludos

Código javascript:

function validaNif(campo)  { if (campo.length == 9 && !checkNIF(campo)) 
alert('Ese NIF no vale ni para abrir puertas'); }
 

El valor del campo va en la variable "campo" que lo toma via this.value. Pienso que tu problema viene en el evento onBlur... Prueba poner onKeyUp

[ferranWEB]

sigue sin salirme.. tiene que haber algo mal en el codigo. he hecho una prueba unicamente con el campo nif, y no me funciona. os dejo el codigo completo:

Código:

<head>
<script>

function checkNIF(nif)
{
if (!nif) return false;

var letras = 'TRWAGMYFPDXBNJZSQVHLCKE';

if (nif.length != 9) return false;
else
{
letra = nif.substr(8, 1); letra = letra.toUpperCase();
dni = nif.substr(0, 8);
dni = dni.toUpperCase(); dni = dni.replace('X','0');
dni -= parseInt(dni/23) * 23;
if (letras.charAt(dni) != letra) return false;
else return true;
}
}

function validaNif(campo)  
{  

if (campo.length == 9 && !checkNIF(campo))

{ 
alert('Ese NIF no vale ni para abrir puertas');  

}
}

</script>

</head>

<body>
<form id="form1" name="form1" method="post" action="">
  <p>
    <input name="nif" type="text" id="nif" onKeyUp="validaNif(this.value)"/>
  </p>
</form>
</body>
</html>

he probado onkeyup,onblur,onfocus.. y ni nguno tira. asi que tiene que haber algo mal en la programacion

[mgusdl]

prueba con

Código:

function validaNif(campo)  
{  
if (campo.length == 9)
 {
  if (!checkNIF(campo)) 
   {
   alert('Ese NIF no vale ni para abrir puertas');
   campo = '';
   }
  else alert('NIF OK');
  }
}

[David]

¿Para qué estás comprobando que la longitud del campo sea 9 para mostrar el mensaje?. Porque si pones esa comprobación cuando no sea 9 no mostrará ningún mensaje de error .

[mgusdl]

Iniciado por David el Grande

¿Para qué estás comprobando que la longitud del campo sea 9 para mostrar el mensaje?. Porque si pones esa comprobación cuando no sea 9 no mostrará ningún mensaje de error .

Como un NIF debe tener longitud 9 lo pongo para que no compruebe si es valido o no hasta que tenga la longitud adecuada.
Yo estoy buscando que alerte cuando es incorrecto.

Si quiere que avise cuando está OK

function validaNif(campo)
{
if (checkNIF(campo)) alert('NIF OK');
}

[ferranWEB]

ahi vamos david...

ahora he hecho el caso. si quito el &&... y dejo solo ==9, el error me salta cuando es 9. pero que sentido tiene este codigo agus? no acabo de entenderlo.

yo lo que quiero es que al irse a otro campo, me verifique si es un DNI o un NIE y si es correcto... que tengo que hacer?

[David]

Yo normalmente las validaciones las hago al enviar el formulario . Y evitar su envío en caso de que el campo es incorrecto.

En el caso de hacer con onblur habría que guardar en una variable cuando el campo sea incorrecto, y no permitir que se envíe el formulario hasta que esa variable sea verdadera.

[ferranWEB]

y como lo harias tu david??? eso que comentas implementado como seria¿

[David]

¿A cuál de las dos formas te refieres?. Si es a validar al enviar el formulario usa el evento onsubmit de este.

Si quieres hacerlo en el onblur, debes como dije, guardar en una variable Boolean el valor de retorno de checkNIF y luego en el onsubmit enviar/cancelar la acción del formulario usando esa variable.

[ferranWEB]

a ver.. te cuento. yo tengo una funcion que al hacer onsubmit del formulario, me valida todos los campos tipo:

if (nombre.value == ""){}... y asi

y tenia una que me validaba que el nif fuese correcto:

Código:

 else if(!nif()) return false;

  return true;
}

function nif() {
  var dni = frm.dni.value;
  numero = dni.substr(0,dni.length-1);
  let = dni.substr(dni.length-1,1);
  numero = numero % 23;
  letra='TRWAGMYFPDXBNJZSQVHLCKET';
  letra=letra.substring(numero,numero+1);
  if (letra!=let)
  {
    alert('NIF erróneo');
    frm.dni.focus();
    return false;
  }
  return true;

y este me funciona perfecta,mente. es mas, si pongo un numero de nif que no concuerda con la letra me lo da como erroneo. cual es el problema? que ahora tb ese campo nif puede ser un NIE, con lo que la programacion del NIF no me comprueba lo del nie...

podriamos añadir que verifique al mismo tiempo si es un NIE o NIF??

saludos,

ferranWEB

[David]

Seguro que se puede, pero es que en mi país no usamos eso del Nif/Nie así que yo no conozco el formato correcto como para hacer que compruebe el Nie. ¿La función que había puesto mgusdl lo hacía?.

[ferranWEB]

buenas,

hagamoslo más facil. pasando de verificar si es correcto o no. creo que el NIE es una letra X + 7 caracteres + otra letra.

como hariamos para verificar que el contenido del campo tiene que ser de 9 caracteres, y tiene que tener minimo una letra?

saludos

[David]

Según lo que leí en Wikipedia el formato sería que empiece con X o Y seguido de 7 dígitos y luego una letra. Por lo que podríamos hacer con una expresión regular:

Código javascript:

function validaNIE(value) {
    niereg = /[x|X|y|Y]\d\d\d\d\d\d\d[a-zA-Z]/;
    if (value.length != 9 || !niereg.test(value)) {
       alert("NIE no válido");
       return false;
   }
   return true;
}
 

[mgusdl]

Nove si os estais complicando la vida!!

Código javascript:

function checkNIF(nif)
    {
    if (!nif) return false;
    var letras = 'TRWAGMYFPDXBNJZSQVHLCKE';
    if (nif.length != 9) return false;
    else {
        letra = nif.substr(8, 1); letra = letra.toUpperCase();
        dni = nif.substr(0, 8);
        dni = dni.toUpperCase(); dni = dni.replace('X','0');
        dni -= parseInt(dni/23) * 23;
        if (letras.charAt(dni) != letra) alert('NIF no valido);
        else alert('NIF correcto');
        }
    }
 

[wmzt]

Hola,

Te recuerdo que la validación para el NIE cambió desde hace algunos meses. Antes únicamente iniciaba con letra X, pero ahora el NIE / NIF de personas extranjeras también inicia con la letra Y.

Te dejo un enlace donde puedes ver el código para crear un validador DNI/NIE/NIF actualizado:

www.web2.0facil.com/2008/12/02/validar-nuevo-nienif-con-letra-y-inicial/

por favor, tenemos que actualizarnos, es una vergüenza que a estas alturas una gran parte de los usuarios no puedan validar sus números de identificación y hagamos perder clientes a nuestros clientes!!

Saludos.