ayuda con un calculo porfavor

[Rasec101]

Tengo un formulario en php y estoy haciendo el calculo con javascript pero tengo el siguiente problema si alguien me puede ayudar por favor en el formulario hago un calculo que me arroja dos resultados que son (total_impo) y (total2) mediante un if si el total_impo es menor me haga un calculo tomando total_impo y en el else me tome total2 para la misma operación pero siempre me toma el resultado de total_impo.
No entiendo que tengo mal acá dejo el código para que alguien me eche una mano por favor. Gracias de antemano.

Código Javascript:

if(total2>total_impo)
{
  var calcular=document.getElementById("calcular").checked;
  
  document.getElementById("seguro2").value = Math.round(document.getElementById("calcular").checked ? (total2)*0.60/100: 0);
 
var calcular1=document.getElementById("calcular1").checked;
 
 document.getElementById("caja").value = Math.round(document.getElementById("calcular1").checked ? (total2)*0.60/100: 0);
 
 var cantidad=a.total2.value;
var precio=a.imposiciones_por.value;
var subt=cantidad*precio;
var total=subt/100;
var total1=(total-1.87)/100;
//var total_general=substring(total,total.length,-2);;
if (isNaN(subt))
{
 a.seguro.focus(); 
}
else
{
a.seguro.value=Math.round(total1); 
 
 
var cantidad=a.total2.value;
var precio=a.por_salud.value;
var subt=cantidad*precio;
var total=(subt/100);
 
if (isNaN(subt))
{
a.sistema_salud.focus(); 
}
else
{
a.sistema_salud.value=Math.round(total); 
}
}
 

Código Javascript:

else
{
 
  var calcular=document.getElementById("calcular").checked;
  
  document.getElementById("seguro2").value = Math.round(document.getElementById("calcular").checked ? (total_impo)*0.60/100: 0);
 
var calcular1=document.getElementById("calcular1").checked;
 
 document.getElementById("caja").value = Math.round(document.getElementById("calcular1").checked ? (total_impo)*0.60/100: 0);
 
 
 
var cantidad=a.total_impo.value;
var precio=a.imposiciones_por.value;
var subt=cantidad*precio;
var total=(subt/100)-seguro;
 
if (isNaN(subt))
{
 a.imposiciones.focus(); 
}
else
{
a.imposiciones.value=Math.round(total);
}
 
 var cantidad=a.total_impo.value;
var precio=a.imposiciones_por.value;
var subt=cantidad*precio;
var total=subt/100;
var total1=(total-1.87)/100;
//var total_general=substring(total,total.length,-2);;
if (isNaN(subt))
{
 a.seguro.focus(); 
}
else
{
a.seguro.value=Math.round(total1); // Control que muestra el total
}
 

[Tecna]

Buenas,

para poder hacer cálculos con el texto que introduzcas deberías asegurarte primero que efectivamente los datos son numéricos para ello puedes simplemente restar 0, o usar la constructora Number( ) con la cadena como argumento (solo para numeros en base 10) , o usar las funciones parseInt o parseFloat (en cualquier base y para números enteros o decimales).
La función isNaN también se usa para comprobar si un dato es un valor numérico legal y devuelve true si el valor analizado es o se puede convertir en el valor especial NaN y false en el resto de casos, por lo tanto cuando usas como condición del if : " if (isNaN(subt)) " ese bloque sólo se ejecutará cuando subt sea NaN y en el resto de casos (null, undefined, cadenas, números, objetos) se ejecutaría el else que no es lo que estás buscando. Sería mejor poner como condición lo que estás buscando y que se ha de hacer mas a menudo en el if y la excepción en el else.
El código según lo pones queda un poco confuso porque no se sabe muy bien cada else a que if pertenece. La norma dice que una cláusula else forma parte de la instrucción if mas cercana así que es probable que tampoco esté haciendo lo que esperas que haga. Y quedaría mas claro si indentaras el código y en lugar de usar else usaras else if para anidar varios if

[Rasec101]

gracias tecna por el aporte por hay buscando encontre algo que me soluciono el problema que tenia gracias.