C言語でファイルを読み込む fgets()

Question

#define WORLD_H 38
#define WORLD_W 38

void fileop(char temp[WORLD_H][WORLD_W]) {

	FILE* fp;
	char txt;
	char fname[] = "sample_data2.txt";
	char str[256];
	int maxchar; //最大文字数

	fp = fopen(fname, "r");
	if (fp == NULL) {
		printf("%s  file not open!\n", fname);

		return;
	}



	// while (fgets(str,256,fp) != NULL)   // 一行ずつ読み込みたい

	for (int rread = 0; rread < WORLD_H; rread++) {

		for (int cread = 0; cread < WORLD_W + 1; cread++) {

			txt = fgetc(fp);

			if (txt == '\n') { break; }

			else if (txt == '1') {

				temp[rread][cread] = 1;
			}
			else if (txt == '.') {

				temp[rread][cread] = 0;

			}


		}

	}
	fclose(fp);

}

sample_data2.txt ファイルの内容
......................................
.........................1............
.......................1.1............
.............11......11............11.
............1...1....11............11.
.11........1.....1...11...............
.11........1...1.11....1.1............
...........1.....1.......1............
............1...1.....................
.............11.......................
......................................
......................................
......................................

---

sample_data2.txt を行ごとに読み込みたいのですが

読み込んだ内容を　if で判定して　. だったら . を 1 だったら1を　一文字ずつ判定したいです
しかし、行で読み込んで　一文字ずつ判定してそれを配列に格納するのが、どのようにすれば良いかあまりイメージが付かなかったのでアドバイス頂けると助かります。

Comments

[Zuishin]

ループと分岐と配列の全てがわからないということですか？
そこは基本中の基本なので、全てわかるようになってからこのようなサイトを使う方が効率的だと思います。
効率を気にせず好きなだけ足踏みしたいならそれでもいいですが。

Answer 1

一応サンプルです。
エラーに関する扱いは、以下のようにしています。
もっと厳密にするか、緩くするかは、あなたが今後自分で判断してください。
行数が３８行を超えた場合は、警告を表示し、３９行以降は読み込まない。
行数が３８行未満なら、エラーを表示しギブアップする。
1行内の文字が１か.のどちらでもないなら、エラーを表示し、ギブアップする。
（ただし３８列までの検査し、以降の列は無視する）

不明点は質問してください。

#include <stdio.h>
#include <stdlib.h>
#define WORLD_H 38
#define WORLD_W 38

void fileop(char temp[WORLD_H][WORLD_W]) {

    FILE* fp;
    char txt;
    char fname[] = "sample_data2.txt";
    char str[256];
    int     i;
    int     j;
    fp = fopen(fname, "r");
    if (fp == NULL) {
        printf("%s  file not open!\n", fname);

        return;
    }
    i = 0;
    while (fgets(str,sizeof(str),fp) != NULL){   // 一行ずつ読み込みたい
        if (i >= WORLD_H){
            printf("行数オーバーなので読み込まない\n");
            break;
        }
        for(j = 0; j < WORLD_W;j++){
            if (str[j] == '1'){
                temp[i][j] = 1;
            }else if (str[j] == '.'){
                temp[i][j] = 0;
            }else{
                printf("%d行 %d列　不正データ\n",i+1,j+1);
                exit(10);
            }
        }
        i++;
    }
    if (i < WORLD_H){
        printf("行数不足\n");
        exit(10);
    }
    fclose(fp);

}
int main()
{
    char    array[WORLD_H][WORLD_W];
    int i,j;
    fileop(array);
    //内容の出力
    printf("12345678901234567890123456789012345678\n");
    for (i = 0; i < WORLD_H;i++){
        for (j = 0; j < WORLD_W;j++){
            printf("%d",array[i][j]);
        }
        printf("\n");
    }
}

Comments

[kokok]

while分の前の　 i = 0; と i++ は　読み込んだファイルの　何行目かということでしょうか？

後もしよかったら、exit(10) の引数１０とはどういう意味なのでしょうか？

[tatsu99]

>while分の前の　 i = 0; と i++ は　読み込んだファイルの　何行目かということでしょうか？
はい、そうです。fgetsを１回実行すると、１行を読みますから、iに１を加えていきます。
最初のiを0にしているのは、1行目が、配列tempのtemp[0]に該当するからです。

＞exit(10) の引数１０とはどういう意味なのでしょうか？
このプログラムの終了コードです。通常（慣習的にですが）は０が正常終了、０以外が異常終了にするのが作法になっています。どのように利用するかは、OSによって違います。あなたの環境のOS（windowsかlinuxのどちらかと思いますが）を提示していただければ、それに沿った説明をします。
尚、exit(値)とmain関数が返す、return 値　は同じことです。

[tatsu99]

追伸：プログラムの終了コードは、慣習的に０以外は異常終了なので、exit(1)でも構いません。
一般的には、値が大きくなるほど、重度のエラーにする方法が採用される傾向にあります。
尚、終了コードの範囲は０～２５５の値を使用するのが無難です。それ以上の値を使用するとOSによりますが、期待した終了コードにならない場合があります。

[kokok]

ありがとうございます。
とても分かりやすくて理解できました。

Answer 2

アドバイス頂けると助かります

その処理の塊を一度に扱えないのであれば，
やりたいことを細かい要素に分割し，１つずつクリアしていけば良いのではないでしょうか．
例えば，

1. ファイルを１行ずつ読む：

fgetsで１行読込んではprintfとかで単に表示してみる
2. .か1かの判定：
ある文字列が与えられたときに，その各文字が.か1かを判定するだけの処理を書いてみる
3. 2.の判定結果を配列に格納するようにしてみる
4. 1～3を組み合わせる

とか．

Comments

[kokok]

回答ありがとうございます。
一つずつクリアしていきたいと思います。

一つ自分の中で引っかかっているのが、fgets で1行ずつ読んで それを一文字ずつ判定するのは、どのようにすれば良いかそこがイメージが付かないのですが、
char str[256];　この変数に　１行ずつ読み込まれていくということなのでしょうか？

[fana]

そういう場合は，fgetsに関してググるなりして，fgets(str,256,fp)と書いた場合に何が起こるのかを把握しましょう．

[kokok]

ありがとうございます。　実際にコードを書いて動きを確かめます。

Answer 3

コードは単に手順の並びです.
「行で読み込んで　一文字ずつ判定してそれを配列に格納する」をそのまま書けばよいだけです.
しかし, いきなり全部を書こうとしますと, どこかで間違えていても影響が全体に及んで間違えた箇所の特定がし難くなりますので, 少しずつ(確認のための表示を入れながら)作って実行を繰り返すほうが, 理解も進むかと思います.

まず, 「行を読み込んで表示」を全行分行うようにされては如何でしょう.
次に, 「行の一文字ずつを表示」するようにします.
最後に, 「表示を判定に変えて, 配列に格納」するように変えます.

Comments

[kokok]

ありがとうございます。
参考にします