如何编写 Const 和 Mutable 重载代码？

Question

我的代码中似乎经常出现这种模式，除了参数/返回的常量性之外，两个函数执行相同的任务。

int& myClass::getData()
{
        return data;
}

// called for const objects 
const int& myData::getData() const
{
        return data;
}

这冒犯了我的 DRY 观念。对于单行代码来说这不是问题，但是随着 getData() 变大，就会出现明显的重复。

我知道为什么我需要这两种方法，但觉得应该有更好的方法来实现它。是否有一个模板技巧可以提供帮助，或者我应该有一种方法来调用另一种方法来根据需要来回转换常量？

添加：作为一个更真实的示例，下面是典型 STL vector::at() 实现的示例：

const_reference at(size_type _Off) const
    {   // subscript nonmutable sequence with checking
    if (size() <= _Off)
        _Xran();
    return (*(begin() + _Off));
    }

reference at(size_type _Off)
    {   // subscript mutable sequence with checking
    if (size() <= _Off)
        _Xran();
    return (*(begin() + _Off));
    }

Answer 1

使用以下技巧（我最初从 Scott Meyers 的书 Effective C++ 中得到）：

int& myClass::getData()
{
    // This is safe because we know from out here
    // that the return value isn't really const
    return const_cast<int&>(const_cast<const myClass&>(*this).getData());
}

const int& myData::getData() const
{
    return data;
}

显然，对于像这样的短函数，您可能会发现复制代码更容易，但是当您具有更长的函数（例如 vector::at 或者它可能会发生很多变化。

Answer 2

提供如下功能

int& myClass::getData()
{
    return data;
}

可能是错误的 - 您不妨公开“数据”。第二个应该写成：

int myData::getData() const
{
   return data;
}

Answer 3

您引用的标准库函数重复代码的原因是标准库函数需要尽可能高性能。在那里调用另一个函数（正如其他人正确建议的那样）可能会搞砸。但您的代码可能受到不太严格的性能限制，因此您应该尽可能使用通用代码。