Я пытаюсь показать только те экземпляры бренда, которые текущий пользователь не пометил, даже если другие пользователи уже пометили тот же бренд. Что-то вроде:

enter image description here

Контроллер

Это мой код контроллера, и хотя он должен работать, в настоящее время он возвращает все экземпляры бренда.

@brand = current_user.brands.includes(:taggings).where( [ "taggings.id IS NULL OR taggings.tagger_id != ?", current_user.id ] ).order("RANDOM()").first

Схема (включая мою модель соединения для хорошей меры)

create_table "brand_users", :force => true do |t|
t.integer  "brand_id"
t.integer  "user_id"
t.datetime "created_at"
t.datetime "updated_at"
end

create_table "taggings", :force => true do |t|
t.integer  "tag_id"
t.integer  "taggable_id"
t.string   "taggable_type"
t.integer  "tagger_id"
t.string   "tagger_type"
t.string   "context"
t.datetime "created_at"
end

add_index "taggings", ["tag_id"], :name => "index_taggings_on_tag_id"
add_index "taggings", ["taggable_id", "taggable_type", "context"], :name => "index_taggings_on_taggable_id_and_taggable_type_and_context"

create_table "tags", :force => true do |t|
t.string "name"
end

end
1
Simpleton 29 Авг 2011 в 22:16

2 ответа

Лучший ответ

Итак, если вы используете драгоценный камень plays-as-taggable-on и имеете следующие модели:

class User < ActiveRecord::Base
  acts_as_tagger
  has_many :brand_users
  has_many :brands, :through => :brand_users
end

Итак, у вас также есть таблицы в вашей схеме, например:

create_table "users", :force => true  do |t|
  t.string "name"
end

create_table "brands", :force => true  do |t|
  t.string "name"
end

Тогда следующий SQL-запрос, надеюсь, должен сделать то, что вы хотите (?):

SELECT brands.*
FROM brands
WHERE brands.id NOT IN (
    SELECT brands.id
    FROM brands
    INNER JOIN brand_users ON brand_users.brand_id = brands.id
    INNER JOIN taggings ON (taggings.tagger_id = brand_users.user_id AND taggings.tagger_type = 'User')
    WHERE brand_users.user_id = 1 AND taggings.taggable_id = brand_users.brand_id
)

Чтобы перевести это в Rails ORM, я не могу приблизиться без жесткого кодирования всей строки SQL sub-select, что-то вроде:

class Brand < ActiveRecord::Base
  has_many :brand_users
  has_many :users, :through => :brand_users

  scope :has_not_been_tagged_by_user, lambda {|user| where("brands.id NOT IN (SELECT brands.id
    FROM brands
    INNER JOIN brand_users ON brand_users.brand_id = brands.id
    INNER JOIN taggings ON (taggings.tagger_id = brand_users.user_id AND taggings.tagger_type = 'User')
    WHERE brand_users.user_id = ? AND taggings.taggable_id = brand_users.brand_id)", user.id) }

end

(Я знаю, что вы могли бы сделать это, а затем использовать рубиновый .map (&: id) .join (',') < / a> но если это большое приложение, я думаю, вы сильно потеряете производительность, если выньте его из базы данных, преобразуя его в строку целых чисел и вернув обратно (насколько я понимаю).

Затем в вашем контроллере я думаю вы бы сделали что-то вроде:

@brand = current_user.brands.has_not_been_tagged_by_user(current_user)

В стороне, я думаю, что тогда на самом деле будет выполнен SQL, как показано ниже (верно?):

SELECT brands.*
FROM users
INNER JOIN brand_users ON brand_users.user_id = users.id
INNER JOIN brands ON brands.id = brand_users.brand_id 
WHERE brands.id NOT IN (
    SELECT brands.id
    FROM brands
    INNER JOIN brand_users ON brand_users.brand_id = brands.id
    INNER JOIN taggings ON (taggings.tagger_id = brand_users.user_id AND taggings.tagger_type = 'User')
    WHERE brand_users.user_id = 1 AND taggings.taggable_id = brand_users.brand_id
) AND users.id = 1
2
Community 23 Май 2017 в 12:26