힙 손상이 감지 : 내가 어떤 연산자를 오버로드 할 시도 후 정상 블록()

Question

그래서, 내가 정의한 템플릿 클래스 후.

template <typename T> class Set 
    { 
    public: 
     Set(void); 
     Set(Set&); 
     ~Set(void); 
     bool contains(T elem); 
     bool add(T elem); 
     bool remove(T elem); 
     bool add(T* tab, int size); 
     T* getSet(); 
     int size(); 
     Set<T> &operator+(Set<T> &snd); 
     Set<T> &operator-(Set<T> &snd); 
    private: 
     T *elements; 
     int numOfElem; 
    }; 

요소를 추가 메서드로 추가하려고하면 모든 것이 정상적으로 작동합니다.

template<typename T> 
    bool Set<T>::add(T elem) 
    { 
     bool found = false; 
     for(int i =0; !found && i<numOfElem; i++){ 
      if(elem == elements[i]) found = true; 
     } 
     if(!found){ 
      numOfElem++; 
      T* tmp = new T[numOfElem]; 
      for(int i =0; i<numOfElem-1; i++){ 
       tmp[i] = elements[i]; 
      } 
      tmp[numOfElem-1] = elem; 
      delete[] elements; 
      elements = tmp; 
     } 
     return !found; 
    } 

    template<typename T> 
    bool Set<T>::add(T* myArray, int size) 
    { 
     bool result = false; 
     for(int i =0; i<size; i++){ 
      add(myArray[i]); 
     } 
     return result; 
    } 
template<typename T> 
Set<T>& Set<T>::operator+(Set<T> &snd) 
{ 
    Set *temp = new Set(*this); 
    temp->add(snd.getSet(), snd.size()); 
    return *temp; 
} 
template<typename T> 
void Set<T>::operator=(Set<T> &snd) 
{ 
    numOfElem = snd.numOfElem; 
    elements = new T[numOfElem]; 
    for(int i =0; i < numOfElem; i++){ 
     elements[i] = snd.elements[i]; 
    } 
} 

template<typename T> 
int Set<T>::size() 
{ 
    return numOfElem; 
} 
template<typename T> 
T* Set<T>::getSet() 
{ 
    return elements; 
} 
template<typename T> 
Set<T>::Set() 
{ 
    numOfElem = 0; 
    elements = nullptr; 
} 

template<typename T> 
Set<T>::Set(Set& old) 
{ 
    numOfElem = old.numOfElem; 
    elements = new T(numOfElem); 
    for(int i = 0; i< numOfElem; i++){ 
     elements[i] = old.elements[i]; 
    } 

} 

template<typename T> 
Set<T>::~Set() 
{ 
    numOfElem = 0; 
    delete[] elements; 
    elements = nullptr; 
} 

대신 + 연산자 (두 개의 개별 세트 추가)를 사용하면 배열 (15 줄)을 삭제하는 동안 오류가 발생합니다. 어떤 아이디어?

int main(){ 
    Set <char> set1, set2, set3; 
    char tab[] = {'a','d','f','g'} ; 
    set1.add(tab, 4); 
    char tab2[] = {'a','d','x','y','z'} ; 
    set2.add(tab2,5); 
    set3= set1+set2; 
} 

Answer 1

당신은 당신의 복사 생성자에 실수 있습니다

elements = new T(numOfElem); 

그것은 당신이 numOfEllem에 초기화 값을 하나 개의 변수 할당 new T(numOfElem);를 작성하여

elements = new T[numOfElem]; 

해야한다.

대신 배열의 표준 : : 벡터를 사용하면 이러한 문제를 방지 할 수 있습니다. 당신은 메모리를 할당하고 당신이 프로그램이 가상 메모리가 부족하여 실행할 수 있습니다 너무 자주 함수를 호출 그래서 만약 당신이 그것을 삭제하지

template<typename T> 
Set<T>& Set<T>::operator+(Set<T> &snd) 
{ 
    Set *temp = new Set(*this); 
    temp->add(snd.getSet(), snd.size()); 
    return *temp; 
} 

:

코드는 또한 더하기 연산자에서 메모리가 누수된다 uncaught std :: bad_alloc 예외로 인해 충돌합니다. 이 함수를 다음으로 변경하십시오.

template<typename T> 
Set<T> Set<T>::operator+(Set<T> &snd) 
{ 
    Set temp(*this); 
    temp.add(snd.getSet(), snd.size()); 
    return temp; 
}