Object Orientated Javascript / Variable declarations / Performance

Question

So I have a rather large object orientated javascript class, with about 120 functions (a lot of getters and setters). Some of these functions have variables that are basically constants.

What I'm wondering, is should I declare these variables in a global scope of the object, so every time the function is run it doesn't have to re-declare the variable?

An example function is below. this.displayContacts is run several times (and will always run within the object), so in this case, there's no point declaring the 'codes' object inside the function?

function orderObject() {

  this.displayContacts = function() {
    var codes  = {'02':'02','03':'03','07':'07','08':'08'};
       // do something with codes
  };

}

So, would this be better, performance wise?

function orderObject() {
var codes  = {'02':'02','03':'03','07':'07','08':'08'};
  this.displayContacts = function() {
    // do something with codes.
  };

}

My other concern is that if I end up with a lot of global variables/objects inside the main orderObject, will that be MORE of a performance hit than simply re-declaring the variables each time?

I wanted advice on garbage collection and any global scope issues before I began changing it over. Theres ~3400 lines of code all up in here so, so it'd take a while lol. — Benno
– Benno, Commented Aug 19, 2011 at 2:48

hvgotcodes · Accepted Answer · 2011-08-19 02:53:15Z

1

absolutely.

function MyClass() {
   this.somevar = ''; // instance scoped variable
};

MyClass.CODES = {'02':'02'...}; // 'Class' scoped variable; one instance for all objects

MyClass.prototype.instanceMethod = function(){
  // 'this' refers to the object *instance*
  // it can still use MyClass.CODES, but can also account for variables local to the class
}

CONSTANT is 'static' so to speak, in java-talk. If your codes are global to the class (and the rest of your application), you will save a lot of overhead this way -- only define the object once. Note that you can have 'static' class-level methods as well, for those cases where the function doesn't need to operate on variables specific to an instance of the class.

Unless your app is really beefy, performance optimization probably wont make it noticeably faster. But that doesn't mean that OO design is not worth-while -- if you are going to use javascript in an object oriented way, its not too hard and never a bad idea to use good OO principals.

edited Aug 19, 2011 at 2:53

answered Aug 19, 2011 at 2:34

hvgotcodes

121k33 gold badges209 silver badges237 bronze badges

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

4 Comments

Benno Over a year ago

Thanks for the insight. I notice you used prototype.instanceMethod, so would that mean MyClass.prototype.displayContacts? or inside the function, this.prototype.displayContacts? What are the benefits of using .prototype? I've been a bit confused on that as well :p

hvgotcodes Over a year ago

@benno -- look at this question: stackoverflow.com/questions/4650513/…

Benno Over a year ago

thanks for that. At the moment I'm not sure if prototyping the methods will make much difference as there's only one call to the object, but, I'm intending on splitting it up into more manageable chunks (i.e. order object and products' object), so prototype would be much more beneficial there I'm guessing when I have multiple product objects. Cheers for that! Will save a lot of performance down the track!

RobG Over a year ago

A better approach would be to make CODES a property of the prototype, so instances can access it as this.CODES.

Schleis · Accepted Answer · 2011-08-19 02:36:56Z

1

I would say that if you have something that you are using in multiple places that it should become a property of your object so that it doesn't have to be redeclared each time. It would also help make the maintenance of the object easier if that constant has to change. Then you are changing it only in one place and not having to hunt down all the locations where you used it.

Don't repeat yourself.

answered Aug 19, 2011 at 2:36

Schleis

44.1k8 gold badges73 silver badges90 bronze badges

1 Comment

Benno Over a year ago

Thanks :) I think I'll go down that track where possible with this

rugg · Accepted Answer · 2011-08-19 02:37:54Z

1

Garbage collection in JavaScript depends on the browser, and most modern browsers handle it pretty well. If you go ahead and make these global, you might see a slight performance increase simply because it's not executing that line of code every time. However, I can't imagine any significant increase in performance by making these static properties on the class, but if they don't change, then it would make more sense.

answered Aug 19, 2011 at 2:37

rugg

5514 silver badges9 bronze badges

1 Comment

Benno Over a year ago

Thanks for that. Yeah I wasn't too sure how garbage collection would handle having many global scope variables as opposed to non-globals. I'm just trying to optimise as much as I can because it gets a tad laggy after a while.

Collectives™ on Stack Overflow

Object Orientated Javascript / Variable declarations / Performance

3 Answers 3

4 Comments

1 Comment

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

4 Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related