Rails ActiveRecord: Найти всех пользователей, кроме текущего пользователя

Question

Я чувствую, что это должно быть очень просто, но мой мозг замыкается на нем. Если у меня есть объект, представляющий текущего пользователя, и вы хотите запросить всех пользователей, кроме текущего пользователя, как я могу это сделать, учитывая, что текущий пользователь может иногда быть nil?

Это то, что я делаю прямо сейчас:

def index 
    @users = User.all 
    @users.delete current_user 
end 

Что мне не нравится то, что я делаю пост-обработку результата запроса. Кроме того, чувствуя себя немного не так, я не думаю, что это сработает хорошо, если я конвертирую запрос на запуск с will_paginate. Любые предложения по тому, как это сделать с запросом? Благодарю.

Answer 1

можно сделать следующее в Rails 4:

User.where.not(id: id) 

Вы можете обернуть его в хорошем объеме.

scope :all_except, ->(user) { where.not(id: user) } 
@users = User.all_except(current_user) 

Или использовать метод класса, если вы предпочитаете:

def self.all_except(user) 
    where.not(id: user) 
end 

Оба метода возвращают объект AR отношение. Это означает, что вы можете цепной метод вызывает:

@users = User.all_except(current_user).paginate 

Вы можете исключить любое количество пользователей, поскольку where() также принимает массив.

@users = User.all_except([1,2,3]) 

Например:

@users = User.all_except(User.unverified) 

И даже через других ассоциаций:

class Post < ActiveRecord::Base 
    has_many :comments 
    has_many :commenters, -> { uniq }, through: :comments 
end 

@commenters = @post.commenters.all_except(@post.author) 

Смотрите в API Docswhere.not().

Answer 2

@users = (current_user.blank? ? User.all : User.find(:all, :conditions => ["id != ?", current_user.id])) 

Answer 3

Что вы делаете, это удаление current_user из массива @users. Это не будет работать, поскольку для массивов не существует метода удаления. То, что вы, вероятно, хотите сделать это

def index 
    @users = User.all 
    @users - [current_user] 
end 

Это возвращает копию массива @users, но со снятым объектом current_user (он содержится в массиве в первую очередь.

Примечание : Это может не сработать, если вычитание массива основано на точном совпадении объектов, а не с содержимым, но оно работает со строками, когда я его пробовал. Не забудьте заключить current_user в [], чтобы вставить его в массив.

Answer 4

более короткий вариант:

User.all :conditions => (current_user ? ["id != ?", current_user.id] : []) 

Answer 5

Вы также можете создать named_scope, например. в модели:

named_scope :without_user, lambda{|user| user ? {:conditions => ["id != ?", user.id]} : {} } 

и в контроллере:

def index 
    @users = User.without_user(current_user).paginate 
end 

Эта область будет возвращать все пользователи при вызове с нуля и всех пользователей, кроме приведены в парам в другом случае. Преимущество этого решения заключается в том, что вы можете связать этот вызов с другими именованными областями или с помощью метода_пагинации paginate.

Answer 6

Одно примечание на ответ GhandaL - по крайней мере, в Rails 3, это стоит изменения в

scope :without_user, lambda{|user| user ? {:conditions => ["users.id != ?", user.id]} : {} } 

(первичного изменения здесь от «users.id = 'ид = ...!'!. .. '; также область вместо named_scope для Rails 3)

Оригинальная версия отлично работает при простом просмотре таблицы Users. При применении области действия к ассоциации (например, team.members.without_user (current_user) ....) это изменение потребовалось, чтобы уточнить, какую таблицу мы используем для сравнения id. Я видел ошибку SQL (используя SQLite) без нее.

Извинения за отдельный ответ ... У меня еще нет репутации, чтобы прокомментировать непосредственно ответ GhandaL.

Answer 7

Очень простое решение я использовал

@users = User.all.where("id != ?", current_user.id) 

Answer 8

Другой простой способ, которым Вы могли бы сделать это:

@users = User.all.where("id NOT IN(?)", current_user.id) 

Answer 9

User.all.where ("? ID NOT IN()", current_user.id) будет через исключение неопределенный метод, где для #<Array:0x0000000aef08f8>

User.where("id NOT IN (?)", current_user.id) 

Answer 10

массив будет больше помощи FUL

arrayID [0] = 1

arrayID [1] = 3

User.where.not (ID: arrayID)