Question

我在使用rails之前从未实现过memoization，我认为我遇到了一个很好的用例，但我不知道该怎么做。以下是我与之合作的内容：

方法

 def manager_rep_code_and_name
    manager = []

    rep_code_list.each do |rep_code|
      context = Finfolio::GetManagerByRepCode.call(rep_code: rep_code)

      manager.push({name: context.manager.response.first["Name"], rep_code: rep_code})
    end

    manager
  end

此方法进行网络调用，并且可能需要一段时间来计算所有这些。我有没有办法memoize这种方法，如果它已经存在，我就不必再出去再提出这个请求了。

像这样：

def manager_rep_code_and_name
   @managers ||= manager = []

   rep_code_list.each do |rep_code|
     context = Finfolio::GetManagerByRepCode.call(rep_code: rep_code)

     manager.push({name: context.manager.response.first["Name"], rep_code: rep_code})
   end

  manager
end

显然，这不起作用，但我现在有点卡住了。

Answer 1

我经常将这些方法分开：

def fetch_manager_rep_code_and_name
  rep_code_list.map do |rep_code|
    context = Finfolio::GetManagerByRepCode.call(rep_code: rep_code)
    { name: context.manager.response.first['Name'], rep_code: rep_code }
  end
end

def manager_rep_code_and_name
  @manager_rep_code_and_name ||= fetch_manager_rep_code_and_name
end

通常，这也使它们更容易测试。

Answer 2

如果要为同一功能记忆单独的输入/输出对，则记忆会稍微复杂一些。由于此处没有输入，并且每次都预期或允许输出相同，因此记忆非常简单。像这样：

  def manager_rep_code_and_name
    return @manager if @manager
    @manager = []
    rep_code_list.each do |rep_code|
      context = Finfolio::GetManagerByRepCode.call(rep_code: rep_code)
      @manager.push({name: context.manager.response.first["Name"], rep_code: rep_code})
    end
    @manager
  end

在这种情况下，将memoized instance var命名为方法名称的下划线版本是@_manager_rep_code_and_name，这是一种常见的约定。

Answer 3

 def manager_rep_code_and_name
    @managers ||= []
    if @managers.blank?
      @managers = rep_code_list.map do |rep_code|
        context = Finfolio::GetManagerByRepCode.call(rep_code: rep_code)
        {name: context.manager.response.first["Name"], rep_code: rep_code}
      end
    end
    @managers
  end

变量名称从@managers命名为@manager，因为它包含数据列表。

Answer 4

您可以使用begin end块来记住多行代码的结果：

def manager_rep_code_and_name
   @manager_rep_code_and_name ||= begin
     manager = []

     rep_code_list.each do |rep_code|
       context = Finfolio::GetManagerByRepCode.call(rep_code: rep_code)

       manager.push({name: context.manager.response.first["Name"], rep_code: rep_code})
     end

    manager
  end
end

记住多行方法 - Rails

方法

像这样：

4 个答案: