Question

I created a javascript “class” as follows:

function MyClass(member1, member2) {
   this.Member1 = member1;
   this.Member2 = member2;
}

All members are Strings.

I want to take an instance of MyClass and “clean” the members by calling

function NoneBecomesNull(item) {
    if (item === "[None]")
        item = "";
    return item;
}

for-each member of the class. Is there an effecient way to accomplish this task? (In the case where MyClass has 30 members).

I would like to avoid doing…

myClassInstance.Member1 = NoneBecomesNull(myClassInstance.Member1);
myClassInstance.Member2 = NoneBecomesNull(myClassInstance.Member2);
//...30+ times

Answer 1

Try the following

for (var name in theObject) {
  if (theObject.hasOwnProperty(name) && theObject[name] === "[None]") {
    theObject[name] = "";
  }
}

I used hasOwnProperty to prevent the reseting of properties higher up in the prototype chain. Your example didn’t show the use of a prototype chain here and hence it’s likely not necessary for this example. But it’s good practice.