求教个 C++ Get 函数怎么写的问题

### 代码

```c++
#include<iostream>
#include <cstdint>
#include <unordered_map>

enum struct Status {
kOk = 0,
};

struct Student {
std::string name;
std::size_t age;
};

class Table {
public:
Table() {
      this->map_.insert(std::make_pair("w1", Student("li", 23)));
      this->map_.insert(std::make_pair("s2", Student("zhao", 18)));
}

Status Get(const std::string& key, Student* value) {
      *value = this->map_[key];
      return Status::kOk;
}

Student Get(const std::string& key) { return this->map_[key]; }

private:
std::unordered_map<std::string, Student> map_;
};

int main(int argc, char* argv[]) {
Table table;

Student stu1;
const Status& status = table.Get("w1", &stu1);
std::cout << stu1.name << ":" << stu1.age << std::endl;

const Student& stu2 = table.Get("s2");
std::cout << stu2.name << ":" << stu2.age << std::endl;
return 0;
}
```

### 结果

```c++
li:23
zhao:18
```

### 问题

```c++
1. Status Get(const std::string& key, Student* value);
2. Student Get(const std::string& key);
```

在 Java/Python 等语言中，个人更喜欢第 2 种写法；但是 C++ 中，一些项目更倾向于第 1 种写法，为啥呢？这样有什么好处吗？

fgwmlhdkkkw

c++的逻辑上“谁申请的谁释放”。

sagaxu

两个区别，一个是内存管理，一个是错误码。

Java/Python 没有那么细致的内存管理，比如 C++中，可以栈上分配 Student ，并且复用

tool2dx

我也喜欢第二种写法。第一种写法是偏向编译效率，以前 C++还没有&&和 move ，所以会多一次临时拷贝。

但其实不写游戏，不考虑效率，第二种更便于理解。

fpk5

C++有一个问题就是怎么区分错误和正常返回，不像 Java 和 Python ，C++的 exception 不一定 work as expected 。你的例子里的 unordered_map::operator[]是有可能失败的（比如内存不足），你的 caller 不一定能正确处理这种 exception 。

第一种写法更接近于 C 风格，返回值用于确定是否有错误，传入一个参数用于接收真正的返回值。很多公司会自己规定使用哪种写法，比如 Google 内部 C++规范实现了一个 StatusOr<T>的类型可以用一个返回值同时表示是否错误和实际返回值。

tyzandhr

要是 std23 ，可以用 expected<Student>

iceheart

Get 语义不太清晰，个人喜欢 fetch
bool fetch(const string &, valuetype &);
或者:
valuetype *fetch(const string &);

henix

首先，这两种写法语义上并不等价，第一种写法多出一个 Status ，第二种写法要加上 Status 的话得返回一个 std::tuple<Status, Student> 或 std::variant<Status, Student>

两者的区别在于，第一种写法，Student 占用的内存由调用方分配，适用于对性能要求较高的场景；第二种写法，每调用一次 Get ，都会为返回的 Student 分配内存（尤其是 Student 包含了一个 string ，string 是动态分配），好处是用起来更方便。

考虑在一个循环中调用 Get ，如果用第一种写法，可以在循环外初始化 Student 并且复用 Student ，从而减少内存分配次数：

Student stu;
for (...) {
Get(key, &stu);
}

wnpllrzodiac

第二种有临时变量效率不高，如果用引用，又有失效问题，当这个类释放后，get 传递的引用变无效了

MoYi123

一般用
const Student& Get(const std::string& key) const { return this->map_.at(key); }
这样拷贝构造发生在外部.

如果有需要再加上 Student& Get(const std::string& key) const { return this->map_.at(key); }
可以支持 move

Status Get(const std::string& key, Student* value); 这种写法是 C 语言的风格. 不建议用