Concuerdo con
Eleazan, pero además podrías mejorar muchísimo tú código si realmente quieres validar un DNI ya que sólo validas que sea numérico...
¿Entonces si yo agrego 8 números eso es un DNI correcto?
Deberías tener en cuenta el NIF, es decir la letra y ver sí se corresponde al número.
Se explica en multitud de páginas y es un ejercicio habitual de programación cuando se empieza y para aprender a usar "mod" --> % .
Por cierto, si estás en españa y lo estás validando
para algo SERIO, recuerda que hay 2 tipos de NIF, para los nacionalizados y para los extranjeros no nacionalizados, Letra X, y que dada la extensa inmigración la normativa cambió, existiendo ahora también la Letra Y. (además va al principio)
Depende de lo serio que quieras hacer tu validador de DNI, pero usando la letra, aunque no puedas saber si pertenece al usuario, al menos sí que compruebas que dicho DNI es válido.