Unexpected token error in for loop, why?

Question

var ccode = ['EUR', 'BRL', 'RUB', 'KRW', 'RON', 'CHF'];
  var i;
  for (var i = 0; i < ccode.length; i++) {
     var ccode[i] = fx.convert(amount, {to: 'ccode[i]'});
  }

The bit above var ccode[i] is causing an error

SyntaxError: Unexpected token '['. Expected ';' after variable declaration.

I am still new to JS so please bear with me.

I am editing my question here since people asked why I was re-declaring var ccode[i] and its because I need to output this:

var EUR = fx.convert(amount, {to: "EUR"});

`for (var i = 0;` re declares `i` again. Remove the `var`, yes and same for `ccode` — Liam, Nov 24 '17 at 10:24
[what does var do in Javascript](https://stackoverflow.com/questions/8059526/what-does-var-do-in-javascript-why-is-it-sometimes-part-of-an-assignment) — Liam, Nov 24 '17 at 10:26
I had var ccode[i] because I need to output this "var EUR = fx.convert(amount, {to: "EUR"});" — user3501825, Nov 24 '17 at 10:33

Machigatta · Answer 1 · 2017-11-24T10:25:08.023

1

var ccode = ['EUR', 'BRL', 'RUB', 'KRW', 'RON', 'CHF'];
var i;
  for (i = 0; i < ccode.length; i++) {
     ccode[i] = fx.convert(amount, {to: 'ccode[i]'});
  }

ccode is already declared.. no need to use var

edited Nov 24 '17 at 10:25

answered Nov 24 '17 at 10:23

Machigatta

118
10

3

same for goes for i – Jamie Eltringham Nov 24 '17 at 10:24

score 0 · Answer 2 · answered Nov 24 '17 at 10:28

Its not about declaring ccode twice, as variable declaration syntax is wrong

var ccode[i]; should not be array, see for reference

You should remove var.

var ccode = ['EUR', 'BRL', 'RUB', 'KRW', 'RON', 'CHF'];
  var i;
  for (var i = 0; i < ccode.length; i++) {
     ccode[i] = fx.convert(amount, {to: 'ccode[i]'});
  }

JackDev · Answer 3 · 2017-11-24T16:19:43.917

0

I would suggest using an array map:

var ccode = ['EUR', 'BRL', 'RUB', 'KRW', 'RON', 'CHF'];
ccode = ccode.map(function(code) {
  return fx.convert(amount, {to: code})
})

edited Nov 24 '17 at 16:19

answered Nov 24 '17 at 10:29

JackDev

4,891
1
39
48

1

map returns a new array, but you do not assign this result. – Nina Scholz Nov 24 '17 at 11:15

Unexpected token error in for loop, why?

3 Answers3