Gereksiz girişi kontrol edin JQuery Code

Question

Bu çalışma koduna sahibim ve sanırım biraz uzun ve gereksiz olabilir mi?

$("#unique").click(function() { 

    if ($(this).is(':checked')) { 
     $(".lotud").show(); 
     $("#add_lot").hide(); 
     $("#lots_rows_contnr").hide(); 

     $(".lotud input").prop({disabled: false}) 
     $("#lots_rows_contnr input").prop({disabled: true}) 
    } 
    else { 
     $(".lotud").hide(); 
     $("#add_lot").show(); 
     $("#lots_rows_contnr").show(); 

     $(".lotud input").prop({disabled: true}) 
     $("#lots_rows_contnr input").prop({disabled: false}) 
    } 

}); 

Answer 1

Sen seçicileri katılmadan ve disabled özellik için temel olarak checked özelliğini kullanarak, DOMElement kendisinde checked özelliğini kullanarak, ternaries kullanımı yoluyla hafifçe kısaltabilir. Bunu deneyin:

$("#unique").click(function() { 
    $(".lotud").toggle(this.checked); 
    $("#add_lot, #lots_rows_contnr").toggle(!this.checked); 
    $(".lotud input").prop({ disabled: !this.checked }); 
    $("#lots_rows_contnr input").prop({ disabled: this.checked }); 
}); 

Orijinal veya yukarıdaki iki sürümden hangisinin daha okunaklı olduğu bir fikir konusudur.