0

我有一个问答应用程序，它与 stackoverflow 的工作方式相似。投票模型是多态的，属于其他 3 个模型。投票是通过每个模型上的成员 POST 路由使用每个控制器中的投票方法创建的。

以下user_reputation方法通过在创建投票时将投票的“第二”值（+5 表示赞成，-3 表示反对）添加到现有用户声誉来工作并设置用户声誉，但感觉设置为不好的做法投票的价值是这样的。

是否有更清洁/最佳实践方法来实现这一目标？

投票.rb

attr_accessible :value, :votable_id, :votable_type
belongs_to :votable, polymorphic: true
belongs_to :user

validates_inclusion_of :value, in: [1, -1]
validates_presence_of :user_id, :value, :votable_id, :votable_type
validates_uniqueness_of :user_id, scope: :votable_id
validates_uniqueness_of :value, scope: :votable_id

after_create :sum_votes

after_create :user_reputation

def user_reputation
    votable = self.votable_type.downcase
    user_rep = self.votable.user
    if self.value == 1
      user_rep.update_attributes(reputation: (user_rep.reputation + 5))
    elsif self.value == -1
      user_rep.update_attributes(reputation: (user_rep.reputation - 3))
    end
  end

投票_form.html.erb

<div class="vote">
    <div id='<%= "#{votes_count}_#{id}" %>' class="vote-box">
        <div class='<%= object.votes_count < 0 ? 'orange-arrows' : 'default-arrows' %>'>
            <h1><%= object.votes_count %></h1>
        </div>
    </div>
    <div id="arrows">
    <%= link_to raw("&#9650;"), vote_path.call(object, value: 1),id: "upvote", remote: true,  method: "post" %><br>
    <%= link_to raw(" &#9660;") , vote_path.call(object, value: -1), id: "downvote", remote: true, method: "post" %>
    </div>
</div>

路线.rb

resources :answers do 
    member { post :vote }
  end

  resources :questions do 
    member { post :vote }
  end 

  resources :comments do 
    member { post :vote }
  end

控制器方法

def vote 
    @vote = current_user.votes.build(value: params[:value], votable_id: params[:id], votable_type: "Answer")
    respond_to do |format|
    if @vote.save
      format.html { redirect_to :back, notice: "Vote submitted" }
      format.js
    else
      format.html { redirect_to :back, alert: "You can't vote on your own content" }
      format.js
    end
  end
 end
4

1 回答 1

1

你可以看看Active Record Observer。它实际上与您正在做的事情相同，但它可以让您将这些逻辑与您的模型分开。

而且我认为您不需要 user_reputation 方法中的第一行。

于 2013-06-10T15:05:30.307 回答