Question

此程序采用两个字符串数组，并从a2的最短长度减去a1的最长长度，然后反之亦然，并返回更大的长度。

它有效（通过了103/103测试），但出现此错误：

NoMethodError: undefined method `length' for nil:NilClass 
main.rb:10:in `mxdiflg'
main.rb:66:in `block (2 levels) in <main>'

问题出在哪里？既然这不是第一次发生，那么我该如何调试一个可行的程序？

def mxdiflg(a1, a2)
    if a1 == '' || a2 == ''
    -1
  end
  a1_order = a1.sort{|left, right| left.length <=> right.length}
  a2_order = a2.sort{|left, right| left.length <=> right.length}

  a_total = a1_order.reverse[0].length - a2_order[0].length
  b_total = a2_order.reverse[0].length - a1_order[0].length

  if b_total > a_total
    b_total
  else
    a_total
  end
end

Answer 1

从修复程序开始。首先，您说您正在接受字符串数组，但是if a1 == '' || a2 == ''检查您是否传递了空字符串。放置-1而不放置return -1基本上没有任何作用。

我假设错误在这一行中（您在stacktrace中有该行，它是main.rb:10:in 'mxdiflg'，因此对于您来说是第10行）：

a_total = a1_order.reverse[0].length - a2_order[0].length

好像您的数组为空，您的array[0]将是nil，因此您无法在其上调用.length（就像您粘贴的错误所暗示的那样）。

对于调试，在某些时候您必须习惯使用Pry，但是现在检查行号和错误消息就足够了。在这种情况下，很明显，您正在.length上调用nil，因此您的a1_order[0]必须为nil，因此您的数组必须为空。您还可以添加简单的看跌期权消息，例如：

puts "a1_order: #{a1_order}"
puts "a2_order: #{a2_order}"
a_total = a1_order.reverse[0].length - a2_order[0].length
b_total = a2_order.reverse[0].length - a1_order[0].length

现在，在运行程序时，您可以检查已排序的数组，应该很清楚，您正在尝试在nil s上调用方法。

现在，覆盖了所有内容，我们可以尝试使您的程序更美观。首先，正如我提到的，您的第一张支票没有多大意义。让我们把它变成：

return -1 if [a1,a2].any?(&:empty)

如果任何数组为空，它将实际上从您的方法中返回-1。

进一步：

a1_order = a1.sort{|left, right| left.length <=> right.length}

可以写为：

a1_order.sort_by(&:length)

致电

a1_order.reverse[0]

效率低下，因为它将以相反的顺序创建数组的副本，您可以简单地执行a1_order.last来代替。

如果要查找最大值/最小值，则可以像这样使用Enumerable#max_by / Enumerable#min_by：

a_total = a1.max_by(&:length).length - a2.min_by(&:length).length
b_total = a2.max_by(&:length).length - a1.min_by(&:length).length

使用Array#max可以获得更高的价值：

[a_total, b_total].max

将所有内容包装在一起，您的方法可能如下所示：

def mxdiflg(a1, a2)
  return -1 if [a1, a2].any?(&:empty?)

  a_total = a1.max_by(&:length).length - a2.min_by(&:length).length
  b_total = a2.max_by(&:length).length - a1.min_by(&:length).length

  [a_total, b_total].max
end

Answer 2

@MarcinKołodziej，你击败了我，但我仍然要发表我的答案。 @Jules问题之一是如何检查空数组。

irb(main):002:0> [] == ''
=> false

即使数组为空，也将始终返回false。空吗是检查空数组的一种方法，请参见下文。

#!/usr/bin/ruby

    def run

    p  mxdiflg(["aa" , "b" , "c"], ["dddf", "r", "u", "ee"] )
    p  mxdiflg([], ["f", "r", "u", "ee"] )


    end


    def mxdiflg(a1, a2)

      # return whatever you want if any of the arrays is empty
      return false  if  a1.empty?  || a2.empty?

      # drop empty elements form array
      a1.reject!(&:empty?)
      a2.reject!(&:empty?)

      a1_order = a1.sort{|left, right| left.length <=> right.length}
      a2_order = a2.sort{|left, right| left.length <=> right.length}

      a_total = a1_order.reverse[0].length - a2_order[0].length
      b_total = a2_order.reverse[0].length - a1_order[0].length

      (b_total > a_total) ? b_total : a_total

    end


    run

程序正常运行，但是我仍然收到NoMethodError'length'nil

2 个答案: