Validar numeros

[rol2007]

hola tengo esta función que me valida que ingresen solo numeros, el problema es que solo funciona en IE y en firefox no puedo ingresar puntos decimal

si alguien sabe como se debiera validar se lo agradezco he buscado los códigos ascii y no encuentro nada para que los tome

gracias

Código:

<script>
function KeyIsDecimal2(evt )
{
	var isNav = (navigator.appName.indexOf("Netscape") != -1)
	var isIE = (navigator.appName.indexOf("Microsoft") != -1)

	if (isNav) {
		if ( evt.which == 13 || evt.wich == 46 || evt.which == 8 || evt.wich == 110 ||(evt.which >= 48 && evt.which <=57) ){
			if (evt.which == 46 && txt.indexOf('.')!=-1) return false;
			return true;
		}
		return false;
	}
	else if (isIE){
		evt = window.event;
		if ( evt.keyCode == 13 || evt.keyCode == 46 || evt.keyCode == 8 || (evt.keyCode >= 48 && evt.keyCode <= 57) ){
			if (evt.keyCode == 46 && txt.indexOf('.')!=-1) return false;
	
			return true;
		}
		return false;
	}
	else {
		alert("Su browser no es soportado por esta aplicación")
		}
	return false
}
</script>
	<tr>
				<td> Monto: </td><td><input type="text" name="monto" 
<input name="txt_rut" maxlength=8 size=8 tabindex=1 onkeypress="KeyIsDecimal2(event)"></td> 
 
 
			 <br>
			 

			<td><input type="button" name="enviar" value="Guardar">
			</tr>

[buzu]

Para empezar estas dejando que solo los navegadores Netscape e IE puedan tener acceso a la funcionalidad. Si el navegador no es ninguno de estos dos, entonces lo estas rechazando.

Yo te sugiero que en lugar de eso, uses una expresión regular.

Mira aquí: http://forums.asp.net/t/897862.aspx

[CMSalvado]

Hola rol2007, tu código es algo antiguo (de la era de IE4 y Netscape 4, 1998-2000!!), en la actualidad podemos simplificarlo de la siguiente manera:

Código javascript:

function isDecimalKey(evt, txt){ 
    var key = evt.which || evt.keyCode;
    // Capturar el codigo de la tecla presionada. 
    // Si no es evt.which, usar evt.keyCode (para IE) 
 
    if ((key == 13 || key == 46 || key == 8) || (key >= 48 && key <=57)){
    // Si la tecla es backspace, enter, punto o digito 
        if (key == 46 && txt.indexOf('.')!=-1){
        // Si es el caracter "." comprobar que sea el unico 
            return false; 
        } 
        return true; 
    } 
    return false; 
}
 

Ah y no olvides poner en tus inputs lo siguiente:

Código html:

<input type="text" name="monto"
          onkeypress="return isDecimalKey(event, this.value)">
 

Básicamente la función isDecimalKey retornara true si la tecla presionada es un dígito, o si es el primer punto ingresado, de lo contrario retornará false, impidiendo que se pueda ingresar cualquier texto que no cumpla con lo anterior.

Puedes probar el código anterior acá: jsbin.com/itute
(Lo siento, no puedo incluir enlaces externos debido a que no tengo mas de 30 mensajes!)


Saludos,

-- CMS

[posman]

y se puede simplificar aun mas

Código:

<script>
function soloNumeros(evt) {
	var tecla = String.fromCharCode(evt.which || evt.keyCode);
	if ( !/[\d.\b\r]/.test(tecla) ) return false;
	return true;
}
</script>

<input type="text" id="texto" onkeypress="return soloNumeros(event);"

La expresion regular acepta digitos, punto, retroceso y enter

[buzu]

habría que hacer un análisis para ver cual de los dos métodos conviene más, algunas veces los métodos primitivos son más efectivos por lo que si lo mismo se cumpliera en este caso el if es mejor que el .test, pero de eso no estoy seguro.

[rol2007]

muchas gracias ...revisare los códigos y les cuento como les fue
y es verdad la funcion es re antigua la encontré en un archivo js de unos proyectos atras

probe la de CMSalvado y resulto sin problemas en FF y IE
muchas gracias a todos!