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

投稿されたコードは、5 つの整数を含む 2 つの行と (予想される) 10 の整数を含む 1 つの行を区別しません。一度に 1 行ずつ読み取り、読み取り行でを使用し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 に答える