代码之家 › 专栏 › 技术社区 › Axil

Ruby on Rails 4 ActudieCordR-结合和独特的连接查询优化

activerecord join ruby-on-rails-4 ruby-on-rails

Axil · 技术社区 · 6 年前

我想弄清楚这个ActuvReCordD查询是好的还是应该有更好的优化呢?

我需要合并3个查询并确保结果是唯一的。我正在使用.uniq

ads1 = Advertisement.where(city: property.city, state: property.state, country_id: property.address_country_id)
ads2 = Advertisement.where(city: nil, state: property.state, country_id: property.address_country_id)
ads3 = Advertisement.where(city: nil, state: nil, country_id: property.address_country_id)

combine_ads = ads1 + ads2 + ads3
uniq_ads = combine_ads.uniq { |ads| ads.id}
uniq_ads = uniq_ads.sort_by { |ads| ads.created_at}
final_ads = uniq_ads.paginate(:page => params[:page], :per_page => params[:page_limit])
status 200
present final_ads, with: Api::Presenters::AdvertisementDetail

我正在使用RubyonRails4

gem 'rails', '4.2.4'

你能回顾一下以上结合了惟一连接的activerecords吗?如果它是好的,或者有更好的优化方法可以这样做,你可以提供反馈吗?

1 回复 | 直到 6 年前

kiddorails 6 年前

你应该改变 combine_ads 致:

combine_ads = Advertisement.where(city: [property.city, nil], 
                                  state: [property.state, nil], 
                                  country_id: property.address_country_id)
                           .order('created_at asc')

上面的唯一额外情况是,它也有那些行,其中 city 是 property.city ,但是 state 是 nil ,这是你的故障没有记录的。

你不需要 uniq_ads 或者,我们已经订购了 created_at .

推荐文章

oneWorkingHeadphone · 检查上次保存的忽略对象的项目。新

6 年前

JBlake · 唯一验证基于datetime属性的存在,而不是值?

6 年前

Serge · Yii-查找具有零相关记录的记录

6 年前

WhoaItsAFactorial · 在创建复合唯一索引时提供文本列长度

6 年前

Eduardo Álvarez · RubyonRails 5随机名称错误:控制台中命名空间模型上的未初始化常量

6 年前

waynard · Rails 5左外连接与WHERE查询相结合

6 年前

David Hempy · Rails SQL跨多个列“select in”:其中(code1,code2)in((“A”,1),(“A”,3),(“Q”,9))

6 年前

Quentin · Rails ActiveRecord:在加入后更改用于集合的ruby类

6 年前

OKMantis · Ruby on Rails中的交叉引用表

6 年前

Grey · 访问不同子控制器中的不同关系

6 年前