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

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

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

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

ループ

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

Q&A

解決済

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

saito_666
saito_666

総合スコア4

C

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

ループ

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

1回答

0グッド

0クリップ

367閲覧

投稿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サクラエディタ

以下のような質問にはグッドを送りましょう

  • 質問内容が明確
  • 自分も答えを知りたい
  • 質問者以外のユーザにも役立つ

グッドが多くついた質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

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

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

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

下記のような質問は推奨されていません。

  • 間違っている
  • 質問になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

適切な質問に修正を依頼しましょう。

2022/11/17 02:43

こちらの質問が他のユーザーから「やってほしいことだけを記載した丸投げの質問」という指摘を受けました。

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 は課題の質問は非推奨です。」

回答1

0

ベストアンサー

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

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

投稿2022/11/17 02:23

y_waiwai

総合スコア86074

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

下記のような回答は推奨されていません。

  • 間違っている回答
  • 質問の回答になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

このような回答には修正を依頼しましょう。

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

ただいまの回答率
86.02%

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

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

質問する

関連した質問

同じタグがついた質問を見る

C

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

ループ

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