c言語でファイル読み込みがうまくいきません。

Question

a.txtには複数行２列のデータが入っています。
最初にファイルを読み込み行数を出し、その行列を２次元配列に格納したいです。
for文で２次元配列を作ったのですが、出力がうまくいきません。
c言語初心者です。どなたかわかる方、教えていただきたいです、、、。

#include<stdio.h>
#define BUF_SIZE 256
int main(int argc, char *argv[])
{
FILE *fp;
char buf[BUF_SIZE];
int line = 0;
if ((fp = fopen("a.txt", "r")) == NULL) {
return -1;
}
while (fgets(buf, BUF_SIZE, fp) != NULL) {
line++;
}
printf("%d\n", line);//行数出力

int i,j,val[line][2];
for(i=0;i<line;i++){
for(j=0;j<2;j++){
fscanf(fp,"%d",&val[i][j]);
}
}

for(i=0;i<line;i++){
for(j=0;j<2;j++){
printf("%d",val[i][j]);
}
printf("\n");
}

fclose(fp);
return 0;
}

Comments

[dodox86]

> int i,j,val[line][2];

のval[line][2]は可変長配列(VLA)と言ってC言語としては取り扱い要注意です。

[dodox86]

また、出力が依然としてうまくいかないのであれば

> a.txtには複数行２列のデータが入っています。

具体的にa.txtの内容を質問文に示しましょう。詳細が分からないと、誰も判断できません。

[iamisa]

回答ありがとうございます。
そのようにします。ご指示ありがとうございます、、

Answer 1

ファイルをオープンして、（行数を求めるために）fgetsで最後まで読んで、
こんどはfscanfしてますが、すでにファイルの最後になっているので、何も読めません。

案1：一度fcloseしてfopenし直してからfscanf（初心者にもわかりやすい）
案2：rewindかfseekでファイルの先頭に戻る（ファイルストリームのファイル位置表示子という概念の理解が必要）

Comments

[iamisa]

回答ありがとうございます。
案１で実行してみたのですが、まだ出力がうまくいきません。他に問題があるのでしょうか？、

[otn]

「うまくいかない」とは具体的にどういうことですか？
否定形で説明されると情報量がほぼゼロです。

[iamisa]

a.txtには
110000528 110000031
110000529 110000059
110000033 110000531
110000031 110000062
、、、といった数値が入っており、（行数は1529行です）実行すると、
-101802376032767
00
00
00
149629177632767
-116759054632767
00
00
00
00
149629216032767
1101284421
という数が無数に並んでいます、、。

[otn]

修正を間違えたんですかね。
あるいは、質問文のコードと違う物を実行しているとか。

Answer 2

質問にコードを書くときは <code> ボタンを押して「コード」に #incude <stdio.h> から
} までのコードを上書きしてください。「ここに言語を入力」には C を上書きしてください。

#include <stdio.h>

#define BUF_SIZE 256

int main(int argc, char *argv[])
{
	FILE *fp;
	char buf[BUF_SIZE];
	int line = 0;
	if ((fp = fopen("a.txt", "r")) == NULL) {
		return -1;
	}
	while (fgets(buf, BUF_SIZE, fp) != NULL) {
		line++;
	}
	printf("%d\n", line);		//行数出力

	rewind(fp); // ★ または fseek(fp, 0, SEEK_SET);

	int i, j, val[line][2];
	for (i = 0; i < line; i++) {
		for (j = 0; j < 2; j++) {
			fscanf(fp, "%d", &val[i][j]);
		}
	}

	for (i = 0; i < line; i++) {
		for (j = 0; j < 2; j++) {
			printf("%d", val[i][j]);
		}
		printf("\n");
	}

	fclose(fp);
	return 0;
}

a.txt

110000528 110000031
110000529 110000059
110000033 110000531
110000031 110000062

実行結果

4
110000528110000031
110000529110000059
110000033110000531
110000031110000062

Comments

[iamisa]

回答ありがとうございます。
もう一度やり直してみたところ出力することができました！
その他コードの書き方など教えていただき、ありがとうございました！

Answer 3

最初の

while (fgets(buf, BUF_SIZE, fp) != NULL) {

で、ファイルからの読出し位置が、ファイルの終わりになっています。そのため、次の fscanf()での読出しが出来ていません。

いくつか方法はありますが、fseek(fp, 0, SEEK_SET); で、ファイル位置をファイルの先頭に戻すか、bufで読み込んだ行データに対し、sscanf()で読み出すか、そのあたりでしょうか。

Comments

[iamisa]

回答ありがとうございます。
fseek(fp, 0, SEEK_SET); を加え実行してみたのですが、まだ出力がうまくいきません。他に問題があるのでしょうか？、、

Answer 4

while (fgets(buf, BUF_SIZE, fp) != NULL) {

line++;
}

行数を出しているんでしょうけど、これによってファイルポインタはファイルの最後尾を指します。
この次のfscanfではファイルの最後を読むため、常に失敗します
ファイルポインタを先頭に設定し直す必要がありますね

Comments

[iamisa]

回答ありがとうございます。
ファイルポインタを先頭に設定し直すとは、具体的にどのように書いたら良いのでしょうか？、、、
c言語初心者なのですみません、。

[y_waiwai]

http://simd.jugem.jp/?eid=58

「C言語 ファイルポインタを戻す」とかでぐぐるとでてきます。

[iamisa]

ありがとうございます。考えてみます！