你的姿势正确吗？你搜一下使用电脑的正确姿势，严格按照那个来，症状大概会减轻一半。

我以为是在一个地方注册了就可以到处登录了，结果并不是。

@ericguo 多谢！

感谢！

@vincent 好的，非常感谢您的建议！

这里的order_item应该不会让用户直接操作，所有没有验证的问题。失败是正常，如果保存失败，能保证回滚和处理异常就好。创建之前都要执行validate的。

嗯，controller加ActiveRecord::Base.transaction do，觉得有些不对味。

感谢。

如果按照您这样 create order 和 create order_items解决了， 还有最后一步逻辑，下单后，清空购物车。

Cart.where(user_id: current_user.id).destroy_all

这个应该包含在事务中？这个逻辑该放在哪里呢？

@vincent , @mengqing 改造了一下，如果使用ActiveRecord::Base.transaction的话

将create逻辑放到了controller中核心代码如下：

def create
    ActiveRecord::Base.transaction do
      @order = Order.new(user_id: current_user.id, orderID: @orderID , message:params[:message], fixed_address: Address.find(address_id).full_address, total_price: @total)
      @order_saved = @order.save!
      current_user.carts.each do |cart|
        OrderItem.new(user_id: current_user.id, orderID: @orderID, product_id: cart.product_id, price: cart.product.price, amount: cart.amount).save!
        # raise "crash for test"
      end
      #raise "crash for test"
      Cart.where(user_id: current_user.id).destroy_all
    end
end

将delete逻辑放到了controller中核心代码如下：

def destroy
    ActiveRecord::Base.transaction do
      OrderItem.where(orderID: @order.orderID).destroy_all
      @order.destroy!
    end
end

@mengqing 这个明白了，多谢！

Any exception that is not ActiveRecord::Rollback or ActiveRecord::RecordInvalid will be re-raised by Rails after the callback chain is halted. Raising an exception other than ActiveRecord::Rollback or ActiveRecord::RecordInvalid may break code that does not expect methods like save and update_attributes (which normally try to return true or false) to raise an exception.