16

以下代码在第一个 for 循环中用 10 个值填充向量。在第二个 for 循环中，我希望打印向量的元素。输出直到 j 循环之前的 cout 语句。给出向量下标超出范围的错误。

#include "stdafx.h"
#include "iostream"
#include "vector"
using namespace std;

int _tmain(int argc, _TCHAR * argv[])
{
    vector<int> v;

    cout << "Hello India" << endl;
    cout << "Size of vector is: " << v.size() << endl;
    for (int i = 1; i <= 10; ++i)
    {
        v.push_back(i);

    }
    cout << "size of vector: " << v.size() << endl;

    for (int j = 10; j > 0; --j)
    {
        cout << v[j];
    }

    return 0;
}
4

3 回答 3

14

无论您如何索引推回，您的向量都包含从0( 0, 1, ..., 9) 索引的 10 个元素。所以在你的第二个循环v[j]中是无效的，when jis 10.

这将修复错误：

for(int j = 9;j >= 0;--j)
{
    cout << v[j];
}

一般来说，最好将索引视为0基于，因此我建议您也将第一个循环更改为：

for(int i = 0;i < 10;++i)
{
    v.push_back(i);
}

此外，要访问容器的元素，惯用的方法是使用迭代器（在这种情况下：反向迭代器）：

for (vector<int>::reverse_iterator i = v.rbegin(); i != v.rend(); ++i)
{
    std::cout << *i << std::endl;
}
于 2013-10-19T11:34:03.927 回答
5

v有10元素，索引从0to开始9。

for(int j=10;j>0;--j)
{
    cout<<v[j];   // v[10] out of range
}

您应该将for循环更新为

for(int j=9; j>=0; --j)
//      ^^^^^^^^^^
{
    cout<<v[j];   // out of range
}

或者使用反向迭代器以相反的顺序打印元素

for (auto ri = v.rbegin(); ri != v.rend(); ++ri)
{
  std::cout << *ri << std::endl;
}
于 2013-10-19T11:34:47.673 回答
2

当您尝试通过未分配数据数据的索引访问数据时，通常会发生此类错误。例如

//assign of data in to array
for(int i=0; i<10; i++){
   arr[i]=i;
}
//accessing of data through array index
for(int i=10; i>=0; i--){
cout << arr[i];
}

该代码将给出错误（向量下标超出范围），因为您正在访问尚未分配的 arr[10]。

于 2018-05-09T05:16:54.037 回答