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

入力した数値を文字列に変更し，任意の桁数に変更して文字列として出力する関数を作ろうとしています．
戻り値の値を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);

}

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

回答2件

C++
1template
2<
3    typename TYPE,
4    std::size_t SIZE,
5    std::size_t DIGIT
6>
7TYPE* hogehoge_public(TYPE (&buf)[SIZE], int num, int DIGIT)
8{
9    constexpr double log10 = 3.32192809489
10    static_assert(SIZE > std::numeric_limits<int>::digits / log10 + 2, "hogehoge usage: buffer is too small");
11    static_assert(SIZE > DIGIT, "hogehoge usage: buffer is too small");
12    hogehoge_private(buf, num, DIGIT);
13    return buf;
14}
15

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

投稿2019/12/22 16:04

編集2019/12/22 16:17

majiponi

総合スコア1720

こんにちは。

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

投稿2019/12/22 05:17

Chironian

総合スコア23272

yunyun0913

2019/12/22 05:37

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

Chironian

2019/12/22 05:48

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