이 할당 연산자 오버로드에 어떤 문제가 있습니까?

Question

하나의 객체를 다른 객체에 할당하려고하면 프로그램이 중단되고 아무 문제가 있습니까?

Screen& operator=(Screen &scr) { 
    if (this != &scr){ 
    for (int i = 0; i < 50; i++) 
     fields[i] = NULL; 

    for (int i = 0; i < scr.numOfFields; i++) 
     fields[i] = scr.fields[i]; 

    numOfFields = scr.numOfFields; 
    currentField = scr.currentField; 
    } 

    return *this; 
} 

Answer 1

당신은 당신의 동적 배열 대신 std::vector를 사용해야하고 나는 당신의 문제가 매우 빠르게 사라집니다 의심 :

Field *fields[50]; 
int numOfFields; 
int currentField; 

기능은 다음과 같습니다

변수입니다.

MadCap에서 제안한 것처럼 공유 포인터를 사용해야합니다.이 포인터는 컨테이너에 포인터를 사용할 때 가장 좋은 방법입니다. 또한 이것은 당신에게 당신이 기대하는 복사 의미를 줄 것이다.

이이에 대한 필요성 부정하는 것 :

for (int i = 0; i < 50; i++) 
     fields[i] = NULL; 

그리고 당신이 대체 할 수있는 : 성병 :: 사본

for (int i = 0; i < scr.numOfFields; i++) 
     fields[i] = scr.fields[i]; 

을, 뭔가 같은 :

fields.clear() 
std::copy(scr.fields.begin(), scr.fields.end(), fields.begin()); 

으로 이렇게하면 포인터 액세스 또는 초기화 오류로 인해 충돌이 발생할 가능성이 제거됩니다.

간단한 조언은 다음과 같습니다. 동적 배열 및 원시 포인터 사용을 중지하고 std::vector 및 boost::shared_ptr을 사용하는 대신 이러한 종류의 문제가 발생하는 것을 방지하는 데 도움이됩니다.

Answer 2

scr.numOfFields가 대상 객체의 필드 수를 초과하는 것이 하나의 문제 일 수 있습니다.

또 다른 문제는 적어도 새 개체에 포인터를 할당하고있는 것으로 보입니다. 즉, 프로그램에서 두 번 같은 위치에 대한 참조를 갖게됩니다. 한 지점에서 삭제되고 다른 지점에서 삭제되는 경우 어떻게됩니까? 메모리에 액세스하려고하면 seg 오류가 발생합니다.

당신은이를 방지하기 위해 자신의 공유 포인터를 사용할 수있는 부스트 한 경우 : http://www.boost.org/doc/libs/1_43_0/libs/smart_ptr/smart_ptr.htm