C++最佳实践:返回引用与对象

Mik*_*owe 42 c++

我正在尝试学习C++,并试图理解返回的对象.我似乎看到了两种方法,并且需要了解什么是最佳实践.

选项1:

QList<Weight *> ret;
Weight *weight = new Weight(cname, "Weight");
ret.append(weight);
ret.append(c);
return &ret;
Run Code Online (Sandbox Code Playgroud)

选项2:

QList<Weight *> *ret = new QList();
Weight *weight = new Weight(cname, "Weight");
ret->append(weight);
ret->append(c);
return ret;
Run Code Online (Sandbox Code Playgroud)

(当然,我也可能不理解这一点).

哪种方式被认为是最佳实践,应该遵循?

Pot*_*ter 48

选项1有缺陷.声明对象时

QList<Weight *> ret;
Run Code Online (Sandbox Code Playgroud)

它只存在于当地范围内.函数退出时会被销毁.但是,您可以使用它

return ret; // no "&"
Run Code Online (Sandbox Code Playgroud)

现在,虽然ret被销毁,但是首先制作副本并将其传递给调用者.

这是通常优选的方法.事实上,复制和破坏操作(其中无所作为,真的)通常省略掉,或优化掉了,你会得到一个快速的,优雅的程序.

选项2有效,但是你有一个指向堆的指针.查看C++的一种方法是该语言的目的是避免手动内存管理等.有时您确实希望管理堆上的对象,但选项1仍然允许:

QList<Weight *> *myList = new QList<Weight *>( getWeights() );
Run Code Online (Sandbox Code Playgroud)

getWeights你的示例函数在哪里 (在这种情况下,您可能必须定义一个复制构造函数QList::QList( QList const & ),但与前面的示例一样,它可能不会被调用.)

同样,您可能应该避免使用指针列表.该列表应直接存储对象.尝试使用std::list...语言功能的实践比实践数据结构更重要.


mis*_*tor 7

使用选项#1略有变化; 而不是返回对本地创建的对象的引用,返回其副本.

return ret;

大多数C++编译器执行返回值优化(RVO)以优化创建的临时对象以保存函数的返回值.


Mic*_*yan 5

通常,您不应该返回引用或指针.而是返回对象的副本或返回拥有该对象的智能指针类.通常,使用静态存储分配,除非在运行时大小不同或对象的生命周期要求使用动态存储分配进行分配.

正如已经指出的那样,通过引用返回的示例返回对不再存在的对象的引用(因为它已超出范围),因此调用未定义的行为.这是你永远不应该返回引用的原因.您永远不应该返回原始指针,因为所有权不明确.

还应该注意的是,由于返回值优化(RVO),按值返回非常便宜,并且由于引入了右值引用而很快会更便宜.