Cita:
Iniciado por caricatos 
No he visto el mensaje, pero no veo bueno ese sistema de validación... ya que el botón no es del tipo submit...
Hombre es otra forma de hacerlo, lo que le faltaría poner lo siguiente para enviar en los datos en caso de que estuviera todo correcto:
Código:
document.forms["form1"].submit();
O eso o enviarlo por AJAX, con lo que no necesitas ningún submit.
Aparte de lo que te comenta Caricatos que está bien, deberías controlar también por ejemplo, si en un cuadro de texto han introducido sólo espacios (busca por aquí alguna función de trim).
Y para números te recomiendo que le eches un vistazo a parseInt(), ya que tiene "sorpresas"... por ejemplo si el usuario introduce "100 200" te devolvería 100, también hay sorpresa si introduce algo como "0x900"... Lo mejor para estos casos es validar toda la cadena carácter a carácter con un bucle o mejor con una expresión regular.
Y repito... cuidadín con parseInt... que si haces lo siguientes verás que la variable "num" es numérica y su valor es NaN (no es un número)...
Código:
var num = parseInt( "" );
alert( typeof(num) + " -> " + num );
Cosas del javaScript...