只允许用户为每部电影创建一个评论 [英] Only allow users to create one review per movie

问题描述

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

  class ReviewsController< ApplicationController 
 before_action：has_reviewed，只有[：new] 
 .... 
 def has_reviewed？ 
如果Review.where（user_id：current_user.id，movie_id：@ movie.id）.any？ 
 redirect_to movie_reviews_path 
 
 flash [：notice] =你已经为这部电影写了评论。 
 end 
 end 
 end 
  

我现在在哪里麻烦的是将这个相同的逻辑转换成我的索引视图模板，使用Devise和CanCanCan的帮助方法。

 <％如果user_signed_in？ &安培;&安培; ...％> ＃current_user已经为此电影创建了评论
<％= link_to编辑评论，edit_movie_review_path（@movie，review）％> 
<％else％> 
<％= link_to撰写评论，new_movie_review_path％> 
<％end％> 
  

另外：有什么办法可以改进我的 has_reviewed？方法？我觉得写一个更好的方法，但无法确定最合适的修复。

解决方案

我想出：

我创建了一个实例方法来使用 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.movi​​e_review（@movie） 
 
如果user_review.present？ 
 redirect_to edit_movie_review_path（@movie，user_review）
 end 
 end 
 end 
  

创建了一个帮助方法，用于显示编辑或创建的相关链接。非常感谢Austio和他的建议：

 模块评论Helper 
 def create_or_edit_review_path（movie）
 user_review = current_user.movi​​e_review（movie）if user_signed_in？ 
 
如果user_signed_in？ &安培;&安培; user_review.present？ 
 link_to编辑评论，edit_movie_review_path（movie，user_review）
 else 
 link_to写评论，new_movie_review_path 
 end 
 end 
 end 
  

最后这是我在我的视图模板中调用帮助器：

  .... 
<％= create_or_edit_review_path（@album）％> 
  

I have my app setup where users can write reviews for a movie. What I'd like to do is limit the user to create only one review per movie. I've managed to accomplish this in my reviews controller as so:

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

Where I'm now having trouble is translating this same logic into my index view template with the helper methods of Devise and CanCanCan at my disposal.

<% 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 %>

Also: Is there any way to improve the lookup in my has_reviewed? method? I feel like there's a better way to write it but can't determine the most appropriate fix.

解决方案

Here's what I came up with:

I created an instance method to retrieve a user's movie review using the find_by method on the Review model:

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

This method also comes in handy when setting up my callback:

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

Created a helper method for showing the appropriate link to edit or create a review. Big thanks to Austio and his 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

And at last this is how I call the helper in my view template(s):

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

这篇关于只允许用户为每部电影创建一个评论的文章就介绍到这了，希望我们推荐的答案对大家有所帮助，也希望大家多多支持IT屋！