c言語が実行しても動作しません

c
1#include <stdio.h>
2#include <stdlib.h>
3
4int main(void) {
5  struct point_list {
6    double x, y;
7    struct point_list *next;
8  };
9  typedef struct point_list List;
10  List *listhead = NULL;
11  List *p;
12  FILE *fp;
13  double n1, n2;
14  
15  if((fp = fopen("input_0407.txt", "r")) == NULL) {
16    fprintf(stderr, "File open error\n");
17  exit(1);
18  }
19
20  while(1) {
21    fscanf(fp, "%lf %lf", &n1, &n2);
22    if(n2 == EOF) break;
23    p = (List*)malloc(sizeof(List));
24    p -> x = n1;
25    p -> y = n2;
26    p -> next = listhead;
27    listhead = p;
28  }
29
30  p = listhead;
31
32  while(listhead != NULL) {
33    printf("(%lf, %lf) ,", p -> x, p -> y);
34    p = p -> next;
35    free(listhead);
36    listhead = p;
37  }
38
39  return 0;
40}

txt
1-4.5 3.0
2-2.0 -3.0
33.5 0.5
41.0 2.0

上のようなコードを書いてターミナルで実行しようとしたのですが、全く何も出力されません。理由をおしえてください。

hoshi-takanori

2021/05/11 17:56

input_0407.txt の中身はどうなってますか？ (if(n2 == EOF) があやしいですね…。)

hoshi-takanori

2021/05/11 18:13

input_0407.txt の最後に 0.0 -1.0 を追加したら動きましたよ。

dodox86

2021/05/11 22:29

デバッグ環境を整えてデバッグしましょう。要望通りに動かないたびに質問する訳にはいきませんね。

行動規範の内容に同意します

回答2件

全く何も出力されません。理由をおしえてください。

いうまでもなく、コンピュータはプログラムは書いてある通りに動いているのだけれど、そのプログラムが間違っている(あなたの期待する動作をするプログラムになっていない)から。

C
1  while(1) {
2    fscanf(fp, "%lf %lf", &n1, &n2);
3    if(n2 == EOF) break;
4    //<略>
5  }

プログラムが進むとここにたどり着いて、fscanfはまず-4.5と3.0を拾ってn1とn2の場所に入れて、2個のデータを取得したので戻り値2を返します(この値は捨てられます)。
n2は3.0になってEOF(大抵のシステムで数値-1としてマクロ定義)とは異なるのでbreakしません。

繰り返しで-2.0 -3.0を拾って<略>
...

繰り返しで1.0 2.0を拾って、
n2は2.0になってEOFとは異なるのでbreakしません。

繰り返しでデータがファイルの終わりに達したので、fscanf関数は戻り値としてEOFを返し(この値は捨てられます)、n1,n2の値は変更しません。
n2は2.0のまま、EOFとは異なるのでbreakしません。

繰り返しで、データがファイルの終わりに達しているので、fscanf関数は戻り値としてEOFを返し(この値は捨てられます)、n1,n2の値は変更しません。
n2は2.0のまま、EOFとは異なるのでbreakしません。

以下ず～っと繰り返し。

与えたデータで出力が行われるプログラムではないことがおわかりでしょうか。

投稿2021/05/11 23:09

thkana

総合スコア7639

K.M.PEANUTS

2021/05/11 23:51

理解できましたありがとうございます。EOFは負の値で、通常の戻り値は読み取った項目数（正の値）であることを用いて該当のwhile文のループ条件をfscanf(fp, "%lf %lf", &n1, &n2) >= 0)としたところうまく動作しました。この条件の書き方は、良いもの(一般的)でしょうか？