Debug-Assertion schlägt auf dynamischem Array fehl

Question

Also hier ist mein Arbeitscode für ein einfaches dynamisches Array. Dies hat ein Beispielcode für eine sehr Einstiegsdatenstruktur Implementierung sein:

#include <iostream> 
using namespace std; 

class AdvancedArray { 
public: 
    AdvancedArray(); 
    ~AdvancedArray(); 
    int get_size() const; // get the number of elements stored 
    double& at(int idx) const; // access the element at idx 
    void push_back(double d); // adds a new element 
    void remove(int idx); // remove the element at idx 
    void clear(); // delete all the data stored 
    void print() const; 

private: 
    double* elements; 
    int size; 
}; 

int main() 
{ 
    AdvancedArray* arr = new AdvancedArray(); 
    cout << "The Array Size is: " << arr->get_size() << endl; 
    cout << "Pusing Values: 1.2, 2.1, 3.3, 4.5 in the Array. " << endl; 
    arr->push_back(1.2); 
    arr->push_back(2.1); 
    arr->push_back(3.3); 
    arr->push_back(4.5); 
    arr->print(); 
    cout << "The Array Size is: " << arr->get_size() << endl; 
    cout << "The Element at Index 2 is: " << arr->at(2) << endl; 
    cout << "Deleting Values: 2.1 from the Array. " << endl; 
    arr->remove(1); 
    cout << "The Array Size is: " << arr->get_size() << endl; 
    arr->print(); 
    cout << "Clearing the Array: " << endl; 
    arr->clear(); 
    cout << "The Array Size is: " << arr->get_size() << endl; 
    arr->clear(); 
    return 0; 
} 

AdvancedArray::AdvancedArray() 
{ 
    size = -1; 
    elements = new double[100]; //Maximum Size of the Array 
} 

AdvancedArray::~AdvancedArray() 
{ 
    delete[] elements; 
} 

int AdvancedArray::get_size() const 
{ 
    if(size < 0) 
    { 
     return 0; 
    } 
    return size; 
} 

double & AdvancedArray::at(int idx) const 
{ 
    if (idx < 100 && idx >= 0 && size > 0) { 
     return elements[idx]; 
    } 
    cout << "Index Out of Bounds." << endl; 
} 

void AdvancedArray::push_back(double d) 
{ 
    if (size >= 100) 
    { 
     cout << "Overflow Condition. No More Space!" << endl; 
    } 
    else 
    { 
     elements[++size] = d; 
     cout << "Element Pushed In Stack Successfully!" << endl; 
    } 
} 

void AdvancedArray::remove(int idx) 
{ 
    if (size >= 100 || size < 0) 
    { 
     cout << "No Such Element Exists!" << endl; 
    } 
    else 
    { 
     for(int i = idx; i <size; i++) 
     { 
      elements[idx] = elements[idx + 1]; 
     } 
     size--; 
     cout << "Element Deleted In Stack Successfully!" << endl; 
    } 
} 

void AdvancedArray::clear() 
{ 
    delete[] elements; 
    size = -1; 
} 

void AdvancedArray::print() const 
{ 
    cout << "[ "; 
    for(int i = 0; i <= size; i++) 
    { 
     cout << elements[i] << " "; 
    } 
    cout << "]" << endl; 
} 

Also jedes Mal, wenn ich versuche, dies zu laufen Ich habe die zwei Probleme:

Was ist falsch an meinem Code? Warum wird der Heap beschädigt (ich habe nach dem Fehlercode gesucht und das muss alles sein)? Verletzt mein Code einige wichtige Zugriffsverletzungen? Ich benutze VS2015.

Answer 1

Sie tun delete [] elementsdrei mal ohne elements-nullptr dazwischen zu setzen. Das führt zu undefined Verhalten zum zweiten Mal (und dritten Mal).

Answer 2

Wenn size == 99, das folgende Stück Code versucht elements[100] zuzugreifen:

if (size >= 100) 
{ 
    cout << "Overflow Condition. No More Space!" << endl; 
} 
else 
{ 
    elements[++size] = d; 
    cout << "Element Pushed In Stack Successfully!" << endl; 
} 

Sie müssen ++size zu size++ ändern.