necesito sumar y me concatena!!

[nataliasoria]

Hola, quisiera saber porque no me suma.......gracias!

<script>

function calculo(campo){

var totalfinal;
var Total;


gTotal = parseInt( document.getElementById('totalGeneral').value);
pCantidad = document.getElementById(['prd'+campo]).value;
tListado = document.getElementById('listado').value;
pPrecio = parseInt(document.getElementById(['txt'+campo]).value);

subTotal = pCantidad*pPrecio;

document.getElementById(['subto'+campo]).value = subTotal;


valor = document.getElementById(['subto'+campo]).value;
Total = Total + valor;
document.getElementById('totalGeneral').value = Total;



}
</script>

[derkenuke]

Hola nataliasoria:

Hay varias cosas que no entiendo:
¿Por qué a document.getElementById() le pasas a veces un String y a veces un Array? Lo correcto es siempre String...
Cuando haces Total = Total + valor; ¿Qué era Total antes? ¿No era undefined? Además, valor es un String, lo que me lleva a la siguiente cuestión:
¿Por qué haces parseInt() a algunos valores y otros no? Si son números y quieres sumarlos debes convertirlos a números, pero todos, no sólo algunos (bien con parseInt(), parseFloat(), Number(), o incluso los he visto que multiplican por la unidad al String :S ).



Veo demasiadas cosas raras como para que funcione...

[nataliasoria]

ya pude solucionarlo, solo que ahora tengo que ver como hacer porque si me modifican algun valor sigue suamandolo, debería restarle el valor anterior y luego sumarle el nuevo.....(son valores dinamicos que traigo desde el php)

<script>

function calculo(campo){

var totalfinal;


gTotal = isNaN( document.getElementById('totalGeneral').value) || document.getElementById('totalGeneral').value=="" ? "0" : document.getElementById('totalGeneral').value;
pCantidad = document.getElementById(['prd'+campo]).value;
tListado = document.getElementById('listado').value;
pPrecio = parseInt(document.getElementById(['txt'+campo]).value);

subTotal = pCantidad*pPrecio;


document.getElementById(['subto'+campo]).value = subTotal;

// valor = parseInt(document.getElementById(['subto'+campo]).value);
gTotal = parseInt(gTotal) + subTotal;


document.getElementById('totalGeneral').value = gTotal;


}
</script>

[derkenuke]

Hola de nuevo:

No sé si has pasado de mi, o te gusta auto responderte pasando por alto los demás posts, pero... ¿has leído mi respuesta?

[nataliasoria]

ahora la he leido, y tienes razon..

respondi cuando aun no vi tu respuesta.


respecto a lo que me preguntas de porque a veces paso un string y a veces un array, es porque estoy mostrando productos que fueron seleccionados previamente mediante php, entonces al ser dinamico el resultado de alguna manera tenia que indicar que ese valor.

yo tengo la cantidad del producto, e ingreso manualmente el precio del producto, y es ahi donde genero el calculo.


lo intentare, gracias.

[venkman]

A lo que se refiere derkenuke en su primera pregunta es a que NO hagas esto:

Código:

document.getElementById(['subto'+campo]).value

Sino que hagas esto:

Código:

document.getElementById('subto'+campo).value

Por cierto, una recomendación. Además de hacer parseInt o parseLong como dice derkenuke, cuando hagas parseInt, ponle la base 10:

Código:

alert(parseInt("025"));
alert(parseInt("025",10));