如何将调用函数作为 strcpy 中的第二个字符串？

Question

我需要从 main 调用一个函数来确定一个字符串值。但是，据我所知，字符串不能采用string_name = call_function() 的形式。字符串的分配必须采用strcpy(str1, str2) 的形式，对吗？所以我想知道这个strcpy 和str1 作为变量名，而str2 是另一个函数的返回字符串值，我做错了什么。

到目前为止，这就是我所拥有的。

#include <stdio.h>
#include <string.h>

char *get_name(int num);

char *get_name(int num) {
    char real_name[30];
    
    if (num == 1)
        strcpy(real_name, "Jake Peralta");
    
    return real_name;
}

void main() {
    char name[30];
    int num;
    
    num = 1;
    
    strcpy(name, *get_name(num));
    printf("%s", name);
}

它没有在我的输出屏幕上打印任何东西。

我尝试过的：

• get_name(num) 不使用指针（还是不行）

p/s：这不是实际代码。这只是我正在尝试做的一个例子，实际代码更长，我已经确定这部分是错误的来源。

Answer 1

您是对的，字符串不能通过return 语句作为数组返回。当您传递或返回一个数组时，只使用指向其第一个元素的指针。因此，函数get_name() 返回一个指向本地定义的自动存储数组的指针（又名在堆栈上）。这是不正确的，因为这个数组一旦超出范围就会被丢弃，即：当函数返回时。

get_name() 可以通过多种方式向调用者提供名称：

• 您可以传递目标数组及其长度：并让函数将名称填充到数组中，小心避免超出数组末尾，但确保它有一个空终止符：

   char *get_name(char *dest, size_t size, int num) {
       if (num == 1) {
           snprintf(dest, size, "Jake Peralta");
       } else {
           snprintf(dest, size, "John Doe");
       }
       // return the destination pointer for convenience.
       return dest;
   }
  
   int main() {
       char name[30];
       int num = 1;
       get_name(name, sizeof name, num);
       printf("%s\n", name);
       return 0;
   }
  

• 您可以在get_name() 中分配内存并返回一个指向您复制字符串的已分配数组的指针。当不再使用该对象时，调用者将负责使用free() 释放该对象。

   char *get_name(int num) {
       if (num == 1) {
           return strdup("Jake Peralta");
       } else {
           return strdup("John Doe");
       }
   }
  
   int main() {
       int num = 1;
       char *name = get_name(num);
       printf("%s\n", name);
       free(name);
       return 0;
   }
  

• 您可以返回一个常量字符串，但只有在编译时所有名称都已知时才能这样做。

   const char *get_name(int num) {
       if (num == 1) {
           "Jake Peralta";
       } else {
           "John Doe";
       }
   }
  
   int main() {
       int num = 1;
       const char *name = get_name(num);
       printf("%s\n", name);
       return 0;
   }
  

Answer 2

您正在从get_name 方法返回real_name 的地址，该方法将在函数返回后超出范围。而是在堆上分配字符串的内存并返回其地址。此外，调用者需要释放在堆上分配的字符串内存以避免任何内存泄漏。

Answer 3

就像你说的，“但是，据我所知，字符串不能采用string_name = call_function() 的形式。”要了解这背后的逻辑，只需查看您的 get_name() 函数：

char *get_name(int num)
{
    char real_name[30];
    
    if (num==1)
        strcpy(real_name,"Jake Peralta");
    
    return real_name;
}

在这里，您尝试返回real_name 的起始地址，但real_name 在超出范围时被销毁（在这种情况下，当函数返回时）。我认为有两种方法可以解决这个问题。一种是添加应该保存返回值的字符串作为参数。在您的情况下，它将是：

void get_name(int num, char *dest)
{
    char real_name[30];
    
    if (num==1)
    {
        strcpy(real_name,"Jake Peralta");
        strcpy(dest, real_name);
    }
}

或者，现在完全避免使用real_name 来缩短函数：

void get_name(int num, char *dest)
{
    if (num==1)
        strcpy(dest, "Jake Peralta");
}

另一种方法是在堆上分配返回值，但我不建议这样做；您必须跟踪每个分配的字符串并最终释放所有字符串。不过，这就是您的情况：

char *get_name(int num, char *dest)
{
    char *real_name = calloc(30, sizeof(char)); // Using calloc to avoid returning an uninitialized string if num is not 1
    if (num==1)
        strcpy(real_name, "Jake Peralta";
    return real_name;
}

顺便说一句，我将您的strcpy 保留在这里，但以后可能希望避免使用它，因为它可能导致缓冲区溢出。 Here's a post with useful answers as to why it's bad。事实上，即使strncpy 也不是完全安全的（参见here（感谢@chqrlie 提供链接））。 @chqrlie 在他自己的回答中使用snprintf 提供了一个干净安全的替代方案，我建议你使用它。