Rails 3 - Ajax和控制器重构

Question

我是新来的Rails，我想知道我是否可以重构下面的代码。

基本上我想为ajax中的问题vote_down ou vote_up。

控制器：

def vote_up 
@question = get_question params[:id] 
if current_user != @question.user 
    render :json => @question.vote(current_user, 'up').to_json 
end 
end 

def vote_down 
@question = get_question params[:id] 
if current_user != @question.user 
    render :json => @question.vote(current_user, 'down').to_json 
end 
end 

型号：

def vote(user, vote) 
if user.voted?(self) 
    'Question already voted' 
else 
    I18n.t('question.voted') if user.send("#{vote}_vote", self) 
end 
end 

查看：

<script> 
$('#question_vote_up').live('ajax:success', function(evt, data, status, xhr) { 
$('#question_vote_up').remove() 
$('#question_vote_down').remove() 
alert(xhr.responseText) 
}) 

$('#question_vote_down').live('ajax:success', function(evt, data,  status, xhr) { 
$('#question_vote_up').remove() 
$('#question_vote_down').remove() 
alert(xhr.responseText) 
}) 
</script> 

<% if current_user != @question.user %> 
<%= link_to t('question.vote_up'), { :action => "vote_up" }, :id => "question_vote_up", :remote => true %> 
<%= link_to t('question.vote_down'), { :action => "vote_down" }, :id => "question_vote_down", :remote => true %> 
<% end %> 

我无法弄清楚如何不重复自己，如果有一个更清洁的方式到

if current_user != @question.user 

在此先感谢

Answer 1

像下面可能工作（没有测试过，只要确保不增加一倍渲染）：

def getQuestion(direction) 
    @question = get_question params[:id] 
    if current_user != @question.user 
     render :json => @question.vote(current_user, direction).to_json 
    else 
     render :nothing => true 
    end 
end 

def vote_up 
    getQuestion "up" 
end 

def vote_down 
    getQuestion "down"  
end