C++関数はNULL文字列リファレンスを返します

Question

関数の戻り値の型がstd::string&で、関数内で条件が一致しない場合NULLを返すにはどうすればいいですか？string&

std::string& SomeClass::getSomething() 
{ 
    if(){...} 
    else if(){...} 
    // return a NULL 
} 

Answer 1

C++の参照がnullにすることはできません。

データメンバのように、関数呼び出しのスコープに関連付けられていないオブジェクトへの参照を返す場合は、生ポインタを安全に返すことができます。ポインタをにすることをお勧めします。

std::string const* foo::bar() const { 
    if (condition) { 
     return &some_data_member; 
    } else { 
     return nullptr; 
    } 
} 

ない場合は、最善の解決策は、boost::optional（またはC++ 17でstd::optional）のようなラッパー型を使用することです。これにより、オプションのオブジェクトを値で返すことができます（これはパフォーマンスが向上する可能性があります）。self-documentingです。

std::optional<std::string> foo::bar() const { 
    if (condition) { 
     return "hello, world"; 
    } else { 
     return std::nullopt; 
    } 
} 

また、ポインタを返すこともできますが、nullでもかまいません。しかし、生ポインタを返すことで、動的に割り当てられた文字列を誰が削除するのかが問われます。この場合、所有者が呼び出し側に明示的に渡されるため、std::unique_ptrを返すことが最適なオプションになります。

std::unique_ptr<std::string> foo::bar() const { 
    if (condition) { 
     return std::make_unique<std::string>("hello, world"); 
    } else { 
     return nullptr; 
    } 
} 

さらに単純なケースでは、空の文字列を返すこともできます。そして正直なところ、これは私の好みのアプローチです（KISS）。

std::string foo::bar() const { 
    if (condition) { 
     return "hello, world"; 
    } else { 
     return ""; 
    } 
} 

Answer 2

がjuanchopanzaによってコメントで述べたように、あなたがすることはできません。

NULLをテストする必要がある場合は、スマートポインタを使用してアプローチを再考することができます。例えばstd::shared_ptr<std::string>：

std::shared_ptr<std::string> SomeClass::getSomething() 
{ 
    std::shared_ptr<std::string> stringPtr; 
    if(){ 
     //... 
     stringPtr = std::make_shared<std::string>("Whatever string goes here"); 
    } 
    else if(){ 
     //... 
     stringPtr = std::make_shared<std::string>("The other string..."); 
    } 
    return stringPtr; 
} 

次に、あなただけのブール値への暗黙的な変換とstd::shared_ptrをテストすることができます：

auto strReturnedPtr = someClassObj.getSomething(); 
if (strReturnedPtr) 
{ 
    // Do stuff 
} 

Answer 3

とにかくあなたはNULLが、nullptrを返しません。 また、関数からの参照を返すことに注意し、参照が有効な生存オブジェクトを参照することを確認する必要があります。ローカルオブジェクトへの参照を返すことが間違っています。

nullptrnullptrはポインタであり、string&は参考資料であるため、返品することはできません。

あなたのオプションは次のとおりです。

1. は、オプションのクラス（boost::optional、などのようなものを使用して例外
2. を投げます私は、関数が失敗する可能性が強い可能性がある知っていれば）

個人的に、私は参照型の引数に結果を渡すと失敗

bool SomeClass::getSomething(std::string& result) 
{ 
    if(){result = "success 1"; return true; } 
    else if(){result = "success 2"; return true; } 
    return false. 
} 

Answer 4

空を返すの成功を示すためにboolを返します文字列 ''はここで意味をなさないかもしれません。

あなたのコードの断片からは、クラスのカバーの下を見て、クラスそのものが作成しなければならないという決定を下すかどうか疑問に思うでしょう。 Martin Fowlerの記事「Tell, don't Ask」も参照してください。これは、The Pragmatic Programmersのオリジナル記事を参照してください。