strncmp/strcpy 损坏源

Question

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

这里是sn-p

#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

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

Answer 2

char * getString(int minChars, int maxChars)
{

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

这里的“字符串”数组只分配给 getString() 函数的作用域。一旦它返回（超出范围），它将不复存在，并将被您的程序的其余部分覆盖。 “return(string)”语句返回不再分配的数据的指针——而不是数据本身。这是由于 C 中隐式的数组到指针的转换。

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

Answer 3

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

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

Answer 4

我发现您的 getString() 函数存在两个问题：

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

即换行：

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

阅读

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

您不想在 scanf() 调用中使用与号的原因是手册页中的以下内容，man 3 scanf：

s 匹配一系列非空白字符；下一个 指针必须是足够长的**指向字符数组**的指针 保存输入序列和终止空字符 ('\0')，它是自动添加的。输入字符串在 空白或最大字段宽度，以发生者为准 第一的。