我有一个地图,可以更改值或将其设置为零。然后我想从列表中删除nil条目。该列表不需要保留。
这就是我目前拥有的:
items.map! { |x| process_x url } # [1, 2, 3, 4, 5] => [1, nil, 3, nil, nil]
items.select! { |x| !x.nil? } # [1, nil, 3, nil, nil] => [1, 3]
我知道我可以做一个循环并有条件地收集另一个数组,如下所示:
new_items = []
items.each do |x|
x = process_x x
new_items.append(x) unless x.nil?
end
items = new_items
但它似乎不是红宝石般的。有没有一种很好的方法可以在列表中运行一个函数来删除/排除nils?
你可以使用compact
:
[1, nil, 3, nil, nil].compact
=> [1, 3]
我想提醒大家的是,如果你得到一个包含nils的数组作为map
块的输出,并且该块试图有条件地返回值,那么你有代码味道并需要重新考虑你的逻辑。
例如,如果您正在执行以下操作:
[1,2,3].map{ |i|
if i % 2 == 0
i
end
}
# => [nil, 2, nil]
然后不要。相反,在map
之前,reject
你不想要的东西或select
你想要的东西:
[1,2,3].select{ |i| i % 2 == 0 }.map{ |i|
i
}
# => [2]
我考虑使用compact
来清理乱七八糟的东西作为摆脱我们没有正确处理的事情的最后努力,通常是因为我们不知道会发生什么。我们应该始终知道在我们的程序中抛出了什么类型的数据;意外/未知数据很糟糕。无论何时我在我正在研究的数组中看到nils,我都会深入了解它们存在的原因,看看我是否可以改进生成数组的代码,而不是让Ruby浪费时间和内存生成nils然后筛选数组以删除他们以后。
'Just my $%0.2f.' % [2.to_f/100]
尝试使用#reduce
或#inject
!
[1, 2, 3].reduce([]) { |memo, i|
if i % 2 == 0
memo << i
end
memo
}
我同意接受的答案,我们不应该映射和压缩,但不是出于同样的原因!
我深深感觉到地图 - 然后 - 紧凑等同于select-then-map。考虑:地图是一对一的功能。如果要从某些值集映射并映射,则输出集中的每个值的输出集中都需要一个值。如果您必须事先选择,那么您可能不希望在集合上有地图。如果你必须事后选择(或紧凑),那么你可能不希望在集合上有地图。在任何一种情况下,你都需要在整个集合上进行两次迭代,而reduce只需要进行一次。
另外,在英语中,您试图“将一组整数减少为一组偶数整数”。
在你的例子中:
items.map! { |x| process_x url } # [1, 2, 3, 4, 5] => [1, nil, 3, nil, nil]
除了被nil
取代之外,它看起来并没有改变。如果是这样的话,那么:
items.select{|x| process_x url}
就足够了。
如果你想要一个更宽松的拒绝标准,例如,拒绝空字符串以及nil,你可以使用:
[1, nil, 3, 0, ''].reject(&:blank?)
=> [1, 3, 0]
如果您想进一步拒绝零值(或将更复杂的逻辑应用于流程),您可以传递一个块来拒绝:
[1, nil, 3, 0, ''].reject do |value| value.blank? || value==0 end
=> [1, 3]
[1, nil, 3, 0, '', 1000].reject do |value| value.blank? || value==0 || value>10 end
=> [1, 3]
@the Tin Man,很好 - 我不知道这个方法。嗯,绝对紧凑是最好的方法,但仍然可以通过简单的减法完成:
[1, nil, 3, nil, nil] - [nil]
=> [1, 3]
each_with_object
可能是最干净的方式:
new_items = items.each_with_object([]) do |x, memo|
ret = process_x(x)
memo << ret unless ret.nil?
end
在我看来,each_with_object
在条件情况下比inject
/ reduce
更好,因为你不必担心块的返回值。
完成它的另一种方法如下所示。在这里,我们使用Enumerable#each_with_object
来收集值,并利用Object#tap
来摆脱nil
检查process_x
方法结果所需的临时变量。
items.each_with_object([]) {|x, obj| (process x).tap {|r| obj << r unless r.nil?}}
完整的插图示例:
items = [1,2,3,4,5]
def process x
rand(10) > 5 ? nil : x
end
items.each_with_object([]) {|x, obj| (process x).tap {|r| obj << r unless r.nil?}}
替代方法:
通过查看您调用process_x url
的方法,目前尚不清楚在该方法中输入x
的目的是什么。如果我假设你要通过传递一些x
来处理url
的值并确定哪个x
s真正被处理成有效的非零结果 - 那么,可能是Enumerabble.group_by
是比Enumerable#map
更好的选择。
h = items.group_by {|x| (process x).nil? ? "Bad" : "Good"}
#=> {"Bad"=>[1, 2], "Good"=>[3, 4, 5]}
h["Good"]
#=> [3,4,5]