mallocを使用して関数からC文字列を返す方法

Question

関数からC文字列を返そうとしています。この関数は3つの整数をコンマで連結し、結果をchar配列として返しますが、ガベージ値が得られます。私はmallocを正しく呼び出さないと仮定しています。誰かが何が問題なのかアドバイスできますか？

using namespace std; 

const char * createCommand(int p1, int p2, int p3){ 
    stringstream sstm; 
    std::string comma = ","; 
    sstm << p1 << comma << p2 << comma << p3; 
    std::string str = sstm.str(); 
    const char *cstr = (const char *)malloc((str.length()+1) * sizeof (char)); 

    cstr = str.c_str(); 
    return cstr;  
} 

int main() { 
    const char *cstr2 = createCommand(1,0,250); //I want to return "1,0,250" 
    printf("char = %s\n",cstr2); 
} 

Answer 1

リテラルの問題に対処するための回答にはすでに他の2つの回答がありますので、私はあなたの問題を引き起こしているかなり重大な設計上の欠陥、すなわちc-ストリングを返すことについてアドバイスします。

提供しているコード例では、c-stringの使用はまったく意味がありません。次のコードは、あなたが支障なくもしくは問題のあるコードをどうするつもり何を達成します：

std::string createCommand(int p1, int p2, int p3){ 
    std::stringstream sstm; 
    std::string comma = ","; 
    sstm << p1 << comma << p2 << comma << p3; 
    return sstm.str(); 
} 

int main() { 
    std::string command = createCommand(1,0,250); //I want to return "1,0,250" 
    std::cout << "char = " << command << "\n"; 
} 

あなたは、この設計はまだ良いです、代わりにC++のiostreamライブラリのprintfを使用に限定している場合でも：

std::string createCommand(int p1, int p2, int p3){ 
    std::stringstream sstm; 
    std::string comma = ","; 
    sstm << p1 << comma << p2 << comma << p3; 
    return sstm.str(); 
} 

int main() { 
    std::string command = createCommand(1,0,250); //I want to return "1,0,250" 
    printf("char = %s\n", command.c_str()); 
} 

古いCベースのライブラリに渡されるC文字列が必要な場合でも、この設計で十分です。つまり、mallocを使用する理由や、文字列自体を除いて基本となるC文字列表現とのインタフェースを使用する理由はありません。

Answer 2

ポインタを返す前に、文字列をstrcpyの形式でコピーする必要があります。 std::string及びその他の目的のために正常に動作します

const char * createCommand(int p1, int p2, int p3){ 
    stringstream sstm; 
    std::string comma = ","; 
    sstm << p1 << comma << p2 << comma << p3; 
    std::string str = sstm.str(); 
    const char *cstr = (const char *)malloc((str.length()+1) * sizeof (char)); 

    strcpy(cstr, str.c_str()); 
    return cstr;  
} 

Answer 3

代入演算子は、ポインタの上書きを持つことはできません。したがって、割り当ては

cstr = str.c_str(); 

は、割り当てたメモリをリークし、文字列のデータでポインタを置き換えます。さらに、関数が現在返すポインタは、関数を終了すると無効にされるメモリを指し、リークに加えて未定義の動作を生成します。

この問題を解決するには、std::strcpy(cstr, str.c_str());に電話してください。呼び出し結果にstd::freeを呼び出すことを忘れないでください。 編集：createCommand関数（WhozCraig、コメントありがとう）の戻り値の型からconstを削除する必要があります。

注：私は、これはあなたがnew[]を使用することが好適であることを知っていること、そしてあなたが関数からstd::stringを返すことができる場合は、上記のいずれかを行う必要がないであろうと、mallocを使用しての唯一の運動であると仮定。