2

我正在制作一个需要我将 36 个数字初始化为向量的 C++ 游戏。你不能用初始化列表初始化向量,所以我创建了一个 while 循环来更快地初始化它。我想让它从 2 到 10 将每个数字的 4 推回,所以我使用一个名为第四的 int 来检查循环的数量是否是 4 的倍数。如果是,它会将推回的数字更改为下一个数字。但是,当我运行它时,我得到了 SIGABRT。不过,肯定是第四个有问题,因为当我把它拿出来时,它没有发出信号。这是程序:

for (int i; i < 36;) {
    int fourth = 0;
    fourth++;
    fourth%=4;
    vec.push_back(i);
    if (fourth == 0) {
        i++;
    }
}

请帮忙!

4

3 回答 3

5

您不初始化i. 使用for (int i = 0; i<36;). forth此外,循环体的每次迭代都会分配一个新变量。因此,测试fourth==0总是会产生false

我想让它把每个数字的 4 从 2 推回 10

我会使用最直接的方法:

for (int value = 2; value <= 10; ++value)
{
  for (int count = 0; count < 4; ++count)
  {
    vec.push_back(value);
  }
}

我要做的唯一优化是在进入循环之前确保向量的容量足够。我会将其他优化留给编译器。我的猜测是,你通过省略内部循环获得了什么,你会因为频繁的模除而失去。

于 2011-05-03T01:08:56.277 回答
1

您没有初始化 i,并且您在每次迭代中重置第四个。另外,对于您的 for 循环条件,我认为它不会满足您的要求。

我认为这应该有效:

int fourth = 0;
for (int i = 2; i<=10;) {
  fourth++;
  fourth%=4;
  vec.push_back(i);
  if (fourth==0) {
     i++;
  }      
}
于 2011-05-03T01:11:38.000 回答
0

我已经能够创建一个静态数组声明并在初始化时将该数组传递给向量,而不会出现问题。也很干净:


const int initialValues[36] = {0,1,2...,35};
std::vector foo(initialValues);

适用于常量,但尚未尝试使用非 const 数组。

于 2011-05-03T02:11:20.410 回答