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 (Vertex вершина):

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

В основном первый добавление вершин отлично работает, но когда Я пытаюсь добавить вторую вершину (используя данный метод), вместо этого меняются первые координаты вершин.

NB! Координаты класса 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(); 
... 

Таким образом, каждый раз, когда вы нажимаете на мышь, вы находитесь переписываете старые координаты. Теперь это, возможно, не проблема, но так как Vertex с вы добавляете в список указатели этим членам класса, вместо простого хранения копии координат, первая вершина, по-видимому, изменила координаты.

Способ борьбы с этим - не использовать указатели вообще (по крайней мере для ваших классов)!

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(); 
    } 
}