值未写入矢量

时间:2013-10-15 07:13:36

标签: c++ stdvector

我正在尝试从对象的构造函数中读取文件中的对值。

该文件如下所示:

4
1 1
2 2
3 3
4 4

第一个数字是要读取的对数。

在某些行中,值似乎已正确写入向量中。在接下来他们走了。我完全糊涂了

inline
BaseInterpolator::BaseInterpolator(std::string data_file_name)
{
    std::ifstream in_file(data_file_name);
    if (!in_file) {
        std::cerr << "Can't open input file " << data_file_name << std::endl;
        exit(EXIT_FAILURE);
    }
    size_t n;
    in_file >> n;
    xs_.reserve(n);
    ys_.reserve(n);

    size_t i = 0;
    while(in_file >> xs_[i] >> ys_[i])
    {
        // this line prints correct values i.e. 1 1, 2 2, 3 3, 4 4
        std::cout << xs_[i] << " " << ys_[i] << std::endl;
        // this lines prints xs_.size() = 0
        std::cout <<  "xs_.size() = " << xs_.size() << std::endl;
        if(i + 1 < n)
            i += 1;
        else
            break;
        // this line prints 0 0, 0 0, 0 0
        std::cout << xs_[i] << " " << ys_[i] << std::endl;
    }
    // this line prints correct values i.e. 4 4    
    std::cout << xs_[i] << " " << ys_[i] << std::endl;
    // this lines prints xs_.size() = 0
    std::cout <<  "xs_.size() = " << xs_.size() << std::endl;
}

这个类定义如下:

class BaseInterpolator
{
public:
    ~BaseInterpolator();
    BaseInterpolator();
    BaseInterpolator(std::vector<double> &xs, std::vector<double> &ys);
    BaseInterpolator(std::string data_file_name);

    virtual int interpolate(std::vector<double> &x, std::vector<double> &fx) = 0;
    virtual int interpolate(std::string input_file_name,
                        std::string output_file_name) = 0;


protected:
    std::vector<double> xs_;
    std::vector<double> ys_;
};

3 个答案:

答案 0 :(得分:1)

您遇到了未定义的行为。看起来它的工作量只有一半,但是这个工作差得多一半。

问题在于:

xs_.reserve(n);
ys_.reserve(n);

您只是保留大小,而不是创建它。 替换为:

xs_.resize(n);
ys_.resize(n);

现在,xs[i] i < n实际上有效。

如果有疑问,请使用xs_.at(i)代替xs_[i]。它执行额外的边界检查,可以省去调试的麻烦,而不知道从哪里开始。

答案 1 :(得分:1)

  1. 您正在使用reserve(),这会增加容量(存储空间),但不会增加向量的大小(即它不会向其中添加任何对象)。您应该使用resize()代替。这将照顾size()为0。

  2. 您在增加xs_[i]后重新打印ys_[i]i 。它自然会是0(或者可能是随机值) - 你还没有初始化它们。

答案 2 :(得分:0)

vector::reserve保留空间以便进一步操作,但不要更改向量的大小,您应该使用vector::resize