c++ - 関数から値を返すときのc_str（）の使用法の違い

Question

次の2つの機能がある場合

std::string foo1()
{
    std::string temp;
    ...
    return temp;
}

と

const char* foo2()
{
    std::string temp;
    ...
    return temp.c_str();
}

constchar*を入力として受け取る関数。

void bar(const char* input) { ... }

どちらが安全ですか：

bar(foo1().c_str());

また

bar(foo2());

文字列を入力としてbarに渡すだけで、どちらの関数の戻り値も気にしない場合、fooそれは実際に重要ですか？

score 8 · Accepted Answer

const char* foo2()
{
    std::string temp;
    ...
    return temp.c_str();
}

foo2()安全ではありませんconst char*。関数が戻るときにガベージを指すローカル変数にを返します。

foo1オブジェクトを返すには、C++で安全で慣用的な方法であるwhichを使用するだけです。NRVOがキックインする可能性があります。これにより、戻ったときにtempのコピーが削除されfoo1ます。

std::string foo1()
{
    std::string temp;
    ...
    return temp;
}

score 6 · Accepted Answer

const char* foo2()
{
    std::string temp;
    ...
    return temp.c_str();
}

破壊されるので、まったく安全ではないtempので、ダングリングポインタを返します。

score 2 · Accepted Answer

bar（foo2（））; foo2が戻ると、temp std :: stringが破棄され、c_str（）によって返されるポインタが無効な場所を指しているためです。

score 2 · Accepted Answer

c_strの文字配列は、元の文字列オブジェクトが破棄されない限り有効です。これは以前に尋ねられました。。関数の最後で、tempは破棄されます。これはfoo2()、無効なポインターを返すことを意味します。

score 1 · Accepted Answer

から戻るときにが破棄されるため、によって返されたポインタが無効になるため、foo2バージョンは安全ではありません。のコピーを返すため、より安全です。c_str()std::string tempfoo2foo1std::string temp

score 0 · Accepted Answer

foo2関数を終了するときに破棄されるローカル変数の内部へのポインターを返します。（それは良くないね）

6 に答える 6