Only the last value of For-of-Loop being passed to JQuery click function

Question

I have the following For Of loop in CoffeeScript that loops through the properties of an object:

for buildingFrame of buildingNames
  $("#bt-#{buildingFrame}").click () => @displayProperties(buildingFrame)

It appears that only the last value of buildingFrame is passed to every call to @displayProperties. Searching the site I found what I think is the reason here: Possible Answer

The reason why only the last value in the loop is used is because JavaScript is a late-binding language and loops do not introduce a new scope. The solution to fix this is given in that answer in JavaScript like so:

for(var i=0; i<barValues.length; i++) function(i){
  ...
}(i);

I have tried using this solution to my coffeScript above to try and solve the problem like so:

for buildingFrame of buildingNames => (buildingFrame)
  $("#bt-#{buildingFrame}").click () => @displayProperties(buildingFrame)
(buildingFrame)

But this just gives my complier errors. Coud someone please advise me how I can tackle this problem in CS. Thanks everyone!

Niko · Accepted Answer · 2013-02-11 22:23:20Z

2

How about something like this?

for buildingFrame of buildingNames
  do (buildingFrame) =>
    $("#bt-#{buildingFrame}").click => @displayProperties(buildingFrame)

This compiles into the following JavaScript:

_fn = function(buildingFrame) {
  return $("#bt-" + buildingFrame).click(function() {
    return _this.displayProperties(buildingFrame);
  });
};

for (buildingFrame in buildingNames) {
  _fn(buildingFrame);
}

answered Feb 11, 2013 at 22:23

Niko

26.8k9 gold badges97 silver badges112 bronze badges

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

1 Comment

Jamie Fearon Over a year ago

Cool, I never new about Do - seams like it was written just for the very problem I was experiencing!

Blender · Accepted Answer · 2013-02-11 22:23:11Z

1

You can use the do keyword:

for buildingFrame in buildingNames
    do (buildingFrame) ->
        $("#bt-#{buildingFrame}").click () => @displayProperties(buildingFrame)

answered Feb 11, 2013 at 22:23

Blender

300k55 gold badges462 silver badges512 bronze badges

Collectives™ on Stack Overflow

Only the last value of For-of-Loop being passed to JQuery click function

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related