“Özellik kıskançlığı” kodu nedir ve neden kod kokusu olarak kabul edilir?

SO hakkındaki bu soru , OP düşüncesinin kıskançlık kodunun ne olduğunu düzeltme konusunda konuşuyor . Bu şık ifadenin alıntılandığını gördüğüm bir başka örnek , programcılarda son zamanlarda verilen cevapta . Bir damla Her ne kadar Yorumlarınız bilgiler soran o cevaba ben terim ne anlama geldiğini anlamak için Q & A aşağıdaki programcılar için genel yardım düşündüm özellik kıskançlık . Lütfen uygun olduğunu düşünüyorsanız ek etiketleri düzenlemekten çekinmeyin.

Özellik kıskançlığı , bir nesnenin başka bir nesnenin alanlarına girdiği bir durumu tanımlamak için kullanılan bir terimdir.

Önemsiz bir örnek olarak, dikdörtgeni temsil eden bir sınıf düşünün. Dikdörtgenin kullanıcısının bölgesini bilmesi gerekebilir. Programcı ortaya çıkarabilir widthve heightalanları açıp Rectanglesınıf dışında hesaplamayı yapabilir . Alternatif olarak ve alanları gizli Rectangletutabilir ve bir yöntem sağlayabilir . Bu tartışmalı daha iyi bir yaklaşım.widthheightgetArea

İlk durumla ilgili sorun ve kod kokusu olarak kabul edilmesinin nedeni, kapsüllemeyi bozmasıdır.

Genel bir kural olarak, ne zaman bir tür mantık veya hesaplama yapmak için kendinizi başka bir sınıftan alanlardan yararlanırken bulursanız, bu mantığı sınıfın kendisinde bir yönteme taşımayı düşünün.

Başka bir sınıf / struct yöntemini kapsamlı bir şekilde kullanmanın uygun olduğu durumlarda olası bir durum vardır - sınıfınız / yapı sisteminiz veri için bir konteyner olduğunda. Genelde bu verilerle dış bağlam olmadan yapabileceğiniz çok az şey vardır.

Bu sınıflar hala bazı iç mantıkları tutabilirler, ancak daha sıklıkla kapsayıcı olarak kullanılırlar:

class YourUid {
 public:
  YourUid(int id_in_workplace_, int id_in_living_place_, DB* FBI_database, int id_in_FBI_database);
  bool IsInvalidWorker() const { return id_in_workplace == consts::invalid_id_in_workplace; }
  bool CanMessWith() const { return !FBI_database_.is_cool(id_in_FBI_database_); }
  int id_in_workplace;
  int id_in_living_place;
 private:
  int id_in_FBI_database_;
  const DB* FBI_database_;
};

Cevabındaki @jhewlett, diğer sınıf üyelerini yoğun bir şekilde kullanmamanız gerektiğini kanıtlamak için bu makaleye atıfta bulunuyor, ancak burada başka bir kod var.

Uzun Parametre Listesi Belirli bir yöntemde ihtiyacınız olan parametre sayısını sınırlayın veya parametreleri birleştirmek için bir nesne kullanın.