Есть ли лучший способ сделать это в рубине

Question

На самом деле это в настоящее время не работает на всех

@category = Category.find(params[:id]) 
@sub_categories = @category.sub_categories 
if @sub_categories 
    @designs = [] 
    @sub_categories.each do |sub_cat| 
    @designs << sub_cat.designs.paginate :page => params[:page], :order => 'name', :per_page => @per_page 
    end 
end 

Его неисправный на этой синтаксической ошибке

categories_controller.rb:21: syntax error, unexpected tSYMBEG, expecting kEND 
...<< sub_cat.designs.paginate :page => params[:page], :order ... 

В принципе у меня есть категория имеет много sub_categories, который имеет много дизайн и его текущую категорию, и я хочу показать все проекты для этой категории ... любую идею о лучших практиках и о том, как исправлять эту проблему

Answer 1

Я хотел бы сделать две вещи, чтобы реорганизовать это ...

Во-первых, чтобы уменьшить сцепление, перемещая варианты нумерации страниц к методу класса на модели проектирования. Ваш контроллер не должен знать об этой модели.

class Design < ActiveRecord::Base 
    def self.paginate_with_params(params, per_page) 
    paginate(:page => params[:page], :order => 'name', :per_page => per_page) 
    end 
end 

Во-вторых, чтобы удалить ненужные переменные и логику из контроллера, которые на самом деле не добавить большое значение:

@category = Category.find(params[:id]) 
@designs = @category.sub_categories.map |sub_cat| 
    sub_cat.designs.paginate_with_params(params, @per_page) 
end 

Я не проверял этот код, поэтому, пожалуйста, идти легко на меня, если он не работает :)

Answer 2

Предполагая, что вы хотели использовать @designs, а не @patterns:

@category = Category.find(params[:id]) 
@sub_categories = @category.sub_categories 
pagination_options = { :page => params[:page], 
         :order => 'name', 
         :per_page => @per_page 
        } 
unless @sub_categories.empty? 
    @designs = [] 
    @sub_categories.each do |sub_cat| 
    @designs << sub_cat.designs.paginate(pagination_options) 
    end 
end 

Я думаю, что не хватало, были скобки для аргументов paginate вызова.

Answer 3

a = [] 
b.each do |c| 
    a << c.d(e) 
end 

Равен:

a = b.map { |c| c.d(e) } 

И если d не нужны параметры:

a = b.map &:d 

Но в вашем случае, я думаю, вам просто нужно добавить () вокруг вашего параметры хэша, как уже ответил Райан Бигг.

Answer 4

@category = Category.find(params[:id]) 
@sub_categories = @category.sub_categories # assumption: this doesn't return nil 
page_opts = {:page => params[:page], :order => 'name', :per_page => @per_page} 
@designs = @sub_categories.map {|sub_cat| sub_cat.designs.paginate(page_opts) }