【ruby】親モデルオブジェクト.子モデルオブジェクト.delete_atメソッドが使えない【Rails】

Question

###no method errorになってしまう

環境
cloud９
rails4.2.8
Mqsql2 0.3.18'
ruby 2.5.3

以下の様にPostテーブル（親）とPosttools（子）テーブルでアソシエーションを組んでいる。

class Post < ActiveRecord::Base

    has_many :posttools, dependent: :destroy
    

    accepts_nested_attributes_for :posttools ,allow_destroy: true

end

class Posttool < ActiveRecord::Base
  belongs_to :post
  belongs_to :tool
  
end
    

###Postインスタンス.Posttoolsインスタンスの時に、配列の要素を消したい・・・ができない

[1] pry(#<PostsController>)> @post
=> #<Post:0x00000006831548
 id: nil,
 user_id: 1,
 title: "なんということだ",
 created_at: nil,
 updated_at: nil,
 likes_count: 0,
 comments_count: 0,
 youtube: "",
 series: nil,
 unread: 0>
[2] pry(#<PostsController>)> @post.posttools
=> [#<Posttool:0x0000000680baa0 id: nil, post_id: nil, tool_id: 1, created_at: nil, updated_at: nil>,
 #<Posttool:0x00000006808aa8 id: nil, post_id: nil, tool_id: nil, created_at: nil, updated_at: nil>,
 #<Posttool:0x00000006801f28 id: nil, post_id: nil, tool_id: nil, created_at: nil, updated_at: nil>]

@postの下にPosttoolsのモデルオブジェクトを配列で複数入れて@post.saveでテーブルにレコードを作っています。

この時、tool_idがnilの時は、そのオブジェクト（要素）だけ消したいのですが
（ここで言うと <Posttool:0x00000006808aa8 id: nil, post_id: nil, tool_id: nil, created_at: nil, updated_at: nil>,
<Posttool:0x00000006801f28 id: nil, post_id: nil, tool_id: nil, created_at: nil, updated_at: nil>]　この２つ）

[23] pry(#<PostsController>)> @post.posttools.delete_at(1)
NoMethodError: undefined method `delete_at' for #<Posttool::ActiveRecord_Associations_CollectionProxy:0x00000005eec7f8>
Did you mean?  delete_all
               delete
from /usr/local/rvm/gems/ruby-2.3.1/gems/activerecord-4.2.8/lib/active_record/relation/delegation.rb:136:in `method_missing'

no methoderrorになってしまいます。

###どうすればよいのか

そもそも他のcompact!などのメソッドも使用できず、調べて使用できるメソッドを表示したらこんな感じでした。

[14] pry(#<PostsController>)> @post.posttools.methods
=> [:method_missing,
 :<<,
 :==,
 :empty?,
 :length,
 :size,
 :to_ary,
 :to_a,
 :new,
 :include?,
 :find,
 :count,
 :select,
 :first,
 :any?,
 :take,
 :replace,
 :clear,
 :concat,
[15] pry(#<PostsController>)> a = []

ということはこれしか使えない？

delete_atのソースを調べてposttools.rbに書こうともしたけど正直よくわかりませんでした。

どうすればよいでしょうか？

Answer 1

@post.posttoolsは、クラスがPosttool::ActiveRecord_Associations_CollectionProxyと出ているように、配列ではありません。そして、これはデータベースから引いてきたそのままになっているので、追加以外の破壊的変更もできません。

したいことは、以下2つのどちらでしょうか。

• データベースからも消したい場合は、@post.posttools[番号].destroyのように1つずつdestroyするか、whereで絞り込んでdeleteするなどしてください。
• Rails上で削減したリストを得たいのであれば、.to_aで配列に変えてから.delete_atなど好きに操作してください。

Comments

[nobinobinontan]

丁寧に回答して頂きありがとうございました。
僕の質問の仕方が悪かったのか1番目の方法では削除できず、
mark_for_destructionというメソッドをpost.rbのコールバック内で使用したら削除することができました。

Answer 2

post.rbに以下の様にコールバック内にmark_for_destructionというメソッドを使用したら削除できた。

class Post < ActiveRecord::Base
    before_save do
        posttools.each { |pt| pt.mark_for_destruction if pt.tool_id.nil? }
        
    end
    

コンソールでは削除してないよに見えても、テーブルにはちゃんと入っていません。