ファイル入力

質問

1 20
2 30
3 95
4 52
5 90

3 Maximum

　上記の様な入力ファイルinput.datのうち1～5行目のデータを表示するプログラムを作っています。
　6行目は空白で、7行目は2列目の中で最大値をとる行の1列目の数値が入っています。
　1列目の数値は最後の行を除いて重複することはありません。
　入力データの行数はファイルによって最大20行まで変動します。列数は2列で固定です。
　以下のプログラムのままでは6行目以降のデータも読み取ってしまい、出力がおかしくなってしまいます。
　1～5行目のデータのみ出力するにはどうしたらよいでしょうか。
　ご存知の方、お手数ですが教えてください。よろしくお願いします。

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

#define row 20
#define col 2

int main(){
int i,j,data[20][2];
FILE *fp;

if((fp=fopen("input.dat","r"))==NULL){
fprintf(stderr,"Cannot open file input.dat\n");
exit(1);
}
for(i=0;i<row;i++){
for(j=0;j<col;j++){
fscanf(fp,"%d",&data[i][j]);
printf("%d ",data[i][j]);
}
printf("\n");
}
fclose(fp);
return 0;
}

回答No.8

mac_res · ベストアンサー

＞ご回答いただいたプログラムを参考に下記のように作ってみましたがどうでしょうか。

停止条件の行を読み込むために、20行のデータがあるとdata[20][0]に読み込み配列領域を超えてしまいます。
for (i = 0; i < row; i++)で切ったほうが安全です。

BLUEPIXYさんがおっしゃるとおり、
fscanf(fp, "%d %d", &data[i][0], &data[i][1]) は、終了条件以外でも!=1になりうるので危険です。

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

#define row 20
#define col 2

int main(){
int i,j, data[row][col];
FILE *fp;
char buf[BUFSIZ];

if ((fp = fopen("input.dat", "r")) == NULL) {
fprintf(stderr, "Cannot open file input.dat\n");
exit(errno);
}

for (i = 0; i < row; i++) {
if (fgets(buf, BUFSIZ, fp) == 0 ||
buf[0] == '\n' ||
sscanf(buf, "%d %d", &data[i][0], &data[i][1]) != 2) break;
}

for (j = 0; j < i; j++) {
printf("%d %d\n", data[j][0], data[j][1]);
}
fclose(fp);
return 0;
}

お礼

何度もご回答していただきありがとうございました。
今回質問させていただき、理解することができました。C言語の勉強をこれからも続けていきたいと思います。

回答No.7

BLUEPIXY

>>exit使っていることはどうなんだ
>すみません。exitを使ってはいけないんでしょうか？
使ってはいけないということではないです。
あんまり気にされませんように・
exitを使うと、自動的に登録した終了処理をしてくれたりして便利ですよね。
int mainなのに、voidで終わるのはどうなのぐらいの意味です。
例外的な終了なんだからいいんだ！で済む話です。
余計なことを言ってすみませんです
ついでに
＃６お礼＞while(fscanf(fp, "%d %d", &data[i][0], &data[i][1]) != 1) {
ですが、fscanfの返した値の要素数で、ループの終了条件にするのはあまりいいとは思えません。
fscanfが何らかの理由でエラーを返すような状態になった時、無限にループすることになります。

お礼

何度もご回答していただきありがとうございました。
今回でかなり勉強になりました。これからもC言語の勉強を続けていきます。

回答No.6

mac_res

＞もしおかしな点があればご指摘お願いします。

定義した変数jが使われていません。
data[row][col]に読み込んだデータは本番では使うのでしょうか？
使わなければ、配列に読み込む意味はありません。
Maximumとの一致を停止条件にするのは、不経済で、安全性に欠けます。
せっかく空行があるのだから、fscanf()の戻り値を有効に活用すべきです。

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

#define row 20
#define col 2

int
main()
{
int i, data[row][col];
FILE *fp;

if ((fp = fopen("input.dat", "r")) == NULL) {
fprintf(stderr, "Cannot open file input.dat\n");
exit(errno);
}
for (i = 0; i < row; i++) {
if (fscanf(fp, "%d %d", &data[i][0], &data[i][1]) != 2) break;
printf("%d %d\n", data[i][0], data[i][1]);
}
fclose(fp);
return 0;
}

お礼

何度もご回答ありがとうございます。
data[row][col]に読み込んだデータは本番で使います。
ご回答いただいたプログラムを参考に下記のように作ってみましたがどうでしょうか。

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

#define row 20
#define col 2

int main(){
int i=0,j, data[row][col];
FILE *fp;

if ((fp = fopen("input.dat", "r")) == NULL) {
fprintf(stderr, "Cannot open file input.dat\n");
exit(errno);
}

while(fscanf(fp, "%d %d", &data[i][0], &data[i][1]) != 1) {
i++;
}

for (j = 0; j < i; j++) {
printf("%d %d\n", data[j][0], data[j][1]);
}
fclose(fp);
return 0;
}

回答No.5

BLUEPIXY

>プログラムを見ていただけるとありがたいです。
exit使っていることはどうなんだとか、hitって名前はどうなのみたいなこと（毎回hit++しないでも、hit==0の時だけhit=1すればいいんじゃないの）とかは、流儀みたいな話なので、そんなことを除けば、
仕様の範囲内でちゃんと動くし特に問題ないと思います。

お礼

何度もありがとうございます。
言われてみれば、hit=0の時だけhit++すればよいですね。

>exit使っていることはどうなんだ
すみません。exitを使ってはいけないんでしょうか？

回答No.4

mac_res

＞このままでは入力ファイルの行数が変化すると対応できないと思うのですが・・・。
データー行が変動しても「最初の5行」を出力するプログラム化と思いました。空行の前までを出力するプログラムなのですね。
#include <stdio.h>
#include <stdlib.h>
#include <errno.h>

int
main(void)
{
char buf[BUFSIZ];
int i, j, k;
FILE *fp;

if ((fp = fopen("input.dat", "r")) == NULL) {
fprintf(stderr, "Cannot open file input.dat\n");
exit(errno);
}

for (;;) {
fgets(buf, BUFSIZ, fp);
if (sscanf(buf, "%d %d", &j, &k) != 2) break;
fputs(buf, stdout);
}
fclose(fp);
return 0;
}

お礼

再びご回答ありがとうございます。
　mac_resさんのプログラムは本を見ながらなんとか理解できました。まだまだ勉強不足だと感じています。
　No.3のお礼に自分なりに作ったプログラムをのせましたので、もしおかしな点があればご指摘お願いします。

回答No.3

BLUEPIXY

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

#define row 20
#define col 2
#define BUFFSIZE 1024

int main(){
int i,j,data[row][col];
FILE *fp;
char buff[BUFFSIZE];

if((fp=fopen("input.dat","r"))==NULL){
fprintf(stderr,"Cannot open file input.dat\n");
exit(1);
}
for(i=0;i<row;i++){
if(NULL==fgets(buff, BUFFSIZE, fp))break;
if(isspace(*buff))break;
sscanf(buff,"%d %d",&data[i][0], &data[i][1]);
printf("%d %d\n",data[i][0], data[i][1]);
}
fclose(fp);
return 0;
}
---------------------------------------------------------------
って感じですかね。
col は、#defineされていること、配列のサイズになっていることからすると、決め打ちしてもいいと思うけど、
１つずつ読み込む必要があるなら、strtokとatoiとかを使えばいいと思います。

お礼

再びご回答ありがとうございます。
　C言語はまだまだ初心者レベルなので分からない事だらけです。BLUEPIXYさんのプログラムは本で調べながら何とか理解できました。今の時点の自分の実力で今日半日かけて作ったプログラムを見ていただけるとありがたいです。最後の行の1列目が、2列目の最大値の行の1列目と同一ということに着目しました。

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

#define row 20
#define col 2

int main(){
int i,j,max_no,max,hit=0,data[row][col];
FILE *fp;

if((fp=fopen("input.dat","r"))==NULL){
fprintf(stderr,"Cannot open file input.dat\n");
exit(1);
}
for(i=0;i<row;i++)
{
fscanf(fp,"%d %d",&data[i][0],&data[i][1]);
if(hit==0){
max_no=data[i][0];
max=data[i][1];
}
else if(max_no==data[i][0]){
break;
}
else if(max<data[i][1]){
max_no=data[i][0];
max=data[i][1];
}
printf("%d %d\n",data[i][0],data[i][1]);
hit++;
}
fclose(fp);
return 0;
}

回答No.2

mac_res

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

int
main(void)
{
char buf[BUFSIZ];
int i;
FILE *fp;

if ((fp = fopen("input.dat", "r")) == NULL) {
fprintf(stderr, "Cannot open file input.dat\n");
exit(errno);
}

for (i = 0; i < 5; i++) {
fgets(buf, BUFSIZ, fp);
fputs(buf, stdout);
}
fclose(fp);
return 0;
}

補足

ご回答ありがとうございます。
作っていただいたプログラムは非常に参考になりましたが、このままでは入力ファイルの行数が変化すると対応できないと思うのですが・・・。

回答No.1

BLUEPIXY

いきなり
fscanfで読み込むのではなくて、
一行読み込んで、
行の先頭が'\n'とか' 'とかあるいは数字でないとかでブレイクするようにして、
fscanfは、sscanfに置き換えるというのはどうでしょう

補足

ご回答ありがとうございます。
sscanfは使ったことがないので、調べてみます。