只允许用户为每部电影创建一条评论

Question

我有我的应用设置，用户可以在其中为电影撰写评论。我想做的是限制用户每部电影只能创建一个评论。我已经设法在我的评论控制器中完成了这一点：

class ReviewsController < ApplicationController
  before_action :has_reviewed, only [:new]
  ....
  def has_reviewed?
    if Review.where(user_id: current_user.id, movie_id: @movie.id).any?
      redirect_to movie_reviews_path

      flash[:notice] = "You've already written a review for this movie."
    end
  end
end

我现在遇到的麻烦是使用 Devise 和 CanCanCan 的辅助方法将相同的逻辑转换到我的索引视图模板中。

<% if user_signed_in? && ... %> # current_user has already created a review for this movie
  <%= link_to "Edit Review", edit_movie_review_path(@movie, review) %>
<% else %>
  <%= link_to "Write a Review", new_movie_review_path %>
<% end %>

另外：有什么方法可以改进我的 has_reviewed? 方法中的查找？我觉得有更好的方法来编写它，但无法确定最合适的修复方法。

Answer 1

Why not use a validation:

#app/models/review.rb
class Review < ActiveRecord::Base
    validates :movie_id, uniqueness: { scope: :user_id, message: "You've reviewed this movie!" }
end

这是考虑您的review 模型belongs_to :movie

---

您也可以使用 ActiveRecord 回调：

#app/models/review.rb
class Review < ActiveRecord::Base
   before_create :has_review?
   belongs_to :user, inverse_of: :reviews
   belongs_to :movie

   def has_review?
      return if Review.exists?(user: user, movie_id: movie_id)
   end
end

#app/models/user.rb
class User < ActiveRecord::Base
   has_many :reviews, inverse_of: :user
end

---

有什么方法可以改进我的 has_reviewed 中的查找？方法？

  def has_reviewed?
      redirect_to album_reviews_path, notice: "You've already written a review for this album." if current_user.reviews.exists?(movie: @movie)
  end

Answer 2

为什么不做一个has_reviewed？用户类的方法？

例如

def has_reviewed?(reviewable)
   # query in here
end

那么你应该可以在你的控制器和视图中使用它。

Answer 3

您将希望对 new 和 create 执行此操作。否则，精明的用户将能够发布可以通过您的新操作的帖子。

我会将 link_to 放在 helper 或 Presenter 对象中。它通常看起来像这样。

def create_or_edit_review_path(movie, current_user)
  return '' if current_user.blank?

  if current_user.review.present?
    #Generate review edit link
  else
    #generate new link
  end 
end

在那之后，在你所有的观点中，它只是

<%= create_or_edit_review_path(@movie, current_user) %>

然后在您的控制器中为 new 和 create 您可以执行之前的操作或仅在每个操作上进行重定向。

before_action :enforce_single_review, only: [:create, :new]

def enforce_single_review
  if current_user.review.present?
    redirect_to review_path(current_user.review)
  end
end

Answer 4

这是我的想法：

我使用Review 模型上的find_by 方法创建了一个实例方法来检索用户的电影评论：

class User < ActiveRecord::Base
  ....
  def movie_review(album)
    Review.find_by(user_id: self, album_id: album)
  end
end

这个方法在设置我的回调时也很方便：

class ReviewsController < ApplicationController
  before_action :limit_review, only: [:new, :create]
  ....
  private
    def limit_review
      user_review = current_user.movie_review(@movie)

      if user_review.present?
        redirect_to edit_movie_review_path(@movie, user_review)
      end 
    end
end

创建了一个辅助方法来显示适当的链接以编辑或创建评论。非常感谢 Austio 和他的suggestion：

module ReviewsHelper
  def create_or_edit_review_path(movie)
    user_review = current_user.movie_review(movie) if user_signed_in?

    if user_signed_in? && user_review.present?
      link_to "Edit review", edit_movie_review_path(movie, user_review)
    else
      link_to "Write a review", new_movie_review_path
    end
  end
end

最后这就是我在视图模板中调用助手的方式：

....
<%= create_or_edit_review_path(@album) %>