Как избежать дублирования/повторения методов в контроллерах - RAILS 3+

Question

Я создаю пример приложения электронной коммерции, используя ruby ​​on rails. Одно из моего имени контроллера - «product_controller». Этот контроллер также размещен внутри как вложенные контроллеры. Действия внутри этих контроллеров одинаковы. Как мы можем представить эти действия без дублирования кодов. Образцы кода приведены ниже.

приложение/контроллеры/products_controller.rb

def index 
@product = Product.all 
@vari = @products.variants 
............. 
............. 
end 

приложение/контроллеры/master_admins/products_controller.rb

def index 
@product = Product.all 
@vari = @products.variants 
............. 
............. 
end 

приложение/контроллеры/master_admins/properties_controller.rb

def product 
@product = Product.all 
@vari = @products.variants 
............. 
............. 
end 

Вышеуказанные действия содержат один и тот же набор кодов. Как мы можем реорганизовать это так t код не будет повторяться.

Заранее спасибо ....

Answer 1

Я предложил бы использовать concerns, которые являются удивительным для DRY.

Для контроллера, общие методы могут быть размещены здесь:

В моем приложение/контроллеры/проблем/common.rb

module Common 
    extend ActiveSupport::Concern 

    module ClassMethods 
    ## This is a Class method, call it just like you call any other class method 
    def get_products_and_variants 
     @product = Self.all 
     @vari = @product.variants 
    end 
    end 

## Instance method, if you don't want aclass method, use this instance method 
def my_instance_method 
    ## code for method 
end 

Затем вызовите его, включив в common.rb контроллер *

include Common 

def index 
    ## This will make @product and @vari available 
    Product.get_products_and_variants 

    # ............ 
end 



## Other method using same method call 
def product 
    ## This will make @product and @vari available 
    Product.get_products_and_variants 

    # ............. 
end 

Если у вас есть несколько классов, используя этот метод класса, вы можно использовать что-то вроде этого (в common.rb):

def get_details   
    if self == Product 
    ## Get products 
    elsif self == Variant 
    ## Get variants 
    elsif self == ProductDetail 
    ## Get product details 
    end    
end 

Answer 2

Вы можете сделать что-то вроде:

class GenericController < ActiveRecord::Base 
    def index 
    do something interesting 
    end 
end 

class PropertiesController < GenericController 
    # index is done 
end 

class ProductsController < GenericController 
    # index is done 
end 

Answer 3

Вы можете использовать before_action в контроллере. Он будет выполнять find_product перед выполнением индекса:

before_action :find_product, only: :index 

def find_product 
    @product = Product.all 
    @vari = @products.variants  
end 

def index 
# some useful code 
end 

Вы можете удалить «только:» части для выполнения find_product перед другими действиями, или переместить его в ApplicationController (или другой родитель класс контроллера) для выполнения before_action во всех соответствующих контроллерах.