ruby - 如何改进这个 ruby​​ 代码更地道？

Question

因为下面的代码看起来非常陈旧和丑陋。但是在以这种方式玩耍之后才有效。

def duplicated? url
  found = false
  current_user.bookmarks.each do |bookmark|
    if bookmark.url.eql? url
      found = true
    end
  end
  found
end

我删除了找到的本地var和这样的if语句

def duplicated? url
  current_user.bookmarks.each do |bookmark|
    bookmark.url.eql? url
  end
end

它返回错误的预期结果。

知道如何改进吗？谢谢

Answer 1

我想你需要Enumerable#any?这里。它应该优先于 map/include 对，因为它不会创建临时数组，并且会在找到第一个元素时提前返回。

def duplicated? url
  current_user.bookmarks.any?{|b| b.url.eql? url }
end