为何无法将return -1移至循环外或改用else块？附代码示例

先看你当前的代码问题：

int index_of(vector<string> &names, string name)
{
    for (int i = 0; i < names.size(); i++)
    {
        if (to_lowercase(names[i]) == to_lowercase(name))
        {
            return i;
        }
        return -1; // 这就是问题根源
    }
}

这个return -1写在for循环的单次迭代里，只要第一个元素不匹配，函数直接就返回-1了，根本不会继续检查后面的元素，完全达不到遍历整个数组找匹配项的目的。

把return -1移到循环外才是正确的，不存在“不可行” #

正确写法应该是把return -1放在for循环结束之后——也就是整个数组都遍历完，确实没找到匹配项的时候，才返回-1：

int index_of(vector<string> &names, string name)
{
    for (int i = 0; i < names.size(); i++)
    {
        if (to_lowercase(names[i]) == to_lowercase(name))
        {
            return i;
        }
    }
    // 所有元素都查过了没找到，才返回-1
    return -1;
}

这种写法逻辑完全通顺：遍历每个元素，找到匹配的立刻返回索引；如果循环正常跑完（说明没找到任何匹配），就返回-1。你觉得“不可行”，大概率是尝试的时候写错了，不是这个思路本身有问题。

用else块的正确姿势 #

如果想用else块，绝对不能把return -1放在else里（不然还是第一次不匹配就返回，和原来的错误一样）。实际上这里else块根本没必要——不匹配的时候，循环本来就会自动进入下一次迭代，继续检查下一个元素。

举个错误的else用法（和你原来的问题一模一样）：

// 错！第一次不匹配就直接返回-1
for (int i = 0; i < names.size(); i++)
{
    if (匹配) { return i; }
    else { return -1; }
}

正确的逻辑里，else块不用写任何返回语句，就让循环继续跑就行，最后还是要在循环外返回-1。

总结：你一开始想的（把return -1移到循环外或者用else）是对的，当前代码的问题是return -1的位置错了导致逻辑崩了，不是你的思路不可行。

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