포인터를 삭제하려고하면 중단 점이 발생합니다.

Question

현재 작업하고있는 프로그램의 목적은 포인터의 동적 배열을 시뮬레이션하여 기본 정수 배열 데이터 형식을 "향상"하는 클래스를 만드는 것입니다. 포인터와 포인터의 배열을 삭제하려고 할 때 "Windows에서 project4.exe의 중단 점을 트리거했습니다."

이는 project4의 버그를 나타내는 힙 손상으로 인한 것일 수 있습니다. EXE하거나로드 DLL 중 어느.

이것은 인해 project4.exe 포커스가있을 F12 가압 사용자 일 수있다. 출력 창보다 진단 정보를 가질 수있다

. "

class Array 
{ 
private: 
    int length; 
int* data; 
public: 
    Array(); 
    Array(const Array &cpy); 
    ~Array(); 
    bool addint(int toadd); 
    bool deletelast(); 
    int getlength(); 
    friend ostream& operator<<(ostream &out, const Array &n); 
}; 

ostream& operator<<(ostream &out, const Array &n); 

Array::Array() 
{ 
    length = -1;  
    data = NULL; 
} 

Array::Array(const Array &cpy) 
{ 
    length = cpy.length;    //value of length is copied 

    if (length < 0) 
     data = NULL; 
    else 
    { 
     data = new int [length]; 

     for (int i=0; i<=length; i++) 
      data[i] = cpy.data[i]; 
    } 

}  

Array::~Array() 
{ 
    if (length != 0) 
     delete [] data; 
    else 
     delete data; 

    data = NULL; 
} 

bool Array::addint(int toadd) 
{ 
    length ++; 
    int* point = new int[length]; 

    for (int i=0; i < length; i++) 
     point[i] = data[i];    

    point[length] = toadd; 

    if (length != 0)  
     delete [] data; 

    data = point; 

    point = NULL; 

    return true; 
}  

bool Array::deletelast() 
{ 
    int* temppoint; 
    if (length > 0) 
     temppoint = new int [length-1]; 
    else 
     temppoint = new int[0]; 

    for (int i=0; i<length; i++)   
     temppoint[i] = data[i]; 

    if (length == 0) 
     temppoint[0] = 0; 

    length --; 
    delete [] data; 
    data = temppoint; 
    temppoint = NULL; 

    return true; 
} 

void menu(Array var) 
{ 
    int selection=0, 
     input; 
    bool success; 
    Array* arrcpy; 
    while (selection != 3) 
    { 
     if (var.getlength() == -1) 
     { 
      cout << "What would you like to demonstrate?" << endl << "1) Add an integer " << endl 
      << "2) Exit" << endl << "Enter your selection: "; 
      cin >> selection; 
      if (selection == 2) 
       selection = 4; 
     } 
     else 
     { 
      cout << endl << "Now what would you like to demonstrate?" << endl << "1) Add an integer " << endl 
      << "2) Delete the last entered integer" << endl << "3) Copy constructor" << endl << "4) Exit" << endl << "Enter your selection: "; 
      cin >> selection; 
     } 

     if (selection==1) 
     { 
      cout << endl << "The length of the array before adding a new value is: " << var.getlength() + 1 << endl; 
      cout << "Please enter the integer that you wish to add: "; 
      cin >> input; 
      success = var.addint(input); 
      if (success) 
       cout << endl << "The data input was a success!" << endl << "The length of the array is now: " 
       << var.getlength() + 1 << endl << "The new value of the array is: " << var << endl; 
      else 
       cout << endl << "The input failed" << endl; 
     }  
     if (selection == 2) 
     { 
      cout << endl << "The lenght of the array before the deletion is: " << var.getlength() + 1 << endl 
       << "and the value held in the array is: " << var << endl; 
      success = var.deletelast(); 
      if (success) 
       cout << endl << "The data deletion was a success!" << endl << "The length of the array is now: " 
       << var.getlength() + 1 << endl << "The new value of the array is: " << var << endl; 
      else 
       cout << endl << "The deletion failed" << endl; 
     }  
     if (selection == 3) 
     { 
       cout << endl << "The lenght of the array being copied is: " << var.getlength() + 1 << endl 
       << "and the value held in the array is: " << var << endl; 
       arrcpy=new Array(var); 
       cout << endl << "The length of the copied array is: " << arrcpy->getlength() +1 << endl 
       << "and the value contained in the array is: " << *arrcpy; 
       delete arrcpy; 
     } 
    } 
} 

이것은 내가 갖고있는 문제와 관련된 모든 관련 소스 코드입니다. 이 경우에는 delete 연산자와 delete [] 연산자의 모든 인스턴스에서이 중단 점 오류가 발생하고 내가 잘못하고 있는지 확실하지 않습니다.

편집 : 길이 값을 -1 대신 0으로 기본 설정하고 모든 것이 작동합니다!

Answer 1

int* point = new int[length]; 

길이가 -1부터 시작하므로이 줄의 첫 번째 호출은 new int[0] 일 것입니다. 당신이 length의 의미를 고정하지 주장하면 .. 문제가 될 수 없습니다, 당신은 여기 length+1 원하는

관련없는 점, 당신은 std::vector가 어떻게하는지 살펴 봐야하는 대신 모든 추가 재 할당, 당신은 overallocating 만 재 할당 시도해야한다 공간이 채워지는 경우.

Answer 2

복사 생성자에있는 에 i<length (이하보다 작거나 같음)이 있어야한다고 생각합니다. 그것은 하나의 분명한 문제입니다.

또한 addint() 메서드로 바인딩을 전달합니다. 배열의 마지막 요소는 인덱스 [길이 -1]에 있습니다.