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

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

c++

Chui Tey · 技术社区 · 5 年前

我从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 回复 | 直到 5 年前

srilakshmikanthanp 5 年前

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;
}

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

谢谢。

推荐文章

AstralHex · 矩阵乘法代码工作不正常

1 年前

Baba Dan Constantin · SSE4.1在矩阵4x4乘法上比SSE3慢?

1 年前

Giogre · 为包含许多数值字段的简单“struct”重载比较运算符

1 年前

einpoklum · 定义一个并不真正提供now()函数的std::chrono Clock是“合法的”吗?

1 年前

Fishie · 作为类成员的智能指针是否仍然自动释放?[关闭]

1 年前

Die4Toast · 递归调用成员箭头运算符->

1 年前

Angle.Bracket · 如何用C++将UTF-8文件名写入MS Windows控制台?

1 年前

Anka HanÄ±m · 关于结构和动态数组地址的问题

1 年前

Adam Barnes · 我如何定义一个基于constexpr函数返回值进行限制的概念?

1 年前

user2138149 · 为什么我不能获取包含多个元素的结构体中某些元素的地址?[副本]

1 年前