1

http://rubymonk.com/learning/books/1/problems/148-array_of_fixnum

红宝石和尚建议：

def array_of_fixnums?(array)
  array.all? { |x| x.is_a? Fixnum }
end

这很好，但是下面的代码在 irb 1.9.2 中工作，但是当 ruby​​monk 传递一个空数组时失败：

def array_of_fixnums?(array)
  result = false
  array.each { |n|
    if n.is_a? Fixnum
      result = true
    else
      result = false
    end }
  result
end

这是 irb 输出：

1.9.2-p320 :001 > array_of_fixnums? []
 => false

以下是 ruby​​monk 对我的解决方案的评价：

returns 'true' for [1,2,3] ✔
returns 'false' for ['a',1,:b] ✔
returns 'true' for []
    RSpec::Expectations::ExpectationNotMetError
    expected false to be true

我想知道为什么会这样？

根据答案更新：

def array_of_fixnums?(array)
  result = true
  array.each { |n| return false unless n.is_a? Fixnum }
  result
end
4

2 回答 2

1

您的代码有两个问题：

  1. 这个问题的措辞有点模糊。他们真正想要的是，如果任何元素不是 Fixnums，则返回 false，否则返回 true——因此空数组应该返回 true。如果您查看您的代码，您会发现它result以 false 开头，因此如果数组为空，即使测试认为它应该为 true，它也会返回 false。你可以从 true 开始解决这个问题。

  2. 您的代码实际上只是检测数组的最后一个元素是否是 Fixnum。我们来看看数组[1, "nope", 3]。它首先会看到 1 并设置result为 true，然后会看到“nope”并设置result为 false，然后会看到 3 并设置result为 true，这就是方法将返回的内容。RubyMonks 测试实际上并没有检测到这个错误，但它会出现在现实世界中。您希望在得到错误结果后立即返回，因为这足以确定数组不是所有的 Fixnums。

于 2013-03-31T20:23:12.307 回答
0

对于这种情况，您将永远不会进入每个循环，因为array. result因此，您返回在上面的行中设置为 false的默认值。但是，如果数组中没有元素，那么它的所有元素都是 Fixnums，因此您应该返回 true。

于 2013-03-31T20:22:39.930 回答