条件に合った商品のみ一覧表示したい

Question

前提・実現したいこと

大学の教科書販売サイトを作成しています。ユーザーがログインした時にそのユーザーのuniversity_idと出品者のuniversity_idが合致する商品のみ、商品一覧で表示したいです。

発生している問題

itemコントローラーのindexアクションの記述方法が分からない状態です。

該当のソースコード

app > controllers > item_controller.rb

class ItemsController < ApplicationController
  before_action :authenticate_user!, only: [:new, :create]

  def index
    @items = Item.order('created_at DESC')
  end

  def new
    @item = Item.new
  end

  def create
    @item = Item.new(item_params)
    if @item.save
      redirect_to root_path
    else
      render :new
    end
  end

  private
  def item_params
    params.require(:item).permit(:name, :price, :year_period_id, :lecture, :teacher, :sales_status_id, :scheduled_delivery_id, :author, :publisher, :image).merge(user_id: current_user.id)
  end
end

app > models >user.rb

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to :university
  belongs_to :department
  has_many :items

  with_options presence: true do
    validates :nickname
    validates :birthday

    with_options numericality: { other_than: 1, message: "を選択してください" } do
      validates :university_id
      validates :department_id
    end
  end
  
  PASSWORD_REGEX = /\A(?=.*[a-zA-Z])(?=.*?\d)[a-zA-Z\d]+\z/i.freeze
  validates_format_of :password, with: PASSWORD_REGEX, message: 'は半角英字と半角数字の両方を含めて設定してください'
end

app > models >item.rb

class Item < ApplicationRecord
  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to :year_period
  belongs_to :sales_status
  belongs_to :scheduled_delivery
  belongs_to :user
  has_one_attached :image

  with_options presence: true do
    validates :image
    validates :name

    with_options numericality: { greater_than: 0,allow_blank: true }, format: { with: /\A[0-9]+\z/,allow_blank: true } do
      validates :price
    end

    with_options format: { with: /\A[ぁ-んァ-ン一-龥０-９]+\z/, allow_blank: true } do
      validates :lecture
      validates :teacher
      validates :author
      validates :publisher
    end

    with_options numericality: { other_than: 1, message: "を選択してください"} do
      validates :year_period_id
      validates :sales_status_id
      validates :scheduled_delivery_id
    end
  end
end

db > migrate > 20210708035000_devise_create_users.rb

# frozen_string_literal: true

class DeviseCreateUsers < ActiveRecord::Migration[6.0]
  def change
    create_table :users do |t|
      ## Database authenticatable
      t.string :nickname, null: false
      t.string :email,              null: false, default: ""
      t.string :encrypted_password, null: false, default: ""
      t.integer :university_id, null: false
      t.integer :department_id, null: false
      t.date :birthday, null: false

省略

db > migrate > 20210713000013_create_items.rb

class CreateItems < ActiveRecord::Migration[6.0]
  def change
    create_table :items do |t|
      t.string :name, null: false
      t.integer :price, null: false
      t.integer :year_period_id, null: false
      t.string :lecture, null: false
      t.string :teacher, null: false
      t.integer :sales_status_id, null: false
      t.integer :scheduled_delivery_id, null: false
      t.string :author, null: false
      t.string :publisher, null: false
      t.references :user, null: false, foreign_key: true
      t.timestamps
    end
  end
end

試したこと

itemコントローラーのindexアクションにwhereを使用したのですが、itemテーブルにuniversity_idがない場合の書き方が分からずにいます。

Answer 1

孫要素の取得は

items = university.users.map{ |user| user.items}.flatten
# または
items = University.find(id).users.map{ |user| user.items}.flatten

みたいな感じでとれます。

ただこのままだと university.users の数分 query が走ってしまうので (N+1問題)

users = university.users.includes(:items)
items = users.map { |user| user.items}.flatten

こんな感じでとれると 2回のクエリでとれます。

(university モデルに has_many :users がある前提です)