jQuery Error - Generating undefined

Question

So I have a list of users registered on my site in 1 column, in the 2nd is their email address with a checkbox next to it. On this page a user can check the box (or multiples) and click a submit button. Once they do that it will generate a list of the emails semicolon separated.

My issue is after they hit submit the lists generates, but the first email address has "undefined" written right next to it.. so instead of saying "[email protected]; [email protected]" it reads "[email protected]; [email protected]".

Here is my jQuery:

jQuery(document).ready(function() {
jQuery('#memberSubmit').click(function() {
    var emailList;
    jQuery('.email-members input:checked').each(function() {
        var $this = jQuery(this);
        emailList += $this.next('a').html() + "; ";
    });
    jQuery('.email-message').hide();
    jQuery('.email-members').hide();
    jQuery('.email-checks').hide();
    jQuery('#memberSubmit').hide();
    jQuery('.email-results a').attr('href', "mailto: " + emailList).fadeIn(2000);
    jQuery('.email-results .email-list p').html(emailList).fadeIn(2000);
    jQuery('.email-results h2').fadeIn(2000);
    jQuery('.email-results p').fadeIn(2000);
    jQuery('.email-list h2').fadeIn(2000);
    //console.info('Emails: ' + emailList);
});
});

I think my error is on the line: emailList += $this.next('a').html() + "; "; But I am not sure... any ideas?

Thanks!

Can you supply a short HTML example of your setup?

Matt
– Matt

2013-01-24 21:46:51 +00:00
Commented Jan 24, 2013 at 21:46 — Matt
– Matt, Commented Jan 24, 2013 at 21:46

Mathew Thompson · Accepted Answer · 2013-01-24 21:47:57Z

3

Initialize the emailList the variable first, that means it doesn't start at undefined when you perform your first go around. Coincidently, when you're calling += for the first time, it's actually converting undefined to a string, thus meaning your string always starting with that.

var emailList = "";

answered Jan 24, 2013 at 21:47

Mathew Thompson

56.5k15 gold badges131 silver badges151 bronze badges

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

2 Comments

tckmn Over a year ago

Darn it you beat me by a couple of seconds :P

Mathew Thompson Over a year ago

@Doorknob Hehe I noticed, sorry dude!

tckmn · Accepted Answer · 2013-01-24 21:48:21Z

0

Try replacing emailList's declaration with this code:

var emailList = "";

That's because emailList starts out as undefined if you don't initialize it. Therefore undefined + "this is a test" would turn out as undefinedthis is a test.

answered Jan 24, 2013 at 21:48

tckmn

59.7k27 gold badges118 silver badges156 bronze badges

1 Comment

Mathew Thompson Over a year ago

Mention me in this answer tomorrow and I'll +1 you, I've hit the vote cap for today!

Collectives™ on Stack Overflow

jQuery Error - Generating undefined

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related