为belongs_to的众多车型的模型CRUD方法的Rails
我是新来的铁轨和我可能会丢失一些非常基本的位置:为belongs_to的众多车型的模型CRUD方法的Rails
用户可以创建一个公司
为两个分支的接触和分裂Branch.rb
class Branch < ApplicationRecord
belongs_to :company
has_many :contacts
end
Division.rb
class Division < ApplicationRecord
belongs_to :company
has_many :contacts
end
Contact.rb
class Contact < ApplicationRecord
belongs_to :branch
belongs_to :division
end
现在,用户可以创建从分支页面不存在division_id并能创造本司网页中的联系人联系。
我定义我的routes.rb这样的:
的routes.rb
resources :companies, :shallow => true do
get 'company_page'
resources :branches, :shallow => true do
get 'branch_page'
resources :contacts
end
resources :divisions, :shallow => true do
get 'division_page'
resources :contacts
end
end
其结果是,如果我创建从任一科或科接触,它去接触#创建方法。
在我contacts_controller.rb,我有:
def create
@newContact = Contact.new(contact_params)
id = @division = @branch = nil
isBranch = false
if params[:branch_id] != nil
isBranch = true
id = params[:branch_id]
else
isBranch = false
id = params[:division_id]
end
if isBranch
branch = Branch.find(id)
@newContact.branch = branch
@branch = branch
else
division = Division.find(id)
@newContact.division = division
@division = division
end
respond_to do |format|
if @newContact.save
format.js
format.html { render :nothing => true, :notice => 'Contact created successfully!' }
format.json { render json: @newContact, status: :created, location: @newContact }
else
format.html { render action: "new" }
format.json { render json: @newContact, status: :unprocessable_entity }
end
end
end
但在@newContact.save
我已经面临ActiveRecord Error
。
我确定我在这里做的事情基本上非常错误,Rails以另一种我不知道的优雅方式处理这些事情。
由于@Anthony注意,你需要让你的belongs_to
协会可选:
# app/models/contact.rb
class Contact < ApplicationRecord
belongs_to :branch, optional: true
belongs_to :division, optional: true
end
但另一个问题是,params[:division_id]
和params[:branch_id]
始终为零。这两个密钥都存在于[:contact]
密钥中。所以你得到的错误应该是ActiveRecord::RecordNotFound: Couldn't find Division with 'id'=
所有那些条件逻辑是不必要的。你可以用任何参数进行新的联系。另外,你应该使用Ruby约定来进行变量命名,这是snake_case而不是camelCase。
最后,我想你会想要将HTML请求重定向到分支或分区展示页面,具体取决于关联的内容。所以我添加了逻辑来做到这一点。
这里的控制器#create
行动的快速重构:
def create
@new_contact = Contact.new(contact_params)
if @new_contact.save
branch = @new_contact.branch
division = @new_contact.division
redirect_path = branch ? branch_path(branch) : division_path(division)
respond_to do |format|
format.js
format.html { redirect_to redirect_path, :notice => 'Contact created successfully!' }
format.json { render json: @new_contact, status: :created, location: @new_contact }
end
else
respond_to do |format|
format.html { render action: "new" }
format.json { render json: @new_contact, status: :unprocessable_entity }
end
end
end
这证明了它的工作原理:
# spec/controllers/contacts_controller_spec.rb
require 'rails_helper'
RSpec.describe ContactsController, type: :controller do
let(:company) { Company.create!(name: 'Company Name') }
let(:division) { Division.create!(name: 'Division Name', company: company) }
let(:branch) { Branch.create!(name: 'Branch Name', company: company) }
describe '#create' do
context 'when created with a division id' do
let(:attributes) { {'division_id' => division.id, 'name' => 'Contact Name'} }
it 'creates a contact record and associates it with the division' do
expect(Contact.count).to eq(0)
post :create, params: {contact: attributes}
expect(Contact.count).to eq(1)
contact = Contact.first
expect(contact.division).to eq(division)
end
end
context 'when created with a branch id' do
let(:attributes) { {'branch_id' => branch.id, 'name' => 'Contact Name'} }
it 'creates a contact record and associates it with the branch' do
expect(Contact.count).to eq(0)
post :create, params: {contact: attributes}
expect(Contact.count).to eq(1)
contact = Contact.first
expect(contact.branch).to eq(branch)
end
end
end
end
谢谢!这清除了我所有的疑虑。 – user122121
有什么错误? – Anthony
'在265ms内完成500次内部服务器错误(ActiveRecord:66。8ms)' – user122121
我猜这个错误是因为'联系'有两个属于关联,但你只给它一个。 log/development.log中的错误是什么? – Anthony