トランザクションを使いたい。。。

Question

@user.point >= @order.priceの時のみ
saveが動くように、トランザクションを設定したいのですが、saveが動いてしまいます。。

どなたか、ご教授願えませんでしょうか。よろしくお願いいたします

def create
  @user = current_user
  @order = Order.new(order_params)
  @order.user = current_user
 Order.transaction do
  if @order.save && @user.point >= @order.price

  order

  else
    redirect_to orders_error_path
  end
 end
end

Answer 1

@user.point >= @order.priceを先に判定しないと、.saveが先に処理されます。

Comments

[engeng]

早速のご回答ありがとうございます！：：
@order.priceは@order.quantityに応じてorder.rbの中でbefore_actionの元、
計算されるように設定してあるので、saveの前だと、@order.priceがnilになってしまうのですが、何かいい方法ございませんでしょうか：：

[maisumakun]

どうしてもトランザクションに頼るのであれば、条件を満たさなければロールバックする（raise ActiveRecord::Rollbackとする）ような形にすることになると思います。

もっとも、このようなレベルの不整合でロールバックを持ち出すのも大げさな気もします。

[engeng]

トランザクションに頼らないやり方で、
@order.save　ではなく Order.new の時点で @order.price を計算をと思い、
before_actionの所をafter_initialize :keisan_priceにしてみた所、うまく機能いたしました！
ありがとうございました！