JQuery code not working

Question

I am trying to find a cell in a table that has a class of 'empty', i am then using a bit of code to find the id(cell number) so i can the find out what cells are next too it.

just too see if it works, i am trying:

console.log($('.empty').attr('id'));

but Firebug just returns 'undefined'

each cell has the class of 'box' and only one has empty as well so 'box empty'.

Any asssitance would be much appreciated.

$('.empty') may return array of DOM elements. In that case, array itself does not have attribute — user482594
– user482594, Commented Feb 12, 2012 at 21:28
$(document).ready(function(){ $('.empty').each(function() { console.log( $(this).attr('id') ); }); }); jquery above, html for cell <td id="B" class="leftbox" name="empty" ondragover="allowDrop(event)" ondrop="drop(event,this.id)"> </td> — nmyster
– nmyster, Commented Feb 12, 2012 at 21:38
@nmyster You need attribute selector, not class. the name is empty, not the class! See my updated answer. — gdoron
– gdoron, Commented Feb 12, 2012 at 21:42

gdoron · Accepted Answer · 2012-02-12 21:41:42Z

2

Probably the DOM isn't ready. Put the code inside the on DOM ready event:

$(function(){console.log($('.empty').attr('id'));});
//or
$(document).ready(function(){console.log($('.empty').attr('id'));});

Update:

Based on the Markup you wrote, the selector you need is attribute selector not class:

<td id="B" class="leftbox" name="empty" ondragover="allowDrop(event)" 
    ondrop="drop(event,this.id)"> </td>

$(function(){console.log($('input[name="empty"]').attr('id'));});

edited Feb 12, 2012 at 21:41

answered Feb 12, 2012 at 21:28

gdoron

151k59 gold badges302 silver badges376 bronze badges

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

1 Comment

nmyster Over a year ago

$(document).ready(function(){ console.log($('.empty').attr('id')); } each cell has class of 'box' and empty cell has class of 'box empty' and each cell has an ID that is its number of cell.

Julien Lafont · Accepted Answer · 2012-02-12 21:31:09Z

1

$('.empty') return an array of dom element, so you must use a loop (for, each...) to analyse results

$('.empty').each(function() {
   console.log( $(this).attr('id') );
}

If you are sure there is only one element, you can use :

console.log( $('.empty:first').attr('id') );

answered Feb 12, 2012 at 21:31

Julien Lafont

7,8772 gold badges34 silver badges52 bronze badges

1 Comment

gdoron Over a year ago

He is sure there is only one. but the problem he mixed up class with name. see my answer and his comment on his question.

Collectives™ on Stack Overflow

JQuery code not working

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related