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

สมมติว่าฉันมีวัตถุสองชนิดคือ A และ B ความสัมพันธ์ระหว่างพวกมันมีมาก - ต่อ - กลุ่ม แต่ไม่มีพวกมันเป็นเจ้าของอีกคน

อินสแตนซ์ A และ B ทั้งสองจำเป็นต้องตระหนักถึงการเชื่อมต่อ มันไม่ใช่แค่ทางเดียว

ดังนั้นเราสามารถทำสิ่งนี้:

class A
{
    ...

    private: std::vector<B *> Bs;
}

class B
{
    private: std::vector<A *> As;
}

คำถามของฉันคือ:ฉันจะวางฟังก์ชันเพื่อสร้างและทำลายการเชื่อมต่อที่ไหน?

ควรเป็น A :: Attach (B) ซึ่งจะอัพเดท A :: Bs และ B :: เป็นเวกเตอร์หรือไม่

หรือควรเป็น B :: Attach (A) ซึ่งดูสมเหตุสมผลพอ ๆ กัน

ไม่มีความรู้สึกใดเลย ถ้าฉันหยุดทำงานกับรหัสและกลับมาหลังจากผ่านไปหนึ่งสัปดาห์ฉันแน่ใจว่าฉันจะจำไม่ได้ว่าฉันควรทำ A.Attach (B) หรือ B.Attach (A) หรือไม่

บางทีมันควรจะเป็นฟังก์ชั่นแบบนี้:

CreateConnection(A, B);

แต่การสร้างฟังก์ชั่นระดับโลกดูเหมือนจะเป็นสิ่งที่ไม่พึงประสงค์ด้วยเนื่องจากมันเป็นฟังก์ชั่นเฉพาะสำหรับการทำงานกับคลาส A และ B เท่านั้น

คำถามอื่น:หากฉันพบปัญหา / ข้อกำหนดนี้บ่อยฉันจะสามารถแก้ปัญหาทั่วไปได้หรือไม่? บางทีคลาส TwoWayConnection ที่ฉันสามารถหามาจากหรือใช้ภายในคลาสที่แบ่งใช้ความสัมพันธ์ประเภทนี้?

อะไรคือวิธีที่ดีในการจัดการกับสถานการณ์นี้ ... ฉันรู้วิธีจัดการสถานการณ์ "C เป็นเจ้าของ D" แบบตัวต่อตัวได้ค่อนข้างดี แต่สถานการณ์นี้ค่อนข้างหลอกลวง

แก้ไข: เพื่อให้ชัดเจนยิ่งขึ้นคำถามนี้ไม่เกี่ยวข้องกับปัญหาการเป็นเจ้าของ ทั้ง A และ B เป็นเจ้าของโดยวัตถุอื่น Z และ Z ดูแลปัญหาการเป็นเจ้าของทั้งหมด ฉันสนใจเพียงแค่วิธีการสร้าง / ลบลิงก์หลายต่อหลายอันระหว่าง A และ B

วิธีหนึ่งคือการเพิ่มAttach()วิธีสาธารณะและวิธีการป้องกันAttachWithoutReciprocating()ในแต่ละชั้นเรียน หาเพื่อนAและBร่วมกันเพื่อให้Attach()วิธีการของพวกเขาสามารถโทรหาอีกฝ่ายAttachWithoutReciprocating():

A::Attach(B &b) {
    Bs.push_back(&b);
    b.AttachWithoutReciprocating(*this);
}

A::AttachWithoutReciprocating(B &b) {
    Bs.push_back(&b);
}

หากคุณใช้วิธีการที่คล้ายกันสำหรับBคุณจะไม่ต้องจำไว้ซึ่งระดับที่จะเรียกAttach()ใน

ฉันแน่ใจว่าคุณสามารถสรุปพฤติกรรมนั้นในMutuallyAttachableชั้นเรียนที่ทั้งสองAและBสืบทอดมาดังนั้นหลีกเลี่ยงการทำซ้ำตัวเองและทำคะแนนคะแนนโบนัสในวันพิพากษา แต่แม้กระทั่งวิธีการติดตั้งที่ไม่ซับซ้อนซับซ้อนก็จะทำให้งานสำเร็จลุล่วงได้

เป็นไปได้หรือไม่ที่ความสัมพันธ์นั้นมีคุณสมบัติเพิ่มเติม?

ถ้าเป็นเช่นนั้นมันควรจะเป็นชั้นแยกต่างหาก

ถ้าไม่เช่นนั้นกลไกการจัดการรายการที่ตอบสนองจะพอเพียง

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

หนึ่งผู้สมัครสำหรับการย้ายการเชื่อมโยงไปเป็นรหัสที่สร้างการเชื่อมโยงในสถานที่แรก บางทีแทนที่จะเรียกมันก็เก็บรายการattach() std::pair<A, B>หากมีหลายสถานที่ที่สร้างการเชื่อมโยงจะสามารถแยกออกเป็นหนึ่งคลาส

ตัวเลือกอื่นสำหรับการย้ายคือวัตถุที่เป็นเจ้าของวัตถุที่เกี่ยวข้องZในกรณีของคุณ

อีกตัวเลือกทั่วไปสำหรับการย้ายการเชื่อมโยงไปเป็นรหัสที่เรียกวิธีAหรือBวัตถุ ตัวอย่างเช่นแทนที่จะโทรหาa->foo()ภายในซึ่งทำอะไรบางอย่างกับBวัตถุที่เกี่ยวข้องทั้งหมดภายในจะทราบถึงการเชื่อมโยงและการโทรa->foo(b)สำหรับแต่ละรายการ อีกครั้งหากมีหลายสถานที่เรียกว่ามันสามารถแยกเป็นชั้นเดียว

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

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

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

ถ้าฉันใช้สิ่งนี้ฉันจะใส่ Attach ทั้ง A และ B ด้านในของ Attach method ฉันจะเรียก Attach บนวัตถุที่ส่งผ่านด้วยวิธีนี้คุณสามารถเรียกทั้ง A.Attach (B) หรือ B.Attach ( A) ไวยากรณ์ของฉันอาจไม่ถูกต้องฉันไม่ได้ใช้ c ++ ในหลายปี:

class A {
  private: std::vector<B *> Bs;

  void Attach (B* obj) {
    // Only add obj if it's not in the vector
    if (std::find(Bs.begin(), Bs.end(), obj) == Bs.end()) {
      //Add obj to the vector
      Bs.push_back(obj);

      // Attach this class to obj
      obj->Attach(this);
    }
  }    
}

class B {
  private: std::vector<A *> As;

  void Attach (A* obj) {
    // Only add obj if it's not in the vector
    if (std::find(As.begin(), As.end(), obj) == As.end()) {
      //Add obj to the vector
      As.push_back(obj);

      // Attach this class to obj
      obj->Attach(this);
    }
  }    
}

วิธีอื่นจะสร้างคลาสที่ 3 C ซึ่งจัดการรายการคงที่ของการจับคู่ AB