Question

因此，作为学校作业的一部分，我必须创建一个应用程序来处理使用结构的各种字段的输入。在main之外定义如下

typedef struct stud {
    struct number {
        int studNum;
    } number;
    struct name {
        char firstName[NAME_SIZE];
        char lastName[NAME_SIZE];
    } name;
    struct cellNum {
        int areaCode;
        int prefix;
        int suffix;
    } cellNum;
    struct courses {
        struct fall {
            char className[NAME_SIZE];
            float mark;
        } fall;
        struct winter {
            char className[NAME_SIZE];
            float mark;
        } winter;
    } courses;
} stud;

NAME_SIZE定义为30

我按以下方式访问结构

stud studArray[100]; // max 100 students
stud *studPtr;

for (int i = 0; i < studCount; i++) {
    studPtr = &studArray[i];
    initializeStrings(studPtr[i]);
    getNum(studPtr[i]);
    getName(studPtr[i]);
    getCell(studPtr[i]);
    getCourses(studPtr[i]);
}

initializeStrings是我添加空终结符的

void initializeStrings(stud student) {
    student.name.firstName[NAME_SIZE - 1] = '\0'; 
    student.name.lastName[NAME_SIZE -1] = '\0';
    student.courses.fall.className[NAME_SIZE -1] = '\0';
    student.courses.winter.className[NAME_SIZE -1] = '\0';
}

现在问题就出现了：在我的各种函数中，当我尝试访问我写入字符串的任何信息时，我得到的垃圾字符输出看起来像是横向T，但仅限于我以不同的功能访问数据。

示例：这是获得2门课程和标记的代码

void getCourses(stud student) {
    getchar();
    printf("\n%s", "Enter Course name - Fall 2016: ");
    fgets(student.courses.fall.className, NAME_SIZE - 1, stdin);
    fflush(stdin); // suggested by my prof to fix the issue, did nothing

    printf("\n%s%s%s", "Enter mark received for ", student.courses.fall.className, ": ");
    scanf("%f", &student.courses.fall.mark);

    getchar();
    printf("\n%s", "Enter Course name - Winter 2017: ");
    fgets(student.courses.winter.className, NAME_SIZE - 1, stdin);

    printf("\n%s%s%s", "Enter mark received for ", student.courses.winter.className, ": ");
    scanf("%f", &student.courses.winter.mark);
}

如果我输入helloWorld作为课程名称，系统会正确打印输入为helloWorld收到的标记

但是当我尝试从我的打印功能打印数据时：

void printData(stud student) {
// various other values (none of which work)
printf("\t%s\t%s\t%f\n", "Fall2016", student.courses.fall.className, student.courses.fall.mark);
}

我获得了垃圾数据输出（image）

任何帮助/代码批评欢迎

编辑：谢谢Eddiem和其他所有人，你指出了我正确的方向。

Answer 1

这段代码并没有多大意义。首先，为什么不将stud*指针传递给您在下面调用的每个函数？无论你在这里使用它的方式，你都要初始化studPtr以指向studArray数组中的特定元素，但仍然将i索引到该指针中。你可以通过studPtr[0]，但这很奇怪。

原始代码：

stud studArray[100]; // max 100 students
stud *studPtr;

for (int i = 0; i < studCount; i++) {
    studPtr = &studArray[i];
    initializeStrings(studPtr[i]);
    getNum(studPtr[i]);
    getName(studPtr[i]);
    getCell(studPtr[i]);
    getCourses(studPtr[i]);
}

我建议修改如下：

void initializeStrings(stud *student) {
    student->name.firstName[NAME_SIZE - 1] = '\0'; 
    student->name.lastName[NAME_SIZE -1] = '\0';
    student->courses.fall.className[NAME_SIZE -1] = '\0';
    student->courses.winter.className[NAME_SIZE -1] = '\0';
}

如果当前采用stud参数的函数，则需要对每个修改进行这些修改。然后，将循环初始化更改为：

for (int i = 0; i < studCount; i++) {
    initializeStrings(&studArray[i]);
    getNum(&studArray[i]);
    getName(&studArray[i]);
    getCell(&studArray[i]);
    getCourses(&studArray[i]);
}

Answer 2

这是不对的：

studPtr = &studArray[i];
initializeStrings(studPtr[i]);

假设您要访问数组的第三个元素，因此i为3.这会为您提供第三个条目的偏移量：

studPtr = &studArray[i];

现在你使用索引......

initializeStrings(studPtr[i]);

所以你要访问的元素比你已经存在的第三个元素更多;你可以看到你要离开数组的末尾，或者访问你没有初始化的元素。

所以使用：

studPtr = &studArray[i];
initializeStrings(studPtr);

或

initializeStrings(&studArray[i]);

这两个相同，指向同一个地方。

Answer 3

问题是你没有通过引用传递（也就是使用指针）。如果不使用指针，那么像getCourses这样的函数会直接修改临时数据而不是数据。因此，原始数据中的垃圾值永远不会被删除。

更改

void getCourses(stud student) { // no pointer = copy data to temporary memory, original data is unmodified in this function.
getchar();
printf("\n%s", "Enter Course name - Fall 2016: ");
fgets(student.courses.fall.className, NAME_SIZE - 1, stdin);
fflush(stdin); // suggested by my prof to fix the issue, did nothing

printf("\n%s%s%s", "Enter mark received for ", student.courses.fall.className, ": ");
scanf("%f", &student.courses.fall.mark);

getchar();
printf("\n%s", "Enter Course name - Winter 2017: ");
fgets(student.courses.winter.className, NAME_SIZE - 1, stdin);

printf("\n%s%s%s", "Enter mark received for ", student.courses.winter.className, ": ");
scanf("%f", &student.courses.winter.mark);
}

要

void getCourses(stud *student) { // notice the pointer used. Original data is to be modified.
getchar();
printf("\n%s", "Enter Course name - Fall 2016: ");
fgets(student->courses.fall.className, NAME_SIZE - 1, stdin);
fflush(stdin); // suggested by my prof to fix the issue, did nothing

printf("\n%s%s%s", "Enter mark received for ", student.courses.fall.className, ": ");
scanf("%f", &student->courses.fall.mark);

getchar();
printf("\n%s", "Enter Course name - Winter 2017: ");
fgets(student->courses.winter.className, NAME_SIZE - 1, stdin);

printf("\n%s%s%s", "Enter mark received for ", student.courses.winter.className, ": ");
scanf("%f", &student->courses.winter.mark);
}

此外，您必须修改for循环以处理指针。即，将getCourses(studPtr[i]);更改为getCourses(&studPtr[i]);

C - 垃圾字符，即使是空终结符，结构

3 个答案: