jQuery simple each/while loop in coffeescript does not work as expected

Question

I've been trying couple of approaches on this problem. Basically I have to modify the content of several dropdowns on the exact same way. So I though I could use an each or a while to be more DRY but I dont have that much experience in coffeeScript.

Here is 'working' undry code to refactor:

$('#someId').val('someVal')
$('#someOtherId').val('someVal')

Here is my first attempt:

arr = ['someId', 'someOtherId']
$.each arr, (i, el) =>
  $('#'+el).val('someVal')

Here is my second attempt:

arr = ['someId', 'someOtherId']
length = arr.length
counter = 0
while counter < length
 $('#' + arr[counter]).val('someVal')
 counter ++

For some reasons, I have been trying multiple variations on each approch, I can assert that I can console.log the selector and it returns the correct value, but when it comes to change the value, the code just seems not to work as expected.

Any help is appreciated.

If you have access to the HTML you could add a class to each element and use a single call: $('.foo').val('someval'); — Rory McCrossan
– Rory McCrossan, Commented Sep 26, 2016 at 10:43
Definitly smart. I will go with this solution. Thanks a lot Rory — gastngouron
– gastngouron, Commented Sep 26, 2016 at 12:02

sbulat · Accepted Answer · 2016-09-26 12:08:29Z

1

Try:

arr = ['someId', 'someOtherId']
for value, index in arr
    $("##{value}").val('someVal')

Or if you don't care about index just:

arr = ['someId', 'someOtherId']
for value in arr
    $("##{value}").val('someVal')

edited Sep 26, 2016 at 12:08

answered Sep 26, 2016 at 10:44

sbulat

1118 bronze badges

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

3 Comments

gastngouron Over a year ago

Thanks for your advice, but I'm getting a Uncaught Error: Syntax error, unrecognized expression: ##{value} here.

sbulat Over a year ago

My fault, should be "##{value}" instead of '##{value}'

gastngouron Over a year ago

Worked! Thanks a lot for this interpolation I didn't know it was possible that way.

Keith · Accepted Answer · 2016-09-26 10:41:12Z

0

I'm not a coffee script user,. But looking at your code it looks like your using jquery to iterate an array. I assume you mean to use Array.forEach instead.. eg.

arr = ['someId', 'someOtherId']
arr.forEach (el) =>
    $('#'+el).val('someVal')

answered Sep 26, 2016 at 10:41

Keith

24.4k2 gold badges32 silver badges48 bronze badges

3 Comments

gastngouron Over a year ago

Thanks for your help, I just gave it a try and it doesn't work aswell. I'll keep you updated as soon as I find a solution. So far I'm going with Rory's solution.

Keith Over a year ago

I assume you have 2 html elements of type INPUT? + are you also running this code on DomReady?..

gastngouron Over a year ago

I do have two HTML elements of type <select>. This code is wrapped into a function called by another function on backbone + Marionette well known pattern 'onShow'.

Collectives™ on Stack Overflow

jQuery simple each/while loop in coffeescript does not work as expected

2 Answers 2

3 Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related