对于C ++中的循环函数参数奇性

时间:2012-08-08 18:22:15

标签: c++

在我目前的项目中,我偶然发现了一个奇怪的现象,即嵌套的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编译的,没有任何特殊的优化标记。

2 个答案:

答案 0 :(得分:13)

y永远不会在x循环中重置,因此一旦它超出第一遍的界限,它就会永远超出界限。

答案 1 :(得分:2)

由于xy也存在于循环之外,并且由于循环没有设置其初始值,因此即使重用它们也会保留最后一个值。 这尤其是关于你的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;
}