C言語 realloc関数に関する質問

Question

質問内容

入力文字数制限なしで文字を入力していくプログラムを作っています。
デバックなしで実行で、文字列中に数値以外が含まれる入力を3,4回繰り返すと全く動かなくなりました。
どこで止まったのか調べたところ29行目のrealloc(number, i + 2);で止まってました。
ちなみにデバックで実行したところ問題なく動作しました。

realloc関数、malloc関数、free関数は使ったことがなく、何がおかしいのかもわかりません。
このプログラムをどう書き直したらエラーや停止状態にならなくできるのか教えてください。

※CheckNumber関数は省略しています。

該当のソースコード

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

int CheckNumber(char *); // 文字列に数字だけが含まれている時に1を返す。それ以外は0。

void main()
{
	char *number = 0;
	char c = 0;
	int jdg = 0;
	int i = 0;

	while (jdg != 1)
	{
		printf("数値を入力してください。\n");
		printf("> ");

		number = (char *)malloc(1);
		jdg = 0;
		i = 0;

		while (1)
		{
			c = getchar();

			if (c != '\n')
			{
				*(number + i) = c;
				realloc(number, i + 2); //ここで止まってました
				i++;
			}
			else
			{
				*(number + i) = '\0';
				break;
			}
		}

		if (*(number + 0) != '\0')
		{
			jdg = CheckNumber(number);
			if (jdg == 0)
			{
				printf("\n数値以外が入力されています。\n");
				free(number);
			}
		}
		else
		{
			printf("\n数字を入力されていません。\n");
			free(number);
		}
	}

	printf("入力された数字は %s です。\n", number);
	free(number);
}

エラーメッセージ

Expression:_CrtlsValidHeapPointer(block)

※エラーメッセージの有無は環境によって異なるようで、エラーメッセージが出るものと出ないものがあるようです。

Answer 1

既に回答がついていますが、一応、、、。

realloc()関数はあまり使った事が無いので、調べてみました。
この辺の解説が分かり易いでしょうか。 realloc関数
(MS サイトは、怪しい日本語だった)

realloc(number, i + 2); //ここで止まってました

こんな感じでしょうか。

	// 適当なところで、 char *old_buff; を宣言
	old_buff = number;
	number = realloc(number, i + 2);
	if (number == NULL) {
		// 確保失敗。 なんかのエラー処理
		// free(old_buff);  exit(1);
	}

あと、ポインタですが、

*(number + i) = c; => number[i] = c;

とした方が、見易いと思いますが、どうでしょうか。

Comments

[Alyn]

if (number == NULL)はどういった時に実行される処理なのでしょうか？
私のソースコードにも加えるべきでしょうか？

[pepperleaf]

realloc()でのメモリ割り当てが失敗した場合です。
この程度のコードだと発生する可能性は低いですが、エラー処理として必要と思います。

[Alyn]

number = (char *)realloc(number, i+2);
if (number == NULL){
free(number);
}
ですか？

[pepperleaf]

free()するは良いですが、そのデータは使えませんので、異常終了として、プログラムを終了すべきかと思います。

[Alyn]

異常終了ということもあるのですね。
if文の中身はgoto以外で終了されるにはどうしたらよいでしょうか？

Answer 2

reallocは割当てなおした新しいポインタを返すのでそちらで上書きしないとダメです。

number = realloc(number, i+2);

この処理をおこたり古いアドレスを使い続けてしまうと
書き込んではいけないとこに書き込んでしまいます。

Comments

[Alyn]

ありがとうございます。
キャストしたnumber = (char *)realloc(number, i+2);でよろしいのでしょうか？
またmalloc関数、free関数に誤りはありませんか？

[asm]

C++ではキャストが必要ですが、C言語の場合はvoid*は任意のポインタ型にキャストなしで代入できます。

malloc/freeに誤りはありませんが、
強いていえば、運用としてメモリの割当というのは若干時間の掛かる処理なので
1バイトずつ増やして再割り当てしていくのは無駄に感じます。

C++にはstd::vectorという可変長配列があり
そちらでは割り当ててある容量を超えると、容量を2倍に増やす感じで実装される事が多いです。

[Alyn]

ありがとうございました！