ベクトル内のオブジェクトへのポインタを見つけて返す

Question

C++の新機能、どこが間違っているのかわからない。まだptrsの周りに私の頭を包んでいる。

私はプロファイルオブジェクトのベクトルを持っています。私はそのプロパティの1つに基づいて特定のオブジェクトを探したい。それから私はそのオブジェクトを返したい。オブジェクトのコピーや何かを作成したくないので、特定のオブジェクトへのポインタを返す必要があると思いますか？

Profile* profileManager::getProfile(const std::string& name) 
{ 
    std::vector<Profile>::iterator it = std::find_if(allProfiles.begin(), allProfiles.end(), [name](Profile &p) { return p.getName() == name; }); 
    return it != allProfiles.end() ? *it : nullptr; 
} 

私は修正する方法がわからないことがいくつか間違ってあります：

は、ここに私のコードです。一致が見つからない場合はnullを返し、呼び出し元に処理させる場合はどうすればいいですか？私は現時点で互換性のないタイプを取得していると私はビット周りにそれを変更しようとしたが、運がなかった。それは正しいタイプではないとも言います。見つかった場合、どのようにプロファイルへのポインタを返すようにすればよいですか？

ありがとうございました。ありがとう。

Answer 1

return it != allProfiles.end() ? *it : nullptr; 

これはほとんどあなたが望むものです。 *itのタイプはProfileです。ポインタを取得するには、アドレス：&*it

また、いくつかの変更をお勧めします。

1. getProfile一定の方法を確認します。を変更するものではありません。
   一定のオーバーロードをgetProfileにすることは良いアイデアですが、変更可能なオーバーロードも維持することをお勧めします。

2. ラムダパラメータを定数参照(const Profile &p)にしてください。これは上記の変更を行う場合にも必要です。

3. 値でnameをキャプチャする必要はありません。参照でキャプチャすると、コピーが保存されます。 [&name]