用于选中/取消选中复选框的弹出窗口

Question

I have prompts that produce a popup when a box is ticked and unticked. There looks to be a line of redundant code, but when removed the functions no longer work. So maybe not so redundant:-) But the #id doesn't match to anything (currently set as CAPS to not match)

Any ideas why this is interfering?

$('#checkbox').click(function() {
  if ($("#checkbox").is(':checked')) {
    if (confirm('Are you sure you want to CONFIRM the order?')) {
      $('#CHECKBOX').click();
    }
  }
});

$("#checkbox").on('change', function() {
  this.checked = !this.checked ? !confirm('Do you really want to change this to NOT RECEIVED?') : true;
});

http://jsfiddle.net/5nd1wj54/1/

Answer 1

Use classes instead ids, so now redundant code needed. You can store informations in the data attribute of a HTML element. jsFiddle .

I've add a class to every checkbox what should be examined.

$(".checkIt").on('change', function() {
  if ($(this).is(':checked')) {
    confirm('Do you really want to change ' + $(this).data('id') + ' to NOT RECEIVED?');
  }
});

HTML

<input type="checkbox" data-id="checkbox1" class="checkIt" />
<input type="checkbox" data-id="checkbox2" class="checkIt" />
<input type="checkbox" data-id="checkbox3" class="checkIt" />

Answer 2

Here is what I think you mean

$('#checkbox').click(function() {
  if (this.checked) {
    this.checked = confirm('Are you sure you want to CONFIRM the order?')); 
  }    
  else {
    this.checked = !confirm('Do you really want to change this to NOT RECEIVED?'); 
  }  
});