Question

66

Don't make functions within a loop

rated 0 times [ 71] [ 5] / answers: 1 / hits: 76069 / 12 Years ago, wed, april 25, 2012, 12:00:00

What would be the correct way to solve the jslint error in this case? I'm adding a getter function to an object which uses this. I don't know how to do this without creating the function inside the loop.

for (var i = 0; i<processorList.length; ++i) {

   result[i] = {

       processor_: timestampsToDateTime(processorList[i]),

       name_: processorList[i].processorName,

       getLabel: function() { // TODO solve function in loop.

            return this.name_;

       }

   };

}

Answers

Only authorized users can answer the question. Please sign in first, or register a free account.

andreguym

Add To Favorites

Follow

Total Points: 125

Total Questions: 112

Total Answers: 103

Location: Wallis and Futuna

Member since Tue, Mar 30, 2021

3 Years ago

andreguym questions

1 Cant run cypress tests in Chrome & Edge browsers(being manageed by organization)

Sun, Dec 5, 21, 00:00, 3 Years ago

1 Bootstrap5 modal not showing

Fri, May 14, 21, 00:00, 3 Years ago

1 Kick/disconnect someone from voice chat in discord

Sun, Apr 4, 21, 00:00, 3 Years ago

1 TypeError(Super expression must either be null or a function) - when App is a class instead of a function

Sat, Apr 3, 21, 00:00, 3 Years ago

1 tooltip to work for mouse hover and click both - ngbootstrap

Wed, Mar 17, 21, 00:00, 3 Years ago

View All

answered 12 Years ago ignacio · Accepted Answer

Move the function outside the loop:

function dummy() {

    return this.name_;

}

// Or: var dummy = function() {return this.name;};

for (var i = 0; i<processorList.length; ++i) {

   result[i] = {

       processor_: timestampsToDateTime(processorList[i]),

       name_: processorList[i].processorName,

       getLabel: dummy

   };

}

... Or just ignore the message by using the loopfunc option at the top of the file:

/*jshint loopfunc:true */