处理尾随\\ n使用feof（）时

Question

我写了一个小程序，它接受一个文件的输入，如：

13,22,13,14,31,22, 3, 1,12,10
11, 4,23, 7, 5, 1, 9,33,11,10
40,19,17,23, 2,43,35,21, 4,34
30,25,16,12,11, 9,87,45, 3, 1
1,2,3,4,5,6,7,8,9,10

并输出每行小于50的最大数字总和。

但是，如果输入的文件具有尾随换行符，则循环运行太多次，因此使用随机数据将另一行添加到数组中。 所以我正在寻找一种更好的方法来进行这种比较以避免这个问题。 我也假设所有线路目前都有10个整数，因为我无法想到更好的方法来进行线路循环比较。

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

void readLineData(int lineNo, int val[][10], FILE *fp);
int findSum(int lineNo, int val[][10], FILE *fp);

int main(int argc, char *argv[]) {
  FILE *fp;
  int val[5][10];

  // Open file.
  if ((fp = fopen(argv[1], "r")) == NULL)
    {
    perror("Cannot open file ");
    exit(EXIT_FAILURE);
    }


  for (int i = 0; !feof(fp); i++) // runs too many times if file ends with '\n'
    {
      readLineData(i, val, fp);
      printf("%d\n", findSum(i, val, fp));
    }

 fclose(fp);
 return EXIT_SUCCESS;
}

void readLineData(int lineNo, int val[][10], FILE *fp) {
  char c;  
  for (int i = 0; i < 10; i++) // assuming line contains 10 integers
    {
      fscanf(fp, "%d,", &val[lineNo][i]);
    }
}

int findSum(int lineNo, int val[][10], FILE *fp) {
  int highVal = 0;
  int value1 = 0;
  int value2 = 0;

  for(int i = 0; i < 10; i++) //each letter
    {
      for(int j = 0; j < 10; j++)// every other letter
    {
      if((val[lineNo][i] + val[lineNo][j]) > highVal && i != j && (val[lineNo][i] + val[lineNo][j]) <= 50)
        {
          highVal = val[lineNo][i] + val[lineNo][j];
          value1 = val[lineNo][i];
          value2 = val[lineNo][j];
        }
    }
    }
  printf("Line %d: largest pair is %d and %d, with a total of: ", lineNo+1, value1, value2);
  return highVal;
}

任何有关循环比较和一般符号提示的帮助都是最受欢迎的。

谢谢

Answer 1

发布的代码不区分具有五个整数的两行和（预期的）具有10个整数的一行。 建议一次读取一行，使用fgets()然后在读取行上使用sscanf()以确保所有读取整数属于同一行。

检查输入操作的返回值。 例如， sscanf() （和fscanf() ）返回所做的赋值数。 仅处理具有预期的10个整数的行，这将检测包括尾随空行的无效行。

例如：

/* Returns 1 on success and 0 on failure. */
int readLineData(int lineNo, int val[][10], FILE *fp)
{
    char line[1024]; /* Arbitrarily large. */
    if (fgets(line, sizeof(line), fp))
    {
        /* %n records position where processing ended. */
        int pos;
        const int result = sscanf(line,
                                  "%d,%d,%d,%d,%d,%d,%d,%d,%d,%d%n",
                                  &val[lineNo][0],
                                  &val[lineNo][1],
                                  &val[lineNo][2],
                                  &val[lineNo][3],
                                  &val[lineNo][4],
                                  &val[lineNo][5],
                                  &val[lineNo][6],
                                  &val[lineNo][7],
                                  &val[lineNo][8],
                                  &val[lineNo][9],
                                  &pos);

        /* 10 integers and full line processed,
           except if new-line character present. */
        return 10 == result &&
               (pos == strlen(line) ||
                (pos + 1 == strlen(line) && '\n' == line[pos]));
    }
    return 0;
}

Answer 2

您可以自己简单地使用换行符：

for (int i = 0; !feof(fp); i++) // runs too many times if file ends with '\n'
{
  readLineData(i, val, fp);
  printf("%d\n", findSum(i, val, fp));
  fscanf(fp, "%*c"); // read a character without storing it in a variable
}

请注意，毫无疑问，更好的方法是立即阅读整行，并简单地检查其内容; 但这是最适合您已有的方式。

Answer 3

你可以检查你的readLineData函数中fscanf失败：

int readLineData(int lineNo, int val[][10], FILE *fp) {
  for (int i = 0; i < 10; i++) {// assuming line contains 10 integers
     if (fscanf(fp, "%d,", &val[lineNo][i]) != 1) {
         return 1;
     }
  }
  return 0;
}