Question

Aka：是否存在“在取消初始化期间调用虚拟”的习语

我正在清理一些旧代码，需要修复在构造函数和析构函数中调用虚方法的情况。我不知道代码库，它是巨大的。重大改写不是一种选择。

构造函数的修复很简单。我将虚拟调用移动到静态Create模板，并使所有构造函数受到保护。然后，我需要做的就是编译并更改导致错误的所有位置以使用Create模板。回归的可能性最小。然而，对于析构函数，没有类似的东西。

你会如何解决这个问题？

示例代码

#include <iostream>

class Base
{
public:
    virtual ~Base()
    {
        DeInit();
    }
protected:
    virtual void DeInit()
    {
        std::cout << "Base" << std::endl;
    }
};

class Derived : public Base
{
protected:
    virtual void DeInit() override
    {
        std::cout << "Derived" << std::endl;
        Base::DeInit();
    }
};

int main()
{
    Derived d;
}

此代码不会调用Derived::DeInit（仅打印“Base”）。我需要解决这类问题。

Working example code

Answer 1

...
virtual Base::~Base()
{
    Base::DeInit();
}
...

...
Derived::~Derived()
{
    // de-initialization code
    // do not call Derived::DeInit() here as otherwise Base::DeInit()
    // will be called two times
}
...

并在发现它们时从析构函数中清除虚函数调用。

Answer 2

这非常棘手，因为在离开范围时会自动调用析构函数，无论是通过正常流，break，continue，return还是throw。这也是你不能将参数传递给析构函数的原因。

直接的解决方案是从Derived::DeInit致电Derived::~Derived。这还有额外的好处，即仍有Derived成员可用。

另一种方法是创建自己的智能指针类，在T::DeInit之前调用T::~T。要防止这被绕过，请从Create。

返回此智能指针

Answer 3

你不需要有虚拟的DeInit乐趣。

    #include <iostream>

    class Base
    {
    public:
        virtual ~Base()
        {
            DeInit(); //this calls Base version
        }
    protected:
        void DeInit()
        {
            std::cout << "Base" << std::endl;
        }
    };

    class Derived : public Base
    {

    public:
         ~Derived()
        {
            DeInit(); //this calls Derived version
        }
    protected:
        void DeInit() 
        {
        std::cout << "Derived" << std::endl;
        }
    };

    int main()
    {
        Derived d;
    }

输出：派生基

这就是你想要的吗？

Answer 4

一个灵感来自MSalters第二个想法的解决方案。

此解决方案仅需要更改Base类和Derived类的实例化。任何Derived实施都不需要进行任何更改。

#include <iostream>
#include <memory>

class Base
{
private:
    template <class T>
    class WithAutoDeInit : public T
    {
    public:
        virtual ~WithAutoDeInit() override
        {
            T::DeInit();
        }
    };

public:
    template <class T>
    static std::unique_ptr<typename std::enable_if<std::is_base_of<Base, T>::value, WithAutoDeInit<T>>::type> Create()
    {
        return std::make_unique<WithAutoDeInit<T>>();
    }

    virtual ~Base() = default;

protected:
    virtual void DeInit()
    {
        std::cout << "Base" << std::endl;
    }
};

class Derived : public Base
{
protected:
    virtual void DeInit() override
    {
        std::cout << "Derived" << std::endl;
        Base::DeInit();
    }
};

int main()
{
    Base::Create<Derived>();
}

Working example code

这不是一个强大的解决方案。您仍然可以直接创建Derived的实例。如果您使用受保护的构造函数更新所有Derived类，那么不知情的开发人员仍然可以创建一个新类，忘记使其构造函数受到保护。我想知道这是否可以通过战略位置的某种断言来强制执行？

static_assert(std::is_constructible<Derived>::value, "Derived class is constructable");

顺便说一句：我最后选择重写代码。我认为它是可管理的，并且生成的代码将会更简单（因此更好）。

在去初始化成语期间是否存在任何动态绑定

4 个答案: