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

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

ただいまの
回答率

90.53%

  • JavaScript

    16322questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Ruby on Rails

    7231questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Railsの非同期通信でいいね機能が反映されるようにしたい

解決済

回答 1

投稿

  • 評価
  • クリップ 1
  • VIEW 855

●実現したいこと

現在、ユーザーの作品を投稿できるInstagramのようなサイトを作成しています。
その中で、作品に「いいね」が押せるように機能を追加しようとしているのですが、「いいねボタン」を押しても、リロードしないと結果が反映されない状態が出ています。

●質問

以下エラー文にあるart.like_userのartがnilになっていることは、何となく感じているのですが、その対処法がわからず悩んでおります。

●「いいねボタン」を押した際に起きているエラー

ActionView::Template::Error (undefined local variable or method `art' for #<#<Class:0x007ff54d1973b0>:0x007ff54b9f6f08>
Did you mean?  arts
               art_url
               @arts):
    1:    <% if user_signed_in? %>
    2:       <% if art.like_user(current_user.id) %>
    3:         <%= button_to art_likes_path(like, art_id: art.id), method: :delete, id: "like-button", remote: true do %>
    4:           <%= image_tag("icon_red_heart.svg") %>
    5:           <span>
  app/views/likes/_like.html.erb:2:in `_app_views_likes__like_html_erb___1033727092812393927_70345769537660'
  app/views/likes/create.js.erb:1:in `_app_views_likes_create_js_erb__3536096750636237475_70345756223260'

●各ファイルの状態 index.html.erb

  <% @arts.each do |art| %>

    <div class="contents col-md-3 clearfix">


        <div class="art_image">
          <%= image_tag art.image.url %>
        </div>

        <span class="user_data">投稿者:</span><%= art.user.nickname %>

        <% if user_signed_in? && current_user.id == art.user_id %>
          <ul class="more_list">
            <li>
              <%= link_to '編集', "/arts/#{art.id}/edit", method: :get %>
            </li>
            <li>
              <%= link_to '削除', "/arts/#{art.id}", method: :delete %>
            </li>
          </ul>
        <% end %>

      <div class="like">
        <%= render partial: 'likes/like', locals: { art: art, arts: @arts, likes: @likes, like: @like } %>
      </div>

    </div>

  <% end %>


●各ファイルの状態 _like.html.erb

   <% if user_signed_in? %>
      <% if art.like_user(current_user.id) %>
        <%= button_to art_likes_path(like, art_id: art.id), method: :delete, id: "like-button", remote: true do %>
          <%= image_tag("icon_red_heart.svg") %>
          <span>
            <%= art.likes_count %>
          </span>
        <% end %>
      <% else %>
        <%= button_to art_likes_path(art), id: "like-button", remote: true do %>
          <%= image_tag("icon_heart.svg") %>
          <span>
            <%= art.likes_count %>
          </span>
        <% end %>
      <% end %>
    <% else %>
      <%= image_tag("icon_heart.svg") %>
        <span>
          <%= art.likes_count %>
        </span>
    <% end %>


●各ファイルの状態 routes.rb

Rails.application.routes.draw do
  devise_for :users
  resources :users, only: [:show, :edit, :update]
  root to: "arts#index"
  resources :arts, only: [:index, :new, :create, :show, :edit, :update, :destroy] do
    resources :likes, only: [:create, :destroy]
  end
end


●各ファイルの状態 likes_controller

class LikesController < ApplicationController
  def create
    @like = Like.create(user_id: current_user.id, art_id: params[:art_id])
    @likes = Like.where(art_id: params[:art_id])
    @arts = Art.all
  end

  def destroy
    like = Like.find_by(user_id: current_user.id, art_id: params[:art_id])
    like.destroy
    @likes = Like.where(art_id: params[:art_id])
    @arts = Art.all
  end
end


●各ファイルの状態 create.js.erb

$("#like-button").html("<%= j(render partial: 'likes/like', locals: { arts: @arts, likes: @likes, like: @like}) %>");


●各ファイルの状態 destroy.js.erb

$("#like-button").html("<%= j(render partial: 'likes/like', locals: { arts: @arts, likes: @likes }) %>");


●各ファイルの状態 art.rb

class Art < ActiveRecord::Base
  belongs_to :user
  mount_uploader :image, ImageUploader
  has_many :likes, dependent: :destroy
  def like_user(user_id)
   likes.find_by(user_id: user_id)
  end
end


各ファイルの状態 like.rb

class Like < ActiveRecord::Base
  belongs_to :art, counter_cache: :likes_count
  belongs_to :user
end
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

とりあえず、これでいいんじゃないですかね?

class LikesController < ApplicationController
  def create
    @like = Like.create(user_id: current_user.id, art_id: params[:art_id])
    @likes = Like.where(art_id: params[:art_id])
    @art=Art.find(params[:art_id])
    @arts = Art.all
  end

  def destroy
    like = Like.find_by(user_id: current_user.id, art_id: params[:art_id])
    like.destroy
    @likes = Like.where(art_id: params[:art_id])
    @art=Art.find(params[:art_id])
    @arts = Art.all
  end
end


create.js.erb

$("#like-button").html("<%= j(render partial: 'likes/like', locals: {art: @art,arts: @arts, likes: @likes, like: @like}) %>");
$("#like-button").html("<%= j(render partial: 'likes/like', locals: {art: @art, arts: @arts, likes: @likes }) %>");

余力があるなら

class LikesController < ApplicationController
  def create
    @art=Art.find(params[:art_id])
    @like = @art.likes.create(user_id: current_user.id)
    @likes =@art.likes
    @arts = Art.all
  end

  def destroy
    @art=Art.find(params[:art_id])
    like = @art.likes.find_by(user_id: current_user.id)
    like.destroy
    @likes =@art.likes
    @arts = Art.all
  end
end


意味を考えつつこのようなリファクタリングをしてみては?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/28 23:11

    mokeさん
    早速ご教示頂きまして、誠にありがとうございます。
    上記方法を試してみたところ、非同期通信で反映されるようになりました。

    「余力があるなら」とお教え頂いた分についても、一度試してみたいと思います。

    ありがとうございました。

    キャンセル

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

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

関連した質問

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

  • JavaScript

    16322questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Ruby on Rails

    7231questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。