Limitar a numeros y "-"

[Vendetta]

Buenas!

He estado haciendo algunas cosillas y un amigo me ha hecho esta funcion

Código Javascript:

function isNumberKey(evt)
      {
         var charCode = (evt.which) ? evt.which : event.keyCode
         if (charCode > 31 && (charCode < 48 || charCode > 57))
            return false;
 
         return true;
      }
 

De esta forma en los input que quiera, solo se pueden poner numeros ahora, no habia caido en que era posible que algunos numeros cayeran en negativo y claro, no me deja. He probado haciendo cosas del tipo

Código Javascript:

if (charCode > 31 && (charCode < 48 || charCode > 57 || charCode != 45))
 

Pero ni se Javascript ni estoy seguro de que el 45 haga referencia al guión. He buscado una tabla pero vamos, no estoy muy metido en el tema este asi que no se si he buscado lo que tenia que buscar. Tambien he mirado por google pero esque tampoco se muy bien que o como deberia buscarlo y no encuentro la forma.

Solo me queda pedir ayuda por aqui : <

[AlvaroG]

No necesitás probar que charCode sea mayor de 31 si luego probás que esté entre 57 y 48.

Código Javascript:

function isNumberKey(evt) {
    var charCode = (evt.which) ? evt.which : event.keyCode
 
    return (charCode >= 48 && charCode <= 57) || (charCode == 45)
}
 

De todas formas ten en cuenta que "-4-5506-7" sería una entrada válida, pero no es un número. Es mejor no inventar y validar el número usando lo que ya está en el lenguaje:

Código Javascript:

function esEntero(num) {
    return ! isNaN(parseInt(num, 10));
}
 

Saludos.

[Vendetta]

Iniciado por AlvaroG

No necesitás probar que charCode sea mayor de 31 si luego probás que esté entre 57 y 48.

Código Javascript:

function isNumberKey(evt) {
    var charCode = (evt.which) ? evt.which : event.keyCode
 
    return (charCode >= 48 && charCode <= 57) || (charCode == 45)
}
 

De todas formas ten en cuenta que "-4-5506-7" sería una entrada válida, pero no es un número. Es mejor no inventar y validar el número usando lo que ya está en el lenguaje:

Código Javascript:

function esEntero(num) {
    return ! isNaN(parseInt(num, 10));
}
 

Saludos.

Pues usando el codigo "inventado" funciona y usando lo que ya está en el lenguaje me deja poner numeros letras y de todo :\

Suponiendo que lo que tu has escrito sobreescribe mi codigo (y cambiando las palabras que se tengan que cambiar)

[dannytm]

Porque no lo compruebas con un parseInt?

[AlvaroG]

Iniciado por Vendetta

Pues usando el codigo "inventado" funciona y usando lo que ya está en el lenguaje me deja poner numeros letras y de todo :\

Claro, uno usa lo que funciona, pero como ves resulta imposible de extender manteniendo funcionalidad.
La única comprobación que se me ocurre es que agreges una comprobación que permita el guión *solamente* en el primer caracter.

Iniciado por Vendetta

Suponiendo que lo que tu has escrito sobreescribe mi codigo (y cambiando las palabras que se tengan que cambiar)

No, es un ejemplo. Sirve para comprobar una cadena, no un evento, y por lo tanto no funciona si lo aplicás directamente.
La forma más directa de usarlo para sustituir tu código es cambiar la línea 4 del primer bloque de código que puse por la línea 2 del segundo, cambiando "num" por "charCode"


Saludos.