Почему я не могу поместить этот объект в свой std :: list?

Это должен быть допустимый фрагмент кода.

#include <iostream>
#include <list>

class Point { 
public:
    int x, y;
    Point(int x1, int y1)
    {
        x = x1;
        y = y1;
    }
};

int main()
{
    std::list<Point> points;

    points.push_back(Point(0, 0));
    points.push_back(Point(1, 1));
    points.push_back(Point(2, 2));

    std::list<Point>::iterator iter;

    for(iter = points.begin(); iter != points.end(); ++iter)
    {
        Point test = *iter;
        std::cout << test.x << ", " << test.y << "; ";
    }
    std::cout << std::endl;

    return 0;
}

Используя этот код:

jasons-macbook41:~ g++ test.cpp
jasons-macbook41:~ ./a.out
0, 0; 1, 1; 2, 2; 
jasons-macbook41:~ 

Хотя я бы не стал создавать временную копию Point, как ваш код. Я бы переписал цикл так:

for(iter = points.begin(); iter != points.end(); ++iter)
{
    std::cout << iter->x << ", " << iter->y << "; ";
}

Итератор синтаксически похож на указатель.

РЕДАКТИРОВАТЬ: Учитывая вашу новую проблему, отбросьте «новый» из строительной строки. Это создает указатель на Point, а не на Point в стеке. Это было бы верно:

Point* temp = new Point(0, 0);

Или это:

Point temp = Point(0, 0);

И тебе будет лучше с последним.

несколько вещей..

• Вы пробовали iter->x и iter->y вместо копирования значения?
• указанную вами ошибку трудно понять. Вы не пытаетесь получить x и y через итератор, вы копируете данные итератора в новую точку.

РЕДАКТИРОВАТЬ:

по новой информации в ОП. Вы пытаетесь создать новый объект без указателя, а затем пытаетесь вставить точку в вектор, который принимает только объекты. Вам нужно либо сделать вектор вектором указателей и не забыть удалить их послесловия, либо создать новую точку в стеке и скопировать их в вектор со стандартным назначением. Попробуй это:

if (button == GLUT_LEFT_BUTTON && state == GLUT_DOWN)
{
    Point currentPoint = Point(x, y);
    pointList.push_front(currentPoint);
}

Если у вас уже есть функция, которую вы хотите применить ко всему списку, std :: for_each - это путь, например,

std::for_each(pointList.begin(), pointList.end(), myGreatFunction);

Если вам нужно написать цикл for, тогда что-то вроде этого:

std::list<Point>::iterator itEnd = pointList.end();
for(std::list<Point>::iterator itCur=pointList.begin(); itCur != itEnd; ++itCur) {
    yourFunction(itCur->x, itCur->y);
}

Примечания:

• ++ itCur может быть более эффективным, чем itCur ++, из-за возвращаемых типов (ссылка vs значение / копия)

Нескалярная проблема заключается в том, что вы назначаете указатель Point (возвращаемое значение оператора new) объекту стека Point (поскольку он не Point * в вашем коде).

Я бы порекомендовал сказать

    Point currentPoint(x, y);
    pointList.push_front(currentPoint);

Обратите внимание, что currentPoint будет скопирован в ваш список; неявно сгенерированный конструктор копирования Point (поскольку вы не объявили конструктор Point (const Point & other) в своем классе, компилятор сделал его за вас) скопирует currentPoint.x и currentPoint.y в список; в этом случае это нормально. Точка маленькая, поэтому затраты на копирование невелики, и она содержит только два целых числа, поэтому прямое копирование целых чисел нормально.

Этот ответ относится к отредактированной версии вопроса.

Как сказал gbrandt в отредактированной версии своего ответа, ваша проблема в том, что вы пытаетесь динамически выделить экземпляр Point, а затем назначить его Point объекту, а не указателю на Point. Результатом new является указатель на Point, а не объект Point - в данном случае вам действительно нужен последний, который вы создаете без new:

Point currentPoint(x, y);
pointList.push_front(currentPoint);

Point *currentPoint = new Point(x, y);
pointList.push_front(*currentPoint);      // Notice the "*"