CRT 检测到应用程序在堆缓冲区（新建/删除）类结束后写入内存答案

【问题标题】：CRT detected that the application wrote to memory after end of heap buffer (new/delete) classesCRT 检测到应用程序在堆缓冲区（新建/删除）类结束后写入内存
【发布时间】：2017-03-29 16:00:01
【问题描述】：

我收到“调试错误”CRT 检测到应用程序在堆结束后写入内存。但我不明白为什么，也找不到任何类似的例子。

完整的代码是... 虽然我认为这可能只是 main 的问题。

#include <iostream>
#include <cmath>
#include <array>

using namespace std;
// Declare global consts
const double pi = 3.1415926583;


// Base class Shape
class Shape{
protected:
    double *sides;
public:
    Shape(const int n){ //parameterized for n dimensional shape
    sides = new double[n];
}
// need virtual destructor
virtual ~Shape(){
    delete[] sides;
}
virtual double area() const = 0; // pure virtual function for area
virtual double volume() const = 0; // pure virtual function for volume
};

//////////////////////////////////////////////////////////
//Derived classes for 2D and 3D Shapes
class Shape2D : public Shape{ // inherit shape
protected:
    int n = 2; //n denotes the number of dimensions
public:
    // default constructor
    Shape2D() :Shape(n){}
    // param constructor
    Shape2D(const double side1, const double side2) :Shape(n){
        sides[0] = side1; sides[1] = side2;
    }
    virtual ~Shape2D(){} //virtual destructor
    double volume() const { cout << "trying to calculate volume of 2d   shape..." << endl; return 0; };
};

/////////////////////////////////////////////////////////////////////
//2D shapes
class Rectangle : public Shape2D{
public:
    // constructors
    Rectangle() :Shape2D() {}
    Rectangle(const double side1, const double side2) :Shape2D(side1, side2){}
    ~Rectangle(){}
    double area() const { return (sides[0] * sides[1]); }
};

int main(){
    Shape **ShapePointer = new Shape*[2];
    ShapePointer[0] = new Rectangle(2, 5);
    ShapePointer[1] = new Rectangle(1, 3);
    // clean up 
    delete ShapePointer[0];
    delete ShapePointer[1];
    delete[] ShapePointer;
    system("pause");
    return 0;
}

【问题讨论】：

在一个不相关的（也许）我认为不需要ShapePointer。只需有一个指向Shape 的普通指针数组，如Shape* ShapeArray[2] = { new Rectangle(...), new Rectangle(...) }
谢谢，我最初尝试过，但遇到了这个问题。
与您的问题更相关，您不能在调试器中运行并在运行程序时捕获错误吗？然后你应该能够在你的代码中找到它发生的位置。或者至少添加大量调试日志来帮助您缩小范围？
double *sides; -- 只需使用std::vector<double> sides;，因为您的Shape 对象既不能安全地复制也不能按其编写方式分配。

标签： c++ c++11 initialization

【解决方案1】：

危险！！！

protected:
    int n = 2; //n denotes the number of dimensions
public:
    // default constructor
    Shape2D() :Shape(n){}

初始化Shape(n)时n未定义

同样的问题

Shape2D(const double side1, const double side2) :Shape(n)

如果可以，将n 定义为static const（或static constexpr，因为您标记了C++11）

 protected:
    static constexpr int n = 2; //n denotes the number of dimensions

否则，您应该定义一个静态的const/constexpr 变量（例如nDef），其值为2，并使用此常量初始化n 和Shape()。

无论如何，考虑一下，现在Shape（Shape2D 的基类）被初始化之前 n 是该类的成员。

【讨论】：

【解决方案2】：

问题在于以下两行代码： int n = 2; //n denotes the number of dimensions Shape2D(const double side1, const double side2) :Shape(n){ ... }

问题是Shape() 在n 初始化之前被执行。详情请见constructors-called-before-initializing-variables。

解决此问题的方法是在 Shape 类中创建以下方法

protected: 
void InitializeBuffer(const int n)
{
sides = new double[n]
}

而不是在父构造函数中初始化sides，而是在Shape2D类的构造函数中调用这个InitializeBuffer

Shape2D(const double side1, const double side2) :Shape()
{
    InitializeBuffer (n);
    sides[0] = side1; sides[1] = side2;
}

【讨论】：