質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.47%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

ループ

ループとは、プログラミングにおいて、条件に合致している間、複数回繰り返し実行される箇所や、その制御構造を指します

Q&A

解決済

1回答

709閲覧

C言語で無限ループしてしまう

saito_666

総合スコア4

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

ループ

ループとは、プログラミングにおいて、条件に合致している間、複数回繰り返し実行される箇所や、その制御構造を指します

0グッド

0クリップ

投稿2022/11/17 02:14

前提

現在下記の課題に取り組んでいるのですが、ループ処理からのbreakが上手くいかない状態です。
修正点を教えて頂けないでしょうか?

■演習問題000 ~構造化プログラミングの基礎3 ファイル入出力~

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

  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

実現したいこと

無限ループから抜け出す

発生している問題・エラーメッセージ

ここの行で無限ループしてしまう。 for (i = 0;i < 1000;i = i + 1) { fgets(sin[i], sizeof(sin[0]), fp); if (sin[i][0] == 0) { break; } ++cnt; }

該当のソースコード

C言語

1ソースコード 2```#include <stdio.h> 3#include <time.h> 4#include <string.h> 5#include <stdlib.h> 6 7 8static char ad[10]; 9 10int cmp_u(const void* a, const void* d) { 11 return strcmp((char*)a, (char*)d); 12} 13 14int cmp_d(const void* a, const void* d) { 15 return strcmp((char*)d, (char*)a); 16} 17 18int main() { 19 int num1, num2; 20 char op; 21 float answer; 22 int r,i,n; 23 FILE* fp; 24 char c[11]; 25 char sin[1000][1000]; 26 char ad[8]; 27 28 29 fp = fopen("log.txt", "a+"); 30 31 if (fp == NULL) { 32 printf("ファイルオープン失敗\n"); 33 return -1; 34 } 35 36 37 while (1) { 38 r = scanf("%d%c%d", &num1, &op, &num2); 39 if (r != 3) { 40 puts("input error"); 41 return 1; 42 } 43 44 if (op == '+') { 45 answer = num1 + num2; 46 } 47 else if (op == '-') { 48 answer = num1 - num2; 49 } 50 else if (op == '*') { 51 answer = num1 * num2; 52 } 53 else if (op == '/') { 54 answer = (float)num1 / num2; 55 } 56 57 time_t t = time(NULL); 58 struct tm* tm = localtime(&t); 59 printf("%d/%02d/%02d ", tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday); 60 printf("%02d:%02d:%02d ", tm->tm_hour, tm->tm_min, tm->tm_sec); 61 printf("%d%c%d,%f\n", num1, op, num2, answer); 62 fprintf(fp, "%d/%02d/%02d ", tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday); 63 fprintf(fp, "%02d:%02d:%02d ", tm->tm_hour, tm->tm_min, tm->tm_sec); 64 fprintf(fp, "%d%c%d,%f\n", num1, op, num2, answer); 65 66 printf("計算を続けますか?"); 67 scanf("%s\n", &c); 68 if (strcmp(c, "no") == 0) { 69 break ; 70 } 71 } 72 73 fclose(fp); 74 75 76 fp = fopen("log.txt", "r"); 77 78 int cnt = 0; 79 80 for (i = 0;i < 1000;i = i + 1) { 81 fgets(sin[i], sizeof(sin[0]), fp); 82 if (sin[i][0] == 0) { 83 break; 84 } 85 ++cnt; 86 } 87 88 89 fclose(fp); 90 91 92 93 printf("ASC or DESC: "); 94 scanf(" %s", ad); 95 96 if (strcmp(ad, "ASC") == 0) { 97 qsort(sin,cnt , sizeof(sin[0]), cmp_u); 98 99 } 100 else { 101 qsort(sin, cnt, sizeof(sin[0]), cmp_d); 102 103 } 104 105 106 for (i = 0;i < 1000;i = i + 1) { 107 if (sin[i][0] == 0) { 108 break; 109 } 110 printf("%s", sin[i]); 111 } 112 113 return 0; 114} 115 116 117 118 119### 補足情報(FW/ツールのバージョンなど) 120 121Visual Studio 122サクラエディタ

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

maisumakun

2022/11/17 02:16

> ここの行で無限ループしてしまう。 どのような条件で脱出することを意図して実装を行いましたか?
int32_t

2022/11/17 02:21 編集

https://ja.stackoverflow.com/a/92256/3475 ここで「fgets() の戻り値をチェックします」と書きましたが、調べたり試してみましたか? あと、無限ループはしないのでは? 1000回ループしたら抜けますよね。
jimbe

2022/11/17 03:49

そう言えば指摘されていませんので一応。 「teratail は課題の質問は非推奨です。」
guest

回答1

0

ベストアンサー

無限ループしている、ってわけじゃなくて、ファイルの中身が1000行なくて入力待ちしているだけでは。
ループの中で、ファイルの終わりの判定をしよう

#あるいはファイルが存在していない?

投稿2022/11/17 02:23

y_waiwai

総合スコア87784

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.47%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問