forEach文で発生するエラーが解決できない

Question

#状況
forEach文を使用する処理を書いているが、Uncaught ReferenceError: element is not definedというエラーが出てしまう。

#ソース

    $(document).ready ->
    # 画面遷移時の入力情報の配列を生成
      @oldMain = Array.from($('.input-main'))
      @oldMaintenance = Array.from($('.input-maintenance'))

      # valueを生成した配列に詰める
      @oldMain.forEach (element) ->
        @oldMainAmounts = new Array
        @oldMainAmounts.push element.value

      @oldMaintenance.forEach (element) ->
        @oldMaintenanceAmounts = new Array
        @oldMaintenanceAmounts.push element.value

    $('.input-main').keyup ->
      # 現状の入力情報の配列を生成
      main = Array.from($('.input-main'))
      # valueを生成した配列に詰める
      main.forEach(element) ->
        mainAmounts = new Array
        mainAmounts.push(element.value)

        # 遷移時と現状で比較し、処理を行う
　　　　# input-mainクラスに何か入力した時に、input-maintenanceクラスのフォームを非活性にする
　　　　# 非活性後、遷移時と同じ状態にすれば再び活性となる
      if @oldMainAmounts != mainAmounts
        $('.input-maintenance').addClass('disabled')
      else if @oldMainAmounts == mainAmounts
        $('.input-maintenance').removeClass('disabled')

入力フォームに文字を入れると、keyupイベントは動くのですが上のエラーが発生します。
main = Array.from($('.input-main'))のあとにconsole.log mainを入れると、
keyup後、webコンソールにちゃんと表示されるのでここまでは問題ないかとおもうのですが・・・。

おそらくmain.forEach(element)のelementでエラーが起きているものと思われます。
このエラーがなぜ起きているかが分からなく、解決できません。
教えていただけると助かります。宜しくお願い致します。

Answer 1

      main.forEach(element) ->
        mainAmounts = new Array
        mainAmounts.push(element.value)

この部分だけを JavaScript にトランスパイルしてみてください。
原因は一目瞭然だと思います。

Comments

[tenten11055]

ご回答ありがとうございます。
試したところ、次のような結果になりました。

```
main.forEach(element)(function() {
var mainAmounts;
mainAmounts = new Array;
return mainAmounts.push(element.value);
});
```

まずeachの中でmainAmountsを毎回newしていたのでそれはまずいなと外に出し、結果的に次のようになりました

```
var main, mainAmounts;

main = Array.from($('.input-main'));

mainAmounts = new Array;

main.forEach(element)(function() {
mainAmounts.push(element.value);
});
```

またjqueryのeach文があったので、最終的に次のような形にしたところ、
エラーが出なくなりました。

```
$.each main, (index, element) ->
mainAmounts.push element.valu
```

each()で指定する変数が足りなかったのでしょうか。
ひとまず解決です。
ありがとうございました。

[2KOH]

原因も対処方法も違います。
「main.forEach と (element) -> の間にスペースを入れ忘れていることが原因であり、main.forEach (element) -> とすればいい」が答えです。

このコードの上では、
@oldMain.forEach (element) ->
のように forEach と (element) -> の間にきちんとスペースを書いていたため、すぐに気付けると思ってわざと遠回りな回答をしたのですが、混乱させてしまったようですみません。

[tenten11055]

そういうことでしたか！
気づけず悔しいです・・・。

いえいえ、ご丁寧にありがとうございました！