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

您可以检查fscanf功能中readLineData是否失败：

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;
}

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

3 个答案: