代码之家 › 专栏 › 技术社区 › Chui Tey

嘲笑复制构造函数的C++类

c++

Chui Tey · 技术社区 · 4 年前

我从C++到C++,当我尝试构造测试时,复制构造函数让我很难模仿我习惯的方式。

#include <iostream>

using namespace std;

class DB {
public:
    virtual int getValue() { return 42; }
    DB(const DB& source) { }
    DB() { }
};

class DB_Mock: public DB {
public:
    virtual int getValue() { return 999; }
};

class A {
public:
    A(DB db) {
        m_db = db;
    }
    void doIt() {
        cout << "Result:" << m_db.getValue() << endl;
    }
private:
    DB m_db;
};

int main() {
    /* prints 42 as expected */
    DB db;
    A a(db);
    a.doIt();

    /* prints 42, expected 999 */
    DB_Mock db_mock;
    A a2(db_mock);
    a2.doIt();
    return 0;
}

你如何处理这样的问题?

1 回复 | 直到 4 年前

srilakshmikanthanp 4 年前

m_db不是一个引用或指针,c++多态性只与基类指针或引用一起工作。

    A(DB db) {
        m_db = db; // m_db just sliced copy of base part of db.
    }

将此更改为引用

class A
{
public:
    explicit A(DB &db) : m_db(db) // it is initializing not copy
    {
    }
    void doIt()
    {
        std::cout << "Result:" << m_db.getValue() << std::endl;
    }

private:
    DB &m_db;  // Now m_db is reference
};

Compile and Run Here

Constructors and member initializer lists
Smart pointer
override
看到了吗 virtual table 要了解vu ptr

编辑:

作为 Pete Becker

A make()
{
    DB db;
    A a(db);
    return a;
}

int main()
{
   const A &a = make();
   // Wrong db is destroyed (dangling reference).
   a.doIt(); // assume doIt(), getValue() are declared as const
   return 0;
}

上面的代码是错误的(未定义的行为),所以我建议使用智能指针(堆存储)来避免这个问题。

谢谢。

推荐文章

patrick · 从boost::绑定1和2占位符运行make失败。。。?

1 年前

rookie · 检查函数模板的所有参数包参数是否属于int

1 年前

ivaigult · -W转换和隐式字符串到布尔类型转换

1 年前

David Bien · declval<_Xp(&)()>()()-这在下面的上下文中意味着什么?

1 年前

khallad sayed · 在%上,C++表达式必须具有整型或无范围枚举类型

1 年前

rainer · 后台插入程序的初始化

1 年前

Community wiki · 以理智、安全和高效的方式复制文件

1 年前

Shefali Kanaujia · 对C中向量的向量进行排序++

1 年前

Prakritish Ghosh · 现在显示正确输出的函数中的Infix到Postfix错误[重复]

1 年前

Ma Joonyoung · 粗粒度和细粒度链表的时间比较

1 年前