รหัส“ คุณสมบัติอิจฉา” คืออะไรและเหตุใดจึงถือว่าเป็นกลิ่นรหัส

นี้คำถามเกี่ยวกับ SOพูดคุยเกี่ยวกับการแก้ไขสิ่งที่คิด OP คือคุณลักษณะอิจฉารหัส อีกตัวอย่างหนึ่งที่ฉันเห็นวลีที่ดีนี้ถูกอ้างถึงในคำตอบที่ได้รับเมื่อเร็ว ๆ นี้ใน programmers.SE ถึงแม้ว่าผมจะไม่ได้ลดลงในความคิดเห็นที่จะตอบว่าขอให้ข้อมูลที่ผมคิดว่ามันจะมีการช่วยเหลือทั่วไปในการเขียนโปรแกรมต่อไปนี้ Q & A ที่จะเข้าใจสิ่งที่หมายโดยระยะคุณลักษณะอิจฉา โปรดแก้ไขแท็กเพิ่มเติมหากคุณคิดว่าเหมาะสม

ฟีเจอร์อิจฉาเป็นคำที่ใช้อธิบายสถานการณ์ที่วัตถุหนึ่งได้รับที่สาขาของวัตถุอื่นเพื่อทำการคำนวณหรือตัดสินใจบางอย่างแทนที่จะขอให้วัตถุทำการคำนวณเอง

เป็นตัวอย่างเล็กน้อยพิจารณาชั้นเรียนที่เป็นตัวแทนของสี่เหลี่ยมผืนผ้า ผู้ใช้สี่เหลี่ยมอาจจำเป็นต้องทราบพื้นที่ของมัน โปรแกรมเมอร์สามารถเปิดเผยwidthและheightเขตข้อมูลจากนั้นทำการคำนวณนอกRectangleชั้นเรียน อีกวิธีหนึ่งคือRectangleสามารถเก็บwidthและheightฟิลด์เป็นส่วนตัวและให้getAreaวิธีการ นี่เป็นวิธีการที่ดีกว่า

ปัญหากับสถานการณ์แรกและเหตุผลที่ถือว่าเป็นกลิ่นรหัสเป็นเพราะมันแบ่ง encapsulation

ตามกฎของหัวแม่มือเมื่อใดก็ตามที่คุณพบว่าตัวเองใช้ประโยชน์จากฟิลด์จากคลาสอื่นเพื่อทำการเรียงลำดับของตรรกะหรือการคำนวณใด ๆ ให้พิจารณาการย้ายตรรกะนั้นไปยังวิธีการในคลาสนั้น

มีสถานการณ์ที่เป็นไปได้เมื่อตกลงที่จะใช้วิธีการ class / struct อื่นอย่างกว้างขวาง - เมื่อ class / struct ของคุณเป็นที่เก็บข้อมูล โดยปกติจะมีเพียงเล็กน้อยที่คุณสามารถทำได้กับข้อมูลนี้โดยไม่มีบริบทภายนอก

คลาสดังกล่าวยังคงสามารถใช้ตรรกะภายในได้ แต่บ่อยครั้งที่มันถูกใช้เป็นคอนเทนเนอร์:

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_;
};

@ jhewlett ในคำตอบของเขาอ้างถึงบทความนี้เพื่อพิสูจน์ว่าคุณไม่ควรใช้สมาชิกชั้นเรียนคนอื่น ๆ อย่างกว้างขวาง แต่มีสถานการณ์รหัสกลิ่นอื่นที่อธิบายไว้ที่นั่นพร้อมกับตัวอย่างของฉัน:

รายการพารามิเตอร์แบบยาว จำกัด จำนวนพารามิเตอร์ที่คุณต้องการในวิธีการที่กำหนดหรือใช้วัตถุเพื่อรวมพารามิเตอร์