C言語でintを文字列に変換する関数が動作しない理由｜ローカル配列をreturnしてはいけない原因と正しい方法

C言語で整数型のintを文字列へ変換したい場合、sprintf関数を使う方法はよく利用されます。しかし、変換処理を書いたのに正常に動作しない場合、その原因の多くは文字列を格納するメモリの扱いにあります。

特に、関数内で宣言したローカル配列をreturnで返そうとすると、C言語では未定義動作となります。この記事では、intを文字列へ変換する処理で発生しやすい問題と、安全な実装方法について詳しく解説します。

関数内のchar配列をreturnすると問題が発生する理由

問題となるコードでは、以下の部分が原因になります。

char buf[20];

この配列は関数int_to_strの中で作られたローカル変数です。ローカル変数は、その関数が終了すると同時にメモリ上から無効になります。

つまり、以下の処理では、存在しなくなった配列のアドレスを返していることになります。

return buf;

C言語では配列名を返すと、その中身ではなく配列の先頭アドレスが返されます。しかし、returnした時点でbufが使用していた領域は有効ではなくなるため、呼び出し側で利用すると予測できない結果になります。

関数内で宣言された通常の変数は、多くの場合スタック領域に確保されます。

例えば、次のような処理を考えます。

char *str = int_to_str(123);

int_to_str関数の実行中はbufが存在していますが、関数終了後にはその領域が別の処理で使われる可能性があります。そのため、strが指している場所には正しい文字列が残っている保証がありません。

たまたま動作するように見える場合もありますが、それはメモリの状態による偶然であり、正しいプログラムではありません。

方法の一つは、呼び出し側で文字列保存用の領域を用意し、それを関数へ渡す方法です。

例えば以下のように実装できます。

void int_to_str(int int_value, char *buf, size_t size) { snprintf(buf, size, "%d", int_value); }

使用例は以下になります。

char str[20]; int_to_str(12345, str, sizeof(str)); printf("%s", str);

この方法では、文字列を保存する場所を呼び出し側が管理しているため、安全に利用できます。

元のコードではsprintfを使用していますが、sprintfは文字数制限がありません。そのため、用意した配列サイズを超える長さの文字列を書き込む危険があります。

例えば、以下のような場合です。

char buf[5]; sprintf(buf, "%d", 123456789);

この場合、bufの容量を超えて書き込みが発生し、メモリ破壊につながる可能性があります。

そのため、現在のC言語プログラミングでは、サイズを指定できるsnprintfを使用することが推奨されます。

関数側で文字列を作成して返したい場合は、動的メモリを利用する方法もあります。

例えば以下のような形です。

char *int_to_str(int int_value) { char *buf = malloc(20); if(buf == NULL) return NULL;

snprintf(buf, 20, "%d", int_value); return buf; }

この場合、bufはmallocによって確保された領域なので、関数終了後も有効です。

ただし、mallocで確保したメモリは使用後に必ずfreeする必要があります。

char *str = int_to_str(100); printf("%s", str); free(str);

ローカル配列ではなくstatic配列を使う方法もあります。

char *int_to_str(int int_value) { static char buf[20]; snprintf(buf, sizeof(buf), "%d", int_value); return buf; }

この場合、bufは関数終了後も存在するため、一見問題なく動作します。

しかし、同じ関数を複数回呼び出すと内容が上書きされます。また、マルチスレッド環境では問題になる可能性があります。そのため、一般的な用途では呼び出し側でバッファを管理する方法が安全です。

intを文字列へ変換する処理で動作しない主な原因は、関数内で作成したローカル配列bufをreturnしていることです。

関数終了後にローカル変数の領域は無効になるため、そのアドレスを返して利用することは危険です。

C言語では、文字列を返す場合に「誰がメモリを管理するのか」を明確にすることが重要です。初心者のうちは、呼び出し側で配列を用意して関数へ渡す方法を使うと、安全で理解しやすい実装になります。