Evalの代替

Question

私はアプリにbazillionコントローラを持っていて、メンテナンスを少なくするためにメタプログラミングを使用することに不安がありました。これは動作しますが、それはevalの形式で危険でスパイクです：

def plural_action(method_name) 
    class_name = self.class.to_s.gsub(%r{^(\w*)Controller}) {|s| $1 } 
    @title = "#{method_name.to_s.titlecase} of #{class_name}" 
    eval "@q = #{class_name.singularize}.where(:client_id => current_user.client_id).search(params[:q])" 
    eval "@#{class_name.downcase} = @q.result(:distinct => true).paginate(:page => params[:page])" 
    eval "session[:query] = @#{class_name.downcase}.map(&:id)" 
    eval "respond_with(@#{class_name.downcase})" 
    end 

私はevalのを使用せずにこれを行うことができますか？私はinstance_variable_set、sendとconst_getを試しましたが、これまでのところ運がありません。

ここに、私が評価する方法の例があります。

def index 
    @title = "Index of Books" 
    @q = Book.where(:client_id => current_user.client_id).search(params[:q]) 
    @books = @q.result(:distinct => true).paginate(:page => params[:page]) 
    session[:query] = @books.map(&:id) 
    respond_with(@books) 
    end 

Answer 1

定数に文字列（そのクラス型は一例です）、それが表すターン壮大方法constantizeあります。このことを念頭に置いて、私はあなたのようにあなたの方法を書き換えることができると思う：

def plural_action(method_name) 
    class_name = self.class.to_s.gsub(%r{^(\w*)Controller}) {|s| $1 } 
    @title = "#{method_name.to_s.titlecase} of #{class_name}" 
    @q = class_name.singularize.constantize.where(:client_id => current_user.client_id).search(params[:q]) 
    self.instance_variable_set("@#{class_name.downcase}", @q.result(:distinct => true).paginate(:page => params[:page])) 
    session[:query] = self.instance_variable_get("@#{class_name.downcase}").map(&:id) 
    respond_with(self.instance_variable_get("@#{class_name.downcase}")) 
end 

Answer 2

def plural_action(method_name) 
    class_name = self.class.to_s.gsub(%r{^(\w*)Controller}) {|s| $1 } 
    @title = "#{method_name.to_s.titlecase} of #{class_name}" 
    @q = class_name.singularize.constantize.where(:client_id => current_user.client_id).search(params[:q]) 
    instance_variable_set class_name.downcase, @q.result(:distinct => true).paginate(:page => params[:page]) 
    session[:query] = @q_result.map(&:id) 
    respond_with(@q_result) 
end 

Answer 3

あなたはあなたのために、このすべてを行うinherited_resource宝石を使用して考えていましたか？