Question

通过 simple_form_for 传递数据时，我遇到了以上错误。我的视图，控制器和数据库文件包含在下面。看来我没有在控制器中得到 simple_form_for 提交的值。请注意，在我的数据库表中，属性status为not null，在视图中我不将其作为输入。我该如何解决这个错误？

数据库

create_table "users", force: :cascade do |t|
t.integer "team_lead_id"
t.string "name", null: false
t.string "email", null: false
t.string "password", null: false
t.date "joining_date", null: false
t.integer "status", null: false
t.integer "role"
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
end

查看

<%= simple_form_for @user , :url => user_createUser_url, :method => :post do |f| %>
    <%= f.input :name %><br />
    <%= f.input :email%><br />
    <%= f.input :password %><br />
    <%= f.input :joining_date, as: :date, order: [:day, :month, :year] %><br/>
    <%= f.submit "Create User" %>
  <% end %>

控制器

def createUser

    fresh = User.new

    fresh.name = params[:name]
    fresh.email = params[:email]
    fresh.password = params[:password]
    fresh.joining_date = params[:joining_date]
    fresh.status = 1
    fresh.role = 3

    if fresh.save
      flash[:notice] = "User Created"
      redirect_to(:action => index)
    else
      flash[:notice] = "Creating failed"
      redirect_to(:action => index)
    end

  end

我收到此错误

PG::NotNullViolation: ERROR: null value in column "name" violates not-null constraint DETAIL: Failing row contains (12, null, null, null, null, null, 1, 3, 2018-11-12 13:37:54.589835, 2018-11-12 13:37:54.589835). : INSERT INTO "users" ("status", "role", "created_at", "updated_at") VALUES ($1, $2, $3, $4) RETURNING "id"

Answer 1

createUser不是CRUD约定，而是使用create，但您也应该使用strong parameters，并且不要使用像fresh这样的愚蠢名称，而要使用模型对象应该以语义命名。

def create
  user = User.new(permitted_params)
  user.status = 1
  user.role = 3

  if user.save
    flash[:notice] = "User Created"
      redirect_to(:action => index)
  else
    flash[:notice] = "Creating failed"
    redirect_to(:action => index)
  end
end

def permitted_params
  params.require(:user).permit(:name, :email, :password, :joining_date)
end

您可能不需要joining_date，因为您有created_at，但是如果您想保留该字段，请至少使用名称joined_date或join_date，因为将永远是过去式。实际上，用户从未“加入”过，他们要么已经加入，要么还没有“加入”。

请注意，永远不要存储未加密的密码，也许请参阅bcrypt

通过Rails中的simple_form_for传递数据时获取NotNullViolation

1 个答案: