Question

好的，这就是发生的事情..

用户模型有一些方法，其功能取决于是否存在名为status_updates的其他对象，这些对象在has_many belongs_to关系中与其关联。

所以现在我意识到status_updates新页面上有一个表单，并且该控制器提供了初始化但未为该表单保存的status_update.build对象。虽然，当从视图中调用用户方法时，它返回未保存的初始化值，而不是命中数据库以返回NilClass。

我通过在需要它的用户方法上添加以下条件来修复该错误。

if self.status_updates.count == 0
  temp_status_update
else
  self.status_updates.last
end

如果用户对象没有任何status_updates，则返回一个初始化的status_update，其中包含默认值 - 此status_update将不会被保存。在实际创建和保存status_update之前，它只是一个占位符。

这就是奇怪的地方......

现在，当我尝试实际保存status_update时，无论我通过表单提交什么，它都只保存初始化的默认值。

我知道正在提交这些值，似乎这些值似乎没有提交给控制器。

这是我的status_updates控制器中的操作。

   def create
      @status_update = current_user.status_updates.build(params[:status_update])
      if @status_update.save
        flash[:success] = "Status Update Saved! #{params[:status_update]}"
      # redirect_to status_update_path(current_user.id)
        redirect_to new_status_update_path
      else
        flash[:error] = "Input Error - Couldn't Save Status Update"
        redirect_to new_status_update_path
      end 
    end  



   def new
     @status_update = current_user.status_updates.build if user_signed_in?
   end

params不应该提交值而不是默认的初始值吗？

我在成功flash中返回params，它会向视图返回正确的值。

这是status_update模型：

    class StatusUpdate < ActiveRecord::Base
      belongs_to :user

      after_initialize :default_values 
      before_create :sanitize 

      #removed stuff from here for brevity

      def default_values
        if self.created_at == nil
          self.current_bf_pct        = 0
          self.current_weight        = 0
          self.current_lbm           = 0    
          self.current_fat_weight    = 0
          self.change_in_weight      = 0 
          self.change_in_bf_pct      = 0
          self.change_in_lbm         = 0
          self.change_in_fat_weight  = 0
          self.total_weight_change   = 0
          self.total_bf_pct_change   = 0
          self.total_lbm_change      = 0
          self.total_fat_change      = 0 
        end
      end

      #removed rest of model from here for brevity

这是用户模型

    class User < ActiveRecord::Base


      before_create :sanitize

      has_many :status_updates, dependent: :destroy

     #removed stuff from here for brevity

    def sanitize
      #inputs
      self.activity_factor       = 1.3
      self.deficit_amnt          = 1
      self.target_bf_pct         = 10 
      self.fat_factor            = 0.45
      self.protein_factor        = 1
    end

    #Removed functions for brevity


      def temp_status_update
        self.status_updates.build
      end
    end

我用来提交status_update的表单

        <%= form_for(@status_update) do |f| %>

          <%= f.label :current_weight %>
          <%= f.text_field :current_weight %>

          <%= f.label :current_bf_pct %>
          <%= f.text_field :current_bf_pct %>

          <%= f.submit "Update", class:"btn btn-large btn-primary" %> 
        <% end %>

以下是从新status_update视图

调用用户模型的方法

        <tr>
          <td> <%= current_user.current_bf_pct %> % </td>
          <td> <%= current_user.target_bf_pct  %> % </td>
        </tr>

但是，提交给create操作的status_update构建存储在@status_update变量中。

因此，为什么还要考虑OTHER status_update构建？它没有插入传递给create动作的变量中。

Answer 1

每当你想在任何操作之后设置属性时你应该像这样设置它们并调用那个方法after_create，after_update等......

模型回调适用于数据库中填充的所有数据。

before_create :sanitize    

def sanitize
self.users.build(user_id, User.current.id, activity_factor:1.3,
  deficit_amnt: 1
  target_bf_pct: 10 
  fat_factor: 0.45
  protein_factor: 1)
end

对于您想要更新或设置值的任何模型执行相同的操作......它只会从控制器调用该方法并从模型中检查它们以便它可以正常工作。

Answer 2

永远不要使用默认模型值来创建临时对象。

不要避开数据库，只需在模型中添加一个名为“temporary”的属性，并在创建临时对象时将其设置为“true”。然后使用find_by_temporary找到它并根据需要进行销毁。

回调弄乱表单提交

2 个答案: