在堆栈C ++上创建的对象中的数据无效

Question

我真的找不到这个问题的更好的标题。

我有三个类： CheckBox ， Button和Background 。

class CheckBox : public Component
{
private:
    Button m_button;
public:
    CheckBox(const Point &pos, int width, int height, std::string text);
    CheckBox();
};

CheckBox::CheckBox(const Point &pos, int width, int height, string text) : 
    Component(pos, width, height),
    m_button(Button(Point(width-height,0), new Background("button_bg_sample.png", true), new Background("button_bg_onclick_sample.png", true), height, height, 10, "")),
{

}


class Button : public Component
{
private:
    std::string m_text;
    Background* m_pBackground;
    Background* m_pBackgroundOnClick;
    int m_fontSize;
public:
    Button(const Point& pos, Background* pBg, Background* pBgOnClick, int width, int height, int fontSize, std::string title);
    ~Button();

};

Button::Button(const Point& pos, Background* pBg, Background* pBgOnClick, int width, int height, int fontSize, string title) : 
    Component(pos, width, height), 
    m_pBackground(pBg),
    m_pBackgroundOnClick(pBgOnClick),
    m_fontSize(fontSize),
    m_text(title)
{
}


class Background
{
private:
    std::string m_pFileName;
    bool m_bTiling;
    std::vector<unsigned char> m_pImageData;
    unsigned int m_width;
    unsigned int m_height;
    GLuint m_texture; 

    bool load(const std::string& pFileName);
public:
    Background(const std::string& pFileName, bool bTiling);
    ~Background();

    bool draw(const Point &pos, unsigned int width, unsigned int height);
    bool draw(const Point &pos);
};

Background::Background(const string& pFileName, bool bTiling) : 
    m_bTiling(bTiling), 
    m_pFileName(pFileName)
{
    load(pFileName);
}

如您所见， CheckBox类包含Button m_button ， Button类包含Background* m_pBg 。 在Background构造函数中，我加载图像数据并将其存储在std::vector ，这并不重要 - 我知道它正在工作，因为它已经被检查过了。

当我创建CheckBox对象时，它的m_button的数据已损坏。 当我尝试在调试模式下检查内部图像数据时，我得到的信息是空的，并且该背景的文件名是"Error reading characters in string" 。 虽然当我看到在调试模式下一步一步地执行代码时，我已经看到数据在构造函数中正确加载，但不知何故，当创建对象时，数据已经损坏。

当我在CheckBox类中更改要在堆上创建的m_button字段（指向Button指针，使用new运算符创建的对象）时，一切似乎都正常工作。 正确加载数据并保持这样。

请问有谁可以解释一下这个问题的原因是什么？

Answer 1

问题是你不遵守三个规则 。

Button类管理内存本身，因此当您创建副本或进行分配时，您可能会遇到麻烦。 我没有挖掘代码，所以不能肯定这是根（尽管如此仍是一个问题），但我很确定它是。

如果您使用智能指针而不是原始指针来管理资源，则可以避免这一切。

Answer 2

您在CheckBox的构造函数的初始化列表中的堆栈上创建的Button正被复制到m_button字段：

m_button = Button();

default = operator正在做什么，将新创建的Button对象中的每个字段复制到m_button ：

m_button.height = b.height // Let's assume, that 'b' is the name of this new Button
m_button.width = b.width
m_button.name = b.name
...

现在如果你有一个指向某个对象的指针p1并且你有另一个指针p2并且你将p1分配给p2什么？

int* p1 = new int(5);
int* p2 = p1;

两个指针现在指向同一个对象！ 同样的事情发生在你的Background* 。 你有两个Button指向内存中的同一个Background对象。 但是你在初始化列表中创建的第二个Button正在被销毁，因为它超出了一个范围，并且发生了什么PROBABLY（因为我还没有看到你的Button类的析构函数），你正在破坏Button析构函数中的Background对象。 这就是为什么当您在调试模式下遵循代码时，您会看到正确创建的Background ，但稍后它会被Button对象销毁。