从函数返回 char* [重复]

Question

我试图从一个函数中返回一个 char*，该函数通过取给它的两个字符串的一部分来形成一个字符串。该字符串必须以 char* 形式返回。但是当我运行以下代码时，不会打印任何输出。请帮助解决此问题。

#include <iostream>
#include<string.h>
using namespace std;

char *findpass(char *s1, char*s2)
{
    int sl1 = strlen(s1);
    int sl2 = strlen(s2);
    int i = 0, in = 0;
    char temp[100];
    char *t;
    if (sl1 % 3 == 0)
    {
        for (i = 0; i<sl1 / 3; i++)
        {
            temp[in] = *(s1 + i);
            in++;
        }
    }
    t = temp;
    return t;
}

int main()
{
    char i1[10], i2[10];
    char *f1, *f2;
    cin >> i1 >> i2;
    f1 = i1;
    f2 = i2;
    char *f = findpass(f1, f2);
    cout << f;
    return 0;
}

Answer 1

你的char temp[100];是局部变量，离开作用域后会被销毁，所以t = temp;会指向一个乱七八糟的数据。 您需要为char *temp = new char[100]; 分配内存，然后将t 指向临时t = temp;。你也可以return temp;。但在那之后不要忘记通过delete [] 释放分配的内存。看这个例子：

char *findpass(char *s1,char*s2)
{
    int sl1=strlen(s1);
    int sl2=strlen(s2);
    int i=0,in=0;
    char *temp = new char[100];
    char *t;
    if(sl1%3==0)
    {
        for(i=0;i<sl1/3;i++)
        {
            temp[in]=*(s1+i);
            in++;
        }
    }
    t=temp;
    return t;
}

当使用测试输入数据调用它时，我们有：

int main()
{
    char temps1[] = "123456789";
    char temps2[] = "abc";

    char* retVal = findpass(temps1, temps2);
    std::cout << retVal <<std::endl; // prints 123

    //Deallocate allocated memory before
    delete [] retVal;

    return 0;
}

输出是：

123

Answer 2

#include <iostream>
#include <cstring>
using namespace std;
char *findpass(char *s1,char*s2)
{
    int sl1=strlen(s1);
    int sl2=strlen(s2);
    int i=0,in=0;
    char temp[100];

    if(sl1%3==0)
    {
        for(i=0;i<sl1/3;i++)
        {
            temp[in]=*(s1+i);
            in++;
        }
    }
    cout<<temp<<endl;
    char *t= new char [strlen(temp)+1]; // added dynamic memory allocation
    strcpy(t,temp); // copy temp to t;

    return t;
}

需要使用动态内存分配来从函数中返回 char*；

int main()
{
   char i1[10],i2[10];
   char *f1,*f2;
   cin>>i1;
   cin>>i2;
   f1=i1;
   f2=i2;
   char *f = findpass(f1,f2);
   cout<<f;
   delete []f;  // delete dynamic memory allocation to avoid memory leak
   return 0;
}