サービスレイヤメソッドはインスタンスまたはIDを要求する必要がありますか？

Question

この質問は、私の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) 
     // ... 
    } 

} 

これらのオプションのいずれかが他のものより優れていると文書化されていますか？

Answer 1

私は、悲観的なロックを使いたい場合があり、Order.get(orderId)からOrder.lock(orderId)に変更するのが簡単なので、私はIDを好む傾向があります。ロックはトランザクション内で発生する必要があります。最初のアプローチを使用すると、読んだ後にロックし、小さな更新のリスクをその間に実行します。

場合によっては、サービスの外部にインスタンスを読み込む必要があります。コントローラーに存在をテストするため、2番目のアプローチはデータベース呼び出しを無駄にするように感じることができます。しかし、get()コールをexists()コールに変更し、インスタンスが存在するかどうかを確認するためだけにインスタンス全体をロードするのではなく、IDの存在を確認することができます。

ヌルIDを許可することは意味をなさないので、メソッドシグネチャにはlong orderIdを使用する必要があります。