質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.51%

フォームで入力された値から該当するidを取得してモデルに外部キーとして記録する方法ってありますか

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 2
  • VIEW 2,795

score 41

前提・実現したいこと

ruby(ruby on rails)でアカウントモデル同士の取引を取引モデルへの記録したいとします。
この場合、取引モデルにアカウント同士の関係を記録するときは、
すでに元のなるアカウント情報があるので、
「出金(支払)するアカウントのid(アカウントモデルを参照する外部キー)」と
「入金(受取)するアカウントのid(アカウントモデルを参照する外部キー)」を
記録することになると思います。

ただユーザーが入力フォームで入力する値は、
「id(主キーや外部キー)」ではなくユーザー名やメールアドレスみたく
変更可能性のある「アカウント番号(文字列)」なので、
そのまま取引モデルの外部キーのカラムに記録することはできないと思います。

その場合、入力した「アカウント番号(受取をするアカウント番号)」からピッタリ該当する「アカウントモデルのid(アカウントモデルの主キー)」を探し出して、
そのidを取引モデルの
「入金(受取)するアカウントのid(アカウントモデルを参照する外部キー)」カラムに
記録したいのですが、どうしたらいいのかがわかりません。

教えて頂ければ幸いです。
よろしくお願いします。

現状

モデル(テーブル)の内容です。
BasicIncomeAccount(アカウントモデル)
・id(デフォルトで作成されるユニークかつ不変な主キー)
・account_number(ユニークだけど変更の可能性ありのアカウント番号)

AccountTransaction(取引モデル)
・withdrawal_account_id(外部キーであり出金側の口座id)
・deposit_account_id(外部キーであり入金側の口座id)
・amount(取引額)

class BasicIncomeAccount < ApplicationRecord

    has_many :withdrawal_account_transaction, class_name: 'AccountTransaction', :foreign_key => 'withdrawal_account_id'
    has_many :deposit_account_transaction, class_name: 'AccountTransaction', :foreign_key => 'deposit_account_id'
class AccountTransaction < ApplicationRecord

  belongs_to :withdrawal, class_name: 'BasicIncomeAccount', :foreign_key => 'withdrawal_account_id'
  belongs_to :deposit, class_name: 'BasicIncomeAccount', :foreign_key => 'deposit_account_id'

end

取引のコントローラです。
app/controllers/account_transactions_controller.rb

class AccountTransactionsController < ApplicationController

  def new
    @account_transaction = AccountTransaction.new
  end

  def create
    @account_transaction = AccountTransaction.new(
      withdrawal_account_id: current_user.basic_income_account,
      deposit_account_id: params[:deposit_account_id],
      amount: params[:amount]
      )
  end

  private

  def account_transaction_params
   params.require(:account_transaction).permit(:withdrawal_account_id, :deposit_account_id, :amount)
  end
end

app/views/account_transactions/new.html.erb(入力フォーム)

<%= form_for @account_transaction do |f|%>
<div class="field">
  <%= f.label :送金相手の口座番号 %>
  <%= f.text_field :deposit_account_id %> 
  <%= f.label :送金額 %>
  <%= f.number_field :amount %> 
  <%= f.submit"送金" %>
</div>
<% end %>

補足情報(言語/FW/ツール等のバージョンなど)

Rails 5.1.3
ruby 2.4.1
devise (4.3.0)

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

BasicIncomeAccount.find_by(account_number: フォームで入力した値)


等をしてidを取得すればよいのでは?

 追記

存在しなかったときのことは考えていませんが、deposit_account_id:    BasicIncomeAccount.find_by(account_number: params[:deposit_account_id]).idとすればidをdeposit_account_idに入れられると思います。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/11/18 08:53

    chelsy7110さん、回答ありがとうございます。
    ということは、def createを以下のようにすればいいということでしょうか?
    def create
    @account_transaction = AccountTransaction.new(
    withdrawal_account_id: current_user.basic_income_account,
    deposit_account_id: BasicIncomeAccount.find_by(account_number: params[:deposit_account_id]),
    amount: params[:amount]
    )
    end

    キャンセル

  • 2017/11/21 10:23 編集

    度々すみません。

    Account_transactionのコントローラのcreateアクションを以下の通りに書き換えてみました。

    def create
    @account_transaction = AccountTransaction.new(
    withdrawal_account_id: 20 #テスト用、idを直接入力
    deposit_account_id: BasicIncomeAccount.find_by(account_number: params[:deposit_account_id]).id,
    amount: params[:amount]
    )

    そして入力フォームでは存在する口座番号を入力してみたのですが、


    NoMethodError in AccountTransactionsController#create
    undefined method `id' for nil:NilClass
    というエラーが発生し、
    deposit_account_id: BasicIncomeAccount.find_by(account_number: params[:deposit_account_id]).id,
    の部分がおかしいと表示されてしまいました。

    キャンセル

  • 2017/11/22 11:29 編集

    解決できました!
    どうやら、パラメータの渡し方が悪くてうまくいかなかったみたいです。
    deposit_account_id: BasicIncomeAccount.find_by(account_number: params[:deposit_account_id]).id,
    ではなくて、
    deposit_account_id: BasicIncomeAccount.find_by(account_number: params[:account_transaction][:deposit_account_id]).id,
    とすることで正常に機能します。

    最終的には、
    def create
    @account_transaction = AccountTransaction.new(
    withdrawal_account_id: 20 #テスト用、idを直接入力
    deposit_account_id: BasicIncomeAccount.find_by(account_number: params[:account_transaction][:deposit_account_id]).id,
    amount: params[:amount]
    )
    となりました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.51%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る

  • トップ
  • Rubyに関する質問
  • フォームで入力された値から該当するidを取得してモデルに外部キーとして記録する方法ってありますか