在以下情况下如何改进代码以删除数组中的项目？

Question

我有这样的代码：

array = ['notice', 'warning', 'error']

array.delete('notice')  if flash[:notice]
array.delete('warning') if flash[:warning]
array.delete('error')   if flash[:error]

由于有重复的名称，为了缩短代码，我可以使用插值来执行这部分代码：

array.delete('notice')  if flash[:notice]
array.delete('warning') if flash[:warning]
array.delete('error')   if flash[:error]

如何一步完成？< /strong>

---

我尝试过这个

array.each { |item|
  array.delete("#{item}") if flash[:"#{item}"]
}

，但效果不佳。

Answer 1

array.reject! { |item| flash[item.to_sym] }

Answer 2

发生这种情况是因为您正在修改要迭代的数组。
像这样的东西应该可以工作

array.clone.each { |item|
  array.delete("#{item}") if flash[:"#{item}"]
}

尝试在示例输入上使用和不使用clone来运行它

array = ['notice', 'warning', 'error']
flash = {:warning => 1, :error => 2}

...

p array

但是不需要手动调用delete，您只需用户拒绝：

array = array.reject! { |item| flash[:"#{item}"] }

Answer 3

不需要使用字符串插值，只需使用 .to_sym 将字符串转换为符号即可：

array = ['notice', 'warning', 'error']
array.each { |item|
    array.delete(item) if flash[item.to_sym]
}

或者使用 array 中的符号到 .to_s 中删除：

array = [:notice, :warning, :error]
array.each { |item|
    array.delete(item.to_s) if flash[item]
}