在我目前的项目中,我偶然发现了一个奇怪的现象,即嵌套的for循环只执行一次内循环然后停止。即使仔细检查了所涉及的所有变量,外部for循环仍然没有正当理由终止。使得这个结构与我程序中包含的其他for循环不同的唯一原因是计数器变量作为参数传递给包含循环的函数,并且从不复制到任何地方。
所以我决定测试问题是否可以重现:
#include <stdio.h>
void someFunction(int x, int y, int width, int length)
{
int endX = x+width;
int endY = y+length;
printf("x will not exceed: %i\n", endX);
printf("y will not exceed: %i\n", endY);
for(; x < endX; x++)
{
for(; y < endY; y++)
{
printf("(%i, %i)\n", x, y);
}
}
}
int main(int argc, const char *argv[])
{
someFunction(1, 1, 5, 5);
return 0;
}
然而,在执行时,应用程序的输出与直观的预期不同:
x will not exceed: 6
y will not exceed: 6
(1, 1)
(1, 2)
(1, 3)
(1, 4)
(1, 5)
切换x和y时行为相似,但y变量从不递增。通过简单地将一个新变量声明为每个循环的计数器来解决该问题。
但为什么会这样呢?是否因特定原因而被禁止?编译器是否禁用某些参数的修改,如果是,为什么它适用于一个变量但不适用于另一个?
提供的源代码是使用GCC / G ++ 4.5.3编译的,没有任何特殊的优化标记。
答案 0 :(得分:13)
y
永远不会在x
循环中重置,因此一旦它超出第一遍的界限,它就会永远超出界限。
答案 1 :(得分:2)
由于x
和y
也存在于循环之外,并且由于循环没有设置其初始值,因此即使重用它们也会保留最后一个值。
这尤其是关于你的y
,它在y循环退出后不会回退到初始值。
为避免此类问题,请避免将循环外部的变量用作索引。
void someFunction(const int x, const int y, const int width, const int length)
{
int endX = x+width;
int endY = y+length;
printf("x will not exceed: %i\n", endX);
printf("y will not exceed: %i\n", endY);
for(int ix=x; ix < endX; ++ix)
{
for(int iy=0; iy < endY; ++iy)
{
printf("(%i, %i)\n", ix, iy);
}
}
}
嘿,通过使参数const我们确保我们不能触摸它们,即使是错误的。然后我们在循环中使用ix和iy local来处理迭代。
此外,除非您必须出于某些其他原因,否则请避免使用后缀增量,并使用前缀。在使用整数之前,这并不是一个很大的变化,但是使用更复杂的变量可以产生差异。
如果您避免像C程序员一样编码,那么这也是一个好主意(因为您将问题标记为C++
而不是C
)。
惯用的C ++等价物是:
#include <iostream>
void someFunction(const int& x, const int& y, const int& width, const int& length)
{
const int endX = x+width;
const int endY = y+length;
std::cout << "x will not exceed: " << endX << std::endl;
std::cout << "y will not exceed: " << endY << std::endl;
for(int ix=x; ix < endX; ++ix)
{
for(int iy=y; iy < endY; ++iy)
{
std::cout << '('<<ix<<", "<<iy<<')'<< std::endl;
}
}
}
int main()
{
someFunction(1, 1, 5, 5);
return 0;
}