C++ 指针/引用

Question

此代码从鼠标单击中获取坐标并创建一个新顶点；

void DrawingWidget::mousePressEvent(QMouseEvent *event){
   if(getCurrentState()==ADD_VERTEX){
      x=event->x();
      y=event->y();

      Vertex p1 {&x,&y};
      m_mainWindow->addVertex(p1);
      update();
   }
}

这是 Vertex 类代码：

class Vertex {

public:

   float *x = 0;
   float *y = 0;

   Vertex() = default;
   Vertex(float *nx, float *ny);
};

所有 Verces 都保存在列表中：

QList<Vector2> vertexList;

和addVertex(顶点顶点)：

addVertex(Vertex vertex){
   vertexList << vertex;
}

基本上第一个顶点添加效果很好，但是当我尝试添加第二个顶点时（使用给定的方法），第一个顶点坐标被改变了。

注意！ Vertex 类的坐标是有意的指针，因为通过使用另一种方法我正在改变它们的坐标。

我错过了什么？

Answer 1

您的代码中的x 和y 可能是DrawingWidget 的字段，而在Vertex 中，您始终存储指向相同变量（那些字段）的指针。因此，在 vertexList 中，指向相同字段的指针始终具有相同的值。
我不明白你为什么对你的代码行为感到惊讶。

Answer 2

您问题中的代码和您观察到的行为暗示您有这样的事情：

class DrawingWidget
{
public:
    void mousePressEvent(QMouseEvent *event);
private:
    float x;
    float y;
    // ...
};

当函数 mousePressEvent() 被调用时，您将新坐标分配给这些成员：

void DrawingWidget::mousePressEvent(QMouseEvent *event){
   if(getCurrentState()==ADD_VERTEX){
      x=event->x();
      y=event->y();
...

因此，每次按下鼠标时，都会覆盖旧坐标。现在，这可能不是问题，但是由于您添加到列表中的Vertexs 包含指向这些类成员的指针，而不是简单地存储坐标的副本，第一个顶点出现改变了坐标。

解决这个问题的方法是根本不使用指针（至少对于您的类而言）！

class Vertex
{
public:
    Vertex() : x(0.0f), y(0.0f) {}
    Vertex(float x, float y) : x(x), y(y) {}

    float getX() const { return x; }
    float getY() const { return y; }

    void setX(float x) { this->x = x; }
    void setY(float y) { this->y = y; }

private:
   float x; // NOT pointers!
   float y;
};

class DrawingWidget
{
public:
    void mousePressEvent(QMouseEvent *event);
private:
    //float x; // Not needed!
    //float y;
    // ...
};

void DrawingWidget::mousePressEvent(QMouseEvent *event) {
   if(getCurrentState()==ADD_VERTEX) {
      Vertex p1{event->x(), event->y()};
      m_mainWindow->addVertex(p1);
      update();
   }
}