-4

下記の課題に取り組んでおり、ソートは一回抜かしてファイルの読み込みに取り組んでいる状態です。

電卓アプリケーションの作成
 以下の機能を満たすアプリケーションを作成してください。
 ※画面の表示やログの形式は原則として例示されている内容に従ってください。

 1) 四則演算が出来ること
 2) コンソールから計算対象となる数値と演算子を受け取る
   例) 5 + 6
 3) 結果をコンソール上に表示する
   例) 11
 4) 演算結果と実行した日、時間、秒をログファイルとして保存する
   ログの上限は1000行とする(上限を超過するケースはひとまず考慮しなくて良い)
   例) log.txt に 以下の内容を記録
     2015/04/27 14:30:51, 5 + 6, 11
 5) コンソールからパラメータを受け取り、ログを実行時間の昇順/降順に並べ替えて
   コンソールに表示する
   例) ASC を入力
     2015/04/26 10:20:00, 5 + 6, 11
     2015/04/27 14:30:51, 7 - 6, 1
     2015/04/27 15:30:00, 7 + 8, 15
     2015/04/28 14:30:51, 8 + 9, 17
   例) DESC を入力
     2015/04/28 14:30:51, 8 + 9, 17
     2015/04/27 15:30:00, 7 + 8, 15
     2015/04/27 14:30:51, 7 - 6, 1
     2015/04/26 10:20:00, 5 + 6, 11

しかしVisual studioでのデバックで

for (i = 0;i < 1000;i=i+1) {
        fgets(sin[1000], sizeof(sin[0]), fp);
        if (strcmp(sin, "EOF") == 0) {
            break;
        }

この行でループが抜け出せなくなってしまいました。
どこを修正すればいいえしょうか?

#include <stdio.h>
#include <time.h>
#include <string.h>
#include <stdlib.h>
/*
static char ad[10];

int cmp_u(const void* a, const void* d) {
    return strcmp((char*)a, (char*)d);
}

int cmp_d(const void* a, const void* d) {
    return strcmp((char*)d, (char*)a);
}
*/
int main() {
    int num1, num2;
    char op;
    float answer;
    int r,i,n;
    FILE* fp;
    char c[11];
    char sin[1000][1000];
    char ad[8];


    fp = fopen("log.txt", "a");

    if (fp == NULL) {
        printf("ファイルオープン失敗\n");
        return -1;
    }
    
    
    while (1) {
        r = scanf("%d%c%d", &num1, &op, &num2);
        if (r != 3) {
            puts("input error");
            return 1;
        }

        if (op == '+') {
            answer = num1 + num2;
        }
        else if (op == '-') {
            answer = num1 - num2;
        }
        else if (op == '*') {
            answer = num1 * num2;
        }
        else if (op == '/') {
            answer = (float)num1 / num2;
        }

        time_t t = time(NULL);
        struct tm* tm = localtime(&t);
        printf("%d/%02d/%02d ", tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday);
        printf("%02d:%02d:%02d ", tm->tm_hour, tm->tm_min, tm->tm_sec);
        printf("%d%c%d,%f\n", num1, op, num2, answer);
        fprintf(fp, "%d/%02d/%02d ", tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday);
        fprintf(fp, "%02d:%02d:%02d ", tm->tm_hour, tm->tm_min, tm->tm_sec);
        fprintf(fp, "%d%c%d,%f\n", num1, op, num2, answer);

        printf("計算を続けますか?");
        scanf("%s\n", &c);
        if (strcmp(c, "no") == 0) {
            break ;
        }
    }

    int count = 0;
    
    for (i = 0;i < 1000;i=i+1) {
        fgets(sin[1000], sizeof(sin[0]), fp);
        if (strcmp(sin, "EOF") == 0) {
            break;
        }
    }
    
    fclose(fp);

/*
    printf("ASC or DESC: ");
    scanf(" %s", ad);

    if (strcmp(ad, "ASC") == 0) {
        qsort(sin, , sizeof(sin[0]), cmp_u);
    }
    else {
        qsort(sin, , sizeof(sin[0]), cmp_d);
    }
*/
    for (i = 1; i <= n; i++) {
        printf("%s", sin[i]);
    }
    
        return 0;
}
3
  • 1
    if(strcmp(sin, "EOF") == 0) は EOF という文字列がないと 0 になりません。 fgets ファイル読み込み とかで検索すればサンプルは出てきます monozukuri-c.com/langc-file-read/#toc8 このあたりを参考にすればやりたいことはできるかと…  あとたまたま動く可能性はありますが sin[1000] がおかしいです。 まず配列のインデックスの範囲について理解しなおしたほうがいいと思います
    – user35303
    Commented 2022年11月16日 3:29
  • 2
    (ループが意図した通り動いたとしても) break する以外何もしていないので、何を目的としたループかがいまいち不明ですが、電卓アプリとは切り離して考えられるはずです。 質問にあたっては「課題の質問」をそのままするのではなく、再現可能な短いサンプルコードの書き方 も参考にしてください。
    – cubick
    Commented 2022年11月16日 3:40
  • おそらく 5) のためにログファイルをファイル終端まで1行ずつ sin に読み込んでこの後ソートしようとしてるんじゃないですか?  sin = standard input のつもり? の変数名を使いまわしてるせいでファイルインプットを入れる場所として名前が不適切だとは思いますが。。。
    – user35303
    Commented 2022年11月16日 4:12

3 件の回答 3

2

Visual Studioだとデフォルトでは色々な解説やサンプルプログラムと相性の悪いコンパイラの警告 (レベル 3) C4996を無効にしたとしても、以下のようなエラーが発生してビルドが正常に終了しないはずです。(メッセージの形式は変えています。)

行  種類    内容
93  エラー  初期化されていないローカル変数 'n' が使用されます
43  警告    '=': 'int' から 'float' への変換です。データが失われる可能性があります。
       46行目と49行目も同じ警告
75  警告    '関数': 間接参照のレベルが 'const char *' と 'char [1000][1000]' で異なっています。
75  警告    'strcmp': の型が 1 の仮引数および実引数と異なります。
24  警告    'ad': ローカル変数は 1 度も使われていません。

何かプロジェクトのプロパティ等を操作して実行ファイルが出来るようにしたのかもしれませんが、IDEで表示されるエラーや警告はキチンと調べて対処するようにしましょう。

加えて93行目のエラーを修正してビルドが通ったとしても以下のような問題があります。

  • 提示された部分の処理はファイルが追加モードでオープンされたままの状態で行われており、エラーになるか何も結果は得られないでしょう。
    →いったんログファイルをクローズして読み取りモードで再度オープンしてから処理する必要があります。
  • 74行目の引数sinの指定方法が間違っています。
    →2次元配列で定義されているのに1次元のインデックスしか指定していません。
    →読み込んだデータの格納位置(インデックス)が常に同じ値しか指定されていません。
  • 75行目のファイル読み込み終了判定が間違っています。
    →おそらくfgetc()による読み込みと終了判定から類推したものと思われますが、自身が使っている関数はfgets()ですし、ファイル読み込みの終了を表すのは"EOF"という文字列ではないのと、判定する対象の変数sinの指定方法がおかしい等、何重にも間違っています。

何かの講座や学習コースで出た課題ならば、それ以前に説明されているはずの内容しか使わないと思われるので、今までの学習内容を見直してみてください。


ちなみにループ終了とは無関係ですが、問題文/条件は全て記載されていますか?
質問記事に記載された内容が全てだとすると、以下の条件があいまいだと思われます。
出題者に質問出来るなら問い合わせた方が良いでしょう。

  • 加減算の例だけがあり、乗除算の例が提示されていません。
  • 扱うものは「数値」とだけありますが、例では整数しか提示されていません。
  • 入力値に浮動小数点数は許容されますか?
  • 乗除算の結果が浮動小数点数の場合、出力値は浮動小数点数/整数のどちらでしょう?
  • 浮動小数点数を許すなら、入出力値の形式に制限や指定はありますか?

質問されることも含めて講座やコースの一環である可能性も考えられます。

0

解決に行き詰まった場合の御参考として,既に回答された方の御指摘を踏まえた修正案を作りました。

    /* fp = fopen("log.txt", "a"); */
    fp = fopen("log.txt", "a+");

        /* scanf("%s\n", &c); */
        scanf("%s", c);

    /* for (i = 0;i < 1000;i=i+1) { */
    /*     fgets(sin[1000], sizeof(sin[0]), fp); */
    /*     if (strcmp(sin, "EOF") == 0) { */
    rewind(fp);
    for (i = 0; i < 1000; i++) {
        if (fgets(sin[i], sizeof(sin[0]), fp) == NULL) {
            count = i;

    /* for (i = 1; i <= n; i++) { */
    for (i = 0; i < count; i++) {
1
  • ありがとうございました。 Commented 2022年11月16日 7:15
0
        fgets(sin[1000], sizeof(sin[0]), fp);
        if (strcmp(sin, "EOF") == 0) {

char sin[1000][1000]; と宣言されてますから、使えるのはsin[0]からsin[999]までです。sin[1000] は使えません。おそらく、fgets(sin[i], sizeof(sin[0]), fp) としたかったのかなと思います。

fgets() の結果には改行が含まれています。「EOFと入力したら終了」としたくても、"EOF\n" と比較したほうがよいかもしれません。文字列 "EOF" ではなくファイル終端を検出したいなら、fgets() の戻り値をチェックします。

また、コード中で fp はデータの追記に使われているので、データを読むためには使用できません。前半で追記したファイルを読みたいなら、fclose() してから fopen() し直すか、最初の fopen() のモードを "a+" にしてファイルポジションを先頭に戻す必要があります。

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。