设计模式 – 服务层方法应该期望实例还是id？

这个问题源自我在Grails应用程序上的工作,但它适用于几乎所有在层中开发的Web应用程序.这是一个简单的例子：

class OrderService {

    // Option 1
    def shipOrder(Order order) {
        order.status = OrderStatus.SHIPPED
        emailService.sendShipmentEmail(order)
        // ...
    }

    // Option 2
    def shipOrder(long orderId) {
        def order = Order.get(orderId)
        order.status = OrderStatus.SHIPPED
        emailService.sendShipmentEmail(order)
        // ...
    }

}

这些选项中的任何一个都记录为比另一个更好吗？

解决方法

我倾向于选择id,因为你有时想要使用悲观锁定,然后很容易将Order.get(orderId)更改为Order.lock(orderId).锁定必须在事务中发生,因此使用您在读取后锁定的第一种方法,在中间运行较小的更新风险.

有时需要在服务之外加载实例,例如测试控制器中是否存在,所以第二种方法可以感觉它浪费了数据库调用.但是您可以将get()调用更改为exists()调用并仅检查是否存在id,而不是仅仅为了查看它是否存在而加载整个实例.

请注意,您应该在方法签名中使用long orderId,因为允许null id没有意义.