从函数返回一个包含另一个对象的对象

Question

为什么第一次调用cout后drawManifestoGlobal中的值会发生变化？看起来 canvas.panel.drawManifestoGlobal 被破坏了 - 为什么？

我该如何解决这个问题？

#include <iostream>

class DrawManifestoGlobal {
public:
    int value = 2;
};

class Panel {
public:
    void setDrawManifestoGlobal(DrawManifestoGlobal & _drawManifestoGlobal);
    DrawManifestoGlobal * drawManifestoGlobal;
};

class Canvas {
public:
    Canvas() {};
    Canvas(DrawManifestoGlobal _drawManifestoGlobal);
    DrawManifestoGlobal drawManifestoGlobal;
    Panel panel;
};


class SerDe {
public:
    Canvas doSerDe();
};

Canvas SerDe::doSerDe() {

    DrawManifestoGlobal drawManifestoGlobal;
    drawManifestoGlobal.value = 99; 
    Canvas canvas(drawManifestoGlobal);

    return canvas;
}

Canvas::Canvas(DrawManifestoGlobal _drawManifestoGlobal) {
    drawManifestoGlobal = _drawManifestoGlobal;
    panel.setDrawManifestoGlobal(drawManifestoGlobal);
}

void Panel::setDrawManifestoGlobal(DrawManifestoGlobal &_drawManifestoGlobal) {
    drawManifestoGlobal = &_drawManifestoGlobal;
}

int main () {
    SerDe serde;
    Canvas canvas;
    canvas = serde.doSerDe();

    std::cout << canvas.panel.drawManifestoGlobal->value << std::endl; // prints 99
    std::cout << canvas.panel.drawManifestoGlobal->value << std::endl; // prints 0 (!!!)
}

关于实现要求：Canvas 拥有 Panel 和 DrawManifestoGlobal，并且 Panel 本身具有指向 Canvas 的 DrawManifestoGlobal 的指针，因此从 Canvas 发生的任何更改对 Panel 都是可见的。

Answer 1

为什么drawManifestoGlobal 中的值在第一次调用cout 后会发生变化？看起来 canvas.panel.drawManifestoGlobal 被破坏了 - 为什么？

doSerDe() 函数内的drawManifesttoGlobal 是一个在作用域末尾过期的局部变量，但您正在设置指向它的指针 (Panel::drawManifestoGlobal) 并将指针传递到函数外部（通过canvas)。所以你有未定义的行为来访问一个已经被破坏的对象的值。编译器完全有权为同一个对象打印两个不同的值。

如果你真的需要一个指针（你不需要），你会想要动态分配drawManifestoGlobal（最好使用unique_ptr）。

class Panel {
public:
  void setDrawManifestoGlobal(std::unique_ptr<DrawManifestoGlobal> _drawManifestoGlobal);
  std::unique_ptr<DrawManifestoGlobal> drawManifestoGlobal;
};

Canvas SerDe::doSerDe() {
  std::unique_ptr<DrawManifestoGlobal> drawManifestoGlobal(make_unique<DrawManifesttoGlobal>());
  drawManifestoGlobal.value = 99; 
  return {std::move(drawManifestoGlobal)};
}

void Panel::setDrawManifestoGlobal(std::unique_ptr<DrawManifestoGlobal> drawManifestoGlobal) {
  drawManifestoGlobal = std::move(_drawManifestoGlobal);
}

事实上，您的代码中的任何地方都不需要指针！