Como puedo mejorar el siguiente codigo

[Montes28]

hola amigos que me recomiendan para mejor el siguiente código, intente utilizar un switch pero no me funciona.

Lo que pretendo es realizar sumas dependiendo de si se marca o no un grupo decheckbox, estos checkbox están agrupados por áreas.

los checkbox los creo desde base de datos

Código PHP:

@foreach($reglas as $key =>  $regla)
 
     @if ($area != $regla->area->nombre) 
$area = $regla->area->nombre;
 
@endif
<input type="checkbox" id="{{ $regla->area->id }}" class="estado" value="{{ $regla->valor_1 }}">
@endforeach
 

Código Javascript:

var   sum1 = 0;
        sum2 = 0;
        sum3 = 0;
        sum4 = 0;
        sum5 = 0;
        totalArea1 = 0;
        totalArea2 = 0;
        totalArea3 = 0;
        totalArea4 = 0;
        totalArea5 = 0;
    $(".estado").change(function() {
    
 
    if(this.checked) {
        var area = this.id;
 
        if (area == 1) {
        sum1 += Number($(this).val());
        totalArea1 = sum1*0.7;
        $('#checkme').attr('checked',true);
        
        };
        if (area == 2) {
            sum2 += Number($(this).val());
            totalArea2 = sum2*0.1;
        };
        if (area == 3) {
            sum3 += Number($(this).val());
            totalArea3 = sum3*0.1;
        };
        if (area == 4) {
            sum4 += Number($(this).val());
            totalArea4 = sum4*0.05;
        };
        if (area == 5) {
            sum5 += Number($(this).val());
            totalArea5 = sum5*0.05;
        };          
 
 
    }else{
        //var area = this.id;
        //alert(area);
        var area = this.id;
        if (area == 1) {
        sum1 -= Number($(this).val());
        totalArea1 = sum1*0.7;
        $('#checkme').attr('checked',false);
        }
        if (area == 2) {
        sum2 -= Number($(this).val());
        totalArea2 = sum2*0.1
        }
        if (area == 3) {
        sum3 -= Number($(this).val());
        totalArea3 = sum3*0.1;
        }
        if (area == 4) {
        sum4 -= Number($(this).val());
        totalArea4 = sum4*0.05;
        }
        if (area == 5) {
        sum5 -= Number($(this).val());
        totalArea5 = sum5*0.05;
        }
    }
 
 
    $('#Sis').html(Math.round(sum1 * 100) / 100);
    $('#Def').html(Math.round(sum2 * 100) / 100);
    $('#Geo').html(Math.round(sum3 * 100) / 100);
    $('#Geof').html(Math.round(sum4 * 100) / 100);
    $('#Geol').html(Math.round(sum5 * 100) / 100);
 
    var ia=(totalArea1 + totalArea2 + totalArea3 + totalArea4 + totalArea5)*100;
 
    $('#IA-1').html(ia.toFixed(2));
});
 

[diurno10]

Me pondrias un print del html, ya se que son todos checkbox por lo que veo en el php, pero con eso me guio mejor.
Gracias.

[Montes28]

Hola diurno10 asi es como se veria



http://i288.photobucket.com/albums/l...psz25y40yn.png

es lo que me pides? o imprimo el codigo html?

[diurno10]

NO, lo que quiero ver es como te quedaria el html, ej :

Código HTML:

<input type="checkbox" id="1" name="lala" />
<input type="checkbox" id="2" name="lala" />
<input type="checkbox" id="3" name="lala" />
--o puede ser asi
 
<input type="checkbox" id="1" name="lala" />
<input type="checkbox" id="1" name="lala" />
<input type="checkbox" id="1" name="lala" />
 
<input type="checkbox" id="2" name="lala" />
<input type="checkbox" id="2" name="lala" />
<input type="checkbox" id="2" name="lala" />
 

[Montes28]

este es el código



http://i288.photobucket.com/albums/l...psz25y40yn.png

[superweb360]

En las últimas versiones de jQuery, para comprobar si un checkbox está marcado o no, se usa prop()

Código Javascript:

$('#myCheckbox').prop('checked', true);
$('#myCheckbox').prop('checked', false);
 

[Montes28]

este es mi html



código
http://s288.photobucket.com/user/mon...8drjt.png.html

[superweb360]

En el código tienes un </tr> de más. Fíjate que está marcado en rojo.