Nesne Sınırlarında Bilgi Dökülmesi

Çoğu zaman iş nesnelerim, bilginin nesne sınırlarını çok sık aşması gereken durumlara sahip olma eğilimindedir. OO yaparken, bilgilerin bir nesnede olmasını ve bu bilgilerle ilgili tüm kodların o nesnede olabildiğince fazla olmasını istiyoruz. Ancak iş kuralları bu ilkeye uymuyor ve bana sorun çıkarıyor.

Örnek olarak, bir fiyatı olan bir InventoryItem anlamına gelen bir dizi OrderItems içeren bir Siparişimiz olduğunu varsayalım. InventoryItem.GetPrice () ile bir miktarı çarpan OrderItem.GetPrice () sonucunu toplayan Order.GetTotal () işlevini çağırıyorum. Çok uzak çok iyi.

Ama sonra bazı eşyaların bir anlaşma için iki ile satıldığını öğreniyoruz. OrderItem.GetPrice () öğesinin InventoryItem.GetPrice (miktar) gibi bir şey yapmasını ve InventoryItem'in bununla ilgilenmesine izin vererek bunu halledebiliriz.

Ancak, iki-bir-bir anlaşmanın sadece belirli bir süre için devam ettiğini öğreniyoruz. Bu zaman dilimi siparişin tarihini temel almalıdır. Şimdi OrderItem.GetPrice () öğesini InventoryItem.GetPrice olarak değiştiriyoruz (quatity, order.GetDate ())

Ancak, müşterinin sistemde ne kadar süredir bulunduğuna bağlı olarak farklı fiyatları desteklememiz gerekir: InventoryItem.GetPrice (miktar, order.GetDate (), order.GetCustomer ())

Ancak, iki-bir-bir anlaşmanın sadece aynı envanter öğesinin birden fazlasını satın almak için değil, aynı zamanda bir InventoryCategory içindeki herhangi bir öğe için birden fazla satın almak için geçerli olduğu ortaya çıkıyor. Bu noktada ellerimizi atıyoruz ve sadece InventoryItem'e sipariş öğesini veriyoruz ve ihtiyaçlarını almak için erişimciler aracılığıyla nesne referans grafiğinin üzerinden geçmesine izin veriyoruz: InventoryItem.GetPrice (this)

TL; DR Nesnelerde düşük bağlantım olmasını istiyorum, ancak iş kuralları genellikle belirli kararlar alabilmek için her yerden bilgiye erişmeye zorluyor.

Bununla başa çıkmak için iyi teknikler var mı? Diğerleri de aynı sorunu bulur mu?

Temelde çalıştığım yerde aynı deneyime sahibiz ve bunu OrderBusinessLogic sınıfına sahip olarak ele aldık. Çoğunlukla, tanımladığınız düzen, işimizin çoğunluğu için işe yarar. Güzel, temiz ve basit. Ancak, bu kategoriden herhangi bir 2 satın aldığınız okkaizyonlarda, bunu bir "iş yürütme" olarak ele alıyoruz ve OrderBL sınıfının gerekli nesneleri gezerek toplamları yeniden hesaplamasını sağlıyoruz.

Mükemmel bir çözüm mü, hayır. Hâlâ bir sınıfımız var, diğer sınıflar hakkında çok fazla şey biliyor, ama en azından bu ihtiyacı iş nesnelerinden ve bir iş mantığı sınıfına taşıdık.

Senin gibi Sesler tüm bu şeyler izler ayrı İndirim nesnesi (veya onların bir listesini) gerekir ve sonra bir şey gibi Siparişine İndirim (ler) uygulamak Order.getTotal(Discount)ya Discount.applyTo(Order)veya benzer.

Diğer sınıflardan verilere erişmek son derece uygundur. Ancak bunun tek yönlü bir ilişki olmasını istiyorsunuz. Örneğin, ClassOrder öğesinin CallItem'e eriştiğini varsayalım. İdeal olarak, ClassItem ClassOrder erişmemelidir. Sipariş Sınıfınızda eksik olduğunu düşündüğüm, Walter'ın önerdiği gibi bir sınıfı garanti edebilecek veya etmeyecek bir tür iş mantığı.

Edit: Winston kullanıcısının yorumuna yanıt

Hiç bir envanter öğesi nesnesine ihtiyacınız olduğunu sanmıyorum ... en azından onu kullandığınız şekilde. Bunun yerine bir envanter veritabanı yöneten bir envanter sınıfı olurdu.

Bir kimlikle envanter öğelerine atıfta bulunacağım. Her sipariş, envanter kimliklerinin bir listesini ve karşılık gelen bir miktarı içerir.

Sonra siparişin toplamını böyle bir şeyle hesaplardım.
Inventory.GetCost (Öğeler, customerName, tarih)

Sonra başka bir yardımcı işlevi olabilir:

Inventory.ItemsLefts (int itemID)
Inventory.AddItem (int itemID, int miktar)
Inventory.RemoveItem (int itemID, int miktar)
Inventory.AddBusinessRule (...)
Inventory.DeleteBusinessRule (...)

Bunu daha düşündükten sonra, kendi alternatif stratejimi buldum.

Bir Fiyat sınıfı tanımlayın. Inventory.GetPrice()bir fiyat nesnesi döndürür

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();
}

Şimdi Price sınıfı (ve muhtemelen bazı ilgili sınıflar) fiyatlandırma mantığını kapsamaktadır ve düzen bunun için endişelenmek zorunda değildir. Fiyat, Order / OrderItem hakkında hiçbir şey bilmez, bunun yerine sadece bilgi beslenir.