Rails ActiveRecord : Trouver tous les utilisateurs sauf l'utilisateur actuel

Question

J'ai l'impression que cela devrait être très simple mais mon cerveau est en train de court-circuiter sur ce point. Si j'ai un objet représentant l'utilisateur actuel, et que je veux interroger tous les utilisateurs sauf l'utilisateur actuel, comment puis-je le faire, en tenant compte du fait que l'utilisateur actuel peut parfois être nil ?

C'est ce que je fais en ce moment :

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

Ce que je n'aime pas, c'est que je fais du post-traitement sur le résultat de la requête. En plus de me sentir un peu mal, je ne pense pas que cela fonctionnera bien si je convertis la requête pour être exécutée avec will_paginate . Avez-vous des suggestions sur la façon de faire cela avec une requête ? Merci.

Answer 1

Il est possible de faire ce qui suit dans Rails 4 et plus :

User.where.not(id: id)

Vous pouvez l'emballer dans un joli scope.

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

Ou utilisez une méthode de classe si vous préférez :

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

Les deux méthodes renverront un objet de relation AR. Cela signifie que vous pouvez enchaîner les appels de méthodes :

@users = User.all_except(current_user).paginate

Vous pouvez exclure un nombre quelconque d'utilisateurs car where() accepte également un tableau.

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

Par exemple :

@users = User.all_except(User.unverified)

Et même par le biais d'autres associations :

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

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

Voir where.not() dans le Docs API .

Answer 2

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

Answer 3

Vous pouvez également créer named_scope, par exemple dans votre modèle :

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

et en contrôleur :

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

Cette portée retournera tous les utilisateurs si elle est appelée avec nil et tous les utilisateurs sauf ceux donnés en paramètre dans les autres cas. L'avantage de cette solution est que vous êtes libre de chaîner cet appel avec d'autres scopes nommés ou la méthode will_paginate paginate.

Answer 4

Voici une version plus courte :

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

Answer 5

Une remarque sur la réponse de GhandaL - au moins dans Rails 3, il vaut la peine de modifier en

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

(le changement principal ici est de passer de 'id != ...' à 'users.id !=...' ; également scope au lieu de named_scope pour Rails 3)

La version originale fonctionne bien lorsqu'on se contente de délimiter la table Utilisateurs. Lorsque l'on applique la portée à une association (par exemple team.members.without_user(current_user).... ), cette modification était nécessaire pour clarifier quelle table nous utilisons pour la comparaison des identifiants. J'ai vu une erreur SQL (en utilisant SQLite) sans cela.

Je m'excuse pour la réponse séparée... je n'ai pas encore la réputation de commenter directement la réponse de GhandaL.