从 csv 文件中读取错误数量的 '\n'

Question

函数读取 csv 文件，用空字符替换分隔符并计算列数和换行符，但返回错误的 '\n' 数，我不知道为什么？请帮忙！

代码末尾附近的注释：应该是 24643 - 实际是 24687

示例数据：我在必要时手动添加了“,”分隔符以显示列更改。

Product,23659,LAUX431GDS X431,图形诊断扫描工具,P,LAUX431GDS,CANDO INTERNATIONAL,权利,OE 级访问国内、欧洲和亚洲制造商，用于读取/清除代码、实时数据绘图和记录定向测试和模块编码（将提供用于 4 通道示波器、点火分析仪、传感器模拟器和万用表的单独模块）

所有车辆系统及其模块的诊断访问和通信速度无与伦比。启用 Wifi，让您可以访问您喜爱的网站以获取故障排除建议、图表和维修信息。

一键更新，一次更新一条或多条车线

无需连接到 PC 进行更新，所有补丁和软件更新都直接通过您订阅每年更新工具。

包括第一年的更新

    int getCurrentData(FILE *current){

    int totalChars = 0, colCount = 0, productCount = 0;
    char *buffer = NULL, next = '\0';
    size_t i = 0;

    fseek(current, 0, SEEK_END);
    size_t fileSize = ftell(current);
    rewind(current);

    buffer = malloc(fileSize);
    fread(buffer, fileSize, 1, current);

    /*replacing delimiter with null character*/
    for (i = 0; i < fileSize; i++){
        if (buffer[i] == ',' && buffer[i+1] != ' '){
            buffer[i] = '\0';
        }
    }
    /*finds column and product count*/
    for (i = 0; i < fileSize; i++){
        if (buffer[i] == '\0' && productCount == 0 || buffer[i] == '\n' && productCount == 0){
            colCount++;
        }
        if (buffer[i] == '\n' && i < fileSize){
            productCount++;                         /*should be 24643 - actual 24687*/
        }
    }

    return productCount;
}

Answer 1

问题数量

1. 主要问题是替换分隔符循环可能会将\n 替换为\0。

2. 次要：long fileSize 应为 size_t fileSize。

3. if (next != EOF){ 应该被删除。

4. 次要：buffer = malloc(fileSize + 1); 应为 buffer = malloc(fileSize);

5. 替换循环搞砸了

   size_t fileSize = ftell(current);
   ...
   buffer = malloc(fileSize);           // later add error checking
   fread(buffer, fileSize, 1, current); // later add error checking
   /*replacing delimiter with null character*/
   for (size_t i = 0; i < fileSize; i++){
     if (buffer[i] == ',') {
       // Not sure why OP has (buffer[i+1] != ' ')?  Maybe \n was originally meant
       if ((i+1) >= filesize) || ((buffer[i+1] != ' ') && (buffer[i+1] != '\n'))) {
         buffer[i] = '\0';
       // Or rather than the above if(), maybe something simple:
       buffer[i] = '\0';
       }
     }
   }  
   

代码未测试 - 抱歉没有足够的时间。