Kod 212, Kompozisyon ve yetkilendirme tamamlandı

Question

Birkaç hafta sonra bu forumda okumaya başladım, ilk mesajımı yapmanın tam zamanı olduğunu düşündüm.

Şu anda Code Complete'i okudum. Ben son kez bu yana 15 yıl olduğunu düşünüyorum ve hala kod yazabilirim buluyorum ;-)

Neyse, sayfa 138'de Kod tamamlandı, bu kodlama dehşet örneğini bulabilirsiniz. (Bazı kodları kaldırdım)

class Emplyee { 
public: 
FullName GetName() const; 
Address GetAddress() const; 
PhoneNumber GetWorkPhone() const; 
... 

bool IsZipCodeValid(Address address); 
... 

private: 
    ... 
} 

Steve'in kötü olduğunu düşündüğü şey, işlevlerin gevşek bir şekilde bağlantılı olmasıdır. Ya da

Tamam ben tamamen ona katılıyorum "çalışanlar ve posta kodlarını, telefon numarası veya iş sınıflandırmaları kontrol rutinleri arasında hiçbir mantıksal bağ yok" yazıyor vardır. Belki de aşağıdaki örnek gibi bir şey daha iyidir.

class ZipCode 
{ 
public: 
bool IsValid() const; 
    ... 
} 

class Address { 
public: 
    ZipCode GetZipCode() const; 
    ... 
} 

class Employee { 
public: 
Address GetAddress() const; 
    ... 
} 

Zip geçerli olup olmadığını denetlerken böyle bir şey yapmanız gerekir.

Ve bu Law of Demeter ile ilgili değil.

Üç noktadan ikisini çıkarmak isterseniz, bu şekilde temsilci ve birkaç sarıcı işlevini kullanmanız gerekir.

class ZipCode 
{ 
public: 
bool IsValid(); 
} 

class Address { 
public: 
    ZipCode GetZipCode() const; 
    bool IsZipCodeValid() {return GetZipCode()->IsValid()); 
} 

class Employee { 
public: 
FullName GetName() const; 
Address GetAddress() const; 
bool IsZipCodeValid() {return GetAddress()->IsZipCodeValid()); 
PhoneNumber GetWorkPhone() const; 
} 

employee.IsZipCodeValid(); 

Ama sonra tekrar mantıksal bir bağlantıya sahip olmayan yordamlarınız var.

Şahsen bu yazının üç örneğinin de kötü olduğunu düşünüyorum. Düşünmediğim başka bir yolu var mı?

Answer 1

Daha sonra ödeme vs şimdi ödemek var.

Sen heyeti ve sarıcı işlevleri ön (şimdi ödeme) ve ardından() sonradan employee.IsZipCodeValid bağırsaklar değişen daha az iş var yazabilirsiniz. Veya, can tünel kodda gerek, ama bu kodu kırar bir şekilde sınıf tasarımını değiştirmeye karar gerekir sonra öde her yerde

employee.GetAddress().GetZipCode().IsValid();

yazarak IsZipCodeValid kadar. Zehirinizi seçmeniz yeterlidir. ; Çalışan sınıfın ve posta kodu doğrulama arasında hiçbir mantıksal bağlantısı olmadığına göre)

Answer 2

Sen mantıksal bağlantısı eksik:

class ZipCode 
{ 
public: 
bool IsValid(); 
} 

class Address { 
public: 
    ZipCode GetZipCode() const; 
    bool IsAddressValid(); 
    bool IsValid() {return GetZipCode()->IsValid() && IsAddressValid()); 
} 

class Employee { 
public: 
FullName GetName() const; 
Address GetAddress() const; 
bool IsEmployeeValid(); 
bool IsValid() {return GetAddress()->IseValid() && IsEmployeeValid()); 
PhoneNumber GetWorkPhone() const; 
} 

employee.IsValid(); 

Answer 3

, bunu daha mantıklı ait adres sınıfa Posta kodu doğrulama koyabilirim. Ardından, Adres sınıfının sizin için Zip kodunu doğrulamasını isteyebilirsiniz.

class Address 
{ 
    public: 
     static IsZipValid(ZipCode zip) { return zip.isValid(); } 
}; 

Sonra

Address::IsZipValid(employee.GetAddress().GetZipCode()); 

bu mantıksal dernek ve Demeter Kanunun sizin kısıtlamalar altında tatmin edici olduğunu düşünüyorum

yapmak.