self.board.update_attributes返回未定义的方法`update_attributes方法

Question

在回复的创造，我想更新主板的：用线self.board.update_attributes(:toppid => reply_max.to_i + 1) toppid列，但是这将返回NoMethodError in RepliesController#create undefined method 'update_attributes' for nil:NilClass

我怎样才能正确地更新:toppid专栏？

休息的我的代码：

reply.rb：

class Reply < ActiveRecord::Base 
    belongs_to :board 
    belongs_to :post 
    after_create :set_pid 
    def set_pid 
    reply_max = self.post.replies.maximum(:pid) 
    board_max = self.board(:toppid) 
    if board_max.to_i > reply_max.to_i 
     self.update_attributes(:pid => board_max.to_i + 1) 
     self.board.update_attributes(:toppid => board_max.to_i + 1) 
    else 
     self.update_attributes(:pid => reply_max.to_i + 1) 
     self.board.update_attributes(:toppid => reply_max.to_i + 1) 
    end 
    end 
end 

replies_controller.rb：

class RepliesController < ApplicationController 
    def create 
    @board = Board.friendly.find(params[:board_id]) 
    @post = @board.posts.friendly.find params[:post_id] 
    @reply = @post.replies.create(reply_params) 
    @post.touch 
    redirect_to @board 
    end 
    private 
    def reply_params 
     params.require(:reply).permit(:name, :email, :subject, :comment, :reply_file) 
    end 
end 

的routes.rb：

resources :boards, :path => '' do 
    resources :posts, :path => 'thread' do 
     resources :replies 

Answer 1

看看您致电@reply = @post.replies.create(reply_params)您的回复对象永远不会与板对象关联。

你可能想用build来代替。喜欢的东西：

@reply = @post.replies.build(reply_params) 
@reply.board = @board 
@reply.save 

编辑

从您的意见，似乎是在你希望你的模型中的关系是什么样的脱节和你上面的代码。使用belongs_to意味着您在一个模型与另一个模型之间具有数据库级外键关系。

根据你的意见，你不想要这个。如果确实如此，请摆脱关系并将board改为post。否则，请按照我最初的建议修正表格，然后在您的respond_table中添加board_id。

这里是你会怎么写代表团：

class Reply < ActiveRecord::Base 
    belongs_to :post 
    after_create :set_pid 
    delegate :board, to: :post 

    def set_pid 
    reply_max = self.post.replies.maximum(:pid) 
    board_max = self.board(:toppid) # have no idea what you're trying to do here, but it's also a syntax error, maybe you mean to write: self.board.toppid 
    if board_max.to_i > reply_max.to_i 
     self.update_attributes(pid: board_max.to_i + 1) 
     self.board.update_attributes(toppid: board_max.to_i + 1) 
    else 
     self.update_attributes(pid: reply_max.to_i + 1) 
     self.board.update_attributes(toppid: reply_max.to_i + 1) 
    end 
    end 
end 

而且，假设你有整列pid和toppid（它看起来像在update_attributes电话），您的to_i使用是不必要的。