C++中find_period函数内for循环无法终止问题求助

嘿，我一眼就揪出问题根源啦——这是典型的**未定义行为（Undefined Behavior）**导致的诡异现象，和迭代器本身、指针参数都没关系！

核心问题：函数未正确返回值 #

你的find_period函数声明返回int类型，但函数体内完全没有return语句。在C++里，这种情况属于未定义行为：程序可能出现任何不可预测的表现，包括循环无法终止、内存乱读、甚至偶尔看似正常运行（具体表现取决于编译器和运行环境）。

而当你把循环移到main()里时，main函数是特殊的——C++标准规定，如果main函数没写显式的return语句，会默认返回0，所以不会触发未定义行为，循环自然能正常终止。

为什么改成指针参数也没用？ #

问题的核心根本不是参数传递方式（值传递/指针传递），而是函数声明了返回类型却不返回值。不管你怎么传参数，只要这个核心问题没解决，未定义行为就会存在，奇怪的现象自然也不会消失。

修复方案 #

根据你的需求，有两种常见的修复方式：

1. 如果函数不需要返回值，改为void类型 #

直接把函数的返回类型改成void，同时去掉main里对它返回值的输出：

#include <iostream>
#include <vector>
using namespace std;

void find_period(const vector<int>& arr){ // 用const引用避免拷贝，提升效率
    vector<int>::iterator iter;
    for(iter=arr.begin(); iter != arr.end(); iter++){
        cout << *iter << endl;
    }
}

int main() {
    vector<int> input = {2,2,3,4,5,6,7,8,9};
    find_period(input); // 直接调用，不需要cout输出返回值
    return 0;
}

2. 如果函数需要返回某个值（比如周期长度），添加正确的return语句 #

假设你原本想返回数组的周期长度（虽然当前代码没实现这个逻辑），那就要在函数末尾添加对应的return语句，比如：

#include <iostream>
#include <vector>
using namespace std;

int find_period(const vector<int>& arr){
    // 这里添加你的周期计算逻辑...
    vector<int>::iterator iter;
    for(iter=arr.begin(); iter != arr.end(); iter++){
        cout << *iter << endl;
    }
    return arr.size(); // 示例：返回数组长度
}

int main() {
    vector<int> input = {2,2,3,4,5,6,7,8,9};
    cout << find_period(input) << endl; // 现在可以正常输出返回值了
    return 0;
}

内容的提问来源于stack exchange，提问作者nimdrak