トランザクションの設定がうまくいきません。。

Question

サイト内でポイントを使って、買い物ができる様な仕組みを作っています。
ユーザーの保持ポイント(@user.point)＞＝注文の値段(@order.price)の時に
orderアクションに飛ぶ仕組みにしてあるのですが、

下記のコードの場合、仮にポイントが足りなくても注文記録としてsaveされてしまいます。
トランザクションが必要になると思い、ずっと格闘しているのですがうまくいきません。。。

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

def create
  @user = current_user
  @order = Order.new(order_params)
  @order.user = current_user

  if @order.save && @user.point >= @order.price

  order

  else
    redirect_to orders_error_path
  end
  end

Answer 1

if @order.save && @user.point >= @order.price
  order

このコードは @order.save を評価した後に @user.point >= @order.price を評価しちゃってます。
単純に順番を入れ替えると良いです↓↓

if @user.point >= @order.price && @order.save
  order

Comments

[engeng]

ご回答ありがとうございます！
saveを先に通さないと、@order.priceがnilになってしまいます。。。
他に何かやり方あったりしませんでしょうか(T . T)

[set0gut1]

いろいろ考えたんですが、

- @order.save じゃなくて Order.new の時点で @order.price を計算しておく
- @user.point >= @order.price の判定は save 中で行うようにする（DBのトランザクションを使うならこちら）
- いったん save して、条件を満たしてなければ取り消しの save をするようにする（cancelカラムとかが必要になって少し面倒かも）

の3つほど思いつきました。

[engeng]

当初の通りトランザクションでやってみようと思います。
アドバイスの方感謝いたしますmm