loop through html table and check if a checkbox is checked or not

Question

What I'm trying to do here is validating HTML table. I need at least one checkbox to be checked before proceeding. It's doing what it's need to do, except when I don't check a checkbox.

It loops the alert "nothing found" by the count of data on the HTML table. How can I make it better? I need to loop inside the table to get the data on the same row with the checkbox checked.

JS

$('#dataTable').find('tr').each(function () {
    var row = $(this);
    if (row.find('input[type="checkbox"]').is(':checked') ) {
        alert "found"
        //im getting data here.
        });
    }else{
        alert "NOthing found"
    };
});

Steven Moseley · Accepted Answer · 2015-11-14 16:10:25Z

10

No need to loop through anything. Just look for a checked checkbox inside dataTable with your selector. If it exists, then your "true" condition is met.

if ($('#dataTable input[type="checkbox"]:checked').size()) {
    // Proceed
}

Now, if you want to do something with the data inside the "checked" rows, you could do something like this:

var checkedItems = $('#dataTable input[type="checkbox"]:checked').each(function() {
    // Do something with the row
    console.log($(this).parent('tr').find('.data-selector').val());
});
if (!checkedItems.size()) {
    // Nothing was checked
}

edited Nov 14, 2015 at 16:10

answered Nov 14, 2015 at 14:39

Steven Moseley

16.4k4 gold badges43 silver badges50 bronze badges

Sign up to request clarification or add additional context in comments.

8 Comments

Andreas Over a year ago

This will always be true

Andreas Over a year ago

$(...) returns an object which never will evaluate to false. You have to check the .length property

Elentriel Over a year ago

This hardly answers the question as he clearly states he needs to loop through the acceptable tr's to do something and not just check if something is checked. Still, not a bad piece of code~

Elentriel Over a year ago

@Steven Moseley 'I need to loop inside the table to get the data on the same row with the checkbox checked.' as quoted above. Quite clearly too

Elentriel Over a year ago

@Steven Moseley yup, this is the stuff. removed my downvote and upvoted~

|

Sasanka Sekhar · Accepted Answer · 2015-11-14 15:51:22Z

3

In stead of alerting throughout the loop, why not use a flag and update it when something is found. Based on the flag, (at the end - after the loop) just display the status!. I hope the following will modification will help you:

var found = false;
$('#dataTable').find('tr').each(function () {   
var row = $(this);
if (row.find('input[type="checkbox"]').is(':checked') ) {
    found=true;
    //alert ("found"); 
    // Do your job here
    }
else{
     //alert "NOthing found"
     //found=false;
    };
});
if (!found){
    alert("Nothing Found");
}
else{
     alert("Found");// Or you can omit the else part
}

answered Nov 14, 2015 at 15:51

Sasanka Sekhar

377 bronze badges

Comments

Elentriel · Accepted Answer · 2015-11-14 18:20:25Z

2

Likeso you will execute your code for every checked row, and if no row is checked, you will get to execute what you decide is apropriate, without much resources going to waste

  var is_table_empty = true;
 jQuery('#dataTable').find('input[type="checkbox"]:checked').each(function(){
    var row = jQuery(this).closest("tr");
    //do what needs to be done to an active row
    is_table_empty= false;
  });
  if(is_table_empty)
  {
     //do what you need to do if nothing is clicked
  }

edited Nov 14, 2015 at 18:20

answered Nov 14, 2015 at 14:46

Elentriel

1,2378 silver badges21 bronze badges

Comments

Azad · Accepted Answer · 2018-10-01 10:06:34Z

2

try this same as your code.

var checkFound = false;

$('#dataTable').find('tr').each(function() {
  var row = ;
  if (!checkFound && $(this).find('input[type="checkbox"]').is(':checked')) {
    checkFound = true;
    return false;
  }
});

if (checkFound)
  alert('checked Found');
else
  alert('nothig checked');

edited Oct 1, 2018 at 10:06

answered Nov 14, 2015 at 14:47

Azad

5,2725 gold badges32 silver badges59 bronze badges

1 Comment

Elentriel Over a year ago

if cost. Selectors cost. Still, as far as correct answers go, this is one

Collectives™ on Stack Overflow

loop through html table and check if a checkbox is checked or not

4 Answers 4

8 Comments

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

8 Comments

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related