Блог

У меня есть точка структуры, она содержит установщики для x и y:

 struct Point {
    int x;
    int y;

    Point() {
        x = 0;
        y = 0;
    }

    Point(int _x, int _y) {
        x = _x;
        y = _y;
    }

    Pointamp; operator=(Point other) {
        swap(x, other.x);
        swap(y, other.y);
        return *this;
    }

    void setX(int _) { x = _; }

    void setY(int _) { y = _; }

};
 

Также у меня есть элемент для двунаправленного списка, его данные — точка:

 struct Element {
    Point data;
    Element *left;
    Element *right;

    Element(Point data) {
        this->data = data;
        left = right = this;
    }

};
 

А также я создал структуру списка, Point get(int i) возвращает точку, которую я хочу изменить:

 struct List {
    Element *first;
    Element *last;
    int length;

    List() {
        first = last = nullptr;
        length = 0;
    }

    Point add(Point data) {
        Element *e = new Element(data);
        length  ;
        if (!last) {
            first = last = e;
            return data;
        }
        e->left = last;
        last = last->right = e;
        return data;
    }

    Point get(int i) {
        if (i >= length amp;amp; i<0) return Point();
        Element *t = first;
        while(i--) t = t->right;
        return t->data;
    }

};
 

Затем в main я создаю список, добавляю точку в этот список, а затем пытаюсь изменить значения x и y, но они не меняются:

 int main() {
    ....
    List snake;
    snake.add(Point(3,4));
    cout << snake.get(0).x << " " << snake.get(0).y << endl;
    snake.get(0).setX(99);
    snake.get(0).setY(99);
    cout << snake.get(0).x << " " << snake.get(0).y << endl;
    .....
}
 

Пожалуйста, помогите! Почему значения x и y не меняются и как это исправить?

Этот метод:

 Point get(int i) {
    if (i >= length amp;amp; i<0) return Point();
    Element *t = first;
    while(i--) t = t->right;
    return t->data;
}
 

возвращает Point значение by . Возвращаемый объект не является объектом в списке. Следовательно, здесь:

 snake.get(0).setX(99);
 

Вы делаете setX(99) из a Point , но это Point временный объект, возвращаемый из get . Вы, вероятно, хотите вместо этого вернуть ссылку. Однако тогда вам нужно найти другое решение if (i >= length amp;amp; i<0) return Point(); , то есть случай, когда индекс выходит за пределы, потому что вы не можете вернуть ссылку на локальную переменную. Вы можете либо создать исключение, либо просто предположить, что переданный индекс является допустимым. Измените его на:

 Pointamp; get(int i) {
   //^  return reference
    //if (i >= length amp;amp; i<0) return Point();
  //^^ DO NOT return a reference to  ^^ local object !!!
    Element *t = first;
    while(i--) t = t->right;
    return t->data;
}