如何避免析构函数调用两次

zhengyj

mathgl.h

1. #include <mgl/mgl_c.h>
   
2. 
   
3. class mglGraph {
   
4.     HMGL gr;
   
5. public:
   
6.     mglGraph(int kind, int width, int height);
   
7.     ~mglGraph();
   
8. };
   

复制代码

mathgl.cpp

1. #include "mathgl.h"
   
2. #include <iostream>
   
3. 
   
4. using namespace std;
   
5. 
   
6. mglGraph::mglGraph(int kind, int width, int height) {
   
7.     std::cout << "new" << std::endl;
   
8.     if (kind == 0)
   
9.         gr = mgl_create_graph_zb(width, height);
   
10.     else if (kind == 1)
    
11.         gr = mgl_create_graph_ps(width, height);
    
12. }
    
13. 
    
14. mglGraph::~mglGraph() {
    
15.     std::cout << "delete" << std::endl;
    
16.     mgl_delete_graph(gr);
    
17. }
    

复制代码

exam.cpp

1. #include <iostream>
   
2. #include "mathgl.h"
   
3. 
   
4. using namespace std;
   
5. 
   
6. int main() {
   
7.     mglGraph w(1,1,1);
   
8.     return 0;
   
9. }
   

复制代码

运行输出如下：
new
delete
delete
Segmentation fault
为什么析构函数调用了两次，如何让它只调用一次？谢谢！

hbxfzzq_hm

回复 1# zhengyj


    这点儿代码，谁能看出来？

cuererer

好像OIOIC能自动避免构造函数和析构函数被调用两次，保证只调用一次。不是为OIOIC做广告呀。。。。

drangon

无代码无真相。。。。

在构造和析构函数加个断点，看看什么时候进到这两个地方来罗

胡乱猜一下，某个地方调用了一个拷贝构造函数（默认生成），所以多一个析构。

digdeep126

谁知道你mgl_delete_graph函数是怎么实现的。

Aquester

有代码没贴出来，从现有的看不出来，正常应当是不会有两次的。由于第一次delete，gr已经变成无效内存了，第二次delete就coredump了。

bs

智能指针？

digdeep126

回复 1# zhengyj


    你将mathgl.cpp的第16行mgl_delete_graph(gr);注释掉，再编译运行一下，不就知道了到底是哪里出了错误了。

crjwlaq

调试一下