Javascript Error (IE): Object doesn't support this property or method

Question

2015 Note: Everyone should go look through their old questions and be encouraged by how far they've come. There are so many things wrong with this code – it's wonderful.

This code works PERFECTLY in Firefox, and in IE the line ending 'negative'); breaks. I have no idea how to fix it! Argghhh!

$(".topline strong").digits().each(function () {
    s = $(this),
    c = "change";
    if ($(s).hasClass(c) & $(s).text().replace(/%/gi, "") > 0) {
        $(s).addClass("positive");
    }
    if ($(s).hasClass(c) & $(s).text().trim().charAt(0) == "-") $(s).addClass("negative");
});

Can you separate the statements to see which one causes the error? — Pekka
– Pekka, Commented Aug 11, 2010 at 10:53

kennytm · Accepted Answer · 2010-08-11 11:01:28Z

2

Please use && instead of & (unless it's meant for minimizing)
String.trim isn't widely implemented yet. Use
```
$(s).hasClass(c) && /^\s*-/.test($(s).text())
```
instead.

I'd rather rewrite the code as:

$(".topline strong").digits().each(function() {
  var s = $(this);
  if (s.hasClass("change")) { // no need to $() an already $()-ed object.
    var value = parseFloat(s.text());
    if (value != 0)
      s.addClass(value > 0 ? "positive" : "negative"); 
  }
});

edited Aug 11, 2010 at 11:01

answered Aug 11, 2010 at 10:54

kennytm

526k110 gold badges1.1k silver badges1k bronze badges

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

1 Comment

Phil Ricketts Over a year ago

Ahh, interesting - though I did try &&. JSLint didn't say anything about this.

Nick Craver · Accepted Answer · 2010-08-11 11:01:51Z

Instead of .trim() you can use $.trim() like this:

$(".topline strong").digits().each(function () {
  var s = $(this), c = "change";
  if (s.hasClass(c) && s.text().replace(/%/gi, "") > 0) { s.addClass("positive"); }
  if (s.hasClass(c) && $.trim(s.text()).charAt(0) == "-") s.addClass("negative");
});

Also note the s changes, there's no need to clone it as another jQuery object each time, it already is one so use it :) As for the error: .trim() isn't in all browsers, this is why jQuery includes the $.trim() function (same reason it has $.inArray(), IE doesn't have .indexOf()).

Also when declaring variable use var, otherwise you're creating global variables.

As an aside for future readers of this, jQuery 1.4.3+ will use the native String.prototype.trim if it's available when calling $.trim().

Collectives™ on Stack Overflow

Javascript Error (IE): Object doesn't support this property or method

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related