質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.37%

Rails部分テンプレートcollectionで変数名_counterが反応してるのにjsエラーに

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 2
  • VIEW 1,650

score 32

前提・実現したいこと

Railsアプリ作成中です。
edit.html.erbでユーザー入力機能をAjaxで実装しています。
ユーザーが文を入力するとそのページで遷移せずに部分テンプレートが一つ追加される形です。

edit.html.erbで入力

item_controllerのcreate作動

create.js.erbを返す

create.js.erbでAjax処理

部分テンプレートを1つ追加しedit.html.erbに帰る

おそらく、部分テンプレートを1つ追加しedit.html.erbを表示する最後の動作で、変数名_counterが定義されていないと出ているのだと思います。

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

chromeの開発者ツールnetworkのPreviewに出てきたエラーです。
item_counterが違うとのこと。
しかし、edit.html.erbでは、ちゃんと連番でitem_counterが機能しています。

NameError in Items#create

Showing /Users/onoryouma/projects/bucket_list/app/views/users/_item.html.erb where line #3 raised:
undefined local variable or method `item_counter' for #<#<Class:0x007fe96bf14ee0>:0x007fe96f8e7af0>
Did you mean?  item_url
Trace of template inclusion: app/views/items/create.js.erb

該当のソースコード

edit.html.erbの入力フォームと部分テンプレート呼び出し元です。

<!-- 入力フォーム-->
<div class="content content-item">
   <ol>
   <%= form_tag('/items', method: :post, remote: true) do %>
   <input type="text" name="text" pattern=".*\S+.*" required placeholder="23文字以内で" size="23" maxlength="23" autocomplete="off" class="header-btn">
    <%= submit_tag "追加" ,:class => "header-btn margin-top10px" %>
    <% end %>
    </ol>
</div>
<!-- テンプレート呼び出し-->
<div class="content content-item">
   <ol>
    <% if @items.present? %>
   <div id="item-content">
    <%= render partial: "item", collection: @items %>
   </div>
 <% end %>
  </ol>
</div>

item_controller該当部

def create
  @item = Item.new(text: item_params[:text], user_id: current_user.id)
  #Ajax仕様
  respond_to do |format|
  if @item.save
  format.html { redirect_to @item, notice: '作成完了'}
  format.json { render :show, status: :created, location: @item }

    format.js
  else
    format.html { render :new }
    format.json { render json: @item.errors, status: :unprocessable_entity }
     end
   end
 end

create.js.erbの全部

$("<%= escape_javascript(render partial: '/users/item', locals: { item: @item }) %>").appendTo("#item-content");

//二重送信防止のためのdisabledを取り除く
$('form').find(':submit').removeAttr("disabled");

//submitの後に、フォームの中身を空にする
$('form').find("textarea, :text, select").val("").end().find(":checked").prop("checked", false);

以下部分テンプレートです。
_item.html.erb

<li class="idea">
  <div class="idea-text">
  <div  class="checked-item_<%= "#{item_counter}"%>" >
    <% if item.check == true %>
      <div class="delete-text">
         <%= item.text  %>
      </div>
    <% else %>
      <%= item.text  %>
    <% end %>
  </div>
  </div>
  <div class="item-menu-right">
    <%= link_to item_path(item), method: :delete , data:{ confirm: '本当によろしいですか?'}, remote: true do %>
       <i class="fa fa-trash fa-fw" aria-hidden="true"></i>
    <% end %>
    <!-- ここをモーダルにする -->
    <%= link_to edit_item_path(item), method: :get ,class: "fa-edit-margin", item_id: item.id do %>
      <i class="fa fa-edit fa-fw" aria-hidden="true" ></i>
    <% end %>

    <%= check_box_tag '', '', item.check, {'data-id' => item.id, 'data-user-id' => item.user_id ,class: "check_#{item_counter} option-input"} %>

  </div>
</li>

<!-- idからclass指定に変えた -->

<!-- チェックつけたらdelete-textつける -->
<!-- item_counterを使い、各テンプレートに連番を振っている -->
<script>

$(function() {
  $('.check_<%= "#{item_counter}"%>').on('click', function() {
    if ( $(this).prop('checked') == false ) {
      $('.checked-item_<%= "#{item_counter}"%>').removeClass("delete-text");
    } else {
      $('.checked-item_<%= "#{item_counter}"%>').addClass("delete-text");
    }
  });
});


</script>

試したこと

item_counterに関するviewを消したところ動きましたが、機能として必要なのでどうにかしたいです。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • Romay

    2019/02/09 16:36

    @perpouhさん
    コメントありがとうございます。引数の型が違うとはどういうことでしょうか。item で渡しているのですが、これはどこかで型定義されているということでしょうか。恥ずかしながら自分で調べてみても検討がつかずにいます。

    キャンセル

  • Romay

    2019/02/09 16:54

    今回の場合ですと、<%= item.class %>で「Item」と表示され、<%= item.text.class %>で「string」と表示されました。<%= item_counter.class %>では、「Fixnum」となりました。この型が怪しいのでしょうか。。。

    キャンセル

  • perpouh

    2019/02/12 10:17

    `undefined local variable or method `item_counter' for #<#<Class:0x007fe96bf14ee0>:0x007fe96f8e7af0>`
    ここを見ると、表示されていない方(追加された部分テンプレート)に渡されている変数のクラスがItemじゃないんじゃないかと思ったのですが……読みを外しましたかね。追加もしてみましたか?

    キャンセル

回答 1

+1

edit.html.erbで定義している

<%= render partial: "item", collection: @items %>


のときは、collectionなので、item_counterが定義されていると思うのですが、
create.js.erbで定義している

<%= escape_javascript(render partial: '/users/item', locals: { item: @item }) %>


のときは、collectionではなく、単体のitemを使って呼び出しているので、item_counterが定義されないのではないでしょうか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/13 06:44

    コメントありがとうございます。おっしゃる通り、<%= escape_javascript(render partial: '/users/item', collection: @item ) %>にすることで、item_counterのエラーはでず送信される様になりました!
    ただ、Ajaxの表示が機能していません。。。データは保存されていますが、部分テンプレートが追加されていないようです。collectionだとこのようなことが起きるということはあるのでしょうか。

    キャンセル

  • 2019/02/13 11:35

    すみません、ちょっとどういう実装が良いかなかなか思いつかないのですが、例えば、呼び出す側を、<%= escape_javascript(render partial: '/users/item', locals: {item: @item, icount: @items.size+1}); %>のような感じにして、呼び出される側(_item.html.erb)では、icountが定義されていたら、item_counterの代わりにicountを使う、といった実装はどうでしょうか?

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.37%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る