オブジェクトの境界を越えて流出する情報

多くの場合、私のビジネスオブジェクトは、情報がオブジェクトの境界を頻繁に越える必要がある状況にあります。オブジェクト指向を実行するときは、情報を1つのオブジェクトに入れ、その情報を処理するすべてのコードをできるだけそのオブジェクトに含める必要があります。ただし、ビジネスルールはこの原則に従っていないため、問題が発生します。

例として、価格を持つInventoryItemを参照するOrderItemの数を持つOrderがあるとします。数量をInventoryItem.GetPrice（）で乗算するOrderItem.GetPrice（）の結果を合計するOrder.GetTotal（）を呼び出します。ここまでは順調ですね。

しかし、その後、一部の商品が2対1で販売されていることがわかりました。OrderItem.GetPrice（）にInventoryItem.GetPrice（quantity）のような処理を行わせ、InventoryItemにこれを処理させることで、これを処理できます。

ただし、2対1の取引は特定の期間のみ続くことがわかります。この期間は、注文の日付に基づく必要があります。次にOrderItem.GetPrice（）をInventoryItem.GetPrice（quatity、order.GetDate（））に変更します。

ただし、お客様がシステムに滞在している期間に応じて、さまざまな価格をサポートする必要があります。

しかし、2対1の取引は、同じ在庫アイテムを複数購入するだけでなく、InventoryCategory内のアイテムの複数購入にも適用されることがわかります。この時点で、私たちは手を挙げて、InventoryItemに注文アイテムを与え、それがアクセサーを介してオブジェクト参照グラフ上を移動できるようにし、必要な情報を取得します：InventoryItem.GetPrice（this）

TL; DRオブジェクトの結合を低くしたいのですが、ビジネスルールでは、特定の決定を行うために、多くの場合、どこからでも情報にアクセスする必要があります。

これに対処するための良いテクニックはありますか？他の人が同じ問題を見つけますか？

私たちが働いているのと基本的に同じ経験があり、OrderBusinessLogicクラスを持つことでそれに対処しました。あなたが説明したレイアウトの大部分は、私たちのビジネスの大部分で機能します。それは素晴らしく、清潔でシンプルです。ただし、このカテゴリから2つを購入した場合は、それを「ビジネス例外」として扱い、OrderBLクラスに必要なオブジェクトをたどって合計を再計算させます。

いいえ、それは完璧な解決策ですか？まだ1つのクラスが他のクラスについてあまりにも多くのことを知っていますが、少なくともその必要性をビジネスオブジェクトからビジネスロジッククラスに移動しました。

あなたのようなサウンドは、すべてのものを追跡し、個別のディスカウントオブジェクト（またはそれらのリスト）を必要とし、その後何かのように、注文にディスカウント（複数可）を適用するOrder.getTotal(Discount)か、Discount.applyTo(Order)または類似しました。

他のクラスのデータにアクセスしても問題ありません。ただし、これを一方向の関係にする必要があります。たとえば、ClassOrderがCallItemにアクセスするとします。理想的には、ClassItemはClassOrderにアクセスすべきではありません。注文クラスで欠けていると思うのは、ウォルターが提案するようなクラスを保証するかどうかわからない、ある種のビジネスロジックです。

編集：ウィンストンのコメントへの応答

私はあなたがインベントリアイテムオブジェクトをまったく必要としないと思います...少なくともあなたがそれを使用している方法で。代わりに、在庫データベースを管理する在庫クラスがあります。

在庫品はIDで参照します。各注文には、在庫IDのリストと対応する数量が含まれます。

それから私はこのようなもので注文の合計を計算します。
Inventory.GetCost（Items、customerName、date）

次に、次のような他のヘルパー関数を使用できます。

Inventory.ItemsLefts（int itemID）
Inventory.AddItem（int itemID、int数量）
Inventory.RemoveItem（int itemID、int数量）
Inventory.AddBusinessRule（...）
Inventory.DeleteBusinessRule（...）

これについてもっと考えた後、私は自分自身の代替戦略を考え出しました。

Priceクラスを定義します。Inventory.GetPrice()価格オブジェクトを返します

Price OrderItem::GetPrice()
{
    return inventory.GetPrice().quantity(quantity);
}

Currency OrderItem::GetTotal()
{
    Price price = empty_price();
    for(item in order_items)
    {
         price = price.combine( item.GetPrice() )
    }
    price = price.date(date).customer(customer);
    return price.GetActualPrice();
}

現在、Priceクラス（および場合によってはいくつかの関連クラス）は、価格設定のロジックをカプセル化しており、注文はそれについて心配する必要がありません。PriceはOrder / OrderItemについて何も知らず、代わりに単に情報が入力されるだけです。