8

为什么会发生这种情况，我的程序说它没有错误，但是当我运行它时，我在抛出一个'std::out_of_range'what():vector:_M_range_check 的实例后被调用终止。我是 C++ 新手，所以我不明白这些错误

#include <vector>
#include <iostream>
#include <random>
#include <time.h>

using namespace std;
using std::vector;

int main()
{

vector<int> deck;
vector<int> nums;
default_random_engine eng(time(0));
uniform_int_distribution<int> dis(0, 51);

int pos1;
int pos2;
int num1;
int num2;
int i;
int n;
int m;

for (i = 0; i < 52; i++)
{
    nums.push_back(i);

}

for(int j = 0; j < 52; j++)
{
    cout << nums.at(i) << "\n";
}


for(n = 0; n < 50; n++)
{
    pos1 = dis(eng);
    pos2 = dis(eng);

    cout << pos1 << "\n" << pos2 << "\n";

    num1 = deck.at(pos1);
    num2 = deck.at(pos2);

}

}
4

2 回答 2

18

在我看来，这似乎是由于拼写错误，您应该在第二个循环中使用变量“j”。在第一个循环之后，

for (i = 0; i < 52; i++)
{
    nums.push_back(i);
}

变量“i”包含值 52，因此调用 nums.at(i) 听起来预期会抛出 std::out_of_range，因为 nums 仅包含 52 个值，从索引 0 开始。

for(int j = 0; j < 52; j++)
{
    cout << nums.at(i) << "\n";
}

通过用 'j' 替换 at() 的参数来修复它，我认为这是最初的意图：

for(int j = 0; j < 52; j++)
{
    cout << nums.at(j) << "\n";
}
于 2013-10-07T06:30:41.843 回答
4

deck您可以在此处访问元素：

num1 = deck.at(pos1);
num2 = deck.at(pos2);

但它是空的。在拨打这些电话之前，您必须在某个时候填写它。您可以使用该方法检查向量是否为空std::vector::empty，并使用std::vector::size. 有关这两种方法的更多信息，请参阅此std::vector参考。

于 2013-10-07T04:31:57.597 回答