Question

所以我有这个代码从csv文件中读取一堆行。我知道最初文档有16行，这就是我在主要功能中指定int noRows = 16;的原因。

void readBeer(int noRows) {
char *oneline, *token;
char oneproduct[256];
char delim[] = ",";
int x = 1;

FILE *fp; //open file
if ((fp = fopen("varor.csv", "r")) == NULL) //can the file be opened?
{
fprintf(stderr, "File varor.csv couldn't be opened\n"); //"couldn't open file"
exit(-1);
}

while(noRows != 0) 
{
    int countTok = 1;

    fgets(oneproduct, 256, fp); //get the first row
    oneproduct[strlen(oneproduct) - 1] = '\0'; // remove end-of-line character

    oneline = strdup(oneproduct); //duplicate oneproduct into oneline because strtok modifies the given string
    token = strtok(oneline, delim); //split oneline into tokens, tokens are separated by ","

    while (token != NULL) 
    {
        if(countTok == 1) beer[x].productNumber = atoi(token);
        else if(countTok == 2) strcpy(beer[x].name, token);
        else if(countTok == 3) beer[x].price = atof(token);
        else if(countTok == 4) beer[x].volume = atof(token);
        else if(countTok == 5) strcpy(beer[x].type, token);
        else if(countTok == 6) strcpy(beer[x].style, token);
        else if(countTok == 7) strcpy(beer[x].packaging, token);
        else if(countTok == 8) strcpy(beer[x].country, token);
        else if(countTok == 9) strcpy(beer[x].manufacturer, token);
        else if(countTok == 10) beer[x].alcohol = atof(token);
        else printf("kossan hoppade!"); //should never be seen in console
        token = strtok(NULL, delim);
        countTok++;
    }

    x++;
    noRows--;
    free(oneline); free(token);

}

fclose(fp);

}

我的问题是如何在不知道文件有多少行的情况下将文件读到最后？我想在文件中有一个特定的单元格，只是为了在启动和关闭控制台之间保存noRows。

我尝试使用char buffer[1000]; while(fgets(buffer, 1000, fp)) {}，但后来读取前8行（不确定它是否总是正好为8）为0,0,0,0,0,0,0,0,0,0。 / p>

Answer 1

如果只是测试fgets的返回值

，你问题的答案

for (;;)   // idiomatic C style for an infinite loop 
{
    int countTok = 1;

    if (NULL == fgets(oneproduct, 256, fp)) break; //get one row and exit loop on EOF
    ...

但正如您在评论中被告知的那样，您的代码中还有许多其他问题：

将索引初始化为1，而C数组索引从0开始。所以我假设x=1;应为x=0;。出于同样的原因，countTok = 1;没有错，但countTok = 0;会更加惯用C
您在打开文件时使用fprintf(stderr, ...注意到错误。虽然没有错，但它没有说明错误的原因。 perror会做...
删除oneproduct的最后一个字符而不控制它是换行符。这个假设是错误的
- 如果一行包含至少256个字符
- 如果最后一行未以换行符结尾
惯用法是使用strcspn：
```
oneproduct[strcspn(oneproduct, "\n")] = '\0';  // erase an optional end of line
```
您将oneproduct复制到oneline。在这里，没有什么是错的，但它没用，因为你从不使用原始的行
else if的长列表可以用开关代替，但这个主要是风格问题
你在循环结束时释放token。因为它是NULL，所以它是一个无操作，但它没有被分配它不应该被释放。

Answer 2

如果您正在使用fgets，则可以测试调用结果是否等于NULL。fgets信号NULL出错或EOF（文件结束）。

char *fgets(char *line, int maxline, FILE *fp);。

即：

while (fgets(buffer, MAXLINE, fp) != NULL) {
    // Process line here.
}

您还可以逐个字符处理整个文件，并测试(c == EOF)。有几种方法可以做到。

在c中读取csv文件

2 个答案: