Vérifier si l'enregistrement existe de contrôleur dans les Rails

Dans mon application, un Utilisateur peut créer une Entreprise. Quand ils déclenchent le index dans mes BusinessesController je veux vérifier si une Entreprise est liée à la current_user.id:

Si oui: affichage de l'entreprise.
Si non: rediriger vers la new action.

J'ai essayé d'utiliser ce:

if Business.where(:user_id => current_user.id) == nil
  # no business found
end

Mais il renvoie toujours vrai même lorsque l'entreprise n'existe pas...

Comment puis-je tester si il existe un enregistrement dans ma base de données?

À l'aide de where retourne un tableau vide si il n'y a pas d'enregistrements. Et [] n'est pas égal à nil
Qu'en est juste un unless Business.find_by_user_id(current_user.id)?
double possible de Vérifier si ActiveRecord trouver renvoie un résultat

InformationsquelleAutor | 2013-05-22

205

Pourquoi ton code ne fonctionne pas?

La where méthode retourne un ActiveRecord::Relation objet (agit comme un tableau qui contient les résultats de la where), il peut être vide, mais il ne sera jamais nil.
```
Business.where(id: -1) 
 #=> returns an empty ActiveRecord::Relation ( similar to an array )
Business.where(id: -1).nil? # ( similar to == nil? )
 #=> returns false
Business.where(id: -1).empty? # test if the array is empty ( similar to .blank? )
 #=> returns true
```
Comment tester si au moins un enregistrement existe?

Option 1: À L'Aide De .existe?
```
if Business.exists?(user_id: current_user.id)
  # same as Business.where(user_id: current_user.id).exists?
  # ...
else
  # ...
end
```
Option 2: à l'Aide de .le présent? (ou .vide?, à l'opposé de .present?)
```
if Business.where(:user_id => current_user.id).present?
  # less efficiant than using .exists? (see generated SQL for .exists? vs .present?)
else
  # ...
end
```
Option 3: affectation de Variable dans l'instruction if
```
if business = Business.where(:user_id => current_user.id).first
  business.do_some_stuff
else
  # do something else
end
```
Cette option peut être considérée comme une odeur de code par certains des linters (Rubocop par exemple).

Option 3b: affectation de Variable
```
business = Business.where(user_id: current_user.id).first
if business
  # ...
else
  # ...
end
```
Vous pouvez également utiliser .find_by_user_id(current_user.id) au lieu de .where(...).first

Meilleure option:
- Si vous n'utilisez pas le Business objet(s): Option 1
- Si vous avez besoin d'utiliser le Business objet(s): Option 3
- Cela ne semble pas fonctionner. Il garde en passant ce test et le chargement de l'index html comme avec l' == nil test (si j'obtiens une erreur: undefined method `nom' for nil:NilClass).
- Essayez d'abord avant d'appeler présents
- J'obtiens le même problème
- Oh que c'est de ma faute, j'ai été confondre, vous avez besoin de tester avec blank?
- Merci ça a fonctionné! Je ne l'aurais pas remarqué cette erreur. Pourriez-vous me dire pourquoi l' == nil case ne fonctionne pas?
- Oui, donnez-moi une minute, je vais mettre à jour ma réponse, aussi, ne pas utiliser le .first avant .blank? 😉
- Je n'ai pas retiré la première. Merci pour votre aide.
InformationsquelleAutor MrYoshiji
24

Dans ce cas, je tiens à utiliser le exists? méthode prévue par ActiveRecord:
```
Business.exists? user_id: current_user.id
```
- Est existe? avec OU possible?
InformationsquelleAutor Hamed
4

avec " existe?':
```
Business.exists? user_id: current_user.id #=> 1 or nil
```
avec 'tout?':
```
Business.where(:user_id => current_user.id).any? #=> true or false
```
Si vous utilisez quelque chose avec .où, être sûr d'éviter les ennuis avec des étendues et une meilleure utilisation
.non délimité
```
Business.unscoped.where(:user_id => current_user.id).any?
```
- Meilleure utilisation d'Affaires.non délimité.où(:user_id => current_user.id).arracher(:id).tout? pour éviter unnecesary charge des relations de l'objet de la vérification.
InformationsquelleAutor user2427993
1

ActiveRecord#où sera de retour une ActiveRecord::Relation d'objet (qui ne sera jamais nul). Essayez d'utiliser .est-il vide? sur le rapport de test si il sera de retour de tous les enregistrements.

InformationsquelleAutor Puhlze
1

Lorsque vous appelez Business.where(:user_id => current_user.id), vous obtiendrez un tableau. Ce Tableau peuvent ne pas avoir les objets ou l'un ou plusieurs objets, mais il n'est pas null. Ainsi, le check == nil ne sera jamais vrai.

Vous pouvez essayer ce qui suit:
```
if Business.where(:user_id => current_user.id).count == 0
```
De sorte que vous vérifiez le nombre d'éléments dans le tableau et de les comparer à zéro.

ou vous pouvez essayer:
```
if Business.find_by_user_id(current_user.id).nil?
```
ce sera le retour de l'un ou nul.

InformationsquelleAutor marimaf

business = Business.where(:user_id => current_user.id).first
if business.nil?
# no business found
else
# business.ceo = "me"
end

InformationsquelleAutor Nino van Hooff

0

Je voudrais le faire de cette façon, si vous avez besoin d'une variable d'instance de l'objet à travailler avec:
```
if @business = Business.where(:user_id => current_user.id).first
  #Do stuff
else
  #Do stuff
end
```
InformationsquelleAutor user3633260

Vous devez vous connecter pour publier un commentaire.

Comment tester si au moins un enregistrement existe?