有极限的稀疏数组

Question

我有 AR 形式的对象数组 我想有限度地精简它们。

当前的方法看起来像：

  def rarefied_values(limit = 200)
    all_values = self.values.all
    rarefied_values = []

    chunk_size = (all_values.size / limit.to_f).ceil
    if all_values.size > limit
      all_values.each_slice(chunk_size) do |chunk|
        rarefied_values.push(chunk.first)
      end
      return rarefied_values
    else
      return all_values
    end
  end

有重构的提示吗？

Answer 1

def rarefied_values(limit = 200)
  all_values = values.all
  return all_values unless all_values.size > limit
  chunk_size = all_values.size / limit
  (0...limit).map{|i| all_values[i*chunk_size]}
end

ruby self 中重构的一些一般要点

• 通常可以省略。在某些情况下，您不能，例如 self.class。在本例中，self.values.all => values.all
• 如果其中一个条件过程比其他过程简单得多，则首先放置该简单情况，然后使用 return 将其从代码的其余部分中删除>。在这种情况下，返回 all_values，除非 all_values.size > limit
• 一般来说，当需要嵌套条件时，应将过程较简单的情况先分开，将复杂的情况放在最后。
• 让代码尽可能的懒惰。在这种情况下，如果 all_values.size > 则不需要 rarefied_values = []。限制。所以把它放在条件部分。

Answer 2

这是一个简单的重构，保留相同的方法，但删除显式返回调用，并且仅在必要时执行某些转换：

def rarefied_values(limit = 200)
  all_values = self.values.all
  if all_values.size <= limit
    all_values
  else
    chunk_size = (all_values.size / limit.to_f).ceil
    [].tap{ |rare| all_values.each_slice(chunk_size){ |c| rare << c.first } }
  end
end

这是一个更快、更简洁的版本：

def rarefied_values(limit = 200)
  all_values = self.values.all      
  if (size = all_values.size) <= limit
    all_values
  else
    all_values.values_at(*0.step(size-1,(size.to_f/limit).ceil))
  end
end