如何使用malloc从函数返回C字符串

Question

我想从函数返回一个C字符串。该函数假设用逗号分解3个整数，然后将结果作为char数组返回，但是我得到的是垃圾值。我假设我没有正确调用malloc。有人可以就这个问题提出建议吗？

using namespace std; 

const char * createCommand(int p1, int p2, int p3){ 
    stringstream sstm; 
    std::string comma = ","; 
    sstm << p1 << comma << p2 << comma << p3; 
    std::string str = sstm.str(); 
    const char *cstr = (const char *)malloc((str.length()+1) * sizeof (char)); 

    cstr = str.c_str(); 
    return cstr;  
} 

int main() { 
    const char *cstr2 = createCommand(1,0,250); //I want to return "1,0,250" 
    printf("char = %s\n",cstr2); 
} 

Answer 1

由于其他两个答案已经回应了处理字面问题的曲调，我将改为建议我认为是一个非常重要的设计缺陷导致您的问题：返回c字符串。

在您提供的示例代码中，使用c字符串是毫无意义的。下面的代码将实现你打算与任何困难或有问题的代码做什么：

std::string createCommand(int p1, int p2, int p3){ 
    std::stringstream sstm; 
    std::string comma = ","; 
    sstm << p1 << comma << p2 << comma << p3; 
    return sstm.str(); 
} 

int main() { 
    std::string command = createCommand(1,0,250); //I want to return "1,0,250" 
    std::cout << "char = " << command << "\n"; 
} 

即使你只限于使用printf而不是C++的iostream库，这个设计还是更好：

std::string createCommand(int p1, int p2, int p3){ 
    std::stringstream sstm; 
    std::string comma = ","; 
    sstm << p1 << comma << p2 << comma << p3; 
    return sstm.str(); 
} 

int main() { 
    std::string command = createCommand(1,0,250); //I want to return "1,0,250" 
    printf("char = %s\n", command.c_str()); 
} 

如果您需要将C字符串传递给一些较旧的基于C的库，则此设计仍然足够。重点是，没有理由使用malloc或与底层C字符串表示接口，除非通过字符串本身。

Answer 2

在返回指针之前，您需要复制某种形式的字符串strcpy。

const char * createCommand(int p1, int p2, int p3){ 
    stringstream sstm; 
    std::string comma = ","; 
    sstm << p1 << comma << p2 << comma << p3; 
    std::string str = sstm.str(); 
    const char *cstr = (const char *)malloc((str.length()+1) * sizeof (char)); 

    strcpy(cstr, str.c_str()); 
    return cstr;  
} 

Answer 3

赋值运算符，这对于std::string和其他对象正常工作，不能有指针覆盖。因此，分配

cstr = str.c_str(); 

泄漏您分配的内存，并用字符串中的数据替换指针。而且，函数返回的指针现在指向退出函数时失效的内存，除了泄漏外，还会创建未定义的行为。

要解决此问题，请致电std::strcpy(cstr, str.c_str());请不要忘记致电std::free以了解有关呼叫的结果。 编辑：你应该从createCommand函数的返回类型中删除const（WhozCraig，感谢你的评论）。

注：我认为这只是在使用malloc练习，你知道，使用new[]最好，而且你不会做任何以上的，如果你可以从函数返回std::string 。