Question

有没有办法重写这个更优雅？我认为，这是一段糟糕的代码，应该重构。

>> a = [2, 4, 10, 1, 13]
=> [2, 4, 10, 1, 13]
>> index_of_minimal_value_in_array = a.index(a.min)
=> 3

Answer 1

我相信这只会遍历数组一次并且仍然易于阅读：

ary = [2,3,4,5,1]        # => [2,3,4,5,1]
ary.each_with_index.min  # => [1, 4]
                         # where 1 is the element and 4 is the index

Answer 2

这只遍历数组一次，而ary.index(ary.min)将遍历它两次：

ary.each_with_index.inject(0){ |minidx, (v,i)| v < a[minidx] ? i : minidx }

Answer 3

阅读其他情况（找到所有且只有最后一个最小元素）会很有趣。

ary = [1, 2, 1]

# find all matching elements' indexes
ary.each.with_index.find_all{ |a,i| a == ary.min }.map{ |a,b| b } # => [0, 2]
ary.each.with_index.map{ |a, i| (a == ary.min) ? i : nil }.compact # => [0, 2]

# find last matching element's index
ary.rindex(ary.min) # => 2

Answer 4

我实际上喜欢@andersonvom的答案，它只需要循环一次数组并仍然获得索引。

如果您不想使用ary.each_with_index.min，请执行以下操作：

ary = [2,3,4,5,1]                                             # => [2,3,4,5,1]
_, index_of_minimal_value_in_array = ary.each_with_index.min  # => [1, 4]
index_of_minimal_value_in_array                               # => 4

Ruby：如何找到最小数组元素的索引？

4 个答案: