0

我在 C++ 中有一个数据结构类,它带有对某个对象(可能很大)的访问器,并且我有使用这个访问器的 const 和非常量方法,所以我需要重载它。我正在寻找对下面代码的批评- 也许有一种方法可以完成更清洁的相同事情?

按照我的理解,在以下情况下,有两种方法可以在不复制访问器中的代码的情况下实现此目的,即方法 get()。我不确定这两种方法中的任何一种是否存在严重问题,我想在这里获得一些指导。

我喜欢方法A,因为:

  • 只有一个 const_cast
  • get() 方法的 const 版本返回一个副本
  • 非常量方法直接获取非常量引用

我不喜欢方法A,因为:

  • 非 const 方法 get() 仅通过合同为 const,(未经编译器检查)
  • 很难获得 const 引用,尽管并非不可能

我喜欢方法 B,因为:

  • const 方法 get() 的 const-ness 由编译器检查
  • 返回对象的副本由用户控制

我不喜欢方法 B,因为:

  • 需要两个难以阅读的 const_cast

这是这两种情况的(最小)示例代码。

/**
 * summary:
 * Two classes with an overloaded method which is
 * guaranteed (by contract) not to change any
 * internal part of the class. However, there is a
 * version of this method that will return a non-const
 * reference to an internal object, allowing the user
 * to modify it. Don't worry about why I would ever
 * want to do this, though if you want a real-world
 * example, think about std::vector<>::front()
 *
 * The difference between A and B can be summarized
 * as follows. In both cases, the second method merely
 * calls the first, wrapped with the needed
 * const_cast's
 *
 * struct A {
 *     int& get();
 *     int  get() const;
 * };
 *
 * struct B {
 *     const int& get() const;
 *           int& get();
 * };
 *
 **/

struct A
{
    int _val;

    A() : _val(7) {};

    // non-const reference returned here
    // by a non-const method
    int& get()
    {
        // maybe lots of calculations that you do not
        // wish to be duplicated in the const version
        // of this method...
        return _val;
    }

    // const version of get() this time returning
    // a copy of the object returned
    int get() const
    {
        // CONST-CAST!!?? SURE.
        return const_cast<A*>(this)->get();
    }

    // example of const method calling the
    // overloaded get() method
    int deep_get() const
    {
        // gets a copy and makes
        // a copy when returned
        // probably optimized away by compiler
        return this->get();
    }
};

struct B
{
    int _val;

    B() : _val(7) {};

    // const reference returned here
    // by a const method
    const int& get() const
    {
        // maybe lots of calculations that you do not
        // wish to be duplicated in the non-const
        // version of this method...
        return _val;
    }

    // non-const version of get() this time returning
    // a copy of the object returned
    int& get()
    {
        // CONST-CAST!? TWO OF THEM!!?? WHY NOT...
        return const_cast<int&>(const_cast<const B*>(this)->get());
    }

    // example of const method calling the
    // overloaded get() method
    int deep_get() const
    {
        // gets reference and makes
        // a copy when returned
        return this->get();
    }
};


int main()
{
    A a;
    a.get() = 8;  // call non-const method
    a.deep_get(); // indirectly call const method

    B b;
    b.get() = 8;  // call non-const method
    b.deep_get(); // indirectly call const method
}
4

1 回答 1

2

成员函数的常量应该基于以下问题来决定:成员函数是否在上下文中用于修改对象?(成员直接修改对象,或者返回指向内部数据的引用/指针,以便外部调用者可以修改对象)。如果是,则将其设为非 const,否则将其设为 const。

编译器将正确选择仅在 constness 上不同的重载。但是,在重载决议中不使用返回类型。此外,按价值/按参考返回应仅根据您要返回的预期成本和预期所有权来决定。幸运的是,C++11 通过提供移动语义让生活更轻松。这意味着您可以愉快地按值返回大型数据结构。仅当引用的对象比外部调用者寿命长时才通过引用返回。

在我看来,您int& get()应该重命名void set(int),并且您可以将您拆分int get() const为计算助手和适当的get()

class C
{
    int val_;
public:
    void modify()   { /* some expensive computation on val_ */ }
    int get() const { return val_; }
    void set(int v) { val_ = v_; }
};

或者,如果你想保持get()功能,你可以做

class D
{
int val_;
public:
    void modify()    { /* some expensive computation on val_ */ }
    int get() const  { modify(); return val_; }
    int& get()       { modify(); return val_; } // no const-cast needed
};
于 2013-06-05T21:39:00.547 回答