Numeros flotantes

[Sirrohan]

Hola amigos foreros...

Tengo la siguiente detalle en estas líneas de código:

function calculo()
{
var a=document.form1.nro_dias.value;
var b=document.form1.dias1.value;
var c=document.form1.UT.value;
var d=document.form1.ctte.value;
var e=document.form1.nro_dias.value;
document.form1.dias1.value=a;
var j=eval(a*c*d);
document.form1.precio_dias.value=j;
var f=document.form1.dias2.value;
var g=document.form1.UT2.value;
var h=document.form1.ctte2.value;
var i=eval(f*g*h);
var k=eval(j+i);
document.form1.precio_pasaje.value=i;
document.form1.precio_total.value=k;
document.form1.monto.value=k;

}

Arroja el siguiente resultado en varias oportunidades:

2040.5000000000002

La pregunta es... ¿Como hago para redondear los valores después del punto?

Algo así: 2040.5

[IsaBelM]

usa toFixed(numero de decimales)

[Sirrohan]

No ha servido esa función... el valor que devuelve los valores es cuando se realiza ese producto

var j=eval(a*c*d);

[IsaBelM]

por qué usas eval??

var j = (parseFloat(a*c*d)).toFixed(2)

[Sirrohan]

porque es la forma de hacer operaciones matemáticas que estaba en la guía del curso de Javascript =)

[_cronos2]

Pues vaya libro ... XD Supongo que no tiene mayor importancia hacerlo en este ejemplo para calcular con JS, pero digamos que no es muy recomendable usar eval.
Saludos :D

[IsaBelM]

sí, bueno, no te lo creas todo. cual es esa guía??

[Sirrohan]

hice lo siguiente:
var j = (parseFloat(a*c*d)).toFixed(2)
var i=(parseFloat(f*g*h)).toFixed(2);
var k=(parseFloat(j+i)).toFixed(2);

y todavía sigue apareciendo ese número punto flotante; probando distintos valores me aparece algo fuera de lo común:

1150.71 + 112.65 = 1273.3600000000001

No sé que estará pasando =S

[_cronos2]

1150.71 + 112.65 = 1273.3600000000001

En Chrome

alert(parseFloat(1150.71 + 112.65).toFixed(2)); // 1263.36

En FF e IE no he probado pero no creo que te añadan ese 0.00...01 porque sí.
Saludos (:

[Sirrohan]

Después de seis meses por fin pude resolver el problema, tuve que repasar este tutorial que encontré en otro tema, y así quedó el código

Código Javascript:

<script type="text/javascript">
function calculopasaje() 
{
    var a=parseFloat(document.getElementById("nro_dias").value);
    var b=parseFloat(document.getElementById("dias1").value);
    var c=parseFloat(document.getElementById("UT").value);
    var d=parseFloat(document.getElementById("ctte").value);
    var e=parseFloat(document.getElementById("nro_dias").value);
    var j=(a*c*d);
    document.getElementById("dias1").value=a;
    document.getElementById("precio_dias").value=j;
    var f=parseFloat(document.getElementById("dias2").value);
    var g=parseFloat(document.getElementById("UT2").value);
    var h=parseFloat(document.getElementById("ctte2").value);
    var i=(f*g*h);
    var k=parseFloat(i.toFixed(2))+parseFloat(j.toFixed(2));
    document.getElementById("precio_pasaje").value=i;   
    document.getElementById("precio_total").value=k.toFixed(2);
    document.getElementById("monto").value=k.toFixed(2);
}
</script>
 

Y funciona a la perfección :>

[Aijoona]

Funcionará pero no deja de ser horrible.

Que se supone que es la variable g? y la h? Los nombres de variables DEBEN tener coherencia semántica con su contenido.

Tarea: Reescribir el script nombrando las variables según su contenido.

[_cronos2]

Iniciado por Aijoona

Funcionará pero no deja de ser horrible.

Exacto, tengo que reconocer que ni si quiera paré a leer el código D:
Debes entender que si quieres que leamos/comprobemos algo no puedes poner esos nombres a las variables; incluso cuando tú dentro de x tiempo necesites ver el archivo otra vez verás que es imposible entenderlo. Para hacerle un poco de publicidad a @Aijoona http://blog.aijoona.com/2011/05/05/comenta-el-codigo/
Saludos (:

[Sirrohan]

Por donde lo veas es HORRIBLE

Vamos a complacerte =>

Código Javascript:

document.getElementById("precio_dias").value=(parseFloat(document.getElementById("nro_dias").value)*parseFloat(document.getElementById("UT").value)*parseFloat(document.getElementById("ctte").value)).toFixed(2);
 
document.getElementById("precio_pasaje").value= (parseFloat(document.getElementById("dias2").value)*parseFloat(document.getElementById("UT2").value)*parseFloat(document.getElementById("ctte2").value)).toFixed(2);
 
document.getElementById("monto").value=(parseFloat(document.getElementById("nro_dias").value)*parseFloat(document.getElementById("UT").value)*parseFloat(document.getElementById("ctte").value)).toFixed(2)+(parseFloat(document.getElementById("dias2").value)*parseFloat(document.getElementById("UT2").value)*parseFloat(document.getElementById("ctte2").value)).toFixed(2);
 

Creo que es preferible seguir con mi notación de variables :>

incluso cuando tú dentro de x tiempo necesites ver el archivo otra vez verás que es imposible entenderlo

6 meses después sabía exactamente lo que hacía el código :>

[Aijoona]

No, no es preferible. De hecho hasta estás teniendo variables inutiles y es algo que no notás por tu sintaxis (caso variables b y e).

Con un pequeño refactor (no aseguro que ande, es algo rapido):

Código Javascript:

function calculopasaje() {
    function value(id, val) {
        document.getElementById(id).value = val;
    }
 
    function floatValue(id) {
        return parseFloat(document.getElementById(id).value);
    }
 
    var
        nroDias = floatValue("nro_dias"),
        inicio = (nroDias * floatValue("UT") * floatValue("ctte")),
        fin = (floatValue("dias2") * floatValue("UT2") * floatValue("ctte2")),
        total = parseFloat(fin.toFixed(2)) + parseFloat(inicio.toFixed(2));
 
    value("dias1", nroDias);
    value("precio_dias", inicio);
    value("precio_pasaje", fin);
    value("precio_total", total.toFixed(2));
    value("monto", total.toFixed(2));
}
 

[Sirrohan]

Iniciado por Aijoona

No, no es preferible. De hecho hasta estás teniendo variables inutiles y es algo que no notás por tu sintaxis (caso variables b y e).

Con un pequeño refactor (no aseguro que ande, es algo rapido):

Código Javascript:

function calculopasaje() {
    function value(id, val) {
        document.getElementById(id).value = val;
    }
 
    function floatValue(id) {
        return parseFloat(document.getElementById(id).value);
    }
 
    var
        nroDias = floatValue("nro_dias"),
        inicio = (nroDias * floatValue("UT") * floatValue("ctte")),
        fin = (floatValue("dias2") * floatValue("UT2") * floatValue("ctte2")),
        total = parseFloat(fin.toFixed(2)) + parseFloat(inicio.toFixed(2));
 
    value("dias1", nroDias);
    value("precio_dias", inicio);
    value("precio_pasaje", fin);
    value("precio_total", total.toFixed(2));
    value("monto", total.toFixed(2));
}
 

Whoooaaaooo.... Pero como ya comenté más arriba (meses atrás), mis nociones de javascript son de un curso desactualizado, hasta se podría decir obsoleto...

De todos modos, gracias por el apoyo ;)