从函数返回向量时 C++ 中止（核心转储）

Question

我正在尝试从函数返回一个向量。我的代码编译并且我检查了我的函数并认为错误来自返回部分。它编译得很好（使用 Cygwin），但是在运行它时，我得到一个 Aborted (core dumped) 错误。 这是我的代码：

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

//function that returns the square
int f(int n)
{
    return n*n;
}
vector<int> myVec;
int counter = 0;
//function that uses f on all elements in a list
vector<int> map(vector<int> something)
{
    //base case
    if(counter == something.size())
    {
        /*cout << "hello" << endl;
        for (int i=0; i<counter; i++)
        {
            cout << "vector: " << myVec[i] << endl;
        }*/
        counter=0;
        return myVec;
    }
    //recursion
    else
    {
        //cout << "counter: " << counter << endl;
        int n = f(something[counter]);
        //cout << "n: " << n << endl;
        myVec.push_back(n);
        //cout << "vector: " << myVec[counter] << endl;
        counter++;
        map(something);
    }
}

int main()
{
    //making vectors
    vector<int> L;
    vector<int> L1;
    vector<int> L2;
    for (int i=0; i<20; i++)
    {
        L.push_back(i);
    }
    L1 = map(L);
}

代码最初来自一个类文件。

Answer 1

在您的递归中，您不返回任何内容。该函数应返回一个向量。

在您的情况下，如果函数在第一次调用时进入“else”情况会发生什么？它重新进入 map() 直到满足条件，然后返回一个向量。该向量被传递到上一个递归调用并立即删除，因为它不再被传递。

这里的解决方案是将 else-case 的最后一行更改为

return map(something);

因此该值不会丢失并正确传递给原始调用者（您的主函数）。

Answer 2

当您的返回类型是向量时，总是返回一个向量。在您的函数中，有一个不会返回任何内容的分支，这会导致问题。

vector<int> map(vector<int>& something)
{
    //base case
    if(counter == something.size())
    {
        /*cout << "hello" << endl;
        for (int i=0; i<counter; i++)
        {
            cout << "vector: " << myVec[i] << endl;
        }*/
        counter=0;
        return myVec;
    }
    //recursion
    else
    {
        //cout << "counter: " << counter << endl;
        int n = f(something[counter]);
        //cout << "n: " << n << endl;
        myVec.push_back(n);
        //cout << "vector: " << myVec[counter] << endl;
        counter++;
        map(something); //you should return a vector here
        return std::vector<int>(); //empty vector
    }
}

还要注意我在函数调用中添加的“&”符号，以便通过引用传递向量。否则，您将传递一个不会更改的副本。我不知道那个“地图”功能是做什么的，所以我不能为你正在做的事情推荐更好的模型。