rails 4 没有数据传输到数据库

Question

我正在尝试进行用户评论，当用户可以向另一个用户写评论时，我使用 :content 创建表评论，使用 :for_user_id 和 by_user_id 创建 user_reviews， 我的路线

devise_for :users
  resources :users, :only => [:show] do
    resources :reviews
  end

class Review < ActiveRecord::Base
    belongs_to :user
    has_many :users, :through => :users_reviews
end

class User < ActiveRecord::Base

  has_many :users_review
  has_many :reviews, :through => :users_review
end

class UsersReview < ActiveRecord::Base
  belongs_to :user
  belongs_to :review
end

class ReviewsController < ApplicationController

    def new
        @user = User.find(params[:user_id])
        @review = @user.reviews.new(params[:for_user_id])
    end

    def create
        @user = User.find(params[:id])
        @review = current_user.reviews.build(review_params)
        redirect_to root_path
    end

    def show
    end

    def index
        @user = User.find(params[:for_user_id])
        @reviews = Review.all
    end

    private
    def review_params
        params.require(:review).permit(:user_id, :user_id, :content)
    end

end

我的看法

<%= form_for([@user, @user.reviews.build]) do |f| %>
<%= f.text_area :content, placeholder: "Your review" %>
<%= f.submit "Go", class: "btn btn-large btn-primary" %>
<% end %>

一切正常，但没有数据发送到数据库：\我做错了什么？

Answer 1

您没有在 create 方法中保存任何内容，因此不会持续存在任何内容。

你会想要这样的东西：

def create
  @user = User.find(params[:id])
  @review = @user.reviews.build(review_params)

  if @user.save && @review.save
    redirect_to root_path
  else
    (handle bad data)
  end
end

我也倾向于同意@marzapower - 如果您想使用current_user，则不需要@review 上面的行。我上面的方法包含了这个变化。

Answer 2

您在 create 操作中缺少对 save 方法的调用：

def create
     @user = User.find(params[:id])
     @review = current_user.reviews.build(review_params)
     @review.save
     redirect_to root_path
end

post 解释了构建和创建之间的区别。

Answer 3

我认为您缺少两件事。 User 和 Review 类都需要引用它们在直通关系中使用的 UserReview 类，例如，

class User < ActiveRecord::Base
  ...
  has_many :user_reviews_received, class_name: 'UserReview', foreign_key: :for_user_id
  has_many :reviews_received, through: :user_reviews_received, class_name: 'Review'

  has_many :user_reviews_written,  class_name: 'UserReview', foreign_key: :by_user_id
  has_many :reviews_written, through: :user_reviews_written, class_name: 'Review'
  ...
end

如上所述，create 操作似乎有点混乱。一般来说，在 new 和 create 操作中构建实例的参数应该是相同的，但你有一些不同之处。您可以通过以下两种方式之一解决此问题。一种选择是在控制器中解决它。

class ReviewsController < ActionController::Base
  def new
    @user = User.find(params[:user_id])
    @review = @user.reviews_received.new(by_user_id: current_user.id)
  end

  def create
    @review = @user.reviews_received.create(params[:review].merge(by_user_id: current_user.id))
    redirect_to root_path
  end
end

第二个选项是在视图中解决它（表单中的新第一行）。

<%= form_for([@user, @user.reviews_received.build(by_user_id: current_user.id)]) do |f| %>
  <%= f.hidden_field :by_user_id %>
  <%= f.text_area :content, placeholder: "Your review" %>
  <%= f.submit "Go", class: "btn btn-large btn-primary" %>
<% end %>

我倾向于第一个选项

Answer 4

控制器有错误，我认为：

def create
    @user = User.find(params[:id])
    @review = @user.reviews.build(review_params)
    @review.save
    redirect_to root_path
end

您正在为current_user 而不是@user 创建新的Reviews。