C++ 쌍의 동적 크기 배열 삭제 중 문제가 발생했습니다.

Question

m_Array 삭제시 많은 문제가 있습니다. 프로그램은 클린업 부분을 수행 할 때 마지막에 segfault합니다. m_Array에 다른 데이터가있는 두 개의 클래스 A 객체가 있고 프로그램의 어느 시점에서 한 객체의 데이터가 다른 배열로 "둘러싸여"시작되어 잘못된 데이터가 발생합니다. T는 내 템플릿 데이터 유형을 나타냅니다.

두 클래스 A는 같은 클래스 선언에 공개적으로 선언

객체를 생성하는 클래스 B도있다 :

template <typename T> 
A<T>::A(int size) { 

    // Set array size 
    m_Array = new pair<T, int>[size]; 

    // Initialize values 
    for (int i = 0; i < size; i++) { 
     m_Array[i] = make_pair(-1, -1); 
    } 

    //... other things defined and initialized...// 
} 

: 같은 클래스 A의 생성자 정의에 정의

template <typename T> class A 
{ 
public: 
    pair<T, int> *m_Array; // Array of type pair 
    A(int size=1);  // constructor 
    ~A();  // A destructor 

    // ... all other definitions // 

}; 

클래스 A의 소멸자 :

template <typename T> 
A<T>::~A() { 

     delete [] m_Array; // Not working as it should 
} 

,515,

오버 할당 연산자

template <typename T> 
const A<T>& A<T>::operator=(const A<T>& rhs) { 
    m_AArraySize = rhs.m_AArraySize; 
    m_currASize = rhs.m_currASize; 

    for (int i = 0; i < m_currASize; i++) { 

     m_Array[i].first = rhs.m_Array[i].first; 
     m_Array[i].second = rhs.m_Array[i].second; 
    } 

    _ptr = rhs._ptr; 

    return *this; 
} 

복사 생성자

template <typename T> 
A<T>::A(const A<T>& other) { 
    m_AArraySize = other.m_AArraySize; 
    m_AHeapSize = other.m_AHeapSize; 

    for (int i = 0; i < m_currASize; i++) { 
     m_Array[i].first = other.m_Array[i].first; 
     m_Array[i].second = other.m_Array[i].second; 
    } 

    _ptr = other._ptr; 
} 

클래스 B 선언

template <typename T> class B{ 
public: 
    //B constructor 
    B(int size); 


    int m_currBSize;  // spots used in array 
    int m_BSize;   // size of array 

    A <T> oneAHolder; 
    A <T> twoAHolder; 

}; 

B 클래스 생성자

template <typename T> 
b<T>::b(int size){ 
    A<T>(size); 

    m_BArraySize = size; 
    m_currBSize = 1; 

    // Create two A objects 
    A<T> oneA(size); 
    A<T> twoA(size); 

    // oneA and twoA go out of scope 
    oneAHolder = oneA; 
    twoAHolder = twoA; 
} 

내 주요 기능은 모든 일을 끝내고 나는 클래스 B 개체를 만드는 것입니다, 그리고 그것의 삽입 기능을 사용하여 두 개의 A 개체에 데이터를 삽입합니다.

배열에서 데이터를 삭제하고 다른 배열로 데이터가 넘치지 만 여러 가지 방법으로 시도했지만 아무 소용이 없습니다.

감사합니다.

PS : 아니

EDIT "그냥 표준 : : 벡터를 사용하여"하십시오

1) : 당신이 게시 된 코드를 감안할 때 내 코드

Answer 1

, 더 추가 과제 연산자는 두 가지 문제가 있습니다 할당 연산자는 이전 메모리를 할당 해제하지 못했기 때문에 메모리를 누설합니다.

2) this->m_Array하지 않는만큼, 당신은 this->m_Array 이후 for 루프에서 메모리를 덮어 쓰는 이미 컸다는 rhs.m_Array보다 작은 버퍼입니다.

이 결함있는 코드 대신, copy/swap idiom을 사용하면됩니다.

#include <algorithm> 
//... 
template <typename T> 
A<T>& A<T>::operator=(const A<T>& rhs) 
{ 
    A<T> temp(rhs); 
    std::swap(temp.m_AArraySize, m_AArraySize); 
    std::swap(temp.m_currASize, m_currASize); 
    std::swap(temp.m_Array, m_Array); 
    std::swap(temp._ptr, _ptr); 
    return *this; 
} 

간접적 할당 연산자를 호출하지 않는 올바른 복사 생성자를 작성한 경우에 작동

및 올바른 소멸자. 이러한 기능 중 하나에 결함이 있으면 위의 방법이 작동하지 않습니다.

복사 생성자와 소멸자이 방법은 당신이 단지 단지 그것을 다시 사용하고 있기 때문에 복사 생성자의 코드를 재 작성 방지 대입 연산자를 쓰고, 어떤 버그가 없다고 가정. 당신이 게시물에 지정하지 않은 A 더 많은 멤버 변수가있는 경우

, 그들은 모두도 교체해야합니다.

간단히 말해서, 이것은 기본적으로 rhs 사본을 만들어 모든 이전 데이터를 this에서 temp으로 바꿉니다. 그런 다음 temp은 이전 데이터로 끝납니다. 당신은 "랩 어라운드"무엇을