我也有同感，感觉比较合适的方式是按功能模块分。

一个功能模块只能由一个团队负责（包括，开发，测试，运维），一个团队可以负责多个功能模块。

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

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

@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