비슷한 코드를 컨트롤러 메서드에서 사용하기

Question

저는 매우 비슷한 코드를 가진 컨트롤러에서 두 가지 방법을가집니다. 내가 어떻게 그들을 말릴 수 있는지 궁금해! 그들은 모두 csv-importer gem을 사용하여 csv 파일을 구문 분석합니다.

sales_controller.rb

def import_csv_test 
    user_id = params[:user_id] 
    import = ImportSaleCSV.new(file: params[:file]) do 
     after_build do |sale| 
     sale.user_id = user_id 
     skip! if sale.email == nil 
     skip! if sale.order_date == nil 
     skip! if sale.amount == nil 
     end 
    end 
    import.run! 
    redirect_to lifecycle_grid_sales_path, notice: import.report.message 
    end 

    def import_ftp 
    user_id = params[:user_id] 
    import = ImportSaleCSV.new(path: './public/uploads/gotcha.csv') do 
     after_build do |sale| 
     sale.user_id = user_id 
     skip! if sale.email == nil 
     skip! if sale.order_date == nil 
     skip! if sale.amount == nil 
     end 
    end 
    import.run! 
    redirect_to lifecycle_grid_sales_path, notice: import.report.message 
    end 

감사합니다! 를 호출 한 후

def import(hash) 
    user_id = params[:user_id] 
    import = ImportSaleCSV.new(hash) do 
    after_build do |sale| 
     sale.user_id = user_id 
     skip! if sale.email == nil 
     skip! if sale.order_date == nil 
     skip! if sale.amount == nil 
    end 
    end 
    import.run! 
    redirect_to lifecycle_grid_sales_path, notice: import.report.message 
end 

을 그리고 :

import({file: params[:file]}) 
import({path: './public/uploads/gotcha.csv'}) 

당신이 추출 할 수 있도록 그 방법은 컨트롤러에 속하는하지 않는 것

Answer 1

당신은 하나 하나에 당신의 방법을 모두 리팩토링 할 수있다 그것은 어딘가에있다. this great article을 확인하고 새로운 서비스 개체로 메서드를 추출하는 것이 좋습니다.

Answer 2

무거운 짐을 들으려면 수업을 추출 할 수 있다고 생각합니다.

class ImportSaleCSVCreator 
    def initialize(csv_options = {}, csv_attributes = {}) 
    @csv_options = csv_options 
    @csv_attributes = csv_attributes 
    end 

    def build 
    ImportSaleCSV.new(csv_options) do 
     after_build do |sale| 
     csv_attributes.each { |k, v| sale.public_send("#{k}=", v) } 
     skip! if sale.email.nil? || sale.order_date.nil? || sale.amount.nil? 
     end 
    end 
    end 

    private 

    attr_reader :csv_options, :csv_attributes 
end 

class Controller 
    def import_csv 
    import = ImportSaleCSVCreator.new({ file: params[:file] }, { user_id: params[:user_id] }) 
    import.run! 
    end 

    def import_ftp 
    import = ImportSaleCSVCreator.new({ path: './gotcha.csv' }, { user_id: params[:user_id] }) 
    import.run! 
    end 
end 

통과 된 속성을 선택했는지 확인하십시오. 특히 파일, 경로 등을 다룰 때 ImportSaleCSVCreator에서 매개 변수를 필터링 할 수 있습니다.