C中的简单字符串程序 - 无法打印字符串

Question

我无法打印字符串。需要一些关于我做错了什么的帮助，以及在这些情况下最好的方法的一些见解。

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

void getName(char *name);

int main()
{
    char name[256];
    getName(name);
    printf("Name = %s\n", name);
    return 0;
}

void getName(char *name)
{
    char line[256];
    printf("Please enter your name! \n");
    gets(line);
    name = (char *) malloc(strlen(line));
    strcpy(name, line);
}

Answer 1

我做错了什么

其他人已经指出了一些错误，但还没有人给出正确的解决方案。

您的代码天生就容易受到缓冲区溢出的影响。 gets() 函数不允许您指定缓冲区大小，因此如果输入超过 256 个字符，这将超出数组的范围。这很糟糕。

你应该做的是

1. 不要为此使用单独的函数（因为在这种特殊情况下它没有任何好处），并且
2. 在您的阵列上调用fgets()：

---

#include <stdio.h>

int main()
{
    char name[256];
    fgets(name, sizeof name, stdin);
    printf("Name = %s\n", name);
    return 0;
}

此外，如果您考虑过动态内存分配：

1. Do not cast. It's a deadly sin.

2. 检查malloc()的返回值。

3. 您将仍然想要跟踪缓冲区大小；这样的功能怎么样？

---

#define NAME_LENGTH 256

char *getName(void)
{
    char *buf = malloc(NAME_LENGTH);
    if (buf == NULL) {
        return NULL;
    }

    fgets(buf, NAME_LENGTH, stdin);
    return buf;
}

但是，正如我在 cmets 中提到的，在这种情况下，您根本不需要动态内存分配。

Answer 2

你不需要这样做，把它注释掉：

name = (char *) malloc(strlen(line));

您已经传入name 的缓冲区。您不需要分配第二个缓冲区。

此外，您没有在分配中添加 1 来说明空终止符，但这无论如何都无关紧要，因为无论哪种方式都不需要分配。

实际上，您可以将gets 直接转换为name 并去掉line 中间人。

Answer 3

您的getname() 过于复杂 - 它只需要：

void getName(char *name)
{
    printf("Please enter your name! \n");
    gets(name);
}

请注意，gets 被广泛认为是unsafe function to use，您应该尝试养成改用fgets 的习惯。

Answer 4

我知道指针对于初学者来说可能很复杂。所以，忘记指针。您希望以下代码应该如何运行？

void change(int a)
{
   a = 10;
}

int main()
{
    int a = 20;
    change (a);
    printf("%d", a);  // It prints 20
}

（如果您确实理解了上面的代码，请继续阅读。否则，请阅读有关函数和变量范围的内容。）

您的代码具有基本相同的结构，只是您有一个 char *name，而不是 int a。请注意，您在此处有两个不同的 name 标识符。一个是 getName 的参数，另一个是 main 中定义的常量指针（数组标识符）。更改函数的参数值不会更改调用者参数的值。