0

我有一些非常相似的控制器方法，我想知道重构它们的最佳方法是什么。首先想到的是以某种方式将两个块传递给辅助方法，但我也不知道该怎么做。

def action_a
  if @last_updated.nil?
    @variable_a = @stuff_a
  else
    @variable_a = (@stuff_a.select{ |item| item.updated_at > @last_updated }
  end
end

def action_b
  if @last_updated.nil?
    @variable_b = @stuff_b.some_method
  else
    @variable_b = @stuff_b.some_method.select{ |stuff| item.updated_at > @last_updated }
  end
end

似乎我一直在检查是否@last_updated为 nil （我将@last_updated实例变量设置在 a 中before_filter。如果我能以某种方式将其中的东西if作为一个块传递，而将其中的东西else作为另一个块传递，那么我可以删除if @last_updated.nil?重复项吗？

对于许多方法来说，实现这一目标的最佳方法是什么？

更新

在我指定@stuff_aand的地方@stuff_b，它们总是返回一个数组（因为我使用了.select）。

4

3 回答 3

3

看看这个。它是 DRYer，应该产生相同的结果。

def action_a
  do_the_processing :"@variable_a", @stuff_a
end

def action_b
  do_the_processing :"@variable_b", @stuff_b.some_method
end

private
def do_the_processing var_name, collection
  if @last_updated.nil?
    instance_variable_set var_name, collection
  else
    instance_variable_set var_name, collection.select{ |item| item.updated_at > @last_updated }
  end
end

更新

这是两个块的方法（只是为了好玩）（使用 1.9 的 stabby lambda 语法）

def action_a
  check_last_updated is_nil: -> { @variable_a = @stuff_a },
                     is_not_nil: -> { @variable_a = (@stuff_a.select{ |item| item.updated_at > @last_updated } }
end

def action_b
  check_last_updated is_nil: -> { @variable_b = @stuff_b.some_method },
                     is_not_nil: -> { @variable_b = @stuff_b.some_method.select{ |stuff| item.updated_at > @last_updated } }
end

private
def check_last_updated blocks = {}
  if @last_updated.nil?
    blocks[:is_nil].try(:call)
  else
    blocks[:is_not_nil].try(:call)
  end
end
于 2013-01-04T04:10:18.077 回答
1

您需要在单独的def块中提取您的条件并稍后使用它：

def select_updates a
  @last_updated.nil? ? a : a.select{ |item| item.updated_at > @last_updated }
end
def action_a; @variable_a = select_updates(@stuff_a) end
def action_b; @variable_b = select_updates(@stuff_b.some_method) end
于 2013-01-04T10:17:19.723 回答
0

如我所见，您可以执行以下操作

每个有两个范围

前任：

class Stuff < ActiveRecord::Base
  scope :updated_at, lambda {|updated_date|
    {:conditions => "updated_at > #{updated_date}"}
  }
end


class Item < ActiveRecord::Base
  scope :updated_at, lambda {|updated_date|
    {:conditions => "updated_at > #{updated_date}"}
  }
end

在您的控制器中执行此操作

def action_a
  @variable_a = update_method(@stuff_a)
end

def action_b
  @variable_b = update_method(@stuff_b)
end

private
def update_method(obj)
  result = nil
  if @last_updated.nil?
    result = obj.some_method 
  else    
    result = obj.some_method.updated_at(@last_updated) 
  end  
  result 
end

高温高压

于 2013-01-04T04:42:53.057 回答