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

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

ただいまの
回答率

91.35%

  • Ruby

    5173questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    5051questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

他のテーブルの値をvalidates(検証)の一つとして利用したい

受付中

回答 1

投稿 2017/12/05 17:06 ・編集 2017/12/07 08:37

flag 質問者が4日前に「まだ回答を求めています」と言っています。

  • 評価
  • クリップ 0
  • VIEW 115

zendendo

score 12

前提・実現したいこと

ruby(ruby on rails)で、ポイント振替機能みたいなものをつくっていているのですが、
2つの問題が起きてしまっています。
一つ目は残高以上の数値を入力しても取引が実行されてしまうので、
モデルのvalidates(検証)で、支払ユーザー(現在ログインしているユーザー)が保有する口座残高以下の取引額のみ検証成功」にしたいと思っています。

ただ、残高を記録するモデルと取引額を記録するモデルは別々に分かれています。
図にするとこんな感じです。

イメージ説明
例えば、AccountTransactionモデルにamount(取引額)を記録する前に、ログインユーザーの保有する残高が記録されたBasicIncomeAccountモデルのbalance(残高)を参照して、
「入力された値が残高以下なら検証成功として記録」
「入力された値が残高以上なら検証失敗として記録しない」
ようにしたいのですのですが、やり方がわからず困っています。

2つ目の問題は、入力フォームでマイナスの取引額を入力してしまうと、出金処理と入金処理が逆転してしまうので、「入力フォームでマイナス数値の入力を不可」にしたいと思っているのですが、フォーム側で制限ってできるのでしょうか?それとも、モデル側で制限した方がいいのでしょうか?

教えて頂ければ幸いです。

現状のコード

入力フォームです。
ユーザーは、口座番号と送金額だけ入力しています。
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 %> 
  <%= f.submit"送金確認" %>
</div>
<% end %>

account_transactions(取引)テーブル、
basic_income_accounts(口座)テーブル、
の内容です。

  create_table "account_transactions", force: :cascade do |t|
    t.decimal "amount"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer "withdrawal_account_id"
    t.integer "deposit_account_id"
    t.index ["deposit_account_id"], name: "index_account_transactions_on_deposit_account_id"
    t.index ["withdrawal_account_id"], name: "index_account_transactions_on_withdrawal_account_id"
  end

  create_table "basic_income_accounts", force: :cascade do |t|
    t.integer "user_id"
    t.string "account_number"
    t.decimal "balance"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["user_id"], name: "index_basic_income_accounts_on_user_id"
  end


また、BasicIncomeAccount(口座モデル)AccoutTransaction(取引モデル)は、
1:多の関連付けが施されています。

app/models/basic_income_account.rb

class BasicIncomeAccount < ApplicationRecord


    belongs_to :user

    validates :user_id, presence:true
    validates :account_number, presence: true, uniqueness: true
    validates :balance, presence: true, numericality: true

    #BasicIncomeAccountは、AccountTransactionを多数保有してる1:多の関係である
    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'


    has_many :deposits_and_withdrawals

今回の問題の本題となる取引モデルのコードです。
app/models/account_transaction.rb

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'


  validates :withdrawal_account_id, presence:true
  validates :deposit_account_id, presence:true
  #amount(取引額)が入力され、入力された内容が数値が小数点で、自前の条件付きバリデーションに検証をクリアしたら検証成功という処理を記述したい
  validates :amount, presence: true, numericality: true, if: :balance_check

  has_many :deposits_and_withdrawals

 private
##おそらくカスタムなバリデーションをつくることになるが、その記述方法がわからない
  def balance_check
    if current_user.basic_income_acccount.balance < ??????????
      errors.add(:amount, "残高以上の送金はできません")
    end
  end
end


追記
account_transactionsのコントローラです。
ここで、AccountTransactionとDepositsAndWithdrawalにレコードを作成し、
BasicIncomeAccountの更新を同時かつトランザクション処理でできるようにしております。
(ポイント振替処理ができるようになっています)
/app/controllers/account_transactions_controller.rb

    AccountTransaction.transaction do
   #取引モデルにレコードを作成保存
      @account_transaction = AccountTransaction.new(        
        withdrawal_account_id: current_user.basic_income_account.id,
        deposit_account_id: BasicIncomeAccount.find_by(account_number: params[:account_transaction][:deposit_account_id]).id,
        amount: params[:account_transaction][:amount]
        )
      @account_transaction.save!

      DepositsAndWithdrawal.transaction do
        #入出金モデルに出金側レコードを作成保存
        @account_transaction.deposits_and_withdrawals.build(
          transaction_type: "出金",
          basic_income_account_id: current_user.basic_income_account.id,
          amount: -1 * params[:account_transaction][:amount].to_f
          )
        @account_transaction.save!
        #入出金モデルに出金側レコードを作成保存
        @account_transaction.deposits_and_withdrawals.build(
          transaction_type: "入金",
          basic_income_account_id: BasicIncomeAccount.find_by(account_number: params[:account_transaction][:deposit_account_id]).id,
          amount: params[:account_transaction][:amount].to_f
          )
        @account_transaction.save!

        BasicIncomeAccount.transaction do
     #出金口座と入金口座の残高を更新する
          @withdrawal_basic_income_account = current_user.basic_income_account
          @deposit_basic_income_account = BasicIncomeAccount.find_by(account_number: params[:account_transaction][:deposit_account_id])

          @withdrawal_amount = params[:account_transaction][:amount].to_f
          @deposit_amount = params[:account_transaction][:amount].to_f

          @withdrawal_basic_income_account.update(balance: @withdrawal_basic_income_account.balance - @withdrawal_amount)
          @deposit_basic_income_account.update(balance: @deposit_basic_income_account.balance + @deposit_amount)
        end
      end
    end
    #トランザクション処理が完全成功なら、root_path(ホーム画面)へ飛ぶ
    redirect_to root_path
  #トランザクション処理で例外発生(失敗)したら、以下の例外処理を実行する
  rescue => e
    render plain: e.message

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

Rails 5.1.3
ruby 2.4.1

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

0

まず、問題1についてですが、複数のモデルに関連するバリデーションを1つのモデル内で行うのは止めた方がよさそう。
デザインパターンの 'Form Object' を使ってみてはいかがでしょう。検索してみてください。

例) ※ちょっと適当すぎますがよしなに

class AccountTransactionForm
  include ActiveModel::Model

  attr_accessor :errors, :amount, :deposit_account_id

  def initialize(amount, deposit_account_id, basic_income_acccount)
    @amount = amount
    @deposit_account_id = deposit_account_id
    @balance = basic_income_acccount.balance
    @errors = ActiveModel::Errors.new(self)
  end

  def valid?(zip = nil)
    errors.clear
    if @balance < amount
      errors.add(:amount, "残高以上の送金はできません")
    end
    errors.empty?
  end
end

問題2についてですが、
<%= f.number_field :amount, pattern: '[1-9][0-9]*' %>  とかでできないですかね。(未検証)

投稿 2017/12/06 10:08

編集 2017/12/06 10:09

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/07 08:20 編集

    _RYO_さん、回答して頂きありがとうございます!
    Form Objectなるものは初めて知ったので、調べてみます!

    ひとまず途中経過報告を。
    2つ目の問題「フォームにマイナス入力をさせない」については、
    入力フォームのviewファイルのamount入力欄の記述を
    <%= f.number_field :amount,min:1,max:999999999999999 %>
    とすることで解決することができました。

    そして一つ目の問題「支払口座の残高以上の取引を無効にする」についてはまだ解決できておりません。
    現状では、app/models/form/account_transaction_form.rbに
    上記のコードを記述しましたが、
    コントローラ側にも書き換えを行う必要があるということでしょうか?

    キャンセル

  • 2017/12/07 09:56

    そうですね。Controller側からForm Object を呼ぶ感じです。
    参考) http://tech.medpeer.co.jp/entry/2017/05/09/070758

    イメージとしては、Modelを跨ったの複雑な処理やバリデーションを `Form Object` 内で行います。
    前回の例で抜けてますが、 複数の登録処理をController側でやっている場合は `save` も実装した方が Controller が見やすくなりますね。

    ```
    class AccountTransactionForm
    include ActiveModel::Model

    attr_accessor :errors, :amount, :deposit_account_id

    def initialize(amount, deposit_account_id, basic_income_acccount)
    @amount = amount
    @deposit_account_id = deposit_account_id
    @balance = basic_income_acccount.balance
    @errors = ActiveModel::Errors.new(self)
    end

    def save
    return false unless valid?

    # 複数のModelを使用する保存処理

    true
    end

    def valid?
    errors.clear
    if @balance < amount
    errors.add(:amount, "残高以上の送金はできません")
    end
    errors.empty?
    end
    end
    ```

    あと私事ですが業務が忙しくなってきたので今後コメントできないです。あとは自力でがんばってください。

    キャンセル

  • 2017/12/07 10:38

    _RYO_さん、お忙しいところありがとうございました。

    キャンセル

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

ただいまの回答率

91.35%

関連した質問

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

  • Ruby

    5173questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    5051questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。