Pregunta sencilla

[Skylin3]

Soy nuevo en js pero he mirado tu codigo y mi pregunta es.

Quieres saber si los campos estan vacios, para que preguntar

if(document.Formulario.Nombre.value.length == 0)

Si con un:

if(document.Formulario.Nombre.value == "")

Seria menos complejo, aparte no estoy seguro pero puede que el problema es que el .lenght si no ahi valor devuelva un NULL y no un 0, pero no estoy seguro de eso.

Y la funcion isNaN() nunca la he utilizado pero supongo que sera para preguntar si es un numero, o un numero float.
Pero por que pones valor=parseFloat(valor); antes, si la siguiente función es para preguntar si es un numero, mejor no seria:

valor = document.Formulario.Precio.value;
if(isNaN(valor))
{
alert ("Debe introducir un valor numerico en el precio");
document.Formulario.Precio.focus();
return;
}
else
{
valor = parseFloat(valor);
}

Puede que no tenga razon, pero yo comento mi punto de vista haber si te ayuda para algo.

Sino haber si alguien nos explica el codigo a los dos entonces xd.

;) un saludo.