結合テーブルに新しいレコードを作成する

Question

私は、基本的に要求と宿泊施設の間の結合テーブルとして機能する通知というモデルを持っています。私の要求コントローラでの私の「作成」方法では

は、私が持っている：

# find associated accommodations, currently matching: location 
    @accommodations = Accommodation.where('location' => :location) 
    @accommodations.each do |accommodation| 
     @notification = @request.notification.build('accommodation_id' => accommodation.id).save 
    end 

新しい通知レコードを作成していないようです。私はここで間違って何をしていますか？

モデル/ accommodation.rb

class Accommodation < ActiveRecord::Base 
    validates_presence_of :title, :description, :thing, :location, :spaces, :price, :photo 
    attr_accessible :photo_attributes, :title, :description, :thing, :location, :spaces, :price 
    has_one :photo 
    has_many :notifications 
    belongs_to :user 
    accepts_nested_attributes_for :photo, :allow_destroy => true 
end 

モデル/

class Notification < ActiveRecord::Base 
    attr_accessible :accommodation_id, :request_id 
    has_one :request 
end 

モデル/ request.rb notification.rb

class Request < ActiveRecord::Base 
    attr_accessible :firstname, :lastname, :email, :phone, :datestart, :dateend, :adults, :children, :location, :status 
    validates_presence_of :firstname, :lastname, :email, :phone, :datestart, :dateend, :children, :adults, :location 
    has_many :notifications 
end 

Answer 1

コントローラに@requestまたは：の位置を定義していません。それはRequestオブジェクトだと仮定すると：ちょうど@AnomalousThoughtよう

@notifications = Array.new 

@accommodations = Accommodation.where('location' => :location) 
Accommodation.find_each(:conditions => { 'location' => :location }) do |a| 
    notification = @request.notifications.create('accommodation_id' => a.id) 
    @notifications << notifications 
end 

は言った：

提供 例について語ったことがたくさんがあります、私は 実装が完全に悪いと思います。しかし、 の質問に関連していない なぜ通知レコードは保存されていません。

Answer 2

@request.notification.buildは、リクエストがhas_many通知であるため、@request.notifications.buildである必要があります。

また、@notificationを.saveが返す値に設定すると、trueまたはfalseのみになります。 @notificationに実際のオブジェクトをポイントさせたい場合は、createを使用します。さらに、あなたが望むかもしれないし、望ましくないかもしれないすべての反復で変数@notificationを上書きし続けることになることは既に知っているでしょう。

提供されている例について言及する必要がありますが、実装は完全に貧弱だと思います。しかし、通知レコードが保存されていない理由とは関係ありません。