модель, которая относится к более чем одной другой модели одновременно

Question

Со следующими ассоциациями:

class User < ActiveRecord::Base 
    has_many :posts 
    has_many :comments 
end 

class Post < ActiveRecord::Base 
    belongs_to :user 
    has_many :comments 
end 

class Comment < ActiveRecord::Base 
    belongs_to :user 
    belongs_to :post 
end 

я могу сделать что-то вроде этого в контроллере:

@comment = current_user.comments.new(comment_params) 
@comment.user 

Но для доступа к соответствующим Post I необходимо вручную установить его родительское лицо:

@comment.post = Post.find params[:post_id] 

Есть ли лучший способ сделать t его, когда создается новый Comment?

Answer 1

Если вы хотите, чтобы объект комментариев был послан к одному или, тогда будет работать полиморфная ассоциация.

http://guides.rubyonrails.org/association_basics.html#polymorphic-associations

class Comment < ActiveRecord::Base 
    belongs_to :commentable, polymorphic: true 
end 

class User < ActiveRecord::Base 
    has_many :posts 
    has_many :comments, as: :commentable 
end 

class Posts < ActiveRecord::Base 
    belongs_to :user 
    has_many :comments, as: :commentable 
end 

Вы не сможете использовать @ comment.post, но вы будете в состоянии сделать @ comment.commentable, который будет либо ваш пост или объект пользователя в зависимости от какой из связанные с этим экземпляром.

Answer 2

Я хотел бы использовать вложенные ресурсы

resources :posts do 
    resources :comments 
end 

Тогда я хотел бы построить свой комментарий через пост и сливают в идентификатор текущего пользователя, так что вам не нужно скрытых полей, которые могут затем явно манипулировать.

class CommentsController 
    def new 
    @post = Post.find(params[:post_id]) 
    @comment = @post.comments.new 
    end 

    def create 
    @post = Post.find(params[:post_id]) 
    @comment = @post.comments.new(comment_params) 
    end 

    private 

    def comment_params 
    params.require(:comment).permit(:content).merge({ user_id: current_user.id }) 
    end 

end 

Вы бы только действительно нужно объединить текущего пользователя о создании комментария, чтобы вы могли иметь метод частного comment_creation_params, который вызывается на создание

def comment_params 
    params.require(:comment).permit(:content) 
end 

def comment_creation_params 
    comment_params.merge({ user_id: current_user.id }) 
end 

Answer 3

Итак, основная идея просто получить избавился от Post.find(params[:post_id]).

Если бы я тебя, я бы вставить post_id в comment_params явно:

def comment_params 
    params.require(:comment).permit(:text).merge(post_id: params[:post_id]) 
end 

Просто чтобы убедиться, что пост существует, я хотел бы добавить проверки на комментарий:

class Comment 
    validates :post, presence: true 
    #... 
end 

Если вы не хотите связать комментарий с сообщением, вы можете просто пропустить эту проверку или написать свой собственный код

validate do 
    errors.add(:post_id, 'There is no post') unless Post.find(post_id).present? 
end 

Один из ответов предлагает решение с вложенными ресурсами, что является еще одним обходным решением.