尝试删除动态分配的对象会导致运行时崩溃

-2

我正在优化当前的CLI项目。在查看代码和调试时，我发现我没有在类中释放一些动态分配的内存。这里是我的代码看起来是：尝试删除动态分配的对象会导致运行时崩溃

#include "Laser.h" 

Laser::Laser(int x, int y) 
{ 
    initCoord = new Coordinate; 
    initCoord->x = x; 
    initCoord->y = y; 
    icon = '~'; 
} 

char Laser::getIcon() const { return icon; } 
Coordinate* Laser::getCoord() { return initCoord; } 

void Laser::move() 
{ 
    ++initCoord->x; 
}

我已经试过

“Coordinates.h”

#pragma once 
#include <Windows.h> 

#define LENGTH 40 
#define WIDTH 15 

struct Coordinate 
{ 
    int x = 1; 
    int y = 1; 
};

“Laser.h”

#pragma once 
#include "Coordinates.h" 

class Laser 
{ 
private: 
    Coordinate* initCoord; 
    char icon; 

public: 
    Laser(int x, int y); 
    char getIcon() const; 
    Coordinate* getCoord(); 
    void move(); 
};

“Laser.cpp”添加一个析构函数（当然首先在头文件中声明它），清理分配给initCoord的内存，这看起来像这：

Laser::~Laser() 
{ 
    if(initCoord != nullpr) delete initCoord; 
}

添加后，它导致运行时错误。 “ProgramName.exe已停止工作...”此类的对象存储在一个简单的向量中，该向量在程序一次被清除。问题在于碰撞发生在它达到lasers.clear()线之前。我真的不知道为什么这个崩溃发生，并会感谢一些帮助。谢谢！ :)

来源

2017-10-21 Zablas

您违反了三/五规则。 – Rakete1111

我很好奇为什么你甚至只用2个'int'成员动态地分配'struct'。在这种情况下，将其存储为“普通”成员应该更好。（如果你真的需要一个指针，使用智能指针几乎总是一个更好的解决方案） – UnholySheep

我不确定，但没有代码“++ initCoord-> x”改变你的指针“x”字段？ – ElChupacabra

如果您认为此代码

int main() { 
    Laser one(0,0); 
    { 
     Laser two = one; 
     cout << two.getCoord()->x << endl; 
    } 

    return 0; 
}

你有什么期望写？

“0”

这意味着坐标点，以相同的结构one这又意味着，当第一}发生，two被摧毁one没有一个有效的协调了。

当你有一个指针成员时，你需要禁用复制/分配或实现它们。

在这种情况下，如果您使用了std :: unique_ptr而不是原始指针（这也可以节省您的删除），那么您也可以获救。

来源

2017-10-21 20:23:00 Surt

感谢或解释！我切换到shared_ptr，因为我需要返回它。 – Zablas

尝试删除动态分配的对象会导致运行时崩溃

回答

相关问题