如何重构这个Ruby代码？

Question

我创建了以下内容，该方法可以正常工作，但似乎很神秘。 有没有办法以更像Ruby或易于理解的方式编写它？

此方法删除数字以下的较低因素。 因此， 10.high_factors返回[6,7,8,9,10] 。 6被2整除，因此2被删除。 列表中没有大于6的倍数，因此保持不变。

class Fixnum
  def high_factors
    # Get the numbers that are not divisible by lower ones below self
    list = (2..self).to_a
    2.upto(self).each do |i|
      ((i+1)..self).each { |j| list.delete i if j.is_divisible_by? i }
    end

    list
  end

  def is_divisible_by? divisor
    self % divisor == 0
  end
end

Ruby 1.9.3

Answer 1

方法的结果将始终是(N/2) + 1至N的数字列表。

对于每个i<=(N/2) ， 2*i也将出现在列表中。

对于列表中的每个j >= (N/2)+1 ，由于2*j > N ，因此k=x*j上的k=x*j不会是k=x*j 1的整数。

因此，如果您的方法仅返回((self/2 + 1)..self).to_a则它也将按您希望的那样工作。

Answer 2

那这个呢？ 只需删除可被高整数整除的数字。

class Fixnum
  def high_factors
    # Get the numbers that are not divisible by lower ones below self
    (2..self).reject do |i|
      (i+1..self).any? { |j| j.divisible_by?(i) }
    end
  end

  def divisible_by?(divisor)
    self % divisor == 0
  end
end

ps：在ruby中，通常可以在布尔函数的开头省略“ is_”，因为我们可以添加？。

Answer 3

这是我的

def high_factors
  ary = (2..self).to_a
  ary.reject do |factor|
    ary.index {|num| num != factor and num % factor == 0}
  end
end

它起作用，因为如果找不到合适的匹配项，则Array＃index返回nil。