哪个是更好的获取成员方法？

Question

我有一个类的成员类型为std:vector

  private:
     std::vector<int> myVector;

我已经创建了 Get 方法来访问myVector

1. const std::vector<int>   GetMyVector() const;
2. const void   GetMyVector(std::vector<int>& vec) const;

分别实现如下：

1. const std::vector<int> MyClass::GetMyVector() const
   {
       return  myVector;
   }

2. const void MyClass::GetMyVector(std::vector<int>& vec) const
   {
       vec =  myVector;
   }

两种 Get 方法中哪一种更好，为什么？

Answer 1

我更喜欢选项 3：

const std::vector<int>& MyClass::GetMyVector() const
{
    return  myVector;
}

您的选项 1 返回了myVector 的副本。这将返回对类成员的 const（只读）引用。

Answer 2

为什么要返回向量？

int MyClass::GetItem(const size_t index) const
{
    return myVector[index];
}

Answer 3

根据经验，总是尝试返回类成员的const reference。 所以使用const std::vector<int> & MyClass::GetMyVector() const

Answer 4

如果您通过复制返回一个对象，那么将其声明为 const 是没有任何意义的。所以不是

const std::vector<int> MyClass::GetMyVector() const
{
    return  myVector;
}

我会写

std::vector<int> MyClass::GetMyVector() const
{
    return  myVector;
}

第二个声明比第一个更糟糕，因为它只会让用户感到困惑。不清楚是类的对应数据成员赋值给了参数，还是这个方法对参数做了一些改动，没有给参数赋值对应的数据成员。

所以考虑到您建议的变体，我会选择声明

std::vector<int> MyClass::GetMyVector() const
{
    return  myVector;
}

Answer 5

首先，当您从成员函数返回类的私有成员时，您会暴露您的实现，这通常是糟糕的设计。查看@JoachimPileborg 的解决方案，了解如何避免这种情况的示例。

如果你想返回一个副本，那么你应该按值返回。

如果要返回对对象的引用，则按引用返回。 但是，请记住，当对象被破坏时，您最终会得到一个悬空引用，例如

class Foo {
public:
    std::vector<int>& getVec() {
        return myVec;
    }
private:
    std::vector<int> myVec;
};

int main() {
    Foo* f = new Foo();
    std::vector<int>& myRef = f->getVec();
    delete f;

    std::cout << myRef.size(); // The demons come! Dangling reference!
}

因此，返回副本而不是引用通常是正确的做法。