rails で、銀行、支店の関係を持ったマスタデータを作成する画面を作りたい

Question

###前提・実現したいこと
銀行(bank)とその支店(branch)を登録するための画面を作ろうとしています。
bankとbranchをscaffoldで以下のようなモデルで作りました。
Banks
string :bank_cd, limit: 4
string :bank_name

Branches
string :bank_cd, limit: 4
string :branch_cd, limit: 4
string :branch_name

また、
bank.rb

class Bank < ApplicationRecord
  has_many :branches, dependent: :destroy
end

branch.rb

class Branch < ApplicationRecord
  belongs_to :bank
end

という状態です。

ここで、view/branches/_form.html.erb 内のbank_cdを入れる部分を以下のようにしました。

  <div class="field">
    <%= form_for @branch do |f| %>
    <%= f.label :bank_cd %>
    <%= f.collection_select :bank_cd, Bank.all, :bank_cd, :bank_name %>
    <% end %>
  </div>

画面上は想定通りに出てくるのですが、支店レコードを保存することができません。

なお、コントローラーは下記のようにscaffoldのほぼデフォルトです。

branches_controller.rb

  before_action :set_branch, only: [:show, :edit, :update, :destroy]

  def index
    @branches = Branch.all
  end

  def show
  end

  def new
    @branch = Branch.new
  end

  def edit
  end

  def create
    @branch = Branch.new(branch_params)

    respond_to do |format|
      if @branch.save
        format.html { redirect_to @branch, notice: 'Branch was successfully created.' }
        format.json { render :show, status: :created, location: @branch }
      else
        format.html { render :new }
        format.json { render json: @branch.errors, status: :unprocessable_entity }
      end
    end
  end

  def update
    respond_to do |format|
      if @branch.update(branch_params)
        format.html { redirect_to @branch, notice: 'Branch was successfully updated.' }
        format.json { render :show, status: :ok, location: @branch }
      else
        format.html { render :edit }
        format.json { render json: @branch.errors, status: :unprocessable_entity }
      end
    end
  end

  def destroy
    @branch.destroy
    respond_to do |format|
      format.html { redirect_to branches_url, notice: 'Branch was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private
    def set_branch
      @branch = Branch.find(params[:id])
    end

    def branch_params
      params.require(:branch).permit(:bank_cd, :branch_cd, :branch_name)
    end
end

どなたか、お力添え願えますでしょうか。

Comments

[mbk]

保存する部分のcontrollerのコードも貼って頂けますか。

[ak_suzuki]

ありがとうございます。元の質問に、branches_controller.rb のコードを付け加えました。よろしくお願いいたします。

Answer 1

モデルの立て方が間違っているのでは？
Rails(レールズ)は読んで字のごとくレールに乗ることを前提とします。

class Bank < ApplicationRecord
  has_many :branches, dependent: :destroy
end

例えばこれ
has_many :branches
は
has_many :branches, primary_key: :id,foriegn_key: :bank_id
を省略(Rails規約に従っていれば書かなくていい)したものです。
Railsでアプリを書くならbrancheはbank_idを持っているのが、前提!!
あえて、レールから外れ、bank_cd(String)で親子関係を定義するには
has_many :branches, primary_key: :bank_cd,foriegn_key: :bank_cd
とすればいいのですが、(もちろんbelongs_toの方も同様に直してください)

しかし、そもそも論、リレーショナルデータベース(RDB)でStringを関連(主キー)に使うのはお勧めできません。
データが増えてくると明らかな速度低下が想定されます。
RDBに関する見識があれば色々とできないことはないのですが…。Railsの利便性を捨ててまで
bank_codeにこだわる意味はないかと思います。

Railsはレールに乗っていれば周辺知識が不十分でもなかなかなものが作れるのがウリですので。

確信がなければ素直にRailsに乗ってモデルを作り直すことをお勧めします。と言っても、bank_idを追加するだけですが。

Comments

[ak_suzuki]

ありがとうございます。何分初心者なもので、ご指摘いただいた前提やStringを主キーに使うのはお勧めでない点など、大変参考になりました。
bank_idを使っていない理由は、全銀コードという所与の値を使いたかったので、idのようにDB側で自動的に付番されるのは難しいかも？と思ったからです。
自分の習熟度が低く、まだやりたいことができてはいませんが、いったんクローズします。
ありがとうございました。

[moke]

私も全銀を運用中です^_^
でも、ユーザーに編集させるものでは無いと思いますよー
マスターを作って管理者が編集するのがいいと思います。