【Rails】モデル内で関数を再帰的に呼び出し、ユニークな文字列を作成する

Question

前提・実現したいこと

Ruby 2.4.5
Rails 4.2.8
Gem: devise, friendly_url

https://サイト名/users/:id
の:idをユニーク制限（被らない）ランダム文字列にしようとしています。

また、friendly_urlを使っており、ユーザーが編集画面で自分の好きなURLを設定できるようにしています。
ただ、その前にユーザー登録時にランダム文字列を自動的に付与しデフォルトのURLにしたいと考えています。
(users/:idだと1から連番になり推測されやすいため)

Userモデル内で、新規ユーザー登録時に、カラムfrienly_urlにランダムな文字列を挿入しています。
しかし、friendly_urlは、ユーザー登録後、ユーザー自身で変更できるので、重複しないようチェックをかけ、重複していた場合は再帰的に呼び出し再作成するようにしたいです。

該当のソースコード

User.rb

class User < ActiveRecord::Base
 before_create :generate_url

(省略)
#friendly_url
  include FriendlyId
  friendly_id :friendly_url, :use => [:finders]

  validates :friendly_url, length: { in: 3..20 },
                         uniqueness: true, #一意性
                         format: { with: /\A[\w@-]*[A-Za-z][\w@-]*\z/ },
                         # on: :friendly_url_user #登録時に入力必要なし
                         allow_nil: true #無入力許可

 #ユーザー登録時のfriendly_id自動生成
  def generate_url
    if friendly_url.nil?
      self.friendly_url = SecureRandom.hex(10)
      self.save
      #ここで、ランダムに作成したfriendly_urlが既に存在する場合、再帰的に呼び出してもう一度関数実行したい
    end
  end
end

試したこと

再帰的な関数を色々調べたのですが分からず詰まってしまっています。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

再帰にする必要は無いでしょう。

self.friendly_url = SecureRandom.hex(10)
untile self.save
  self.friendly_url = SecureRandom.hex(10)
end

厳密には save　失敗の原因が uniqueness　のみで有ることも調べる必要はあるでしょう。

ところで、saveで調べなくても

friendly_url = SecureRandom.hex(10)
while User.find_by(friendly_url: friendly_url)
  friendly_url = SecureRandom.hex(10)
end

でも ほぼ 同じことができますし、saveでやるのとちがって、uniqunessであることが確実ですが、saveでやる必要が有るほどクリティカルですか？

Comments

[Romay]

コメントありがとうございます。
1つめは、until文でsaveができるまで繰り返すということですね。
2つめは、作成したランダム文字列が存在するかどうかfind_byして、falseになるまで繰り返す、という認識で間違い無いでしょうか？
2つめの場合、その後にfriendly_url.saveをしてあげる必要もありますでしょうか？
2つめのほうが分かりやすくていいなと個人的には思いました！

[winterboum]

２つめはそれでユニークな friendly_url を得て、そしてだいにゅうしてsaveが必要です。
SecureRandom　使っているから大丈夫だろうと思いますが、この方法は注意すべきところが有ります。　ユニークであることを確認してSAVEする間に他の人がユニークであることを確認してsaveしてしまうかもしれない。ニックナームとか名前とかユーザに自由に入れさせるものでのユニーク性を確保する場合は、そこに注意が必要です

[Romay]

なるほど、おっしゃるとおりその可能性も考えられますね。
それを考慮する場合、どのような設計をすべきだとお考えでしょうか？

[Romay]

def generate_url
if friendly_url.nil?
friendly_url = SecureRandom.hex(10)
while User.find_by(friendly_url: friendly_url)
friendly_url = SecureRandom.hex(10)
end
self.friendly_url = friendly_url
end
end
というかたちで、得られたユニークなfriendly_urlをselfで代入する形にしたところ、保存もされておりうまくいきました！しかし、self.saveを追加したところエラーが出てしまいました。この場合、saveは必要ないのでしょうか？self.friendly_url = friendly_urlのみで保存されていたため、疑問に思いました。

[winterboum]

ああ、うっかりしてました。
has_*, belngs_to が定義してあり、かつ この例では self.friendly_url の　self がDBにsave済であった場合は、　self.friendly_url = friendly_url　でsaveされます

[winterboum]

＞それを考慮する場合、どのような設計をすべきだとお考えでしょうか？
case by case です。一般的に説明できるほどは私もベテランではないので、、、、

[Romay]

なるほど！アソシエーション定義しているかつsave済みだからだったのですね！
今回の場合は、saveする間に他のユーザーがsaveするという状況は考えにくいので無視することとします。
ちなみに、上記で成功したのですが、ランダム文字列が20文字になっております。SecureRandom.hex(10)で、10文字と認識していたのですが、2回生成されて繋がってしまっているのでしょうか？

[winterboum]

大抵の説明には 16進10文字　と読める説明なのですが、どうも １０Byteの１６進表現の様です。
ソース読んでいないので結果からの判断です

[Romay]

10Byteだとしても、半角文字だと10文字になるとの認識なのですが、違いますでしょうか。自分は知識が乏しく、2回生成されているのでは？と思ってしまい。。。

[winterboum]

１６進文字10Byteではないです。
10Byteの整数の１６進表現です。
1Byte の整数を16進で表現すると２文字です。

[Romay]

なるほど！そういうことですか。大変勉強になりました！！ご回答いただきありがとうございました！