Как достать данные при связи has_many through?

Question

[brikstar]

Имеются модели

class Category < ActiveRecord::Base
  has_many :products
end

class Product < ActiveRecord::Base
  belongs_to :category
  has_many :product_storages
  has_many :storages, through: :product_storages
end

class Storage < ActiveRecord::Base
  has_many :product_storages
  has_many :products, through: :product_storages
end

class ProductStorage < ActiveRecord::Base
  belongs_to :product
  belongs_to :storage
end

В связующей таблице данные storage_id, product_id, amount, т.е. количество товара на каждом складе, при чем один и тот же товар может находиться на разных складах.

Вопрос в следующем, как правильно получить такие данные не прибегая к find_by_sql, т.е. средствами ActiveRecord основанных на связях?
Может /надо что-то еще проделать с моделями или я что-то не так понимаю, или буду рад информации где почитать с примерами про сложные связи между таблицами в RoR ?

select products.id, products.name,product_storages.amount,product_storages.storage_id  from products 
inner join product_storages on product_storages.product_id = products.id 
inner join categories on categories.id = products.category_id 
where product_storages.storage_id=1 and amount >0 and categories.id =1

select products.id, products.name, product_storages.amount,product_storages.storage_id  from products 
inner join product_storages on product_storages.product_id = products.id 
where product_storages.storage_id = 1 and amount >0

select products.id, products.name,SUM( product_storages.amount),product_storages.storage_id  from products 
inner join product_storages on product_storages.product_id = products.id 
where product_storages.storage_id <>1 and amount >0 
group by products.id

Answer 1

[vsuhachev]

Product.joins(:product_storages, :categories).with_nonzero_amount.where(
  product_storages: { storage_id: 1 }, categories: { id: 1}
)

И определить скоп with_nonzero_amount в классе Product

scope :with_nonzero_amount, -> where('amount > 0') }

Comments

[vsuhachev]

Ну и вообще умные люди рекомендуют все выносить в скопы. Тогда ваш запрос мог бы выглядеть как-то так:

Product.at_storage(1).with_category(1).with_nonzero_amount

[brikstar]

а смысл тогда описывать связи в таблице, если так делать? Можно в контроллере @storages = Storage.all
во вьювере
Список товаров со всех складов с остатками

<% @storages.each do |storage| %>
<%= storage.name %>

<% storage.products.select("*").all.each do |pr| %>
<%= pr.name %>
<%= pr.amount %>

<% end %>
но, если я все правильно понял, то появляется проблема n+1 запроса

[vsuhachev]

Описание связей нужно для валидации и сохранения объекта.

Сложные запросы делаются напрямую через SQL, простые - примерно так как я описал. В вашем примере проблема n+1 запроса есть, но вообще-то вы не описали что вам нужно получить на выходе. Если товары, то лучше через Product. Если что-то другое, то возможно по другому лучше сделать.

В принципе связи вы описали нормально.

[brikstar]

Получить нужно те данные, которые получаем при sql запросах, описаные ваше

[vsuhachev]

ActiveRecord инстанцирует объекты того класса к которому вы обращаетесь, и по другому никак.

Если вам нужно получить солянку из нескольких таблиц и агрегатные функции - используйте SQL и получайте двумерный массив.

Но например для первого запроса - если вы выберите товары, а потом для каждого товара будете вызывать:

product.product_storages.amount
product.product_storages.storage_id

то "n+1" не будет