상속을 사용하는 복사 생성자

Question

학교 과제에 문제가 있습니다. 스포츠 경기에서 경쟁자를 추적하는 시스템을 만들어야합니다. 나는 기본 클래스 경쟁자와 그로부터 파생 된 두 클래스, 즉 Professional과 Exerciser을 가지고있다.
그리고 나서 Contender **contenders을 포함하는 레지스터 클래스가 있습니다. 그리고이 클래스에 대한 복사 생성자를 만들어야하지만 어떻게 해야할지 모르겠습니다.

은 내가 contenders 포인터의 배열에 대한 포인터 것을 이해이

Register::Register(const Register& original) 
{ 
    this->kap = original.kap; 
    this->currentAmount = original.currentAmount; 
    for (int i = 0; i < this->currentAmount; i++) 
    { 
     if (Professional* pro = dynamic_cast<Professional*>(this->contenders[i])) 
     { 
      this->contenders[i] = new Professional(*original.contenders[i]); 
     } 
     if (Exerciser* pro = dynamic_cast<Exerciser*>(this->contenders[i])) 
     { 
      this->contenders[i] = new Exerciser(*original.contenders[i]); 
     } 
    } 
    this->initiate(this->currentAmount); 
} 

Answer 1

처럼 뭔가에 대해 생각했다. 당신이 contenders을 초기화하기 위해 필요

• :

  는 세 가지 문제가 있습니다.

• 는 dynamic_cast<> 체크
• ProfessionalExerciser 및 복사 생성자가 확실히 각각되지 Contender 형 Professional 및 Exerciser의 객체를 복사로 정의 된 원래 객체를 사용한다. 여기

제안 된 보정 :하지만

Register::Register(const Register& original) 
{ 
    kap = original.kap; 
    currentAmount = original.currentAmount; 
    contenders = new Contender[currentAmount]; // was missing 
    for (int i = 0; i < currentAmount; i++) 
    { 
     if (Professional* pro = dynamic_cast<Professional*>(original.contenders[i])) 
     { 
      contenders[i] = new Professional(*pro); // use the pro instead of contender 
     } 
     if (Exerciser* exe = dynamic_cast<Exerciser*>(original.contenders[i])) 
     { 
      contenders[i] = new Exerciser(*exe); // use exe instead of contender 
     } 
    } 
    initiate(currentAmount); 
} 

세의 recomendations가 :

• 모든 개체 변수를 접두사로 this->를 사용할 필요가 없습니다. 멤버 함수 자체의 지역 변수와 모호한 경우에만. 배열에 대한 포인터 대신 vector를 사용
• 생각은
• 은 코멘트에서 제안 된 복제 방식은 dynamic_cast<> 검사에 정말 좋은 강력한 대안이 될 것