小数点以下の浮動小数点数を設定する

Question

floatをconst char *に変換するC++関数を記述したい。この関数では、出力値が表すべき小数点以下の桁数を指定する引数が渡されます。私はこの機能を思いついたし、うまく機能する。

私はこの機能を書いた方がいいですか？

static const char* getString(float value, int decimalPlaces) 
{ 
    char strValue[sizeof value]; 
    sprintf(strValue, "%.%df", value, decimalPlaces); 
    return strValue; 
} 

Answer 1

あなたの機能を持つ2つの大きな問題があります使用しないことをお勧めします。

まず、floatのサイズは、ほとんどのプラットフォームの4バイトにあります。これは、配列が4つの要素であることを意味します。浮動小数点値のすべての桁で十分ではありません。範囲外を書き込むと、の定義されていない動作になります。

2番目の問題は、ローカル変数へのポインタを返すことです。配列strValueは、関数が戻ったときに有効範囲外になり、ポインタは現在という迷惑ポインタとなります。これを省略すると、未定義の動作にもつながります。

つまり、あなたの機能はではありません。は「うまく動作します」。それは実際には全く機能しません。

明白な解決策は、std::stringとstd::to_stringを使用して浮動小数点値を文字列に変換することです。そして、何らかの理由でCスタイルのヌル終了文字列が必要なときは、文字列c_str()を使用してそのようなポインタを取得します。

小数点以下の桁数が必要な場合は、std::ostringstreamと標準I/O manipulatorsを使用して文字列の書式を設定します。あなたの目標が出力する文字列を取得する場合は、もちろん、この手順をスキップして、出力を書き込むときにマニピュレータを直接使用することができます。

Answer 2

return strValue;を使用すると、ローカル変数へのポインタを返すことになります。つまり、関数が終了するとその変数がなくなり、現在は何もないポインタがあります。そのポインタを使用することは未定義の動作です。

は、次のいずれかのバッファを割り当てる new[]を使用する必要があるとしていると、あなたは delete[]に呼び出しサイト内のポインタを覚えているか std::stringのようなものを使用して文字列自体は、メモリ管理を扱うようにする必要があります。

個人的に私は次のようにstd::stringとstd::stringstreamを使用します。あなたはgetMStringを残して、あなたが削除されたデータへのポインタを返したらchar strValue[]が削除されているので、あなたの関数に問題があり

static std::string getMString(float value, int decimalPlaces) 
{ 
    std::stringstream stream; 
    stream << std::fixed << std::setprecision(decimalPlaces) << value; 
    return stream.str(); 
} 

Answer 3

、これはUBです。

は、私が何かを発明するが、C++ 11 std::to_string(float)