Question

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "queue.h"
#include "stack.h"

#define RECORDS_SIZE 100
#define NAME_SIZE    20

typedef struct Student
{
    char nameStudent[NAME_SIZE];
    int  TimeIn;
    int  TimeUpdate;
}STUDENT;

typedef struct TUTOR
{
    char nameTutor[NAME_SIZE];
    int TutorTIme;
    STUDENT *ptr;
}TUTOR;


QUEUE *queue1;
STACK *stack1;

void getData(STUDENT *studentArr[RECORDS_SIZE]);

int main (void)
{

    STUDENT *studentArr[RECORDS_SIZE];
    FILE *fp = NULL;

    getData(studentArr);

    return 0;
} 

void getData(STUDENT *studentArr[RECORDS_SIZE])
{
    FILE *fp;
    char fileName[NAME_SIZE];
    char buffer[RECORDS_SIZE];
    int count = 0;

    printf("Enter file name: ");
    gets(fileName);
    fp = fopen(fileName, "r");
    if (fp == NULL)
    {
       printf("Error! The file does not exist!\n");
    }

    fgets(buffer, RECORDS_SIZE, fp);
    *studentArr = (STUDENT*) malloc(buffer[0]*sizeof(STUDENT));
    while (fgets(buffer, RECORDS_SIZE, fp) != NULL)
    {
        printf("%s", buffer[count]);
       *studentArr[count]->nameStudent = (char*) malloc(strlen(buffer)*sizeof(char));
        studentArr[count]->TimeIn = (int) malloc(strlen(buffer)*sizeof(int));
        sscanf(buffer,"%[,],%d", *studentArr[count]->nameStudent, &studentArr[count]->TimeIn);

        printf("%s%d\n", studentArr[count]->nameStudent, &studentArr[count]->TimeIn);
       count++;
    }

    return;
}

有一个警告，表示所述赋值从指针生成整数而没有强制转换[默认启用] |在我为* studentArr [count] - ＆gt; nameStudent分配内存的行上，为什么我收到此警告？

这就是我的文件

4
A,10
B,12
C,60
D,120
tutorY

我尝试在第一行读取然后使用第一行的数字来分配指向结构数组的指针并继续读取然后分配结构的其余成员

我认为我做了while循环错误，可能是我不想调用fgets然后在while循环中重用它，因为它似乎有错误。我该如何修复这个功能？

提前谢谢

Answer 1

修订问题

nameStudent是一个20个字符的数组，我认为这可能会在分配时出现问题。

在这种情况下，您根本不需要动态分配nameStudent字段。创建学生结构时，名称的所有空间都将作为学生结构的一部分进行分配。

早期的观察

如果你真的有：

*studentArr[count]->nameStudent = (char*) malloc(strlen(buffer)*sizeof(char));

然后是合理类型的nameStudent

 struct Xxxx
 {
     ...
     char *nameStudent;
     ...
 };

你前面有*，取消引用一个未定义的指向char的指针。这就是你收到警告的原因：

 assignment makes integer from pointer without a cast [enabled by default]

对于具有固定规模学生姓名的结构，您可能会得到更好的服务：

 struct Xxxx
 {
     ...
     char nameStudent[32];
     ...
 };

Answer 2

*studentArr[count]->nameStudent = (char*)...

虽然你没有提供STUDENT的定义，但这肯定是错误的......如果nameStudent是char*，你试图通过该初始{{1}存储到它的第一个字节}。您的代码还有其他错误，例如下一行的*强制转换。我建议更多地研究C语言，并更仔细地关注您的程序是否以及如何满足其要求。 C需要很高的精度。

更新

由于(int)是一个数组，所以它不能也不应该指定一个指针。

您的代码也包含许多其他错误。

nameStudent

这会读取RECORDS_SIZE个字节，而不是一个STUDENT记录。

fgets(buffer, RECORDS_SIZE, fp);

*studentArr = (STUDENT*) malloc(buffer[0]*sizeof(STUDENT));在这里毫无意义。只设置buffer[0]的第一个元素是没有意义的......你不要在循环中设置任何其他元素。循环外不应该进行studentArr调用...从不检查错误或EOF的调用。

您的程序包含太多错误和误解，无需为您编写代码即可解答您的问题。

为指向结构数组的指针分配内存

2 个答案:

修订问题

早期的观察