(Ｃ言語)fscanfで数値をすべて読み取りたい

Question

前提・実現したいこと

fscanf構文を用いて数値を大きい順に並べるプログラムを作成したい。

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

fscanfでファイル内の数値を読み込もうとした際に、最初の数値のみしか読みこんでくれず、数値を大きい順に並べることが出来ない。

該当のソースコード

#include <stdio.h>
#include <stdlib.h>

int main( void)
{
	int i;
	int j;
	int k;
	double a[20];
	FILE *fp;
	
	fp = fopen("xxxxxx.txt","r");
	
	if (fp == NULL) {
		printf("ファイルが存在しません。\n");
		exit (1);
	}
	
	fscanf( fp, "%lf", &a[i]);
	
	for ( i=0; i<20; i++ ){
		for ( j=i+1; j<20; j++ ){
			if (a[i] < a[j]){
				k = a[i];
				a[i] = a[j];
				a[j] = k;
			}
		}
	}
	
	printf("トップ10は\n");
	for ( i=0; i<10; i++ ){
		printf("TOP %d: %f  \n", i+1, a[i]);
	}
	return 0;
	fclose(fp);
}

試したこと

根本として並べ替えをするソースコードが間違っているのかと思い、　double a[20] = {2.8, 10.4, 13.2, 6.0, .....};　としてファイルからの読み取りではなく入力した数値で並べ替えをしてみたのですが、その際はしっかりと並べ替えをしてくれました。

補足情報

読み込みたいテキストファイルの数値は
1.3
6.4
5.3
....
の様に1行ずつ改行して数値を入力してあります。

また、コンパイル時にエラー等は表示されません。

表示された実行結果

TOP 1: 1.300000
TOP 2: 0.000000
TOP 3: 0.000000
TOP 4: 0.000000
TOP 5: 0.000000
TOP 6: 0.000000
TOP 7: 0.000000
TOP 8: 0.000000
TOP 9: 0.000000
TOP 10: 0.000000

###教えて頂きたいこと
fscanfの使い方を学び、応用をしてみようと思いましたが意図した通りの挙動をしてくれませんでした。
どのように改善をしたら、fscanfが意図した通りに動くか教えて頂きたいです。

また、当サイトの使用が初めてで拙い質問方法であるかと思いますがどうかよろしくお願いいたします。

Answer 1

fscanfを1回しか実行していないので1つしかデータを読んでいないのは当然です。
総てのデータについてfscanfを実行しましょう。
その際にはiに適切な値をセット。

Comments

[Gabriel_]

ありがとうございます。
ご教授頂いた通り1回しか実行できていなかったので
fscanf( fp, "%lf", &a[i]);
を
for ( i=0; i<20; i++ ){
fscanf( fp, "%lf", &(a[i]) );
}
として試みてみたところすべての値を読み取ってくれました。

また、その際読み込むファイルの中身を
1.4
6.3
5.3
4.7
10.6
9.8
2.3
4.5
6.6
3.3
17.9
22.3
2.9
6.1
56.8
44.3
33.7
7.8
9.1
106.3
としていたのですが、実行結果が小数のあるものやないもので統一されていないのですが、これはどのような改善をしたらよろしいでしょうか？
重ね重ね申し訳ありません。
実行結果↓
TOP 1: 106.300000
TOP 2: 56.000000
TOP 3: 44.000000
TOP 4: 33.000000
TOP 5: 22.000000
TOP 6: 17.000000
TOP 7: 10.000000
TOP 8: 9.000000
TOP 9: 9.000000
TOP 10: 7.000000

[matukeso]

それはkがintだから起きてるバグ。

[Gabriel_]

matukeso様
ご回答ありがとうございます。
k をintから別の型に変えたところ無事意図した通りに動いてくれました。
　　　　　　k = a[i];
a[i] = a[j];
a[j] = k;
としていて、a[i]やa[j]の中身が小数点を含む数なので、kの型を適切な型にすれば良い
という解釈でよろしいでしょうか？

[otn]

Cは整数型変数と浮動小数点型変数が別なので、自分がこの場面でどの型を使うのかを決めて、その型にします。

[otn]

あと、このプログラムは学習用でしょうから良いですが、通常のプログラムの場合はfscanfなどscanf系の関数は、返り値を必ずチェックしてエラー処理をしましょう。

[Gabriel_]

otn様
詳しいご返信ありがとうございます。
まだ、Ｃ言語は学習段階で実用できるものでないですが、今後scanf系の関数を使用する際は返り値をチェックしてエラー処理をするよう心掛けたいと思います。
ありがとうございました。

Answer 2

fscanf( fp, "%lf", &a[i]);

iが初期化されてません

Comments

[Gabriel_]

ご回答ありがとうございます。
iの初期化についても加えてみます。
ありがとうございます。