Dans les Rails, ce qui est la meilleure façon de mettre à jour un enregistrement ou d'en créer une nouvelle si elle n'existe pas?

J'ai une instruction de création de certains modèles, mais c'est la création d'un enregistrement dans une table de jointure, indépendamment de savoir si l'enregistrement existe déjà.

Voici ce que mon code ressemble à ceci:

@user = User.find(current_user)
@event = Event.find(params[:id])
for interest in @event.interests
 @user.choices.create(:interest => interest, :score => 4)
end

Le problème est qu'il crée des enregistrements n'importe quoi. Je tiens à créer un enregistrement uniquement si aucun enregistrement n'existe déjà; si un enregistrement n'existe pas, j'aimerais qu'il prenne l'attribut de l'enregistrement trouvé et d'ajouter ou de soustraire 1.

J'ai été en regardant autour ont vu quelque chose qui s'appelle find_or_create_by. Qu'est-ce quand il trouve un record? Je tiens à prendre le courant :score attribut et ajouter 1.

Est-il possible de trouver ou de créer par id? Je ne suis pas sûr de ce que l'attribut je voudrais trouver en, car le modèle que je suis à la recherche d'une jointure modèle qui n'a id clés étrangères et le score d'attribut.

J'ai essayé

@user.choices.find_or_create_by_user(:user => @user.id, :interest => interest, :score => 4)

mais il a obtenu

méthode non find_by_user

Que dois-je faire?

Est current_user déjà une instance de l'Utilisateur modèle? Si oui, vous n'avez pas besoin de re-trouver et pouvez simplement utiliser current_user au lieu de @user = ....

InformationsquelleAutor ChrisWesAllen | 2011-01-06

conditional-statements ruby-on-rails

23

En supposant que le Choice modèle a un user_id (à associer avec un utilisateur) et un interest_id (à associer avec un intérêt), quelque chose comme cela devrait faire l'affaire:
```
@user = User.find(current_user)
@event = Event.find(params[:id])

@event.interests.each do |interest|
  choice = @user.choices.find_or_initialize_by_interest_id(interest.id) do |c|
    c.score = 0 # Or whatever you want the initial value to be - 1
  end

  choice.score += 1
  choice.save!
end
```
Quelques remarques:
1. Vous n'avez pas besoin d'inclure les user_id colonne dans la find_or_*_by_*, comme vous l'avez déjà demandé à des Rails à la seule extraction choices appartenant à @user.
2. Je suis en utilisant find_or_initialize_by_*, qui est essentiellement la même que find_or_create_by_*, avec une différence essentielle étant que initialize ne crée pas le dossier. Ce serait semblable à Model.new par opposition à Model.create.
3. Le bloc qui définit c.score = 0 est exécuté seulement si l'enregistrement n' pas existent.
4. choice.score += 1 permettra de mettre à jour la valeur du score pour l'enregistrement, peu importe si il existe ou pas. Par conséquent, le nombre de points par défaut c.score = 0 devrait être la valeur initiale moins un.
5. Enfin, choice.save! sera soit mise à jour de la fiche (si elle existe déjà) ou de créer des initiés de la fiche (si elle n'a pas).
- en liant cette question semblable stackoverflow.com/questions/18747062/...
- Cette réponse doit être mis à jour ,certaines méthodes ne sont pas adaptés pour les rails 4
InformationsquelleAutor vonconrad
60
```
my_class = ClassName.find_or_initialize_by_name(name)

my_class.update_attributes({
   :street_address => self.street_address,
   :city_name => self.city_name,
   :zip_code => self.zip_code
})
```
- Pourquoi en bas de vote ?
- je ne sais pas pourquoi cela a été downvoted. cela a été très utile pour moi, merci pour la contribution!
- Je l'ai trouvé utile, aussi. Je suppose que le downvote est parce qu'il a seulement partiellement remédié à ce que l'OP a été demandé.
InformationsquelleAutor krunal shah
7

find_or_create_by_user_id sonne mieux
- Quelle est la différence tho?
- ce n'est pas de mise à jour de l'enregistrement? Il le trouve, ou qu'il crée.
InformationsquelleAutor apneadiving
3

Aussi, dans Rails 3 que vous pouvez faire:
```
@user.choices.where(:user => @user.id, :interest => interest, :score => 4).first_or_create
```
- first_or_create est préféré dans les Rails 4, en attendant find_or_intialize_by_xxx(attribute) est dépréciée en faveur de find_or_intialize_by(xxx:attribute)
InformationsquelleAutor Zuhaib Ali
0

Si vous êtes à l'aide de rails 4 je ne pense pas qu'il crée le finder méthodes comme autrefois, de sorte find_or_create_by_user n'est pas créé pour vous. Au lieu de cela, vous auriez du faire comme ceci:
```
@user = User.find(current_user)
@event = Event.find(params[:id])
for interest in @event.interests
 @user.choices.find_or_create_by(:interest => interest) do |c|
  c.score ||= 0
  c.score += 1
 end
end
```
InformationsquelleAutor rainkinz
0

Dans Les Rails 4
Vous pouvez utiliser find_or_create_by pour obtenir un objet(s'il n'existe pas,il va créer), puis utilisez update pour enregistrer ou mettre à jour l'enregistrement, la méthode de mise à jour va persister enregistrement si elle n'est pas exister, sinon mettre à jour l'enregistrement.

Par exemple
```
@edu = current_user.member_edu_basics.find_or_create_by(params.require(:member).permit(:school))

if @edu.update(params.require(:member).permit(:school, :majoy, :started, :ended))
```
InformationsquelleAutor Albert.Qing

Vous devez vous connecter pour publier un commentaire.