重构每个循环

Question

更新：我设法重构了一下代码，但它仍然是一个嵌套循环。

我想我已经找到了一种使用 map 和 keep_if 改进代码的方法。我不知道它是否理想，因为它仍然是一个嵌套循环。 输出 = {}

array.map do |a|
  output[a[0]] = another_array.dup.keep_if do |b|
    a[1].include?(b["name"])
  end
end

这是一个有点n00b的问题。我试图弄清楚如何重构嵌套的每个循环，如下所示，这样我就不会声明以后不需要的额外变量，以便我的代码运行得更快。

some_array = [["one", 2, 3], ["two", 3, 4], ["three", 4, 5]]
output = {}
some_array.each do |a|
  current_group = []
  another_array.each do |b|
    current_group << b if something == true
  end
  output[a[0]] = current_group
end

output作为数组的哈希返回。 some_array 是一个嵌套数组，其中每个子数组中的第一个元素是一个字符串，another_array是一个哈希数组。

Answer 1

我无法从该代码中确切地判断您在做什么 - group是什么？ 这种情况如何将some_array和other_array联系在一起？ 但一般来说，如果你想建立一个新的数组或哈希，要达到的成语是带有inject的东西（又名 reduce ）。 模式是这样的：

output = some_array.inject({}) do |partial_output, item| 
   ...
   new value of partial_output after this loop iteration
end