C++无法填充数据

int main()
{
    // don't need global variables, just define local ones here
    int x,y;
    Point point;
    string options;
    // You shouldn't store the vector of Points in the Point class itself.
    // It doesn't have anything to do with a Point. classes should generally
    // only contain relevant information (ex. Point contains only x and y coords).
    vector<Point> pointsVector;
    // do-while will do the contents of the loop at least once
    // it will stop when the while condition is no longer met
    do
    {
        cout << "Please Enter x-Cordinate"<< endl;
        cin >> x;
        cout << "Please Enter y-Cordinate" << endl;
        cin >> y;
        pointsVector.push_back(Point(x, y));
        cout << "Enter cords again? yes/no"<< endl;
        cin >> options;
    } while (options == "yes")
    // don't really need to check if options is "no"
    // if you have exited the do/while loop above, the assumption is that you don't 
    // want to enter more coordinates.
    doSomethingWithTheVectorOfPoints(pointsVector);
    return 0;
}

在函数doSomethingWithTheVectorOfPoints中，您可以放置用于输出X和Y坐标的代码。(您也可以直接循环遍历 main 函数中的向量。

此外，还可以向 Point 类添加一个名为 ToString 或 Print 的成员函数来为您完成工作。

编辑：我实际上并没有编译这个，它只是让你知道如何重写你的代码。

你应该有：

• 无全局变量
• 支持流输入(输出(的点类
• 从点类中存储的数据(为什么一个糟糕的点应该管理它？
• 使用验证流式传输输入。

例：

#include <iostream>
#include <stdexcept>
#include <sstream>
#include <vector>
struct Point {
    int x;
    int y;
};
std::istream& operator >> (std::istream& in, Point& point) {
    return in >> point.x >> point.y;
}
typedef std::vector<Point> PointStorage;
int main()
{
    PointStorage point_storage;
    Point point;
    while(true) {
        std::cout << "Please enter X and Y xordinates or 'no' to stop input" << std::endl;
        std::string line;
        if( ! std::getline(std::cin, line))
            throw std::invalid_argument(line);
        else {
            std::istringstream point_input(line);
            // Skip leading white spaces, read a point, skip trailing white apace
            // and ensure no additional character is left.
            if(point_input >> point >> std::ws && point_input.eof()) {
                point_storage.push_back(point);
            }
            else {
                std::string no;
                std::istringstream no_input(line);
                // Skip leading white spaces, read "no", skip trailing white apace
                // and ensure no additional character is left.
                if(no_input >> no >> std::ws && no_input.eof() && no == "no") {
                    break;
                }
                throw std::invalid_argument(line);
            }
        }
    }
    for(PointStorage::const_iterator pos = point_storage.begin();
        pos != point_storage.end();
        ++pos)
    {
        std::cout << pos->x << ", " << pos->y << 'n';
    }
    return 0;
}

注意：抛出异常可能是一个错误的决定，但它简化了示例。

每次输入"no"时，您都会使用最终坐标重新创建Point对象。这就是为什么你只保留最后一对。

在不相关的说明中，您可能应该大大简化代码。Point对象没有理由首先保留Point对象的向量。您可能希望在那里保留原始坐标的历史记录/序列，并具有类似以下内容的内容：

Point mypt;
while (options == "yes") {
    mypt.AddCoords(x, y);
    // read more coords/options
}
// do stuff on filled mypt object