transaction処理の方法について

Question

前提・実現したいこと

bicycle tableのnumberを変更したときに、history tableのbicycle_idを更新処理を書いています。

1　bicycle tableのnumberがすでに存在する場合
numberに該当するidを、bicycle_idとしてhistory tableを更新。

2　bicycle tableのnumberが存在しない場合
bicycle tableに新たに登録、作成されたidを、history tableの該当レコードを更新。

ここで、
2番のbicycle tableに登録後、history tableの更新時にエラーがあった場合、
bicycle tableに登録されたものは、rollbackされるようにするためにどうすればよいでしょうか？

transaction処理を書いているんですが、うまく処理されませんでした。

該当のソースコード

  def correction
    logger.info "update start"
    record = History.find(params[:id])
    bicycle = Bicycle.find_by(number: params[:number])

    # ------- transaction start -------------
    ActiveRecord::Base.transaction do
    
    # number あり
    if bicycle
      begin
        record.update!(bicycle_id: vehicle.id, correction_flg: 1)
        render json: { status: 200, message: "Success correction" } 
      rescue
        render status: :internal_server_error
      end
    else
      # number なし
      logger.info "no result. insert new record."
      begin
        bicycle = Bicycle.create!(number: params[:number])
        if bicycle
           record.update!(bicycle_id: bicycle.id, correction_flg: 1)
           render json: { status: 200, message: "Success correction" }
         end
      rescue
        render status: :internal_server_error
      end
    end #if bicycle end
  end # transaction end
end # correction

Comments

[maisumakun]

> うまく処理されませんでした。

具体的に、どうなっているのでしょうか。

Answer 1

前提としてActiveRecordではtransaction do ～ endの中で例外が起きた時にロールバックが行われます。
したがって、その中でbegin, rescueで例外を握りつぶしてしまったら、
ロールバックのトリガーが発動しません。

ざっくり言うとこういう事です。
begin～endはトランザクションの外に持っていきます。

begin
  ActiveRecord::Base.transaction do
    `ロールバックするためにはここで例外を出さなければならないので、
    ここにはbegin-rescue-endは書いてはいけない`
  end

  # 成功(コミット)
  render json: { status: 200, message: "Success correction" }
rescue Exception => e
  # 失敗(ロールバック)
  render status: :internal_server_error
end