テキストファイルから読み込みたい

[ノダッシー]

こんにちは、お久しぶりです
テキストファイルから文字列を読み込み、１行ずつchar message[0]～message[3]に代入しようと思いますが
今のままデバッグして実行をするとプログラムが応答しなくなってしまいます
どのようにコードを書きかえればいいのでしょうか
よろしくお願いします

コード:

#pragma warning(disable:4996)
#include <stdio.h>
#include <stdlib.h>

#define MAX_STRINGS 80

void main(void)
{
	char message[3];
	char filename[] = "test.txt";
	FILE *fp;
	char data[MAX_STRINGS];
	int i=0;

	/* ファイル・オープン */
	if ((fp = fopen(filename, "r")) == NULL)
	{
		printf("ファイル「%s」のオープンに失敗しました。プログラムを終了します。\n", filename);
		exit(1);
	}

	/* データ処理 */
	while (fgets(data, MAX_STRINGS, fp) != NULL)
	{
		printf("%s", message[i]);
		i++;
	}

	/* ファイル・クローズ */
	fclose(fp);

}
/*test.txtの中身
あいうえお
かきくけこ
さしすせそ
たちつてと
*/

[Rittai_3D]

while中の i が3以上になるから範囲外アクセスで落ちるのではないでしょうか。
iの値を確認してみてください。

[ノダッシー]

早速の返信ありがとうございます
iの値を確認してみました
配列の数を上回ってしまうという事までは分かったのですが、コードの書き方がわかりません
検索しても出てこないので、ご教示願えませんでしょうか

[みけCAT]

テキストファイルから文字列を読み込み、１行ずつchar message[0]～message[3]に代入しようと思いますが

現状のコードでは
・message[3]は確保されていません。アクセスは避けるべきです。
・message[0]～message[2]には1文字(1バイト?)しか入りません。「あいうえお」などのデータは代表的な環境では入らないでしょう。
・せっかく読み込んだdataが読み出されていません。
・message[0]～message[2]は明示的に初期化されていません。
　入っているのが0であっても不定の値であってもポインタとして%sで「そこに格納されている」文字列を出力しようとしたら、落ちる可能性が高いと思います。
　たまたま応答なしになったのは不思議ですね。

修正してみました。

コード:

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

#define MAX_LINES 4
#define MAX_STRINGS 80

int main(void)
{
	char message[MAX_LINES][MAX_STRINGS];
	char filename[] = "test.txt";
	FILE *fp;
	int i=0;

	/* ファイル・オープン */
	if ((fp = fopen(filename, "r")) == NULL)
	{
		printf("ファイル「%s」のオープンに失敗しました。プログラムを終了します。\n", filename);
		exit(1);
	}

	/* データ処理 */
	while (i<MAX_LINES && fgets(message[i], MAX_STRINGS, fp) != NULL)
	{
		printf("%s", message[i]);
		i++;
	}

	/* ファイル・クローズ */
	fclose(fp);

	return 0;
}
/*test.txtの中身
あいうえお
かきくけこ
さしすせそ
たちつてと
*/

[Rittai_3D]

messageの要素を増やせばいいと思います。

オフトピック

現在のコードではmessageは無くても困らないと思いますが、なにをするための変数なのでしょうか？代入も初期化もなにもしてない気がします。

[みけCAT]

オフトピック

現在のコードではmessageは無くても困らないと思いますが、なにをするための変数なのでしょうか？代入も初期化もなにもしてない気がします。

テキストファイルから文字列を読み込み、１行ずつchar message[0]～message[3]に代入しようと思いますが

これが目的ではないでしょうか？

[Rittai_3D]

オフトピック

現在のコードではmessageは無くても困らないと思いますが、なにをするための変数なのでしょうか？代入も初期化もなにもしてない気がします。

テキストファイルから文字列を読み込み、１行ずつchar message[0]～message[3]に代入しようと思いますが

これが目的ではないでしょうか？

すいません、見落としていました。ご指摘ありがとうございます。

[ノダッシー]

Rittai_3DさんみけCATさんありがとうございました！
みけＣＡＴさんのコードを使って今作っているプログラムに組み込んだところ
テキストファイルから拾った文字列は表示できるのですが、語尾に文字化けのような　「・」が表示されてしまいます

例
おはようございます・←

何か解決方法はありますでしょうか？

[みけCAT]

みけＣＡＴさんのコードを使って今作っているプログラムに組み込んだところ
テキストファイルから拾った文字列は表示できるのですが、語尾に文字化けのような　「・」が表示されてしまいます

例
おはようございます・←

何か解決方法はありますでしょうか？

組み込んだプログラムがわからないので予想ですが、末尾の改行文字を消すといいかもしれません。

コード:

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

#define MAX_LINES 4
#define MAX_STRINGS 80

int main(void)
{
	char message[MAX_LINES][MAX_STRINGS];
	char filename[] = "test.txt";
	FILE *fp;
	int i=0;

	/* ファイル・オープン */
	if ((fp = fopen(filename, "r")) == NULL)
	{
		printf("ファイル「%s」のオープンに失敗しました。プログラムを終了します。\n", filename);
		exit(1);
	}

	/* データ処理 */
	while (i<MAX_LINES && fgets(message[i], MAX_STRINGS, fp) != NULL)
	{
		char *lf;
		for (lf = message[i]; *lf != '\0'; lf++)
		{
			if (*lf == '\n')
			{
				*lf = '\0';
				break;
			}
		}
		printf("%s\n", message[i]);
		i++;
	}

	/* ファイル・クローズ */
	fclose(fp);

	return 0;
}
/*test.txtの中身
あいうえお
かきくけこ
さしすせそ
たちつてと
*/

[ノダッシー]

'￥n'を見つけたら'\0'文字列終端マークを代入するのですね
解決致しました
ありがとうございました！