jQuery using variable with string inside a selector

Question

This code prints about 15 rows of data with this select box near each row. I need to capture user's selected value for each row to send it of to the next function. I'm receiving this error: Uncaught TypeError: Object # has no method 'val'. Is there a better way to achieve this?

count = 1;
$.each(data, function(index, element) {
    $('<div class="hd_field">Payment Format: <select name="Payment_Format" id="Payment' + count + '" class="Payment_Format"><option value="CTX" selected="selected">Company to Company</option><option value="PPD">Company to Person</option></select><div id ="Selected_Format' + count + '"></div></div>').appendTo(newDiv3);
    $('select#Payment' + count).change(function() {
        alert(this.val());
        PaymentFormat = $(this).val();
        element.PmtRec.PmtFormat = PaymentFormat;
        $('#Selected_Format' + count).text(PaymentFormat);
    });
    console.log(data);
    count = count + 1;
});

Is the data variable an array?

kevin628
– kevin628

2012-06-06 15:48:54 +00:00
Commented Jun 6, 2012 at 15:48 — kevin628
– kevin628, Commented Jun 6, 2012 at 15:48
Yes, its a multidimensional array.

user1413248
– user1413248

2012-06-06 16:19:15 +00:00
Commented Jun 6, 2012 at 16:19 — user1413248
– user1413248, Commented Jun 6, 2012 at 16:19

James Allardice · Accepted Answer · 2012-06-06 15:49:55Z

1

Your error is thrown on this line:

$('select#Payment'+count).change(function() { 
    alert(this.val()); //Here
    //...
});

The problem is that in the change event handler, this refers to the actual DOM node, not a jQuery object, and the DOM node will not have a val method. You need to pass it to jQuery if you want to use val:

$('select#Payment'+count).change(function() {
    alert($(this).val()); //Pass `this` to jQuery
    //...
});

answered Jun 6, 2012 at 15:49

James Allardice

166k22 gold badges335 silver badges315 bronze badges

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

3 Comments

iddo Over a year ago

or change to alert(this.value)

user1413248 Over a year ago

Thank you that worked! But, the alert was just a test to see if the .change is working, the .text is still not doing anything.

user1413248 Over a year ago

Also, the object assignment isn't changing every time I select a different option.

Collectives™ on Stack Overflow

jQuery using variable with string inside a selector

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related