Actualizar valores en suma

[Briss]

Hola a todos tengo un formulario donde voy a realizar una suma
son 3 inputs a lado tienen un checkbox con el texto Activar por defecto esta seleccionado, si se desmarca pone 0 en el input correspondiente

Esto lo hago asi

Código HTML:

<script type="text/javascript">
    function Cambia(porcentaje)
  {   
   if(porcentaje.checked)
    {
      document.for1m.p1.value="";
	  
    }
   else
    {
      document.form1.p1.value="0";
	 
    }
	
  }
</script>

<script type="text/javascript">
    function Cambia2(porcentaje)
  {   
   if(porcentaje.checked)
    {
      document.for1m.p2.value="";
	  
    }
   else
    {
      document.form1.p2.value="0";
	 
    }
	
  }
</script>

<script type="text/javascript">
    function Cambia3(porcentaje)
  {   
   if(porcentaje.checked)
    {
      document.for1m.p3.value="";
	  
    }
   else
    {
      document.form1.p3.value="0";
	 
    }
	
  }
</script> 

Y la suma asi

Código HTML:

<script> 
function sumar() {
	elem_1 = (isNaN(document.form1["p1"].value) || document.form1["p1"].value == "") ? "0" : document.form1["p1"].value;
	elem_2 = (isNaN(document.form1["p2"].value) || document.form1["p2"].value == "") ? "0" : document.form1["p2"].value;
	elem_3 = (isNaN(document.form1["p3"].value) || document.form1["p3"].value == "") ? "0" : document.form1["p3"].value;
	
	document.form1.total.value = parseInt(elem_1) + parseInt(elem_2)+parseInt(elem_3) ;
}
</script> 

Ambos script funcionan Pero si primero por ejemplo en el input p1 pongo 5
y desactivo el checkbox (osea lo cambio a 0)
en la suma me sigue tomando ese mismo 5 (cuando en realidad ya es 0 porque se desactivo el checkbox) esto afecta el resultado de la suma pues no actualiza el valor del total....



Algun me puede decir que estoy haciendo mal
Gracias de antemano

[Dradi7]

no sera porque en tu funcion cambia esta escrito esto for1m en vez de esto form1 ademas veo demasiado código para algo muy simple

Saludos

[Briss]

Iniciado por Dradi7

no sera porque en tu funcion cambia esta escrito esto for1m en vez de esto form1 ademas veo demasiado código para algo muy simple

Saludos

Gracias por responder he cambiado lo que me indicas y sigo sin actualizar
por ejemplo si en los tres pongo 5
en Total tengo 15
pero si desactivo algun input y paso a 0 el valor
el total no queda en 10 sino en 15

Respecto a la funcion de cambia porcentaje... no pude hacerla mas corta (soy nueva en esto)
xD

[Dradi7]

es que para que quede en 10 como dices debes presionar nuevamente el boton si es que lo tienes para que te haga ejecute la funcion sumar.

Otra manera seria poner al final de la funcion Cambia lanzar el evento sumar asi

Código Javascript:

function Cambia(porcentaje){
// AQUI VA OTRO CODIGO
sumar();
}
 

[Briss]

Iniciado por Dradi7

es que para que quede en 10 como dices debes presionar nuevamente el boton si es que lo tienes para que te haga ejecute la funcion sumar.

Otra manera seria poner al final de la funcion Cambia lanzar el evento sumar asi

Código Javascript:

function Cambia(porcentaje){
// AQUI VA OTRO CODIGO
sumar();
}
 

Con esto funciono perfecto Gracias mil