Question

我正在努力让这项工作适用于我的工作，但我一直遇到分段错误错误，但我无法看到问题。

struct line{
        int source;
        int dest;
        int type;
        int port;
        char data[51];
};


int main(){
    struct line *dataIn;
    dataIn = malloc(sizeof(struct line));
    int nRecords = 0;

    readFile(&nRecords, dataIn);

    int i=0;
    for(i = 0; 1 < 100; i++){
        printf("Source: %d Data: %s\n", dataIn[i].source, dataIn[i].data);
    }

    return 0;
}

void readFile(int *nRecords, struct line *dataIn){
    FILE *fileIn;
    fileIn =fopen("data.txt","r");
    if (!fileIn){
        puts("File Open Error");
        return 1;
    }
    while(fscanf(fileIn, "%d:%d:%d:%d:%[^\n]", &dataIn[*nRecords].source, &dataIn[*nRecords].dest, &dataIn[*nRecords].type, &dataIn[*nRecords].port, dataIn[*nRecords].data) == 5){
        nRecords++;
        dataIn = realloc(dataIn,(*nRecords+1)*sizeof(struct line));
    }
    fclose(fileIn);
}

当我在顶部添加函数原型时：

void readFile(int*, struct line*);

我收到错误：

＆＃39; readFile＆＃39;
的冲突类型

Answer 1

C使用pass-by-value进行函数参数传递。在您的代码中，dataIn本身使用pass-by-value传递给readFile()。

现在，dataIn本身就是指针，您可以从函数中更改dataIn的内容，但不能自行更改dataIn（请查看realloc()）从函数中，并期望反映回main()。

因此，在返回后，您的dataIn只有一个元素，因为之前是malloc()。然后，for循环显然尝试访问超出范围的内存，创建undefined behavior。

如果要从函数中更改dataIn，则需要将指针传递给函数。

那就是说，

nRecords作为指针，nRecords++;将不更新相应的int值。
void函数readFile()不应该有return语句，其值为return 1;

将动态结构传递给函数

1 个答案: