Автогенерация id когда id primary key строка?

Question

[Андрій Степанюк]

Добрый день, вопрос такой, как лучше сделать автогенерацию id когда это строка?

миграция:

def change
    create_table :names, id: false do |t|
      t.string :id, limit: 21, null: false
      ...
    end
    add_index :names, :id, unique: true
end

В моделе соответсвенно:

class Name < ActiveRecord::Base
  self.primary_key = :id
  before_create :set_id

private
  def set_id
    if self.id.blank?
      for attemps in 0..3
        self.id = generate_id
        return if Name.find_by(id: self.id).nil?
      end
    end
  end
end

Функция generate_id генерирует случайный строковой id длинной 21 символ с помощью SecureRandom.

Очевидно, что это решение может упасть с исключением если одновременно будет сгенерирован одинаковый id, или если количества попыток будет недостаточно. Всвязи с этим вопрос: как сделать что бы было надежно? Я понимаю, что вероятность того что это мое решение даст сбой ничтожно мала, но тем не менее интересно какой путь правильный.

Прочитал статью:
ruby-journal.com/how-to-override-default-primary-k...
Там автор рекомендует не делать execute %Q{ ALTER TABLE "products" ADD PRIMARY KEY ("sku"); }

Comments

[thepry]

А эти строковые id расположенны по порядку, или это просто случайные знаки?

[Андрій Степанюк]

просто случайные знаки типа (на самом деле генерируются по другому, но да, случайные):

p SecureRandom.uuid #=> "2d931510-d99f-494a-8c67-87feb05e1594"

Answer 1

[rutaka nashimo]

Не совсем понимаю зачем делать id строкой, но если нужно просто уникальное строковое значение, то можно использовать uuid

Comments

[Андрій Степанюк]

Зачем так делать - это вопрос предметной области в которой я работаю.

Нет, uuid я использовать не могу, на id есть ограничения на используемый алфавит в строке. впрочем, так же как и с uuid вероятность совпадения двух случайно сгенерированных id ничтожно мала. Но вопрос был не в этом.

[rutaka nashimo]

тогда можно отловить исключение и сгенирировать id еще раз, тогда вероятность коллизии будет еще меньше.

[Андрій Степанюк]

да Вы правы, но это не то о чем я спрашиваю, по факту вот этим
for attemps in 0..3
self.id = generate_id
return if Name.find_by(id: self.id).nil?
end

я тоже уменьшаю вероятность фак-апа, а вопрос в том как сделать что бы эта вероятность была нулевой.

транзакии? но как правильно это сделать?

[rutaka nashimo]

я не уверен, но попробуй self.transaction {} если он вообще доступен на момент инициализации.