strncmp/strcpy 破坏源代码

Question

今天我试图与 char * string 变得友好......但似乎我失败了:) 每次我调用 strcmp/strncmp/strcpy 函数时，我的源代码都会损坏...

这是代码片段

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

struct student
{
    int UID;
    char name[20];
    char surname[20];
};

char * getString(int minChars, int maxChars);

struct student * myStud;

int main(int argc, char** argv) {

    myStud = (struct student*)malloc(sizeof(struct student));
    while(1)
    {
        printf("\nEnter new name: ");
        strcpy(myStud->name,getString(1,19));
        printf("\n The values is now %s",myStud->name);
    }
    return (EXIT_SUCCESS);
}

char * getString(int minChars, int maxChars)
{

    char string[maxChars+1];
scanAgain:
    scanf("%s",&string);
    if(strlen(string)<minChars)
    {
        printf("\nToo few symbols, try again: ");
        goto scanAgain;
    }
    if(strlen(string)>maxChars)
    {
        printf("\nToo many symbols, try again: ");
        goto scanAgain;
    }
    string[maxChars]='\0';
    return(string);
}

输出：

Enter new name: Alekasdasd

 The values is now Alekasda�#
Enter new name: 

我只是一个初学者，所以它可能非常简单...可能不是。 哦，顺便说一句，使用linux和netbeans作为SDK，gcc作为编译器。

Answer 1

char * getString(int minChars, int maxChars)
{

    char string[maxChars+1];
    ...
    return(string);
}

这里的“string”数组仅分配给 getString() 函数的范围。一旦它返回（超出范围），它就不再存在，并将被程序的其余部分覆盖。 “return(string)”语句返回不再分配的数据的指针——而不是数据本身。发生这种情况是由于 C 中的隐式数组到指针转换。

您的 getString() 函数不应这样做，而应采用 char* 作为参数，该参数在调用函数中分配。

Answer 2

我发现 getString() 函数有两个问题：

1. 字符串变量必须声明为静态，以便函数返回时不会释放（堆栈、弹出）用于它的内存。
2. scanf() 的参数不需要 & 标记，而只是指向缓冲区的指针，string。

也就是说，将以下行更改为：

char string[maxChars+1];
scanf("%s",&string);

阅读

static char string[maxChars+1];
scanf("%s",string);

手册页 man 3 scanf 中您不希望在 scanf() 调用中使用 & 符号的原因如下：

      s      Matches a  sequence  of  non-white-space  characters;  the  next
              pointer must be a **pointer to character array** that is long enough
              to hold the input sequence and the  terminating  null  character
              ('\0'), which is added automatically.  The input string stops at
              white space or at the  maximum  field  width,  whichever  occurs
              first.

Answer 3

240行不是“片段”。
正如詹姆斯在评论中建议的那样，将代码减少到重现问题所需的最少行数。在这个阶段，问题的原因对您来说应该很明显——如果没有尝试再次发帖。

Answer 4

您将返回一个指向堆栈变量的指针。

char * getString(int minChars, int maxChars)
{

    char string[maxChars+1];

当 getString 返回时，string 无效。您的返回值指向这个无效字符串。

使用：

char * getString(int minChars, int maxChars, char * string) {

    return string;
}
...
char string[100];
getString(1, 2, string);

另外，goto？请停止这种行为 - 使用 for、while do、do while 而不是 goto