数値を任意の桁数の文字列に変更する関数

Question

入力した数値を文字列に変更し，任意の桁数に変更して文字列として出力する関数を作ろうとしています．
戻り値の値をchar*型で返したいのですが．変換がうまくいっていないのか値が返されません．
アドバイスよろしくお願いいたします．

char* hogehoge(int num, int digit) {

char* k;

//0を詰めて8桁にする
std::ostringstream ss;
ss << std::setw(digit) << std::setfill('0') << num << "\n";
std::string s(ss.str());

k = const_cast<char*>(s.c_str());

return (k);

}

Answer 1

template
<
    typename TYPE,
    std::size_t SIZE,
    std::size_t DIGIT
>
TYPE* hogehoge_public(TYPE (&buf)[SIZE], int num, int DIGIT)
{
    constexpr double log10 = 3.32192809489
    static_assert(SIZE > std::numeric_limits<int>::digits / log10 + 2, "hogehoge usage: buffer is too small");
    static_assert(SIZE > DIGIT, "hogehoge usage: buffer is too small");
    hogehoge_private(buf, num, DIGIT);
    return buf;
}

こんな感じで、十分なバッファの確保を強制してみてはいかがでしょうか? (ドラフトなので、コンパイルできるとは限りませんが、方針は使えると思います。)

Answer 2

こんにちは。

ローカル変数は、ローカル変数を確保した関数からreturnする時に開放されます。
なので、戻り値としてローカル変数へのポインタや参照を返却しては行けません。
今回の場合は、std::string側を返却するのがバランスの取れた（性能とメンテナンス性）返却方法です。

Comments

[yunyun0913]

早速に回答ありがとうございます．

dllとして他のプログラムに読み込ませる際に，stringに対応していなかったため，char*で関数を作成したのですが，それでは返却できないのでしょうか．

[Chironian]

その場合、文字列を返却するメモリをどこで確保するのか要検討です。

staticローカル変数を使うとマルチ・スレッドで使えない関数になりますのでやめたほうが良いです。
newで確保して返却することはありと思います。（開放が面倒ですが。）
呼び出し側で十分な領域を確保して、引数として領域へのポインタを渡す方法もあります。（十分な領域って何バイト？という問題があります。）
このように「問題のある方法」か「面倒な方法」しか選択できませんが、頑張るしか無いと思います。

[SaitoAtsushi]

> dllとして他のプログラムに読み込ませる際に，stringに対応していなかったため，

これは本体 (exe) 側が決めた規約に沿って DLL を作らなければならない、 exe をいじることは出来ないという状況を意味していると考えてよいのでしょうか？

[episteme]

文字数は既知なのだから、呼び側で確保して引数に渡すのが無難に思えます。 < 領域確保