Rails ActiveRecord: Buscar todos los usuarios excepto el usuario actual

Question

Creo que esto debería ser muy simple, pero mi cerebro está cortocircuitando. Si tengo un objeto que representa al usuario actual y deseo consultar para todos los usuarios, excepto el usuario actual, ¿cómo puedo hacer esto, teniendo en cuenta que el usuario actual a veces puede ser nil?

Esto es lo que estoy haciendo ahora:

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

Lo que no me gusta es que estoy haciendo post-procesamiento en el resultado de la consulta. Además de sentirme un poco mal, no creo que esto funcione bien si convierto la consulta para que se ejecute con will_paginate. ¿Alguna sugerencia de cómo hacer esto con una consulta? Gracias.

Answer 1

Es posible hacer lo siguiente en los carriles 4:

User.where.not(id: id) 

se puede envolver en un ámbito agradable.

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

O utilice un método de clase, si lo prefiere:

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

Ambos métodos devolver un objeto relación con AR. Esto significa que puede método de la cadena de llamadas:

@users = User.all_except(current_user).paginate 

puede excluir cualquier número de usuarios, ya where() también acepta una matriz.

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

Por ejemplo:

@users = User.all_except(User.unverified) 

E incluso a través de otras asociaciones:

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

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

Ver where.not() en el API Docs.

Answer 2

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

Answer 3

Lo que está haciendo es eliminar el usuario_actual de la matriz @users. Esto no funcionará ya que no hay un método de eliminación para las matrices. Lo que es probable que desee hacer es esto

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

Esto devolverá una copia de la matriz @users, pero con el objeto current_user eliminado (que estaba contenida en la matriz en el primer lugar.

Nota : Esto puede no funcionar si la resta de matriz se basa en coincidencias exactas de objetos y no en el contenido. Pero funcionó con cadenas cuando lo probé. Recuerde incluir current_user en [] para forzarlo a una matriz.

Answer 4

Aquí está una versión más corta:

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

Answer 5

También puede crear named_scope, p. Ej. en su modelo:

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

y en el controlador:

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

Este alcance devolverá todos los usuarios cuando se le llama con nula y todos los usuarios excepto dada en parámetro en otro caso. La ventaja de esta solución es que puede encadenar esta llamada con otros ámbitos nombrados o con el método will_paginate de paginate.

Answer 6

Una nota sobre la respuesta de GhandaL - por lo menos en los carriles 3, vale la pena modificar a

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

(el cambio primordial en este caso es de a 'users.id = 'id = ...!'!. .. '; también ámbito en lugar de named_scope para Rails 3)

La versión original funciona bien cuando simplemente se analiza la tabla Usuarios. Al aplicar el alcance a una asociación (por ejemplo, team.members.without_user (current_user) ....), este cambio fue necesario para aclarar qué tabla estamos usando para la comparación de id. Vi un error de SQL (usando SQLite) sin él.

Disculpas por la respuesta separada ... todavía no tengo la reputación de comentar directamente sobre la respuesta de GhandaL.

Answer 7

solución muy sencilla que utiliza

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

Answer 8

Otra manera fácil que podía hacerlo:

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

Answer 9

User.all.where ("? Id NO EN()", current_user.id) voluntad a través excepción método no definido donde por #<Array:0x0000000aef08f8>

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

Answer 10

una matriz sería más ayuda ful

arrayID [0] = 1

arrayID [1] = 3

User.where.not (id: arrayID)