我将如何重构此代码以使用proc？

Question

原始代码：

  def self.user_admin_links
    ADMIN_PAGES.inject([]) do |result, page|
      result << Page.new(controller: page[:name]) if page[:menu] && (page[:group_admin] || page[:company_admin])
      result
    end
  end

  def self.super_admin_links
    ADMIN_PAGES.inject([]) do |result, page|
      result << Page.new(controller: page[:name]) if page[:super_admin]
      result
    end
  end

我试图像这样重构：

array_builder = Proc.new do |conditional|
    ADMIN_PAGES.inject([]) do |result, page|
      result << Page.new(controller: page[:name]) if conditional
      result
    end
  end

  def self.user_admin_links
    array_builder.call(page[:menu] && (page[:group_admin] || page[:company_admin]))
  end

  def self.super_admin_links
    array_builder.call(page[:super_admin])
  end

但是我得到这个错误：

Error: undefined local variable or method `array_builder' for Page:Class.

当我将array_builder变成一个类方法时，如下所示：

  def self.array_builder 
    Proc.new do |conditional|
      ADMIN_PAGES.inject([]) do |result, hsh|
        result << Page.new(controller: hsh[:name]) if conditional
        result
      end
    end
  end

我收到一个错误，在self.user_admin_links方法中无法识别“页面”。

Answer 1

在Ruby中，方法不是闭包。 即，它们不能使用周围范围的局部变量，例如array_builder 。

而且，如果您希望针对ADMIN_PAGES每个元素对page进行评估，则应使用一个块参数（基本上只是将proc作为参数传递的某种语法）。 否则，将仅在未定义page的links方法中对它进行一次评估。

def array_builder &conditional
  ADMIN_PAGES.inject([]) do |result, page|
    result << Page.new(controller: page[:name]) if conditional[page]
    result
  end
end

另外，这是一个更好的Ruby习惯用法。 使用Enumerable#select和Enumerable#map而不是手动推送到数组并使用inject：

def array_builder &conditional
  ADMIN_PAGES.select(&conditional).map {|page| Page.new(controller: page[:name])}
end

然后，其他方法传递一个块：

def self.user_admin_links
  array_builder {|page| page[:menu] && (page[:group_admin] || page[:company_admin])}
end

def self.super_admin_links
  array_builder {|page| page[:super_admin]}
end