C言語　文字型配列について

前提・実現したいこと

最近配列や文字列を勉強したばかりの初心者です。

１画面のサイズが80*24として、４画面分の文字列を２次元配列で以下のように宣言しています。

char title[24*4][80] = {"・・・（省略）"}

エンターを押すたびにこの文字列を１画面分ずつprintf()関数で表示するプログラムを作っています。
ただし、printf()関数の呼び出しは４回に抑えたいです。

発生している問題・エラーメッセージ

１画面分の文字列を格納する変数(str)を用意して、strcat()関数でその中に文字列(title)を順次複写していき、１画面分の複写が終わったらprintf()関数を実行してから変数を初期化する、というプログラムを作ろうと思いました。

そこで以下のプログラムを作成しましたが、メモリ破壊が起きていることが分かりました。

C
1char str[] = "";
2
3// タイトル画面処理
4void DrawTitle(void)
5{
6	/* 画面描画処理 */
7	for (int y = 0; y < 24 * 4; y++)
8	{
9		//文字列の連結
10		strcat(&str[0], &title[y][0]);
11
12		if ((y + 1) % 24 == 0)
13		{
14			//1画面ずつ表示する
15			printf("%s", str);
16
17			//文字列の初期化
18			strcpy(&str[0], "");
19
20			/* ウェイト */
21			rewind(stdin);
22			_getch();
23
24			/* 画面クリア */
25			system("cls");
26		}
27	}
28}

試したこと

メモリ破壊について調べたところ、配列の要素数を超えて要素を代入したり、存在しない添え字を指定したりすることで発生するという風に理解しました。
なので、まずこの部分で「要素数を省略すると要素数は初期値の数と同じになる」ということを忘れていたので、１画面分の文字列＋ヌル文字用の１文字を要素数として宣言するように修正しました。

//修正前
char str[] = "";

//修正後
char str[24 * 80 + 1];

また、文字列の初期化のやり方が間違っているのではないかと考えて、以下のように修正しました。

//修正前
strcpy(&str[0], "");

//修正後
strcpy(&str[0], "\n");

修正後のソースコード

C
1//要素数を省略すると初期値の数と同じになる
2//１画面分の文字列を格納する変数。ヌル文字用に＋１。
3char str[24 * 80 + 1];
4
5// タイトル画面処理
6void DrawTitle(void)
7{
8	/* 画面描画処理 */
9	for (int y = 0; y < 24 * 4; y++)
10	{
11		//文字列の連結
12		strcat(&str[0], &title[y][0]);
13
14		if((y + 1) % 24 == 0)
15		{
16			//1画面ずつ表示する
17			printf("%s", str);
18
19			//文字列の初期化
20			strcpy(&str[0], "\n");
21
22			/* ウェイト */
23			rewind(stdin);
24			_getch();
25
26			/* 画面クリア */
27			system("cls");
28		}
29	}
30}

質問したいこと

・自己流で修正してみたんですが、これでメモリ破壊は治っていますでしょうか？
・printf()関数は４回以内という条件で、自分が考えた方法よりもっと良い方法がありましたら教えていただきたいです。

neconekocat

2021/05/27 14:11

DrawTitleに限って言えば、バッファオーバーランが起きるかどうかはtitle次第です。試しにDrawTitleの先頭にmemset(title, 0x30, sizeof(title));と書いてみてください。余裕で死にます。

hal335

2021/05/27 14:18

書いてみました。死にました。

行動規範の内容に同意します

回答3件

ベストアンサー

バッファオーバーランを防ぐ方法としては一旦別の変数に確保する等があります。
Cの文字列について調べながら、なぜ防ぐことができるか(元のコードはなぜ防げないか)を考えていけば、他の方が指摘している内容も理解できるようになると思います。

C
1// タイトル画面処理
2void DrawTitle(void)
3{
4    char str[24 * 80 + 1];
5    char tmp[80 + 1];
6    //終端文字を設定
7    tmp[80] = 0x00;
8
9    /* 画面描画処理 */
10    for (int y = 0; y < 24 * 4; y++)
11    {
12        //1行分の文字列を抽出
13        strncpy(tmp, title[y], 80);
14
15        //文字列の連結
16        strcat(str, tmp);
17
18        if((y + 1) % 24 == 0)
19        {
20            //1画面ずつ表示する
21            printf("%s", str);
22
23            //文字列の初期化
24            str[0] = 0x00;
25
26            /* ウェイト */
27            rewind(stdin);
28            _getch();
29
30            /* 画面クリア */
31            system("cls");
32        }
33    }
34}