0

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

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

非常欢迎有关这些循环比较和一般符号提示的任何帮助。

谢谢

4

3 回答 3

2

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

检查输入操作的返回值。例如，sscanf()(and 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;
}
于 2012-11-12T16:28:59.683 回答
0

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

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
}

请注意，无疑有更好的方法，即一次读取整行并简单地检查其内容。但这是适合您已有的最简单的方法。

于 2012-11-12T16:28:59.840 回答
0

您可以检查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;
}
于 2012-11-12T16:29:57.533 回答