ruby Rails API中的链接.where方法

nvbavucw 于 5个月前发布在 Ruby

关注(0)|答案(1)|浏览(65)

我有一个用Rails API构建的Web应用程序来收集提交。我有一个正在提交的表单，它会向具有匹配属性（位置，大小等）的某些商店发送电子邮件。它们基于以下字段进行匹配

位置
容量（最小和最大）
商店类型（酒店，餐饮，独特）

我也有一个属性的商店接收所有线索，我想覆盖任何前面的选项，如果设置为真。
在我的Rails API中的hard_worker.rb中，我让它检查以下字段，以查询将Lead发送到哪些商店。我知道条件的链接不正确，但无法弄清楚如何重新格式化以正确发送。任何帮助都将非常感谢，谢谢！

def build_filters_obj
        filters = []
        filters.push 'location_north' if @lead.location_north
        filters.push 'location_east' if @lead.location_east
        filters.push 'location_south' if @lead.location_south
        filters.push 'location_west' if @lead.location_west
        filters.push 'location_other' if @lead.location_other
        
        return filters
    end

    def perform(lead_id)
        @lead = Lead.find(lead_id)

        lead_email = ValidEmail2::Address.new(@lead.email)
        UserNotifierMailer.send_signup_email(@lead).deliver if lead_email.valid?

        @stores = Store.all
        @stores = @stores.where.not(email: [nil, ''])

        n = @lead.guests_total.to_i
        @stores = @stores.where("capacity_min <= ? AND capacity_max >= ?", n, n)
        
        @stores = @stores.where(:type_unique => true) if @lead.store_type_unique
        @stores = @stores.where(:type_dining => true) if @lead.store_type_dining
        @stores = @stores.where(:type_hotel => true) if @lead.store_type_hotel
        
        filters = build_filters_obj
        filters.each do |filter|
            @stores = @stores.where(filter.to_sym => true)
        end

        @stores = @stores.or(Store.where(:receive_all => true))

        @stores.each do |store|
            store_email = ValidEmail2::Address.new(store.email)
            UserNotifierMailer.send_lead_email(store, @lead).deliver if store_email.valid?
        end
    end

字符串
如果我的代码很糟糕，我道歉，Ruby不是我通常工作的地方，我肯定我犯了一些初学者的错误！

ruby

来源：https://stackoverflow.com/questions/77603041/chaining-where-method-in-rails-api

1条答案

按热度按时间

xj3cbfub1#

我注意到的最大的代码气味是有很多if检查，这使得你的代码更难推理。试着创建哈希并调用方法来清理哈希。
我还建议你通过rubocop来传递这些代码，这会让你的代码看起来更地道，错误更少。
1.构建过滤器对象

filters.push 'location_north' if @lead.location_north
filters.push 'location_east' if @lead.location_east
filters.push 'location_south' if @lead.location_south
filters.push 'location_west' if @lead.location_west
filters.push 'location_other' if @lead.location_other

字符串
也许我们至少可以让它准备好使用。在ruby习惯的命名方式中，这将是一个 * 名词 *，并直接使用它。注意，我们没有创建字符串，也没有在以后将它们转换为符号。

def location_filters
  {
    location_north: @lead.location_north,
    location_east: @lead.location_east,
    location_south: @lead.location_south,
    location_west: @lead.location_west,
    location_other: @lead.location_other
  }.filter { |key, value| value.present? }
end

# later on you would use this like this

@stores = @stores.where(location_filters)

型

if检查store_type* 相关方法
您可以构建一个哈希值，然后删除虚假值

type_filters = {
  type_unique: @lead.store_type_unique
  type_dining: @lead.store_type_dining
  type_hotel: @lead.store_type_hotel
}.filter { |key, value| value.present? }

# later on we can directly pass it to where method

@stores = @stores.where(type_filters)

型

因此，总体而言，这是您可以调用查询的方式

def perform(lead_id)
    @lead = Lead.find(lead_id)

    lead_email = ValidEmail2::Address.new(@lead.email)
    UserNotifierMailer.send_signup_email(@lead).deliver if lead_email.valid?

    capacity = @lead.guests_total.to_i

    type_filters = { # consider moving to a private method called `type_filters`
      type_unique: @lead.store_type_unique
      type_dining: @lead.store_type_dining
      type_hotel: @lead.store_type_hotel
    }.filter { |key, value| value.present? }
      
    @stores = Store.where.not(email: [nil, ''])
                   .where("capacity_min <= ? AND capacity_max >= ?", capacity, capacity)
                   .where(type_filters)
                   .where(location_filters)
                   .or(Store.where(receive_all: true))

    @stores.each do |store|
        store_email = ValidEmail2::Address.new(store.email)
        UserNotifierMailer.send_lead_email(store, @lead).deliver if store_email.valid?
    end
end

型
https://rubyapi.org/3.2/o/hash#method-i-filter

赞(0）回复(0）举报 5个月前

我来回答

ruby Rails API中的链接.where方法

1条答案

因此，总体而言，这是您可以调用查询的方式

相关问题

热门标签

最新问答