jQuery `.attr()` is not a function

Question

I am trying to get values of specific option tags using jQuery.

This is my HTML:

<select name="modele" class="wyszselect" onchange="przenies(this[this.selectedIndex].value);"> <option value="0">Choose model</option> <option value="242">242</option> <option value="243">243</option> <option value="244">244</option> <option value="246">246</option> <option value="320">320</option> <option value="324">324</option> <option value="328">328</option> <option value="33">33</option> </select>

This is my jQuery code:

$(document).ready(function () { var options = []; $("option").each(function (index, oneOption) { options.push(oneOption.attr("value")) }); console.log(options); });

I got this error:

Uncaught TypeError: oneOption.attr is not a function

oneOption is a dom object, .attr() is a jquery function. You could use oneOption.value — billyonecan
– billyonecan, Commented Jun 18, 2015 at 10:55

Oleksandr T. · Accepted Answer · 2015-06-18 10:54:21Z

Conver oneOption to jQuery Object, like so

 $("option").each(function (index, oneOption) { options.push($(oneOption).attr("value")) });

Example

Adil · Accepted Answer · 2015-06-18 10:59:37Z

The each function will pass the DOM object instead of jQuery object. Either convert to jQuery object or use the DOM object, like options.push(oneOption.value), this will be faster then converting the DOM object to jQuery object and then calling attr to get the value. You can also use this.value

$("option").each(function (index, oneOption) { //options.push($(oneOption).attr("value")); options.push(oneOption.value); //options.push(this.value); });

For getting comma separated value

$("option").map(function(){ return this.value; }).get().join(',');

@RazvanDumitru do you mean to do console.log(map(options)) ?
@Adil no i mean that the console should print something like this: optinos = ["value1", "value2", "value3", "value4", ....]

Balachandran · Accepted Answer · 2015-06-18 11:06:34Z

1

use map()in jquery() ,and get the option value using this object

options = $("option").map(function (index, oneOption) { return this.value; }).get().join(',');

DEMO

or in each()

options =[]; $("option").each(function (index, oneOption) { options.push(this.value); })

edited Jun 18, 2015 at 11:06

answered Jun 18, 2015 at 10:54

Balachandran

9,6271 gold badge19 silver badges26 bronze badges

3 Comments

Marco Dinatsoli Over a year ago

how can I get the result in a form separated in comma ?

Balachandran Over a year ago

join () developer.mozilla.org/en/docs/Web/JavaScript/Reference/…

Marco Dinatsoli Over a year ago

your update gives me this result 0,242,243,244,246,320,324,328,33, while i would like this result: ["value", "value", "value" ......]

Razvan Dumitru · Accepted Answer · 2020-09-08 10:23:32Z

Maybe you need to convert it to a jQuery Element:

$("option").each(function (index, oneOption) { options.push($(oneOption).attr("value")) });

Collectives™ on Stack Overflow

jQuery `.attr()` is not a function

4 Answers 4

Comments

2 Comments

3 Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

2 Comments

3 Comments

Comments

Related