C言語でファイルを読込入力して、最大値を求める方法が分かりません。

Question

前提・実現したいこと

C言語で、csvファイルから2つの実数データを１次配列x,yに読込入力して、それぞれの最大値を表示するプログラムを作ろうとしています。Max1がｘの最大値、Max2がｙの最大値を格納する変数です。

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

Max1＝x[0]、Max2=y[0]でそれぞれの最大値を初期化して、while文で最大値を調べているのですが、printfで出力したMax1とMax2が初期値のx[0]とy[0]から変わりません。おそらく、ファイル読込ができていないのですが、どこが間違っているのかわかりません。

該当のソースコード

#include<stdio.h>
#include<stdlib.h>
#define _USE_MATH_DEFINES
#include<math.h>
#define MaxN 100
FILE* fp1;
int main(void)
{
	double x[MaxN] = { 0.0 };
	double y[MaxN] = { 0.0 };
	int i, N;
	double Max1,Max2;

	fopen_s(&fp1, "data3.csv", "r");
	

	i = 1;
	Max1 = x[0];
	Max2 = y[0];

	while (fscanf_s(fp1, "%lf,%lf", &x[i], &y[i]) !=EOF) {
		
		if (Max1 < x[i])
			Max1 = x[i];

		if (Max2 < y[i])
			Max2 = y[i];
		
		i++;
	}
        fclose(fp1);
        printf("%lf %lf",Max1,Max2);

### 試したこと

初期値を0にしたりしましたが、Max1が0になるだけでした。


### 補足情報（FW/ツールのバージョンなど）

visual studio 2019 を使用しています。

Comments

[dodox86]

文中のコードはマークダウン記法を用いて読みやすくするようにしてください。インデントも崩れ、読みづらいとそれだけで閲覧も回答も離れがちになります。
https://teratail.com/help/question-tips#questionTips3-5-1
https://teratail.com/questions/238564
あたりを参考にしてソースコードを適切に表示するようにしてください。

あと、プログラムを書いてデバッグしましたか？　ただ動作させているだけとか。ファイルをちゃんと読んでいますか？

[genchan2000]

ありがとうございます。
マークダウン記法を用いて修正したつもりですが、大丈夫でしょうか。
ファイルがちゃんと読み込めていないということが分かりました。しかし、その原因がよくわかりません。もう一度自分でも考えてみます。

[dodox86]

> マークダウン記法を用いて修正したつもりですが、大丈夫でしょうか。

お疲れ様でした。できています。あと、プログラミング言語名をつければ完璧です。
C言語なら、上の```に"C"をつけて、以下のようにします。

```C
int main() {
/* hoge */
}
```
投稿の際はプレビューで確認しましょう。
ファイルが読めていない件については既に回答をいただいているので、そちらで。

[genchan2000]

ありがとうございました！

Answer 1

fopen_s(&fp1, "data3.csv", "r");

これの戻り値はどうなっているでしょうか。
ファイル名しか指定していないので、ファイルが読み込めているかどうかが気になります。

Comments

[genchan2000]

ファイルがちゃんと読み込めていませんでした。
ご回答ありがとうございます。

Answer 2

※ 回答ではありません

まずは
while (fscanf_s(fp1, "%lf,%lf", &x[i], &y[i]) !=EOF) {　// この直後に
printf("%d : %lf %lf\n", i, x[i], y[i]); // コレ追加して
...

ファイルから値が読み込めていることを確認してみて。いかがですか?

Comments

[genchan2000]

ありがとうございます。
やってみましたが、やはりファイルから値が読み込めていませんでした。
なにが原因なのでしょうか？
もう少し自分でも考えてみます。

[episteme]

fopen_s(...) の戻り値が 0でないとき、ファイル・オープンに失敗してます。確認して。

[genchan2000]

実は、最大値を求める前に
　　　int i;
　　　double sum1,sum2;
fopen_s(&fp1, "data3.csv", "r");

　　　i = 0;
sum1 = 0;
sum2 = 0;
while (fscanf_s(fp1, "%lf,%lf", &x[i], &y[i]) != EOF) {

sum1 = sum1 + x[i];
sum2 = sum2 + y[i];

i++;
}
というコードで、配列要素の和を求めており、その後にそのまま、
i = 1;
Max1 = x[0];
Max2 = y[0];

while (fscanf_s(fp1, "%lf,%lf", &x[i], &y[i]) !=EOF) {

if (Max1 < x[i])
Max1 = x[i];

if (Max2 < y[i])
Max2 = y[i];

i++;
}
fclose(fp1);
と続けていたのですが、ファイルオープンに失敗しているという指摘を受け、配列要素の和を求めた後に、fcloseでファイルを閉じ、もう一度ファイルオープンをするようにしたら成功しました。
一度fscan_fを使うと、その後にそのままfscan_fは使えないということなのでしょうか。
初歩的な質問ですみません。

[episteme]

前段：「配列要素の和」を求めた時点でファイルのナカミを読み尽くしてるから、
そのあといくら読もうとしても"弾切れ"です。

[genchan2000]

そうだったんですね。知りませんでした。
素早いご返信ありがとうございました！