Rails 処理でリセットする場合、ロールバックを使ってもよいでしょうか

Question

Rails6、 MySql8で先着順の商品購入サイトをつくっています。

同一の商品(item)を複数のユーザーが購入できないようにしています。
購入の際、悲観的ロックをかけて、itemのstatusがavailableのものだけを購入できるようにしています。
もし、ほかのひとが購入などしてしまって、itemのstatusが、availableではなくなった場合は、
else以降で、changedに追加したもともとの値をつかってリセットし、breakしています。

実際には、購入時は、関連テーブルの処理など複雑になりますが、
これを例えば、else以降で、raiseでロールバックを起こして、処理をリセットさせるほうが
コードを書く際は簡単なのですが、これは行わない方がよいでしょうか。

確か処理が遅くなるという記述をどこかでみたような気がするのですが、
よろしくお願いいたします。

def update_items(item_ids:)
  changed = []
  results = { success: true, count: 0, price: 0 }
  Item.transaction do
    item_ids.each do |item_id|
      item = Item.lock.find(item_id)
      if item.status == "available"
        changed << ticket.dup
        results[:count] += 1
        results[:price] += t.price
        item.update!(status: "purchased")
      else
        changed.each do |c|
          item = Item.lock.find(c.id)
          item.update!(status: c.status)
        end
        results[:success] = false
        break
      end
    end
  end
  results
end

Answer 1

例外を起こして ロールバックというのが（速度はわかりませんが）一般的ですね。
transactionという仕掛けが用意されています。

Comments

[orori]

なるほど、通常はそのようにされているのですね！

お聞きしたいのですが、エラーを補足して、ＤＢの現状を返す場合、
例えば、コントローラは以下のようになりますでしょうか？

def update
Item.transaction do
(処理)
@item = Item.all
　　　 @message = ''商品を購入しました'
end
rescue ActiveRecord::RecordInvalid
@item = Item.all(処理されてない現状を返す）
@message = "該当の商品は既に購入されました"
end

すみません、いろいろやり方はあるかと思いますが、
よろしくお願いいたします。

[orori]

rescue ActiveRecord::RecordInvalidだと、補足されないですね・・・

例外はtransactionの外側では捕捉されないので、
transaction内部でsuccessなどのパラメータをつくって、それを用いるしかないようですね。

ありがとうございました！！