0

我正在编写一个代码，其中我从一个 CSV 文本文件中读取，该文件在命令行中作为参数给出。我必须计算给定文件的实验平均值：
例如，如果文件是

Bob's experiment,12,33,55,8
Mary's experiment,99,21,12,0

我必须打印出 Bob 的实验（数字的平均值） Mary 的实验（数字的平均值）

这是我的代码：

#include<stdio.h>
#include<stdlib.h>
#include<stdlib.h>
#include<string.h>

int main (int argc, char *argv[]){
FILE* ptr=fopen(argv[1], "rt");
int i=0;
double sum=0;
double count=0;
double ave=0;
if (ptr==NULL){
    perror("Error while opening file");
    exit(EXIT_FAILURE);   
}
while(!feof(ptr)){
                char s='a';
                while(s!=','){
                             s=fgetc(ptr);
                              printf("%c", s);
                  }
                while((char) *ptr)!='\n'){
                                    fscanf(ptr, "%d", &i);
                                    sum+=i;
                                    count++;
                  }
                    ave=sum/count;
                    printf("%.2f", ave);
            }
        fclose(ptr);
}

}

我得到一个奇怪的无限循环类型的结果。请告诉我我做错了什么！

}

4

2 回答 2

2 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>

int main (int argc, char *argv[]){
    FILE* ptr=fopen(argv[1], "rt");
    double sum, count, ave;
    int i=0;

    if (ptr==NULL){
        perror("Error while opening file");
        exit(EXIT_FAILURE);
    }
    while(1){
        int s;
        while((s=fgetc(ptr)) != ',' && s != EOF){
            printf("%c", s);
        }
        if(s == EOF)
            break;
        printf("\t");
        count = sum = 0;
        while(1==fscanf(ptr, "%d%*c", &i)){//%*c skip ',' and '\n'
            sum += i;
            count++;
        }
        ave = sum / count;
        printf("%.2f\n", ave);
    }
    fclose(ptr);
    return 0;
}
于 2014-11-12T01:09:22.983 回答
0

正如上面的评论所暗示的，您从 FILE* 指针检查字符值的语法是无效的。您可以将 ((char) ptr* != '\n') 替换为 (fgetc(ptr) != '\n')

另外需要注意的是，像这样使用双嵌套循环进行解析通常是糟糕的设计并且很难调试。无限循环可能是由许多极端情况引起的（例如，在您阅读了最后一行之后？）。我建议为每种情况使用一个带有条件的while循环，例如：

while(!feof(ptr)) {
    char s = fgetc(ptr);
    if(s == '\n') {
       ...
    } else if(s == ',') {
       ...
    } else {
       ...
    }

}

多个循环只会增加复杂性，因此最好避免。

如果您绝对必须使用上述算法，您可以编写安全措施来检测超时，例如：

int timeout = 0;
while(s!=',' && timeout < 500) {
    ...
    timeout++;
}
if(timeout >= 500) {
   printf("First loop timeout, s:%c\n", s);
   ... some other useful state checking if you wish..
}

这样，您可以轻松检测到哪个循环进入无限循环，并确定此时变量的状态。

于 2014-11-12T01:06:28.647 回答