การรับมรดกผิดไป

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

class Animal  
class Parrot : Animal 
class Elephant : Animal 
class Cow : Animal

เป็นต้น

คุณมีวิธีกิน (), รัน (), ฯลฯ และทุกอย่างดี วันหนึ่งมีคนเข้ามาและพูดว่า - คลาส CageBuilder ของเราทำงานได้ดีและใช้ animal.weight () และ animal.height () ยกเว้น African Bison ใหม่ที่แข็งแรงเกินไปและสามารถทุบกำแพงได้ดังนั้นฉันจะเพิ่ม อีกคุณสมบัติหนึ่งของคลาส Animal - isAfricanBizon () และใช้สิ่งนั้นเมื่อเลือกวัสดุและแทนที่มันสำหรับคลาส AfricanBizon เท่านั้น บุคคลต่อไปมาและทำสิ่งที่คล้ายกันและสิ่งต่อไปที่คุณรู้ว่าคุณมีคุณสมบัติเหล่านี้ทั้งหมดสำหรับชุดย่อยของลำดับชั้นในคลาสพื้นฐาน

เป็นวิธีที่ดีในการปรับปรุง / refactor รหัสดังกล่าวคืออะไร อีกทางเลือกหนึ่งที่นี่คือเพียงแค่ใช้ dynamic_casts เพื่อตรวจสอบประเภท แต่ที่ clutters ผู้โทรและเพิ่มพวงของ if-then-else ทั่วทุกสถานที่ คุณสามารถมีอินเทอร์เฟซเฉพาะเพิ่มเติมได้ที่นี่ แต่ถ้าคุณมีการอ้างอิงคลาสพื้นฐานที่ไม่ได้ช่วยอะไรมาก ข้อเสนอแนะอื่น ๆ ? ตัวอย่าง?

ขอบคุณ!

ดูเหมือนว่าปัญหาจะเกิดขึ้นแทนการใช้ RequireConcreteWall () พวกเขาใช้การตั้งค่าสถานะการเรียก IsAfricanBison () แล้วย้ายตรรกะบนผนังหรือไม่ควรเปลี่ยนนอกขอบเขตของชั้นเรียน คลาสของคุณควรเปิดเผยพฤติกรรมและข้อกำหนดไม่ใช่ตัวตน ผู้บริโภคในชั้นเรียนเหล่านี้ของคุณควรทำงานจากสิ่งที่พวกเขาได้รับการบอกเล่า

isAfricanBizon () ไม่ใช่แบบทั่วไป สมมติว่าคุณขยายฟาร์มเลี้ยงสัตว์ของคุณด้วย hyppopotamus ซึ่งแข็งแกร่งเกินไป แต่กลับเป็นจริงจาก isAfricanBizon () เพื่อให้ได้ผลที่เหมาะสมจะเป็นเพียงโง่

คุณต้องการเพิ่มวิธีการในอินเทอร์เฟซที่ตอบคำถามเฉพาะเสมอในกรณีนี้มันจะมีความแข็งแรง ()

ฉันคิดว่าปัญหาของคุณคือ: คุณมีไคลเอนต์ต่าง ๆ ของไลบรารีที่สนใจเฉพาะชุดย่อยของลำดับชั้น แต่ที่ผ่านตัวชี้ / อ้างอิงไปยังคลาสพื้นฐาน อันที่จริงแล้วปัญหาที่ dynamic_cast <> อยู่ที่นั่นเพื่อแก้ไข

มันเป็นเรื่องของการออกแบบของลูกค้าเพื่อลดการใช้ dynamic_cast <>; พวกเขาควรใช้มันเพื่อตรวจสอบว่าวัตถุนั้นต้องการการดูแลเป็นพิเศษหรือไม่และถ้าเป็นเช่นนั้นการดำเนินการทั้งหมดในการอ้างอิงแบบดาวน์ - คาสท์

หากคุณมีคอลเลกชันประเภท "มิกซ์อิน" ของฟังก์ชั่นที่ใช้กับหลาย ๆ ชั้นย่อยที่แยกจากกันคุณอาจต้องการใช้รูปแบบอินเตอร์เฟสที่ Java และ C # ใช้ มีคลาสพื้นฐานเสมือนที่เป็นคลาสเสมือนจริงและใช้ dynamic_cast <> เพื่อตรวจสอบว่าอินสแตนซ์มีการนำไปใช้งานหรือไม่

สิ่งหนึ่งที่คุณสามารถทำได้คือแทนที่การตรวจสอบที่ชัดเจนของประเภทเช่นมีการตรวจสอบคุณสมบัติที่คุณสนใจจริงคือisAfricanBison()isTooStrong()

สัตว์ไม่ควรสนใจผนังคอนกรีต บางทีคุณสามารถแสดงมันด้วยค่าง่ายๆ

class Animal {
public:
  virtual ~Animal() {}
  virtual size_t height() const = 0;
  virtual size_t weight() const = 0;
  virtual bool isStrong() const = 0;
};

Cage *CreateCageFromSQL(Animal &a);
Cage *CreateCageFromOrangePeelsAndSticks(Animal &a);

ฉันสงสัยว่าไม่ได้เป็นไปได้ นั่นคือปัญหาเกี่ยวกับตัวอย่างของเล่น

ฉันจะไม่ต้องการเห็น RequireConcreteWalls () หรือบรรทัดและบรรทัดของตัวชี้แบบไดนามิกปลดเปลื้องในอัตราใด ๆ

นี่เป็นวิธีแก้ปัญหาราคาถูก ง่ายต่อการบำรุงรักษาและกำหนดแนวคิด และจริงๆปัญหาระบุว่ามันผูกติดอยู่กับประเภทของสัตว์ต่อไป

class Animal {
public:
  virtual ~Animal() {}
  virtual CageBuilder *getCageBuilder() = 0;
};

สิ่งนี้ไม่ได้ขัดขวางคุณจากการใช้รหัสที่ใช้ร่วมกันเพียงแค่ทำให้สัตว์สกปรก

แต่วิธีการสร้างกรงอาจเป็นนโยบายของระบบอื่นและบางทีคุณอาจมีผู้สร้างกรงมากกว่าหนึ่งประเภทต่อสัตว์ มีชุดค่าผสมที่แปลกและซับซ้อนมากมายที่คุณสามารถหาได้

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

Double Dispatchเป็นอีกตัวเลือกหนึ่งแม้ว่าฉันจะลังเลที่จะกระโดดเข้าไปเสมอ

ยิ่งไปกว่านั้นมันยากที่จะคาดเดาปัญหา

attemptEscape()ดีแน่นอนสัตว์ทุกคนมีธรรมชาติของสถานที่ให้บริการ ในขณะที่บางวิธีการที่อาจก่อให้เกิดfalseผลในสถานการณ์ทั้งหมดในขณะที่คนอื่น ๆ อาจมีโอกาสตามออกของการวิเคราะห์พฤติกรรมของลักษณะที่แท้จริงของพวกเขาอื่น ๆ เช่นและsize weightแล้วแน่นอนในบางจุดกลายเป็นน่ารำคาญในขณะที่มันแน่นอนที่สุดจะกลับมาattemptEscape()true

ฉันเกรงว่าฉันจะไม่เข้าใจคำถามของคุณอย่างสมบูรณ์แม้ว่าสัตว์ทุกตัวจะมีการกระทำและลักษณะที่เกี่ยวข้อง ควรแนะนำคนที่มีลักษณะเฉพาะกับสัตว์ในที่ที่เหมาะสม การพยายามเชื่อมโยง Bison to Parrots โดยตรงไม่ใช่การตั้งค่าที่ดีและไม่ควรมีปัญหาในการออกแบบที่เหมาะสม

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

วิธีการเกี่ยวกับ RecommendCageType () ตรงข้ามกับ RequireConcreteWall ()

ทำไมไม่ทำอะไรแบบนี้

class Animals { /***/ } class HeavyAnimals{} : Animals //The basic class for animals like the African Bison

ด้วยคลาส HeavyAnimals คุณสามารถสร้างคลาส Bison ของแอฟริกาโดยขยายคลาส HeavyAnimals

ดังนั้นตอนนี้คุณเป็นผู้ปกครอง (สัตว์) ซึ่งสามารถใช้ในการสร้างคลาสฐานอื่นเช่นคลาส HeavyAnimal ด้วยสามารถใช้ในการสร้างคลาสกระทิงกระทิงแอฟริกาและสัตว์หนักอื่น ๆ ดังนั้นด้วย Bison แอฟริกาตอนนี้คุณสามารถเข้าถึงวิธีการและคุณสมบัติของ Animal Class (นี่คือพื้นฐานสำหรับสัตว์ทั้งหมด) และการเข้าถึงคลาส HeavyAnimals (นี่คือพื้นฐานสำหรับสัตว์หนัก)