【調査】accept_nested_attributes_for について調査する

[ms2sato]

今後なくなっていくとも言われていますが、便利な機能で多くのプロジェクトに導入されている機能です。一度調べてみてください。調査の結果をこのIssueに貼ってくれると嬉しいです。

[jonakp]

@ms2sato
調査結果を以下に記載します。
ご確認お願いします。
（コード書いてみての調査はまだなので、後程更新するかもしれません。）

accepts_nested_attributes_for について

概要

• Relationを利用して、親子関係にあるModelのAttributeを一括で作成・更新できるモジュール(?)
• Modelと画面上のフォームが1対1で一致しない場合に使える実装方法の1つ
  • 他のやり方としては
    Form オブジェクト
    Service クラス（Form オブジェクトとほぼ一緒？）
    があるらしい(未調査)
    https://discourse.clean-rails.org/t/model-form-1-1/14

使い方

主に以下を参照
https://api.rubyonrails.org/classes/ActiveRecord/NestedAttributes/ClassMethods.html

1. Model(通常は親)で"accepts_nested_attributes_for :(モデル名)"を宣言

class Member < ActiveRecord::Base
  has_one :avatar
  accepts_nested_attributes_for :avatar
end

2. view(form)を修正して、入れ子になったパラメータが送信されるようにする。

<%= form_with @article do |f| %>
    <%= f.text_field :title %>
    <%= f.text_area :body %>
    <%= f.field_for :categories |g| %>
      <%= g.text_field :name %>
    <% end %>
  <% f.submit %>
<% end %>

3. controller上でcreate なり、updateなり行う

member = Member.create(params[:member])
member.update params[:member]

注意点

• 子から親を作成・更新する際は、親のフィールドのname属性を自力で指定しないといけない？
  (古い情報なので、真偽不明。実装時に分かるはず)
  http:https://d.hatena.ne.jp/donghai821/20110804/1312475967

疑問点

• macro-shareにおいては、子から親を更新しようとしているが、この形はアリなのでしょうか？
  • 一応上の”注意点”の記載から、できそうだなぁという気はしています。
• accepts_nested_attributes_forはどういった点が問題視されて、今後使われなくなりそうなのでしょうか？
  • DHH氏が無くしたいと発言している記載を見つけました。
    Add form_with to unify form_tag/form_for. rails/rails#26976 (comment)
  • accepts_nested_attributes_forのメリットデメリットの記載も見つけました。
    https://discourse.clean-rails.org/t/model-form-1-1/14/7
  • ただ読んでみても、イマイチaccepts_nested_attributes_forが良くない理由がピント来ませんでした。

[ms2sato]

@jonakp 詳細な調査をありがとうございます。 機能自体の理解は十分そうですね。

使い方について概ねOKです。一つ注意点としては、StrongParameterに飛んでくる子の引数も入れなければならないのでこの辺り気をつけると良いと思います。

疑問点について

macro-shareにおいては、子から親を更新しようとしているが、この形はアリなのでしょうか？

この場合、formの作り方を工夫すると良いですよ。コントローラ名は確かに扱うモデルについて記述するものではありますが、特に中身について限定はしないものと考えると良いです。

userを親、userfeatureを子としてformを作成し、それをUserFeatureController#update で受け取るのは特に問題ありません。こう書くとRailsらしい整ったコードになるでしょう。

accepts_nested_attributes_forはどういった点が問題視

これについては、以下が大きな理由です。プロジェクトオーナーがそう表明しているので、今後非推奨になるでしょう。

DHH氏が無くしたいと発言している

DHH氏がどのように考えて元の発言をしたのかは定かではありませんが、少なくとも彼がそう発信したこと自体には意味があるという考え方で良いと思います。

私個人はこの機能は忌み嫌うレベルのものではありません。むしろコスト削減には良い機能だなと思ってもいます。FormObjectなどで代替できる、ということは背景にあるかもしれません。

[ms2sato]

理解としてはOKそうなのでCloseします。疑問があれば引き続きこの続きでコミュニケーションOKです！

[jonakp]

コメントありがとうございます。
疑問点は解消しました。

userを親、userfeatureを子としてformを作成し、それをUserFeatureController#update で受け取るのは特に問題ありません。こう書くとRailsらしい整ったコードになるでしょう。

この内容はaccept_nested_attributes_for も含めて、
Issue #16 で対応する、というつもりでいます！

[ms2sato]

@jonakp #16 の意識はこちらでバッチリです。適度なタイミングでやりましょう！