0

El total se genera multiplicanco cantidad x precio en la siguiente funcion

El total se genera multiplicanco cantidad x precio en la siguiente funcion

      function calcularMult(con){
      $("#resultado_" + con).val($("#nn_" + con).val() * $("#mm_" + con).val());
      var sum1 = 0;
      $("input[id^='resultado_']").each(function() {
        sum1 += Number($(this).val());
       });  
      sum1.toFixed(2);
      var sum=sum1;         
      $("#total").val(sum)
      }

<button type="button" name="crea" id="+" onClick="cargaProd();" class="btn btn-primary" )>
                  Nueva partida
                  </button>
                  <button type="button" name="envia" id="-" onClick="eliminar()" class="btn btn-danger")>
                  Eliminar Partida
                  </button>
                  <input type="button" name="process" class="btn btn-primary" onclick="pregunta()" value="<?php echo __("Terminar Orden de Compra"); ?>" >





                  <script type="text/javascript">
                    var con = 1;
                     var can = '<?php echo $quantity;?>'-1;


                    function cargaProd()
                      {
var linea = "<tr id='"+con+"'><td>"+con+"</td><td><input type='text' name='articulo["+can+"]' placeholder=' Articulo' class='span12 inputbox'/></td><td><input name='can["+can+"] ' id='nn_"+con+"' type='text' class='span6 inputbox' size='15' value='0' onkeyup='calcularMult("+con+")'></td><td><select name='medida["+can+"]' id='medida' class='selectpicker span9' ><option value=''></option><?php foreach ($resultado as $key => $value) {?><option value='<?php echo $value["Unidad"];  ?>'><?php echo $value["Unidad"];  ?></option><?php  } ?></select></td><td><input name='dp["+can+"]' id='mm_"+con+"' type='text' class='span6 inputbox' size='15' value='0' onkeyup='calcularMult("+con+")' ></td><td><input  name='to["+can+"]' id='resultado_"+con+"' type='number' step='0.01' class='span6 inputbox' size='15' ></td></tr>";

$('#listaArticulos').append(linea);
con = con+1
can = can+1
$("#quantity").val(can);
}
function eliminar(){
  //Determinamos Cuantas filas posee la lista
  var filas = $("#listaArticulos tr").length;

  //Eliminamos la última
  $("tr[id="+filas+"]").remove();
  //Si el contador no es igual a 1, le restamos la fila eliminada
  if(con>1){
    con = con-1
    can = can-1
    $("#quantity").val(can);

  }

}
3
  • 1
    Intenta omitiendo la línea 7 y asignándola a sum directamente var sum = sum1.toFixed(2);
    – DjCrazy
    Commented el 12 dic. 2018 a las 18:21
  • Por favor sería más fácil ayudarte si agregas un ejemplo mínimo verificable Commented el 12 dic. 2018 a las 19:25
  • numero.toFixed(2) te lo deja con 2 decimales. Pero ojo: sera una cadena de texto. Así que si lo necesitas para más cosas dale un parseFloat
    – enxaneta
    Commented el 12 dic. 2018 a las 19:28

2 respuestas 2

0

tu problema es sencillo.

function calcularMult(con){
  $("#resultado_" + con).val($("#nn_" + con).val() * $("#mm_" + con).val());
  var sum1 = 0;
  $("input[id^='resultado_']").each(function() {
    sum1 += Number($(this).val());
   });  
  //Cambiar
  var sum=sum1.toFixed(2);        
  $("#total").val(sum)
}

No estás asignando sum1.toFixed(2); a nada, por ello el resultado de este valor queda en el aire, puede hacerlo como te puse en el código anterior o sencillamente;

  sum1 = sum1.toFixed(2);
  var sum=sum1; 

Espero te haya ayudado.

Por cierto, si el input es de tipo number, es mejor hacer un cast a la variable;

  sum1 = Number(sum1.toFixed(2));
  var sum=sum1;
4
  • he hecho lo que has explicado en tu respuesta pero siguen apareciendo muchas decimales
    – Henry 95
    Commented el 12 dic. 2018 a las 19:57
  • Te agradezco si compartes la estructura html y hago una prueba aquí. Una estructura de ejemplo, si tu html es muy grande. Gracias! Commented el 12 dic. 2018 a las 20:05
  • ya he subido la parte donde se genera la linea html
    – Henry 95
    Commented el 13 dic. 2018 a las 14:19
  • Regalame por favor la estructura final, no el código php. Esto es importante para que pueda copiar y pegar código, verificando lo que dices. Commented el 14 dic. 2018 a las 22:07
0

UPDATE:

Es cierto que en Firefox no funciona. Te pongo un pequeño script para que también te funcione ahí:

$('#total').keyup(function(){
  var total = $(this).val();
  var separador;

  if(total.indexOf(',') != -1){
    separador = ',';
  }
  else if(total.indexOf('.') != -1){
    separador = '.';
  }

  if(separador){
    var decimales = total.split(separador);
    if(decimales[1].length>2){
      decimales[1] = decimales[1].substr(0, 2);
    }
    total = decimales.join(separador);
  }

  $(this).val(total);
});
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<input id="total" type="text">

5
  • no cambia , sigue mostrando muchos decimales
    – Henry 95
    Commented el 12 dic. 2018 a las 18:44
  • @Henry95 Funciona en Chrome, tal vez deberías de especificar que navegador usas Commented el 12 dic. 2018 a las 19:24
  • uso chome , utilizo tu código y aun así siguen apareciendo muchas decimales
    – Henry 95
    Commented el 12 dic. 2018 a las 19:56
  • @Henry95 has probado el update?? acabo de probar en chrome y me está funcionando bien...
    – cooper
    Commented el 12 dic. 2018 a las 20:02
  • @cooper probablemente los dos tengan razon , cambia el keyup por addEventListener Commented el 13 dic. 2018 a las 14:30

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.