Question

require 'active_support'
require 'active_support/core_ext'

def subtract_two_ranges(range_a, range_b)
  return [range_a] unless range_a.overlaps?(range_b)
  [
    ((range_a.begin..(range_b.begin - 1)) if range_a.begin < range_b.begin),
    (((range_b.end + 1)..range_a.end)     if range_b.end   < range_a.end)
  ].compact
end

Answer 1

ABC大小通常表示您的方法过于复杂，难以阅读，并且可能会破坏SRP。

它应该是Range的方法。
将数组中的表达式移动到自己的私有方法中。
使用refinements来避免猴子修补基类。

像这样试试smth。

require 'active_support'
require 'active_support/core_ext'

class Range
  def -(other)
    return [self] unless overlaps?(other)
    [substract_lower_bound(other), substract_upper_bound(other)].compact
  end

  private

  def substract_lower_bound(other)
    first..(other.begin - 1) if first < other.first
  end

  def substract_upper_bound(other)
   (other.last + 1)..last if other.last < last
  end
end

Answer 2

我在下面介绍了两种方法。两者都是纯Ruby，并且在任意有限范围内运行。¹。第一个是效率更高，但更复杂。

高效但复杂

<强>代码

def subtract_range(r1, r2)
  return [r1] if r2.end   < r1.begin  || r2.begin > r1.end
  return []   if r2.begin <= r1.begin && r2.end  >= r1.end
  if r2.begin <= r1.begin
    r2.end < r1.begin ? [r1] : [r2.end.succ..r1.end]
  else # r1.begin < r2.begin <= r1.end
    e = pred(r1.begin, r2.begin)
    r2.end >= r1.end ? [(r1.begin)..e] :
      [r1.begin..e, (r2.end.succ)..r1.end]
  end
end

def pred(start_at, e)
  (return e-1) if e.kind_of? Numeric
  loop do
    break start_at if start_at.succ == e
    start_at = start_at.succ
  end
end

范围的所有元素（无论其类别）都有方法succ（例如String#succ），但通常没有可比较的前任方法。数字类是例外：n的前身是n-1。对于非数字元素，必须通过从某个已知的较小值开始计算前驱值，并连续应用succ，直到找到前一个值。

<强>实施例

subtract_range 15..25, 20..30  #=> [15..19]
subtract_range 15..25, 10..20  #=> [21..25]
subtract_range 15..25, 17..23  #=> [15..16, 24..25]
subtract_range 15..25,  5..10  #=> [15..25]
subtract_range 15..25, 30..35  #=> [15..25]
subtract_range 15..25, 25..30  #=> [15..24]
subtract_range 15..25, 16..30  #=> [15..15]
subtract_range 15..25, 10..30  #=> []
subtract_range 'd'..'j', 'g'..'m'  #=> ["d".."f"]
subtract_range 'd'..'j', 'f'..'h'  #=> ["d".."e", "i".."j"]

效率低但更简单

<强>代码

def subtract_range(r1, r2)
  arr = r1.to_a - r2.to_a
  return [] if arr.empty?
  return [arr.first..arr.first] if arr.size == 1
  arr.slice_when { |m,n| m.succ < n }.
      map { |a| a.size == 1 ? a.first..a.first : a.first..a[-1] }
end

<强>实施例

此方法为上面给出的示例生成相同的返回值。

<强>解释

假设（来自最后一个例子）

r1 = 'd'..'j'
r2 = 'f'..'h'

然后

arr = r1.to_a - r2.to_a
  #=> ["d", "e", "f", "g", "h", "i", "j"] - ["f", "g", "h"] 
  #=> ["d", "e", "i", "j"]
arr.empty?
  #=> false (so do not return [])
arr.size == 1
  #=> false (so do not return [arr.first..arr.first])
enum = arr.slice_when { |m,n| m.succ < n }
  #=> #<Enumerator: #<Enumerator::Generator:0x00000001d2c228>:each>

我们可以将enum转换为数组，以查看将传递给map的元素。

enum.to_a
  #=> [["d", "e"], ["i", "j"]]

最后，

enum.map { |a| a.size == 1 ? a.first..a.first : a.first..a[-1] }
  #=> ["d".."e", "i".."j"]

^{1 2.0..6.0 - 4.0..8.0 #=> 2.0...4.0，不是问题（请注意三个点）。但是，2.0..4.0 - 1.0..3.0 is a problem, as the resulting range, 3.0 .. 4.0，不包括3.0，无法表示为Range个实例。} < / p>

您如何重构此方法以将ABC大小减小到15以下？

2 个答案:

高效但复杂

效率低但更简单