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

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

ただいまの
回答率

87.49%

主キーとして使えるユニークでランダムな数字だけの乱数を作成し、被ったら再生成できるようにしたい

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 2
  • VIEW 1,875

score 41

・前提・実現したいこと

ruby(rails)で1:1の関連付けをしたユーザーモデルと口座モデルがあり、
deviseでユーザーが新規会員登録と同時に自動的に口座を作成する実装をしました。
このときランダムかつユニークで主キーとなり得る11桁の数字だけの乱数(口座番号)を作成して登録するのですが、
万が一に口座番号が被って登録ができなかった場合、再び乱数を生成して登録を成功させるように
するにはどうすればよろしいでしょうか?

現状のコード

・ユーザーモデルと口座モデルは1:1で関連付けをしてあります。
・deviseのカスタマイズした新規会員登録コントローラに、口座への同時作成の指示を記述しています。
・口座モデルには、口座番号がユニークであるかどうかを検証しています。

app/controllers/users/registrations_controller.rb

class Users::RegistrationsController < Devise::RegistrationsController

   def new
     super
   end

  # POST /resource
   def create
     super
     #Userの子要素であるbasic_income_accountを関連付ける
     resource.build_basic_income_account
     #同時につくられるBI口座の登録内容
     #12桁のランダムでユニークな口座番号を作成する
     require 'securerandom'
     n = 12
     resource.basic_income_account.account_number = format("%0#{n}d", SecureRandom.random_number(10**n))
     #同時に作られる口座の残高は0とする。
     resource.basic_income_account.balance = 0
     resource.save
   end

/home/vagrant/megterra/app/models/basic_income_account.rb

class BasicIncomeAccount < ApplicationRecord
    #BasicIncomeAccountはuserから1:1の関係で所有されている
    belongs_to :user

    #ユーザーIDが存在するなら検証成功
    validates :user_id, presence:true
    #口座番号が存在し、値(口座番号)がユニーク(被らない番号)なら検証成功
    validates :account_number, presence: true, uniqueness: true
    #口座残高は数値か小数点のみ有効
    validates :balance, presence: true, numericality: true
end

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

Rails 5.1.3
ruby 2.4.1
devise (4.3.0)

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

+2

口座番号が11桁か12桁かで表記がゆれているようです。
意図的に重複を起こす等して、そのような場合にコードがどのような挙動となるかを確認されると、重複した場合のリカバリ処理が書けるかと思います。
おそらく主キーの重複に関する例外が発生するのではないかと思いますので、例外が発生した場合に再処理を行うといった形が考えられます。

口座番号が完全にランダムな数値となる場合、正しい口座番号かどうかは必ずデータベースで確認しなくてはいけません。
チェックディジットを付与するなどすると、口座番号自体からある程度適切な番号かどうかが確認ができるようになるので、多少チェックしやすくなります。
また、何らかの理由で口座番号を変更する理由が発生した場合のことも考えると、口座番号はユニークではあっても主キーとしての使用はしない方がよい場合もあるかもしれません。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/16 12:21 編集

    suyamaさん。
    アドバイスありがとうございます。

    質問の前提を崩してしまってすみませんが、
    アドバイスを参考に主キーの変更はやめて、railsがデフォルトで作成してくれるidを主キーのままにしました。
    ただし、口座番号は電話番号の様にユニークな(被ることのない)数字だけの文字列にしたいと思います。


    models/basic_income_account.rb(口座モデルファイル)には、
    validates :account_number, presence: true, uniqueness: true
    と記述し、口座番号がユニークじゃないと登録できないようにしてあります。

    registrations_controller.rbに例外処理を記述しました(例外処理の記述内容は、2017/11/16 10:47のコメントに書いた通りです)
    例外を発生させようと既に登録済みの番号を入れたのですが、
    例外処理(新しく乱数を作成してresource.basic_income_account.account_numberに代入する)は行われず、
    口座は作成されませんでした。

    ということは、同じ口座番号は登録できないようにはなっているみたいです。
    しかし、登録されないのならば例外処理を実行するハズなのですが、していないみたいです。

    蒸し返し的な質問ですみませんが、
    どうすれば例外処理が正しく行われるようになるのでしょうか?

    キャンセル

  • 2017/11/16 17:45

    横からですが、保存する処理は、必ずビックリマーク付きのメソッドで実行してください。
    save!, create!, update! です。
    ビックリマーク付きのメソッドは、rubyの文化ではオブジェクトの内容が変更される、または失敗時に例外が発生するというメソッドの印として用いられます。
    railsもこれに従っていて、ビックリマークなしのsaveは結果をtrue/falseで返すだけで例外は発生させません。

    キャンセル

  • 2017/11/16 20:58 編集

    mingosさん、アドバイスありがとうございます。
    saveの部分をsave!に書き換えました。

    さらに、account_numberの作成部分だけでなく、balanceと保存を実行するsave!も
    begin以下(エラーが発生する可能性のある処理として)に記述しました。
    しかし、それでも新規会員登録だけが成功し、口座登録はできませんでした。

    def create
    super
    resource.build_basic_income_account
    begin
    #例外を起こす処理
    resource.basic_income_account.account_number = '231229948564'
    resource.basic_income_account.balance = 0
    resource.save!
    rescue
    #例外発生時の処理。
    require 'securerandom'
    n = 12
    resource.basic_income_account.account_number = format("%0#{n}d", SecureRandom.random_number(10**n))
    resource.basic_income_account.balance = 0
    resource.save!

    end

    end

    キャンセル

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

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

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

  • トップ
  • Rubyに関する質問
  • 主キーとして使えるユニークでランダムな数字だけの乱数を作成し、被ったら再生成できるようにしたい