C 文字列配列の中身 確認方法

Question

前提・実現したいこと

C言語初心者です。
fgetsを使って、txtファイルから文字列の配列を作りたいのですが、配列の中身の確認の仕方が分かりません。

以下のコードでbuf = "A, B, C, D"
のようになっているでしょうか？
またそれの確認方法をご教授ください。

sample.txtは以下のようになっています。

sample.txt
A
B
C
D

該当のソースコード

#include <stdio.h>

int main(void)
{

	FILE *fp;
	char buf[50];

	fp = fopen("sample.txt","r");

    while(fgets(buf,30,fp) != NULL){
    }

	fclose(fp);


		
	return 0;
}

Answer 1

fgetsを使って、txtファイルから文字列の配列を作りたいのですが、配列の中身の確認の仕方が分かりません。

配列というのは char buf[50]; と宣言した buf ですね。
fgets の後は、読み込んだ文字列が入っているわけですから、
printf("[%s]\n", buf); で確認できます。

以下のコードでbuf = "A, B, C, D" のようになっているでしょうか？

なっていません。
fgets で読み込んだ文字列から最後の '\n' (改行文字) を取り除いて、
その文字列を ", " (コンマとスペース) でつないたものにしたいのですよね。

次のコードが理解できますか？
疑問点をコメントしてください。

#include <stdio.h>   // fopen, fclose, fgets, printf
#include <string.h>  // strlen

#define N 50

int main(void)
{
    char buf[N];
	int i = 0;  // buf への読み込み位置

    FILE *fp = fopen("sample.txt","r");
	if (fp == NULL) return 1;

	while (fgets(buf + i, N - i, fp) != NULL) {
		i += strlen(buf + i) + 1;
		buf[i-2] = ',';
		buf[i-1] = ' ';
    }
	buf[i-2] = '\0';
    fclose(fp);

	printf("[%s]\n", buf);
}

入力が多すぎてバッファオーバーランしないように
while (fgets(buf + i, N - i, fp) != NULL) { は
while (i < N - 20 && fgets(buf + i, N - i, fp) != NULL) {
のように制限したほうが良いでしょう。

Answer 2

C言語のコードを書くなら、デバッグできる環境を整えましょう。
Eclipseや、WindowsならVisualStudioなど。
コードの任意の場所で実行を止め、変数のナカミを見ることができます。そこから１行づつ実行して、コードの流れを見れるようになります

Answer 3

配列bufは質問で書かれている意図通りの結果にはなりません。

配列bufの中身はfgetを呼ぶたびに上書きされるので、「A,B,C,D」のような文字列を得たいのだとしたらbufに読みこんだ後に別の文字配列へコピーしていくような処理が必要だと思います。

ソースコードにprintf()を入れてあげれば手っ取り早く確認できると思います。

#include <stdio.h>

int main(void)
{

    FILE *fp;
    char buf[50];

    fp = fopen("sample.txt","r");

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

        printf( "buf=(%s)\n", buf );
    }

    fclose(fp);

    printf( "loop end\n" );
    printf( "buf=(%s)\n", buf );

    return 0;
}

$ ./hoge
buf=(A
)
buf=(B
)
buf=(C
)
buf=(D
)
loop end
buf=(D
)

#コメントを受けて追記

sampleという文字列配列に格納する処理の一例を書いてみました。
printf()で途中経過を表示するようにしてみたので確認してみてください。

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

int main(void)
{

    FILE *fp;
    char buf[50]; /* 制限： 入力ファイルの1行の長さは49文字以外でなければならない */
    char sample[1024]; /* 制限： 最終的な結果文字列のサイズは1023文字に収まらなければならない */

    fp = fopen("sample.txt","r");
    if ( fp == NULL ) {
        perror( "fopen" );
        return 1;
    }

    /* sampleを文字列長が0の文字列となるように初期化する */
    sample[0] = '\0';

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

        printf( "buf(before)=(%s)\n", buf );

        /* 行末の改行文字をカンマに置き換える */
        buf[ strlen( buf ) - 1 ] = ',';

        printf( "buf(after)=(%s)\n", buf );

        /* sampleの末尾に現在のbufを連結する */
        strcat( sample, buf );

        printf( "sample=(%s)\n", sample );

        printf( "\n" );
    }

    fclose(fp);

    /* sampleの末尾にあるカンマを削除する */
    sample[ strlen( sample ) - 1 ] = '\0';

    printf( "loop end\n" );
    printf( "sample=(%s)\n", sample );

    return 0;
}

$ ./hoge
buf(before)=(A
)
buf(after)=(A,)
sample=(A,)

buf(before)=(B
)
buf(after)=(B,)
sample=(A,B,)

buf(before)=(C
)
buf(after)=(C,)
sample=(A,B,C,)

buf(before)=(D
)
buf(after)=(D,)
sample=(A,B,C,D,)

loop end
sample=(A,B,C,D)

Comments

[amezea]

配列の中身の確認方法は理解することができました。ありがとうございます。

「bufに読みこんだ後に別の文字配列へコピーしていくような処理」とは具体的にコードで表すとどのようになるのでしょうか？

[hidezzz]

> fgetsを使って、txtファイルから文字列の配列を作りたい

> 以下のコードでbuf = "A, B, C, D"のようになっているでしょうか？

「文字列の配列」とは具体的はどのようなどのようなデータ構造とか変数(の配列)に格納されることを想定していますか？
具体的なコードを作るためにはそのへんが明確になっている必要があると思います。

それが決まればたぶん、fget()するたびにbuf入っている文字列を「文字列の配列」へstrcpy()とかmemcpy()とかでコピーするような感じになるんじゃないかという気がします。

[amezea]

char sample[] =　"A, B, C, D"
のようにしたいのですが、以下のコードのままだと今まで通り、sampleの値が上書きされてしまうだけのようです。
strcpyを使ってsampleに上のような値を入れていくにはどうしたらよいのでしょうか？

#include <stdio.h>
#include <string.h>

int main(void)
{

FILE *fp;
char buf[50];
char sample[] = {};

fp = fopen("sample.txt","r");

while(fgets(buf,30,fp) != NULL){
strcpy(sample, buf);
}

fclose(fp);

return 0;
}

[hidezzz]

回答に追記しました。