将三种相似的方法变成一种方法

Question

我想将这三种方法合二为一，但目前我无法思考。 1/3 需要稍微不同的调用，这一事实使情况变得复杂。它们都很相似，我知道有更好的方法，但超出了我现在的技能水平。通过传入一个额外的变量（获取、检查或处理），我可以将其变成一个，但我不知道如何做到这一点。

如果您要将这些重构为一种方法，您会怎么做？

def fetch(subjects = current_subjects, queues = QUEUES)
  subjects.each do |s|
    queues.each { |x| fetch_results(x, s) } unless queue.nil?
  end
end

def check(subjects = current_subjects, queues = QUEUES)
  subjects.each do |s|
    queues.each { |x| check_results(s["#{x}_recent"]) } unless queue.nil?
  end
end

def process(subjects = current_subjects, queues = QUEUES)
  subjects.each do |s|
    queues.each { |x| process_results(s["#{x}_recent"]) } unless queue.nil?
  end
end

编辑：一种解决方案与我之前的想法很接近，但我没有明确表示我想将 what 作为一个小数组传递，它可能是可扩展的并可用于指示是否获取、检查、处理或这些的任意组合。因此，本质上，我试图用一种方法循环执行三件事：

• 一个操作：IE、获取、检查或处理。
• 任意数量的科目。
• 任意数量的队列，目前是一个常量。

另外，这里还有其他解决方案：

http://refactormycode.com/codes/2002- Three-into -一个

Answer 1

@Lucapette 提出了一个自上而下的解决方案（我认为这在大多数情况下非常有效）。然而，@Tony 正确地指出，方法可能会演变，因此它可能过于僵化。另一种解决方案是自下而上的方法：

def iter_queues(subjects, queues)
  subjects.each do |subject|
    (queues || []).each { |queue| yield(queue, subject) }
  end
end

def fetch(subjects = current_subjects, queues = QUEUES)
  iter_queues(subjects, queues) { |q, s| fetch_results(q, s) }
end

其他方法同上。顺便说一句，双精度 each 也可以写成：

subjects.product(queues).each { ... }

Answer 2

def execute(what, subjects = current_subjects, queue = QUEUES)
  subjects.each do |s|
    queue.each { |x| send("#{what}_results", s["#{x}_recent"] ) } unless queue.nil?
  end
end

是一种方法。当然，命名由你决定。

Answer 3

我可能会做这样的事情：

def with(subjects,queues)
    subjects.each do |subject|      
        queues.each do |queue|
            yield subject, queue
        end
    end
end

with(my_subjects, my_queues){|s, q| fetch_results(q, s)}

Answer 4

为什么要将它们重构为单个函数？它们现在很相似，但如果它们后来进化并变得不同怎么办？每个函数都有其自己的用途，应将其保留为单独的函数。