[C言語]ファイルからfscanfを用いて数値を読み込み、ファイル末尾になったらwhile(1)ループを抜け出したい

Question

前提・実現したいこと

ファイルからfscanfを用いて数値を読み込み、ファイル末尾になったらwhile(1)ループを抜け出したい

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

whileループを抜けられない

該当のソースコード

#include<stdio.h>
#include<stdlib.h>
int main(void){
 FILE *fp;
 double x[5],y[5];
 int i=0;
 if((fp=fopen("input.txt","r"))==NULL){
   fprintf(stderr,"FILE OPEN ERROR");
   exit(1)
 }
while(1){
  if(fscanf(fp,"(%lf,%lf)",&x[i],&y[i])==EOF) break;
  else{
   printf("(%lf,%lf)",x[i],y[i]);
   i++;
   }
 }
fclose(fp);
return 0;
}

困っていること

テキストファイルからfscanfで数値を読み込み、ファイル末尾の場合終了、そうでない場合読み込んだ数値を画面表示するプログラムを書きたいのですが、whileループを抜けられずセグメンテーション違反となってしまいます。

またif文の条件文について、if(fscanf~)とした場合、その時点でfscanfが実行されたことになるのでしょうか。（つまりelse文でfscanfを書く必要があるのかどうか）

なおよくある方法としてwhile((fscanf~)!=EOF)とする方法がありますが、今回はwhile(1)としてif文を使ってwhileを抜ける方法を考えたいです。

補足情報

input.txtの内容

(1.0,2.0)
(-3.0,4.5)
(2.5,5.0)
(4.0,2.8)

Answer 1

if(fscanf(fp,"(%lf,%lf)",&x[i],&y[i])==EOF) break;

これ、ダメです。 fscanf()の仕様を確認しましょう。
戻り値は、読み込んだ数です。 今回の場合、2個読んでいるので、(ざっくり) fscanf() != 2でしょうか。
既に回答があるように、\nも必要でしょうが、、。

[追記]

(1.1,1.2)
(1.3,1.4)

みたいに正しい数値のみがあるファイルでは、EOFが返りました。
ただ、最後に ()みたいに不正なデータがある場合、その場所での戻り値は、0でした。

Comments

[itagagaki]

入力が終端に達したらEOFが返るので間違いではないです。

[y_waiwai]

まあ、ファイルに変換できない文字があったらそれでは無限ループになりますけどね

[pepperleaf]

手元だと、返らないですね。(Visual studio 2019環境)
方法が悪いのかは未確認ですが、、。

[dodox86]

itagagakiさんのご回答での処置後、あえて加えるならループの回数上限を設ける、でしょうか。現状、5行を超えるファイルのデータだと問題が出ます。

[sine_sign]

原因分かりました、テキストファイルの方の記述が間違っていた（(4,0),(2.8)になっていました。
変なミスで申し訳ありませんでした。回答ありがとうございました。

Answer 2

fscanf(fp,"(%lf,%lf)\n"...にすれば（\nを加えれば）とりあえずは解決すると思います。

if(fscanf~)とした場合、その時点でfscanfが実行されたことになるのでしょうか

これはそのとおりです。

Comments

[sine_sign]

回答ありがとうございます。fscanf(fp,"(%lf,%lf)\n"...として実行してみましたが、症状は変わりませんでした、、、

[itagagaki]

質問に書かれたコードに\nを加えてをコンパイルし（exit(1)にセミコロンを付ける必要がありましたが）、
質問に書かれたinput.txtを置いて実行してみたら問題なく実行できました。
ご報告まで。

[sine_sign]

試しにvisual studio code 2019で実行してみたら問題なく動きました。環境による違いなのでしょうか。。。

[sine_sign]

原因分かりました、テキストファイルの方の記述が間違っていた（(4,0),(2.8)になっていました。
変なミスで申し訳ありませんでした。回答ありがとうございました。