[已解决]C++ 堆区类析构报错

Eden00

补充，似乎问题出在继承，改变继承顺序后，就变成delete c0无法析构，是否有什么问题？



如下代码，没有报错，但是每次运行到delete v0;（157行）的时候，会报错Exception has occurred.Unknown signal，请问是为啥？
谢谢！

1. #include <iostream>
   
2. 
   
3. #include <string>
   
4. 
   
5. #include <cmath>
   
6. 
   
7. #include <stdlib.h>
   
8. 
   
9. #include <ctime>
   
10. 
    
11. using namespace std;
    
12. 
    
13. class CPU
    
14. {
    
15. 
    
16. public:
    
17.     string cname;
    
18. 
    
19.     virtual void calculate() = 0;
    
20.     ~CPU()
    
21.     {
    
22.         cout << cname << " CPU is destructed" << endl;
    
23.     }
    
24. };
    
25. 
    
26. class VideoCard
    
27. {
    
28. 
    
29. public:
    
30.     string vname;
    
31. 
    
32.     virtual void display() = 0;
    
33.     ~VideoCard()
    
34.     {
    
35.         cout << vname << " video card is destructed" << endl;
    
36.     }
    
37. };
    
38. 
    
39. class Memory
    
40. {
    
41. 
    
42. public:
    
43.     string mname;
    
44. 
    
45.     virtual void store() = 0;
    
46.     ~Memory()
    
47.     {
    
48.         cout << mname << " memory is destructed" << endl;
    
49.     }
    
50. };
    
51. 
    
52. class Supplier : public CPU, public VideoCard, public Memory
    
53. {
    
54. 
    
55.     string name;
    
56. 
    
57.     void calculate()
    
58.     {
    
59. 
    
60.         cout << name << " CPU is calculating" << endl;
    
61.     }
    
62. 
    
63.     void store()
    
64.     {
    
65. 
    
66.         cout << name << " memory is stoing" << endl;
    
67.     }
    
68. 
    
69.     void display()
    
70.     {
    
71. 
    
72.         cout << name << " video card is displaying" << endl;
    
73.     }
    
74. 
    
75. public:
    
76.     Supplier()
    
77.     {
    
78. 
    
79.         cout << "Supplier is structed without argument" << endl;
    
80.     }
    
81. 
    
82.     Supplier(string n) : name(n)
    
83.     {
    
84. 
    
85.         cout << "Supplier is structed with name as " << n << endl;
    
86.     }
    
87. 
    
88.     string SetName(string n)
    
89.     {
    
90. 
    
91.         cout << "Supplier name is set by Class Supplier as " << n << endl;
    
92. 
    
93.         name = n;
    
94. 
    
95.         return name;
    
96.     }
    
97. 
    
98.     string GetName()
    
99.     {
    
100. 
     
101.         return name;
     
102.     }
     
103. 
     
104.     ~Supplier()
     
105.     {
     
106. 
     
107.         cout << name << " supplier is destructed" << endl;
     
108.     }
     
109. };
     
110. 
     
111. class Computer
     
112. {
     
113. 
     
114.     CPU *c0;
     
115. 
     
116.     VideoCard *v0;
     
117. 
     
118.     Memory *m0;
     
119. 
     
120. public:
     
121.     Computer(Supplier c, Supplier v, Supplier m)
     
122.     {
     
123. 
     
124.         c0 = new Supplier(c);
     
125. 
     
126.         v0 = new Supplier(v);
     
127. 
     
128.         m0 = new Supplier(m);
     
129. 
     
130.         c0->cname = c.GetName();
     
131. 
     
132.         v0->vname = v.GetName();
     
133. 
     
134.         m0->mname = m.GetName();
     
135. 
     
136.         cout << "This computer is assembled with " << c0->cname << " CPU, " << v0->vname
     
137. 
     
138.              << " video card, " << m0->mname << " memory" << endl;
     
139.     }
     
140. 
     
141.     ~Computer()
     
142.     {
     
143. 
     
144.       /* if (c0 != NULL)
     
145.         {
     
146.             cout << c0->cname << endl
     
147.                  << c0 << endl;
     
148.             delete c0;
     
149. 
     
150.             c0 = NULL;
     
151.         }*/
     
152. 
     
153.         if (v0 != NULL)
     
154.         {
     
155.             cout << v0->vname << endl
     
156.                  << v0 << endl;
     
157.             delete v0;
     
158. 
     
159.             v0 = NULL;
     
160.         }
     
161. 
     
162.         if (m0 != NULL)
     
163.         {
     
164. 
     
165.             delete m0;
     
166. 
     
167.             m0 = NULL;
     
168.         }
     
169.     }
     
170. 
     
171.     void work()
     
172.     {
     
173. 
     
174.         c0->calculate();
     
175. 
     
176.         v0->display();
     
177. 
     
178.         m0->store();
     
179.     }
     
180. };
     
181. 
     
182. void fun0()
     
183. {
     
184. 
     
185.     Supplier q0("Qualcom");
     
186. 
     
187.     Supplier i0;
     
188. 
     
189.     i0.SetName("Intel");
     
190. 
     
191.     Supplier a0("AMD");
     
192. 
     
193.     Computer c0(q0, i0, a0);
     
194. 
     
195.     c0.work();
     
196. }
     
197. 
     
198. int main()
     
199. {
     
200. 
     
201.     fun0();
     
202. 
     
203.     system("pause");
     
204. 
     
205.     return 1;
     
206. }

复制代码

最佳答案

月排行榜 / 总排行榜

isdkz

2023-4-20 21:27:07

Eden00 发表于 2023-4-20 22:32

经过我重新审查代码，找到了问题所在。问题出在析构函数的调用顺序。首先，`Computer`的析构函数被调用，接着调用`Supplier`的析构函数。但是在`Supplier`的析构函数中，由于它继承了`CPU`、`VideoCard`和`Memory`类，所以它们的析构函数也会被调用。因此，在`Computer`的析构函数中，我们需要确保在删除`c0`、`v0`和`m0`之前调用它们的析构函数。

为了解决这个问题，我们可以在`Supplier`类中添加一个名为`cleanUp`的函数，该函数在`Computer`的析构函数中调用，确保调用`CPU`、`VideoCard`和`Memory`的析构函数。

在`Supplier`类中添加`cleanUp`函数：

1. void cleanUp()
   
2. {
   
3.     cname.clear();
   
4.     vname.clear();
   
5.     mname.clear();
   
6. }
   

复制代码

在`Computer`类的析构函数中调用`cleanUp`函数：

1. ~Computer()
   
2. {
   
3.     if (c0 != NULL)
   
4.     {
   
5.         c0->cleanUp();
   
6.         delete c0;
   
7.         c0 = NULL;
   
8.     }
   
9. 
   
10.     if (v0 != NULL)
    
11.     {
    
12.         v0->cleanUp();
    
13.         delete v0;
    
14.         v0 = NULL;
    
15.     }
    
16. 
    
17.     if (m0 != NULL)
    
18.     {
    
19.         m0->cleanUp();
    
20.         delete m0;
    
21.         m0 = NULL;
    
22.     }
    
23. }
    

复制代码

现在，您的程序应该能够正常运行，不会报错。

跳转到最佳答案楼层

isdkz

Eden00 发表于 2023-4-20 22:32

经过我重新审查代码，找到了问题所在。问题出在析构函数的调用顺序。首先，`Computer`的析构函数被调用，接着调用`Supplier`的析构函数。但是在`Supplier`的析构函数中，由于它继承了`CPU`、`VideoCard`和`Memory`类，所以它们的析构函数也会被调用。因此，在`Computer`的析构函数中，我们需要确保在删除`c0`、`v0`和`m0`之前调用它们的析构函数。

为了解决这个问题，我们可以在`Supplier`类中添加一个名为`cleanUp`的函数，该函数在`Computer`的析构函数中调用，确保调用`CPU`、`VideoCard`和`Memory`的析构函数。

在`Supplier`类中添加`cleanUp`函数：

1. void cleanUp()
   
2. {
   
3.     cname.clear();
   
4.     vname.clear();
   
5.     mname.clear();
   
6. }
   

复制代码

在`Computer`类的析构函数中调用`cleanUp`函数：

1. ~Computer()
   
2. {
   
3.     if (c0 != NULL)
   
4.     {
   
5.         c0->cleanUp();
   
6.         delete c0;
   
7.         c0 = NULL;
   
8.     }
   
9. 
   
10.     if (v0 != NULL)
    
11.     {
    
12.         v0->cleanUp();
    
13.         delete v0;
    
14.         v0 = NULL;
    
15.     }
    
16. 
    
17.     if (m0 != NULL)
    
18.     {
    
19.         m0->cleanUp();
    
20.         delete m0;
    
21.         m0 = NULL;
    
22.     }
    
23. }
    

复制代码

现在，您的程序应该能够正常运行，不会报错。

isdkz

根据您提供的代码，问题出在`Computer`类的构造函数中，您试图将已经构造的`Supplier`对象复制给`CPU`、`VideoCard`和`Memory`指针，这将导致析构时多次删除同一对象。为了解决这个问题，您可以将构造函数参数的类型改为指针，并在`main`函数中分配相应的对象。

首先，将`Computer`的构造函数参数类型更改为指针：

1. class Computer
   
2. {
   
3. 
   
4.     CPU *c0;
   
5. 
   
6.     VideoCard *v0;
   
7. 
   
8.     Memory *m0;
   
9. 
   
10. public:
    
11.     Computer(Supplier *c, Supplier *v, Supplier *m)
    
12.     {
    
13. 
    
14.         c0 = c;
    
15. 
    
16.         v0 = v;
    
17. 
    
18.         m0 = m;
    
19. 
    
20.         c0->cname = c->GetName();
    
21. 
    
22.         v0->vname = v->GetName();
    
23. 
    
24.         m0->mname = m->GetName();
    
25. 
    
26.         cout << "This computer is assembled with " << c0->cname << " CPU, " << v0->vname
    
27. 
    
28.              << " video card, " << m0->mname << " memory" << endl;
    
29.     }
    
30. 
    
31.     // ...其他成员函数不变
    
32. };
    

复制代码

接下来，在`main`函数中分配相应的对象并传递指针：

1. void fun0()
   
2. {
   
3. 
   
4.     Supplier *q0 = new Supplier("Qualcom");
   
5. 
   
6.     Supplier *i0 = new Supplier;
   
7. 
   
8.     i0->SetName("Intel");
   
9. 
   
10.     Supplier *a0 = new Supplier("AMD");
    
11. 
    
12.     Computer c0(q0, i0, a0);
    
13. 
    
14.     c0.work();
    
15. }
    
16. 
    
17. int main()
    
18. {
    
19. 
    
20.     fun0();
    
21. 
    
22.     system("pause");
    
23. 
    
24.     return 1;
    
25. }
    

复制代码

最后，您需要更新`Computer`类的析构函数，以删除这些动态分配的`Supplier`对象。

1. ~Computer()
   
2. {
   
3. 
   
4.     if (c0 != NULL)
   
5.     {
   
6.         delete c0;
   
7. 
   
8.         c0 = NULL;
   
9.     }
   
10. 
    
11.     if (v0 != NULL)
    
12.     {
    
13.         delete v0;
    
14. 
    
15.         v0 = NULL;
    
16.     }
    
17. 
    
18.     if (m0 != NULL)
    
19.     {
    
20. 
    
21.         delete m0;
    
22. 
    
23.         m0 = NULL;
    
24.     }
    
25. }
    

复制代码

这样修改后，您的程序应该不会再报错，并按预期运行。

Eden00

isdkz 发表于 2023-4-20 22:05
根据您提供的代码，问题出在`Computer`类的构造函数中，您试图将已经构造的`Supplier`对象复制给`CPU`、`Vi ...

改了之后还是不行

Eden00

isdkz 发表于 2023-4-20 22:05
根据您提供的代码，问题出在`Computer`类的构造函数中，您试图将已经构造的`Supplier`对象复制给`CPU`、`Vi ...

1. #include <iostream>
   
2. 
   
3. #include <string>
   
4. 
   
5. #include <cmath>
   
6. 
   
7. #include <stdlib.h>
   
8. 
   
9. #include <ctime>
   
10. 
    
11. using namespace std;
    
12. 
    
13. class CPU
    
14. {
    
15. 
    
16. public:
    
17.     string cname;
    
18. 
    
19.     virtual void calculate() = 0;
    
20.     ~CPU()
    
21.     {
    
22.         cout << cname << " CPU is destructed" << endl;
    
23.     }
    
24. };
    
25. 
    
26. class VideoCard
    
27. {
    
28. 
    
29. public:
    
30.     string vname;
    
31. 
    
32.     virtual void display() = 0;
    
33.     ~VideoCard()
    
34.     {
    
35.         cout << vname << " video card is destructed" << endl;
    
36.     }
    
37. };
    
38. 
    
39. class Memory
    
40. {
    
41. 
    
42. public:
    
43.     string mname;
    
44. 
    
45.     virtual void store() = 0;
    
46.     ~Memory()
    
47.     {
    
48.         cout << mname << " memory is destructed" << endl;
    
49.     }
    
50. };
    
51. 
    
52. class Supplier : public CPU, public VideoCard, public Memory
    
53. {
    
54. 
    
55.     string name;
    
56. 
    
57.     void calculate()
    
58.     {
    
59. 
    
60.         cout << name << " CPU is calculating" << endl;
    
61.     }
    
62. 
    
63.     void store()
    
64.     {
    
65. 
    
66.         cout << name << " memory is stoing" << endl;
    
67.     }
    
68. 
    
69.     void display()
    
70.     {
    
71. 
    
72.         cout << name << " video card is displaying" << endl;
    
73.     }
    
74. 
    
75. public:
    
76.     Supplier()
    
77.     {
    
78. 
    
79.         cout << "Supplier is structed without argument" << endl;
    
80.     }
    
81. 
    
82.     Supplier(string n) : name(n)
    
83.     {
    
84. 
    
85.         cout << "Supplier is structed with name as " << n << endl;
    
86.     }
    
87. 
    
88.     string SetName(string n)
    
89.     {
    
90. 
    
91.         cout << "Supplier name is set by Class Supplier as " << n << endl;
    
92. 
    
93.         name = n;
    
94. 
    
95.         return name;
    
96.     }
    
97. 
    
98.     string GetName()
    
99.     {
    
100. 
     
101.         return name;
     
102.     }
     
103. 
     
104.     ~Supplier()
     
105.     {
     
106. 
     
107.         cout << name << " supplier is destructed" << endl;
     
108.     }
     
109. };
     
110. 
     
111. class Computer
     
112. {
     
113. 
     
114.     CPU *c0;
     
115. 
     
116.     VideoCard *v0;
     
117. 
     
118.     Memory *m0;
     
119. 
     
120. public:
     
121.     Computer(Supplier *c, Supplier *v, Supplier *m)
     
122.     {
     
123. 
     
124.         c0 = c;
     
125. 
     
126.         v0 = v;
     
127. 
     
128.         m0 = m;
     
129. 
     
130.         c0->cname = c->GetName();
     
131. 
     
132.         v0->vname = v->GetName();
     
133. 
     
134.         m0->mname = m->GetName();
     
135. 
     
136.         cout << "This computer is assembled with " << c0->cname << " CPU, " << v0->vname
     
137. 
     
138.              << " video card, " << m0->mname << " memory" << endl;
     
139.     }
     
140. 
     
141.     ~Computer()
     
142.     {
     
143. 
     
144.        if (c0 != NULL)
     
145.         {
     
146.             cout << c0->cname << endl
     
147.                  << c0 << endl;
     
148.             delete c0;
     
149. 
     
150.             c0 = NULL;
     
151.         }
     
152. 
     
153.         if (v0 != NULL)
     
154.         {
     
155.             cout << v0->vname << endl
     
156.                  << v0 << endl;
     
157.             delete v0;
     
158. 
     
159.             v0 = NULL;
     
160.         }
     
161. 
     
162.         if (m0 != NULL)
     
163.         {
     
164. 
     
165.             delete m0;
     
166. 
     
167.             m0 = NULL;
     
168.         }
     
169.     }
     
170. 
     
171.     void work()
     
172.     {
     
173. 
     
174.         c0->calculate();
     
175. 
     
176.         v0->display();
     
177. 
     
178.         m0->store();
     
179.     }
     
180. };
     
181. 
     
182. void fun0()
     
183. {
     
184. 
     
185.     Supplier *q0=new Supplier("Qualcom");
     
186. 
     
187.     Supplier *i0=new Supplier("Intel");
     
188. 
     
189. 
     
190.     Supplier *a0=new Supplier("AMD");
     
191. 
     
192.     Computer c0(q0, i0, a0);
     
193. 
     
194.     c0.work();
     
195. }
     
196. 
     
197. int main()
     
198. {
     
199. 
     
200.     fun0();
     
201. 
     
202.     system("pause");
     
203. 
     
204.     return 1;
     
205. }

复制代码

Eden00

isdkz 发表于 2023-4-20 23:38
经过我重新审查代码，找到了问题所在。问题出在析构函数的调用顺序。首先，`Computer`的析构函数被调用， ...

c0是CPU指针，v0是videocard指针按照定义应该不能调用supplier类的函数。
但是思路是对的，应该要确保supplier析构时，父类们也一起析构，因此感谢啦