std::vector 未调整大小以在一定数量的条目后容纳更多条目 C++

Question

我有一个自定义向量类，它封装了std::vector。我在 while 循环中使用它逐行读取 CSV 文件并将列存储到名为 columns 的 Vector 中，并对这些值执行一系列操作。一切正常，但是在循环了一些行之后，它会抛出错误：

在抛出 std::out_of_range 的实例后调用终止

我假设我的 Vector 出于某种原因停止调整大小，但它会按照我想要的方式读取每一行，然后再抛出此错误并停止。我使用std::cerr 语句来查看我的几个列值是否被正确读取并使用它，我可以看到它在文件结束之前停止。为什么会这样？

while(getline(datafile, line))
{
    string token;
    Vector<string> columns;
    WindLogType windlog2;
    stringstream ss(line);
    columns.add(string());
    while(getline(ss, token, ','))
    {
        columns.add(token);
    }
    stringstream date(columns[1]);
    string windspeed = columns[11];
    string solar1 = columns[12];
    cerr << solar1 << endl;
    string temperature1 = columns[18];
    cerr << temperature1 << endl; 
}

我的向量类：

#ifndef VECTOR2_H
#define VECTOR2_H
#include <iostream>
#include <string>
#include <sstream>
#include <vector>

using namespace std;

template <class T>
class Vector
{
public:
    Vector(){};
    ~Vector();

    void add(const T &obj);
    int vecsize() const{return data.size();}

    T& operator[](const int index);
    const T& operator[](const int index) const;

private:
    vector<T> data;

};

template <class T>
T& Vector<T>::operator[](int index){
    if(index < 0 || index > data.size()){
        throw("Out of bounds");
    }

    return data.at(index);
}

template <class T>
const T& Vector<T>::operator[](int index) const{
    if(index < 0 || index > data.size()){
        throw("Out of bounds");
    }

    return data.at(index);
}

template <class T>
Vector<T>::~Vector(){    
    data.clear();    
}

template <class T>
void Vector<T>::add(const T &obj){    
    data.push_back(obj);    
}

#endif // VECTOR_H

Answer 1

通常建议避免使用using namespace ...; in a header。如果您要实现已在该命名空间中找到的功能（例如向量），则尤其如此。

您应该考虑抛出一个实际的异常类型，而不是throw("str");，例如std::out_of_range，或者甚至是您自己的异常类型。 throw("str") 有点不寻常，这意味着捕获站点必须使用catch(const char *)。这不是无效的，但也不是常见的做法。

在您的 T& Vector<T>::operator[](int) 函数（和 const-qualified 对应项）中，您正在检查索引是否小于 0（这是正确的），但您正在检查索引是否 大于 em> 内部向量的大小。如果您考虑内部向量为空的情况，那么这意味着根本没有有效索引（因为没有元素），但是，您的验证将允许索引 0 到内部向量，因为它不是大于尺寸。如果内部向量有 1 个元素，那么您的验证允许索引 0 和索引 1，这是不正确的，它应该只允许索引 0。您可以将此检查更改为使用 index >= data.size()。这称为“逐一”错误。

除非您计划提供负索引的实现，否则通常最好使用size_t 或其他无符号整数类型。

最后，不需要在析构函数中显式调用data.clear()，因为向量的析构函数已经这样做了。