Question

我在使用动态分配的内存进行处理时出现问题。我一直在寻找类似的问题，但遗憾的是没有一个解决方案帮助了我。我班的代码如下所示：

我的班级宣言：

class StructuralElement
{
    private:
    short rows;
    short columns;
    short *se;
    friend class Morph;
public:
    StructuralElement(char *path);
    ~StructuralElement();
    short getAt(short row, short column);
    int getSize();
};

我班级的定义：

StructuralElement::StructuralElement(char *path)
{
std::string line;
short rows=0, elements=0;
short it = 0;
std::string token;
try
{
    std::ifstream file(path);                   //counting rows and columns in a file!
    if (file.is_open() == NULL){ CannotOpenException ex; throw ex; }
    if (file.fail()) { CannotOpenException ex; throw ex; }
    while (getline(file,line))
    {
        rows++;
        std::stringstream ss(line);
        while (getline(ss, token, ' '))
        {
            elements++;
        }

    }
    file.close();
    this->rows = rows;
    if (rows!=0)
    this->columns = (elements/rows);
    se = new short[elements];

    std::ifstream file2(path);
    if (!file2.is_open()) throw;
    while (getline(file2, line))
    {
        std::stringstream ss(line);
        while (getline(ss, token, ' '))
        {
            this->se[it++] = (static_cast<int>(token.at(0))-48);        
        }
    }
    file2.close();
}
catch (CannotOpenException &ex)
    {
    std::cerr << "Error occured! Unable to load structural element!";
    }
}

StructuralElement::~StructuralElement()
{
    if (se != NULL) delete[] se;
}

当程序已经完成工作并出现文本时：＆＃34;小心任何钥匙......＆＃34;在控制台中，然后我收到错误：

Debug Assertion Failed

.......

表达式：_BLOCK_TYPR_IS_VALID（pHeap-＆gt; nBlockUse）

当我换行时 se =新的短[元素]; 到这一个： se = new short [];
然后我收到消息：程序已在断点处触发。

我做错了什么？提前感谢您提供任何提示。

Answer 1

您很有可能进行双重删除。

您的班级没有复制构造函数或赋值运算符，但您以非RAII方式管理资源。这很可能是导致问题的原因。实现一个适当的复制构造函数和赋值运算符，它实际上会为您的数据成员创建深层副本。

Answer 2

蒂莫 - 好消息！创建复制构造函数已经解决了我的问题。非常感谢你，因为我被困在这个地方。再次谢谢。这是一个代码：

 StructuralElement::StructuralElement(const StructuralElement &obj)
    {
        short *newse = new short[obj.columns*obj.rows];
        for (int i = 0; i < obj.columns*obj.rows; i++)
            newse[i] = obj.se[i];
        this->rows = obj.rows;
        this->columns = obj.columns;
        this->se = newse;
    }

但是告诉我，当我使用复制构造函数时？我想我永远不会创建基于现有的新对象？

调试断言失败！错误的记忆释放

2 个答案: