Last element in .each() set

Question

I have an issue, where by I am doing a simple form validation, that needs some custom stuff that the validation plugin could not do for me. Basically, I have a name, email and message field, all are required, but only email is really validated, the others just need to check if theyre not empty. Here is my current code:

$("#contactMe form").submit(function() {
    var email = $('.requiredEmail').val();
    if(email != 0)  {
        if(isValidEmailAddress(email))  {
            $('.requiredText').each(function() {
                thisVal = $(this).val();
                var $this = $(this);
                if(thisVal != 0) {
                    console.log('Valid Field: '+thisVal);
                    if ($(this) == $(this).parent(':last')) {
                        console.log('Last field, submit form here');
                    }
                }
            });
        } else {
            console.log('Email Not Valid');
        }
    } 
    return false;
});

Just to explain, I am first checking the email address is valid via the isValidEmailAddress function, which is working. Then I am going through using each(), all the requiredText fields and checking if theyre not empty. When I get to the last requiredText field, I want to submit the form using post or whatever.

if ($(this) == $(this).parent(':last')) { What I have there is obviously incorrect but I am not sure what can be used to check if it is the last in the each result set, and perform an action if true.

Can anybody help me out here?

is requiredText is the class name for all the input text boxes? if so you might be having id also right, why don't you simply verify the last element with id and then submit the form. — kobe, Oct 24 '10 at 01:30

score 161 · Accepted Answer · edited Jun 27 '17 at 09:21

161

each passes into your function index and element. Check index against the length of the set and you're good to go:

var set = $('.requiredText');
var length = set.length;
set.each(function(index, element) {
      thisVal = $(this).val();
      if(parseInt(thisVal) !== 0) {
          console.log('Valid Field: ' + thisVal);
          if (index === (length - 1)) {
              console.log('Last field, submit form here');
          }
      }
});

edited Jun 27 '17 at 09:21

fabpico

2,628
4
26
43

answered Oct 24 '10 at 01:29

Jacob Relkin

161,348
33
346
320

1

Doesn't look nice to write the selector twice but seems to work here. – netAction Nov 13 '14 at 12:02
Why the need to check last element? Just put the code after the loop. – Marco Nov 06 '17 at 11:08
@Marco sometimes we may want to do something on every iteration except the last. – Andrew Kozak Jan 05 '18 at 16:37
But it's not what's the OP's questioned for. – Marco Jan 05 '18 at 16:51

score 14 · Answer 2 · answered Oct 06 '16 at 07:21

For future Googlers i've a different approach to check if it's last element. It's similar to last lines in OP question.

This directly compares elements rather than just checking index numbers.

$yourset.each(function() {
    var $this = $(this);
    if($this[0] === $yourset.last()[0]) {
        //$this is the last one
    }
});

score 3 · Answer 3 · answered Nov 06 '17 at 10:48

3

A shorter answer from here, adapted to this question:

var arr = $('.requiredText');
arr.each(function(index, item) {
   var is_last_item = (index == (arr.length - 1));
});

Just for completeness.

answered Nov 06 '17 at 10:48

Avatar

14,622
9
119
198

Last element in .each() set

3 Answers3

Linked