[SOLUCIONADO] Funcion que Aumenta no Suma

[emilio_viguri]

Ya no se que paso ni por que lo hace pregunto por que me hace esto

tengo un formulario que validadar donde paso parametros los cuales quiero hacer una simple operacion como son varios uso un id para identificarlos y que sean diferentes para depsues hacer la operacion y mostrar bueno todo esta bien pero no se por que no lo suma y me hace cosa rara.

Mi input con PHP y resultado abajo...

Código HTML:

<input name="pago<?php echo $row_pendiente['idcliente']; ?>" type="text" id="pago<?php echo $row_pendiente['idcliente']; ?>" onKeyUp="valida(<?=$row_pendiente['saldo'].','.$cobro.','.$row_pendiente['idcliente']; ?>)" value="<?=$pago;?>">
 
<input name="pago4" type="text" id="pago4" onKeyUp="valida(182.00,350,4)" value="168">
 

Código Javascript:

function valida(saldo, cobro, id){
    var valpago = calculo = pago = null;
    pago = document.getElementById('pago'+id);
    valpago = pago.value;
 
    calculo = saldo - cobro + valpago;
    
    document.getElementById('saldo'+id).value = calculo;
 

si yo pongo 200
lo que me hace esto con esos valores es ponerme en saldo id: -168200

y lo que tendria que hacer es
calculo = saldo - cobro + valpago;
32 = 182 - 350 + 200;


no -168200

[ArturoGallegos]

el problema es bastante claro, estas usando input de texto por lo que al momento de querer sumar en realidad lo que haces es concatenar

182 - 350 = -168 este valor es convertido a texto y se concatena con tu 200 ya que es una cadena de texto lo que has enviado.

Tu solución es parseInt o parseFloat según el tipo de valor enviado, ya sea entero o con decimales según corresponda.

calculo = saldo - cobro + parseInt(valpago);// enteros

o bien
calculo = saldo - cobro + parseFloat(valpago); // con decimales

[emilio_viguri]

Bueno como pelicula de terror tengo un final feliz, a qui les dejo el codigo por si alguien lo quiere re utilizar funciona. Gracias aprendi algo nuevo !

Código Javascript:

function valida(saldo, cobro, id){
    var varsaldo = pago = calculo = pago = null;
    pago = document.getElementById('pago'+id);
    varsaldo = document.getElementById('saldo'+id);
    
    calculo = saldo - cobro + parseFloat(pago.value);
    
    if(!isNaN(calculo)) varsaldo.value = calculo;
    else varsaldo.value = 0;
    
    if(pago.value != ''){//Estilo Verde Bien
        pago.style.color="#006600";
        pago.style.background="#D9FFEC";
        pago.style.border = "thin solid #006600";
    }else{ // Estilos Rojos Errores
        pago.style.background="#FFE1E1";
        pago.style.color="#990000";
        pago.style.border = "thin solid #990000";
    }
    
    if(calculo>=0){//Estilo Verde Bien
        varsaldo.style.color="#006600";
        varsaldo.style.background="#D9FFEC";
        varsaldo.style.border = "thin solid #006600";
        document.getElementById('id_'+id).checked=true;
//      document.getElementById('id_'+id).disabled=false;
//      document.getElementById('submit_'+id).disabled=false;
    }else{ // Estilos Rojos Errores
        varsaldo.style.background="#FFE1E1";
        varsaldo.style.color="#990000";
        varsaldo.style.border = "thin solid #990000";
        document.getElementById('id_'+id).checked=false;
//      document.getElementById('id_'+id).disabled=true;
//      document.getElementById('submit_'+id).disabled=true;
    }
}
 

Código HTML:

<input name="pago13" type="text" id="pago13" onKeyUp="valida(854.00,350,13)" value="0" size="5" maxlength="13">
<input name="saldo13" type="text" id="saldo13" value="" size="5" maxlength="13" readonly>
<input type="radio" name="idcliente" value="13" id="id_13">
<input type="submit" value="Registra" id="submit_13">