Question

这里有一个简单的问题，我的代码中有几个类，但是其中只有一个表现出这个问题而且我不能为我的生活找出原因。当我创建一个类的实例时，直接调用析构函数，但实际上该类的实例似乎没有被删除。

如果析构函数中没有影响该类实例的delete []操作，也许我可以忍受。

我读到了关于“三个规则”的某个地方，所以试图看看我错过了什么。我已经有一个默认构造函数和一个用户定义的构造函数。然后我添加了我认为称为复制构造函数的东西，如下所示：

MyClass::MyClass(const MyClass &duplicate)
{
    variable1 = duplicate.variable1;
    variable2 = duplicate.variable2;
    // etc
}

我在这里错过了什么可能导致这个问题？

编辑：请求的代码。我已经将所有内容重命名，以便一目了然（此代码仍然可以解决问题）。首先，头文件MyClass.h：

#ifndef MYCLASS_H
#define MYCLASS_H
#ifndef UNICODE
#define UNICODE
#endif
#include <string>

class MyClass
{
public:
    MyClass();
    MyClass::MyClass(const MyClass &);
    MyClass(int, std::wstring inputWord, int);
    ~MyClass();

    int intOne;
    int intTwo;
};
#endif

下一个MyClass.cpp：

#include "MyClass.h"
#include <Windows.h>


MyClass::MyClass(const MyClass &duplicate)
{
    intOne = duplicate.intOne;
    intTwo = duplicate.intTwo;
}
MyClass::MyClass()
{
}

MyClass::~MyClass()
{
    MessageBox(NULL, TEXT("TEST"), TEXT("TEST"),0);
}

MyClass::MyClass(int intOneInput, std::wstring stringInput, int intTwoInput)
{
    intOne = intOneInput;
    intTwo = intTwoInput;
}

最后我是如何创建我的对象的：

MyClass test(0, TEXT("TEST"), 0);

[从op的评论中复制]

实际上，抓住我的最后一条评论，解构函数不会被该特定行调用（直到它超出范围），执行的行words.push_back(MyClass(0, TEXT("TEST"), 0));被声明为std::vector<MyClass> words

Answer 1

当对象被销毁时，应该调用析构函数。如果使用new创建对象，则在对象上调用delete时将调用析构函数。否则，当它超出其范围时应该调用它。您可以在其析构函数中设置断点，并查看callstack以检查调用析构函数的内容。希望它有所帮助。

[更新] 尝试为所有ctor和dtor添加以下printf，以确保您不会与临时创建的对象混淆。

printf("ctor %p\n", this); // in constructors

printf("dtor %p\n", this); // in destructor

[更新]

words.push_back(MyClass(0, TEXT("TEST"), 0));

这会创建一个临时对象，因为stl容器（如vector）总是“复制”要存储的东西。（除非发生“移动”。我不想在这里开始解释“移动”和右移。）

Answer 2

此：

MyClass test(0, TEXT("TEST"), 0);

声明一个自动对象当对象超出范围时，该对象将被销毁。

{
   MyClass test(0, TEXT("TEST"), 0);   // constructed here
}                                      // destructed here.

Answer 3

一个可能的问题（但我们需要看到更多）是你的复制构造函数执行卷影副本，你构建了一个临时的对象复制构造。当临时销毁时，临时存放的内存被释放，新对象现在在已删除的内存上有指针。

一种解决方案是：复制构建时执行深层复制 http://www.learncpp.com/cpp-tutorial/912-shallow-vs-deep-copying/

另一个是：使用智能指针而不是原始指针 What is a smart pointer and when should I use one?

Answer 4

我知道这是一个旧线程，但是如果将来可以帮助某个人，那么在未定义复制构造函数的情况下，在测试本文中的代码时，我会看到相同的问题：https://www.viva64.com/en/w/v690/：

1 raise                                    0x7ffff4e320e0
2 abort                                    0x7ffff4e336c1
3 __libc_message                           0x7ffff4e75427
4 malloc_printerr                          0x7ffff4e7bc43
5 MyArray::Clear         mainwindow.cpp 50 0x4204a5
6 MyArray::~MyArray      mainwindow.cpp 53 0x4204e2
7 MainWindow::MainWindow mainwindow.cpp 73 0x41416b
8 main                   main.cpp       10 0x40c372

我在您的代码中看到复制构造函数的头定义为：

MyClass::MyClass(const MyClass &);

而不是简单地：

MyClass(const MyClass &);

我不确定您的副本构造函数是否正确声明和使用。

无论如何，在当前的GCC版本中，都不会发生这种特殊情况，因为由于成员'MyArray [-fpermissive]上的“额外限定'MyArray ::'”错误而无法通过，但是它可以帮助那些使用复制构造函数的人丢失。

C ++类析构函数在构造函数后直接调用

4 个答案: