undefined local variable or method エラーを解消したいです

Question

前提・実現したいこと

ruby on railsのアプリにおいて、Google Maps APIを使用し、マップと、マップ上に複数のマーカー表示をしようとしています。

発生している問題・エラーメッセージ

複数のマーカーを表示できるよう、javascript内にeach文を配置したところ、以下のエラーが発生しました。

エラーメッセージ

undefined local variable or method `damage' for #<#<Class:0x00007ff93a5b1b30>:0x00007ff93b5ccc68>
Did you mean?  @damages

エラー箇所

position: {lat: #{damage.latitude} , lng: #{damage.longitude} },

該当のソースコード

index.html.slim

.page-title 被害一覧

= link_to '被害の新規登録', new_damage_path, class: 'btn btn-primary'

table.table.table-hover.index-table
  thead.thead-light
    tr
      th = Damage.human_attribute_name(:title)
      th = Damage.human_attribute_name(:place)
      th = Damage.human_attribute_name(:amount)
      th = User.human_attribute_name(:department)
      th = User.human_attribute_name(:name)
      th
    tbody
      - @damages.each do |damage|
        tr
          td = link_to damage.title, damage_path(damage), class: 'damage-title'
          td = damage.place
          td = damage.amount&.to_s(:delimited)
          td = damage.user.department
          td = damage.user.name
          - if damage.user_id == current_user.id
            td = link_to '編集', edit_damage_path(damage), class: 'btn btn-primary'
          - else
            td =
      tr
        td = "合計"
        td = "#{@total_place}箇所"
        td = @total_amount.to_s(:delimited)
        td =
        td =
        td =




#map
javascript:
  var map;
  function initMap() {
    var target = document.getElementById('map');
    // 中央に表示する座標
    var center = {lat: 38.698953,lng: 139.925736}

    map = new google.maps.Map(target, {
      center: center,
      zoom: 10
    });

    //複数のマーカーを表示するための記載
    - @damages.each do |damage|
      //マーカー
      var marker = new google.maps.Marker({
        position: {lat: #{damage.latitude} , lng: #{damage.longitude} },
        map: map,
        title: "#{damage.title}",
        icon: ( "#{damage.description}".indexOf('現地確認未了') == -1 ) ? "http://maps.google.com/mapfiles/ms/micons/red-dot.png" : "http://maps.google.com/mapfiles/ms/micons/blue-dot.png"
      });

  }

script async="" defer="defer" src="https://maps.googleapis.com/maps/api/js?key=AIzaSyDD8_DF2AUAtXfZVcQYFUi3T3AslQKAXaw&callback=initMap"

試したこと

インデントの深さを確認しましたが、問題はないと認識しています。

補足情報（FW/ツールのバージョンなど）

javascript文の外側であるtbodyで- @damages.each do |damage| と同じ記載方法をしているのですが、tbodyに置いてはエラーは発生していません。

Google Maps APIの使用に再しては、以下のサイトの情報を参考にさせて頂きました。
https://qiita.com/yamaotoko4177/items/30b72766d013904452fa
http://www.webdesignleaves.com/pr/plugins/googlemap_01.html

Answer 1

エラーメッセージの表現が JS のものに見えません。
メッセージがどのfileのどの行かというのが一緒にあると思います。そこも見せて下さい。
ああ、
map 以下のcodeはその上のindexと思われる slimの中に書かれているのでしょうか？

だとすると
position: {lat: "#{damage.latitude}" , lng: "#{damage.longitude}" },
かもしれない

Comments

[t.togashi]

こんばんは。

>map 以下のcodeはその上のindexと思われる slimの中に書かれているのでしょうか？

仰るとおりです。

[winterboum]

position: {lat: "#{damage.latitude}" , lng: "#{damage.longitude}" },
を試してみて下さい

[t.togashi]

おはようございます。
ご連絡くださりありがとうございました。
ご教示いただいたコードを試してみたのですが、エラーは解消されませんでした。
エラーメッセージ
```
undefined local variable or method `damage' for #<#<Class:0x00007faa74506740>:0x00007faa7444e050>
Did you mean? @damages
```

[winterboum]

ああ、、
position: {lat: "#{damage.latitude}" , lng: "#{damage.longitude}" },
の部分が rubyの式であることが宣言されてないですね。
さて、、、
slim　の中の javascript の中でどう宣言するのだろう。erbでは経験ありますがslimでは未経験です。
その辺りをkeyに検索してみて下さい。

[rhiroe]

普段Rails使っててHTML内に直接JavaScript書かないのでよくわかってないですけど
問題はjavascriptを書くべきところにRubyのコードが書かれていることじゃないですかね？
具体的には以下の箇所なんですけど
- @damages.each do |damage|
これって勝手にjsのコードに変換されるんですか？多分されないはず。

[winterboum]

erbで<%= %><% %>で囲めばやってくれるのは経験済ですが、slimではやったことがないのでどうなるものやら、、、

[rhiroe]

やるなら@damagesをどっかのタグにデータとして渡しておいて、それに対してjsでループ回すとかじゃないとうまくいかないかなと思います。

[rhiroe]

> <%= %><% %>で囲めばやってくれる
なるほど、サンプルコードって書けますか？

[rhiroe]

ちょっと調べてきました。
slimだと#{raw @damages.to_json}に対して順当にjsでfor文を書く他ないみたいですね。

var damages = #{raw @damages.to_json};
for(var i = 0; i < damages.length; ++i) {
... ;
}

こんな感じかな。

Answer 2

coffeeの方が見易いかなと思ってcoffeeで書いてみました。
動作確認はしてないので参考程度にしてください。

javascript:
    damages = #{raw @damages.to_json};
coffee:
  for key, damage of damages
    marker = new (google.maps.Marker)(
               position:
                 lat: damage.latitude
                 lng: damage.longitude
               map: map
               ...
             )