strstreamをsstreamのコンストレイントに変換するc_str（）

Question

このコードブロックはstrstreamと書かれています。そして、それを以下のようにsstreamに変換しました。私は確信していませんが、printStream->str()はprintStreamで指されるストリームバッファ内の内容のコピー（一時的）を持つ文字列オブジェクトを返していると思うし、c_str()を呼び出してconst char *を呼び出してキャストしていますその関数のスコープの外側にポインタを戻します。私はそれが一時的な値printStream->str()から戻ってきているので、あなたはこの機能の外に割り当て解除されたメモリへのポインタを使用していると思います。私はこれをどのようにするべきですか？

char * FieldData::to_string() const 
{ 
    if(printStream) 
    return printStream->str(); 
    FieldData* notConst = (FieldData*) this; 
    notConst->printStream = new std::ostrstream; 
    // check heap sr60315556 
    if (notConst->printStream == NULL) 
    return NULL; 
    *(notConst->printStream) << "Invalid Field Type"; 
    *(notConst->printStream) << '\0'; 
    return printStream->str(); 
} 

char * FieldData::to_string() const 
{ 
    if(printStream) 
    return const_cast<char *>(printStream->str().c_str()); 
    FieldData* notConst = (FieldData*) this; 
    notConst->printStream = new std::ostringstream; 
    // check heap sr60315556 
    if (notConst->printStream == NULL) 
    return NULL; 
    *(notConst->printStream) << "Invalid Field Type"; 
    *(notConst->printStream) << '\0'; 
    return const_cast<char *>(printStream->str().c_str()); 
} 

Answer 1

変更が戻りstd::stringのタイプとは直接std::stringオブジェクトを返します。

Answer 2

私はto_stringという関数が実際にはというのはというstd::stringを返すべきだと思います。

そして、すべてこのジャンクは、変数を変更する*特に* `const_cast`を使用していない、C形式のキャストを使用していない、` new`を使用しないでください

std::string FieldData::to_string() const 
{ return "Invalid Field Type"; }