0

我正在做一个项目，我需要将文本文件中的 CSV 行读取到我的程序中。我得到了一个代码框架，并要求我填写功能。我有一个结构，其中包含我要接收的每种类型的值的变量，但是我的 char 数组导致分段错误。这是我的代码的摘录。没有任何摘录是给定代码的一部分，这都是我的：由于获取时间戳空间内的代码，我的错误是分段错误（核心转储）。我的测试文件只包含一行，5, 10:00:10, 1, 997

/*
*  YOUR CODE GOES HERE:
*  (1) Read an input csv line from stdin
*  (2) Parse csv line into appropriate fields
*  (3) Take action based on input type:
*        - Check-in or check-out a patient with a given ID
*        - Add a new health data type for a given patient
*        - Store health data in patient record or print if requested
*  (4) Continue (1)-(3) until EOF
*/

/* A new struct to hold all of the values from the csv file */
typedef struct {
    int iD;
    char *time[MAXTIME + 1];
    int value;
    int type;
}csv_input;

/* Declare an instance of the struct, and assign pointers for its values */
csv_input aLine;
int *idptr;
char timeval[MAXTIME + 1];
int *valueptr;
int *typeptr;

/*Note: because the time char is already a pointer, I did not make another one for it but instead dereferenced the pointer I was given */
idptr = &aLine.iD;
int j; /* iterator variable */
for(j; j < MAXTIME; j++){
    *aLine.time[j] = timeval[j];
}
valueptr = &aLine.value;
typeptr = &aLine.type;

/* Get the Patient ID */
*idptr = getchar();
printf("%c", aLine.iD); /* a test to see if my pointers worked and the correct value was read */

/*Skip the first comma */
int next;
next = getchar();

/* get the timestamp */
int i;
for(i = 0; i < MAXTIME; i++)
{
    while ((next = getchar()) != ',')
     {
    timeval[i] = next;
    //printf("%s", aLine.time[i]);
     }
}
4

2 回答 2

2

第一的：

int j; /* iterator variable */
for(j; j < MAXTIME; j++){

您需要设置j一些值，j=0这是有道理的。如果没有这个，您将访问一个具有未初始化值的数组，并且您将获得 UB。

第二：

/*Note: because the time char is already a pointer,

不，time是一个指向字符的指针数组，那里有区别。

这一行：

*aLine.time[j] = timeval[j];

行不通，因为一方面，您的陈述but instead dereference the pointer I was given做出了不正确的假设。是的，你得到了一个指针数组，但它们不指向任何东西，它们是未初始化的，因此你不能尊重它们，直到你将它们初始化为一个有效的非 NULL 值。

我认为你试图做这样的事情：

aLine.time[j] = &timeval; //set the pointer to the local static array

但这仅适用于本地函数范围。如果你 malloc 到你的指针数组会更好。

于 2013-04-29T14:45:17.993 回答
0 
char *time[MAXTIME + 1];

这是一个指针数组（指向 char 数组的指针），而不是一个 char 数组

崩溃来自这条线

*aLine.time[j] = timeval[j];

因为正如我所说aLine.time[j]的是一个指针，并且在填充它的内容之前你还没有为这个指针分配内存

于 2013-04-29T14:40:22.140 回答