我有一个搜索页面,用户可以根据标签搜索事件。
我有以下方法,该方法根据传递给该方法的标签 id 返回事件列表:
def self.tagged_with_id(taglist)
# convert taglist to an array if ids
taglist = taglist.chomp.split(',').map { |x| x.to_i }
#get tag name from each id
normalizedlist = taglist.collect{|id| Tag.find(id).name}
Event.includes([:tags]).where("tags.name IN (#{normalizedlist.map{|tagname| "'#{tagname}'"}.join(',')})")
end
。这是显示事件的视图...
<%- @events.each do |event| -%>
...
<td><div class="tag-wrapper"><%= display_tags(event.tags) %></div></td>
...
就像一个魅力,并且完全按照我需要它做的事情(大部分)。但是,我遇到的一个问题是在显示event.tags
时视图中,仅显示搜索的标签。我希望每个事件的所有标签都显示在返回的搜索结果中。
只是为了上下文,如果我使用 tagged_with_id 方法查询事件,然后查看标签,我得到这个......
2.1.5 :007 > e=Event.tagged_with_id('44').first
2.1.5 :008 > e.tags
=> [#<Tag id: 44, name: "winning"]
这是使用轨道查找方法的相同事件...
2.1.5 :009 > e=Event.find 80
2.1.5 :010 > e.tags
=> [#<Tag id: 44, name: "winning">, #<Tag id: 91, name: "air">, #<Tag id: 114, name: "management">
我希望我的tagged_with_id方法返回包含所有标签的事件,而不仅仅是搜索的标签。
我将提供一个解决方案,但首先,让我们清理一下您的方法。首先,您一次查找一个标签,让我们在一个查询中找到它们:
normalizedlist = Tag.where(id: taglist).pluck :name
这做同样的事情,但都在一个查询中,速度要快得多。
然后让我们清理最后一个查询:
Event.includes(:tags).where tags: { name: normalizedlist }
这也与您之前的功能相同,但它使用更干净的哈希语法来匹配关系条件,您也不需要join
列表。ActiveRecord 知道,当它获取数组作为值时,您要执行IN
。
,由于最后一个查询仅加载具有选定标签的事件,因此您需要重新加载event
对象,以便它从数据库加载其所有tags
:
display_tags(event.reload.tags)
这将加载该事件的所有标记。