为belongs_to的众多车型的模型CRUD方法的Rails

Question

我是新来的铁轨和我可能会丢失一些非常基本的位置：

用户可以创建一个公司

为两个分支的接触和分裂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以另一种我不知道的优雅方式处理这些事情。

Answer 1

由于@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