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

您不需要这样做，将其注释掉：

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

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

另外，您也没有在分配中加1来说明空终止符，但这一点都不重要，因为无论哪种方式分配都是不必要的。

你可以gets直接进入name和切出的line中间人，事实上。

Answer 2

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

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

请注意， gets被广泛认为是使用不安全的函数 ，您应该尝试养成使用fgets的习惯。

Answer 3

我究竟做错了什么

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

您的代码天生就容易受到缓冲区溢出的攻击。 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. 不要投。 这是致命的罪过。

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;
}

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

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。 请注意，这里有两个不同的名称标识符。 一个是getName的参数，另一个是main中定义的常量指针（数组标识符）。 更改函数的参数值不会更改调用方参数的值。