생성자에서 포인터를 할당하십시오. 무엇이 좋은 연습입니까?

Question

클래스에서 객체 A을 생성했다고 가정 해 봅시다. Object A은 생성자에 대해 많은 인수를 취합니다.

오브젝트 BB'에서 A'의 인스턴스를 저장하고 싶습니다. 그러나 클래스 B'도 생성자에서 상당히 많은 인수를 원합니다. A'에 대한 모든 인수를 생성자에 전달하는 대신 먼저 A'의 인스턴스를 만든 다음 더 큰 인수를 생성하는 대신 B'의 생성자에 대한 참조를 보내야합니다.

class A' 
{ 
    A'(arg1, arg2,...argn); 


}A(arg1, arg2,...argn); 


class B' 
{ 
    public: 
    B'(arg1, arg2,...argn, A'* aptr) 
    A'* myA; 

}B(arg1, arg2,... argn, &A); 

이 좋은 코드 연습 : C에서

는 다음과 같이 보일 것 ++? 또한 내가 어떻게 생성자를 통해 포인터를 가장 잘 할당 할까? 나는 지금까지 이와 비슷한 것을 해왔다.

myConstructor(object* __pointerToAnotherObjectStoredInThisObject) 
{ 
     pointerToAnotherObjectStoredInThisObject =  __pointerToAnotherObjectStoredInThisObject; 

}; 

확실히 작동하지만 깔끔하고 좋은 연습 등인가?

Answer 1

포인터를 생성자에 전달한 다음 필드에 할당하는 데주의하십시오. 신중하더라도 포인터가 가리키는 데이터가 아직 사용 중일 때 해제되지 않도록하는 것이 어려울 수 있습니다. 또한 메모리를 해제하는 것이 안전 할 때 해결하기가 어려울 수 있습니다.

개체가 포인터로 가리키는 데이터의 자체 복사본을 유지하거나 shared_ptr을 사용하는 것이 좋습니다.

Answer 2

일반적으로 포인터를 피할 수 있으면 객체에 포인터를 저장하고 싶지는 않습니다. 가능하면 참조를 저장하십시오. 그것은 개체를 복사 할 수 없도록 만듭니다. 따라서 종속성이 파손 된 후에도 두 번째 클래스를 계속 사용하지 않게됩니다.

또한, 포인터를 포함한 멤버 변수를 초기화하는 적절한 표기법이 ...

myConstructor (object __foo) : foo(__foo) { }

는 여기서 참조를 사용하는 부재로서 다른 클래스 저장 클래스 정말로 기본적인 예가있다.

// Let's suppose we're implementing an XML file handling class. 
// We decide to put the file handling operations in one class and 
// let the XML stuff wrap around that file class. 

class File { 
    // Implement file opening/writing here 
}; 


class XMLFile { 
    // Implement XML opening/writing nonsense here. 
    File & m_file; 

    XMLFile (File & f); 
}; 

XMLFile::XMLFile (File & f) : m_file(f) { 
}