[已解决]C++快速入门 - 36课 副本构造器 delete未初始化的指针 导致 程序出错 怎么改

mandala

vc++6.0编译器下 delete未初始化的指针（0xcccccccc） 导致 程序出错；
gcc编译器 这段代码正常运行。
有什么办法可以让代码在vc++6.0编译器下正常运行，比如校验指针地址之类的方法。

最佳答案

月排行榜 / 总排行榜

Krant5

2017-8-19 16:35:29

你的 Stack<type> stack2 = stack; 调用        

1.        Stack(const Stack &stack) {
   
2.                 *this = stack;
   
3.         }

复制代码

随后调用
      

1. Stack &operator=(const Stack &stack) {
   
2.         if (this == &stack) {
   
3.             return *this;
   
4.         }
   
5.         length = stack.length;
   
6.         sp = stack.sp;
   
7.         delete[] data;        // vc++6.0编译器报错，gcc竟然能过。这里如何改？
   
8.         data = new T[length];
   
9.         for (unsigned int i = 0; i < sp; i++) {
   
10.             data[i] = stack.data[i];
    
11.         }
    
12.         return *this;
    
13.         }

复制代码

但是事实上,T * data只是被定义了而已,但是并没有初始化,这时候delete[] data;就等于对野指针指向的空间删除,原则上一定会报错的.至于解决这个问题,可以在拷贝构造函数中添加 data=0;
即为:

1.         Stack(const Stack &stack) {
   
2.                 *this.data=0;
   
3.                 *this = stack;
   
4.         }

复制代码

跳转到最佳答案楼层

Krant5

你初始化为空呗?

mandala

Krant5 发表于 2017-8-18 15:28
你初始化为空呗?

差不多是你这个意思，但是要 怎么改 呢?

Krant5

mandala 发表于 2017-8-18 21:56
差不多是你这个意思，但是要 怎么改 呢?

你要把那一块那个类的完整代码贴出来,视频里没有解决还是怎么的

mandala

Krant5 发表于 2017-8-19 08:11
你要把那一块那个类的完整代码贴出来,视频里没有解决还是怎么的

1. // 以下为stack.h里的代码
   
2. #ifndef STACK_H
   
3. #define STACK_H
   
4. 
   
5. #include <iostream>
   
6. 
   
7. template <typename T>
   
8. class Stack {
   
9. public:
   
10.     Stack(unsigned int length = 4) {
    
11.         this->length = length;
    
12.         this->sp = length;
    
13.         this->data = new T[length];
    
14.     }
    
15. 
    
16.         Stack(const Stack &stack) {
    
17.                 *this = stack;
    
18.         }
    
19. 
    
20.     ~Stack() {
    
21.         delete []data;
    
22.     }
    
23. 
    
24.     void push(T value) throw (const char *) {
    
25.         if (!sp) {
    
26.             throw "Stack overflow!\n";
    
27.         }
    
28.         data[--sp] = value;
    
29.     }
    
30. 
    
31.     T pop() throw (const char *) {
    
32.         if (sp == length) {
    
33.             throw  "Stack out of bounds!\n";
    
34.         }
    
35.         return data[sp++];
    
36.     }
    
37. 
    
38.         Stack &operator=(const Stack &stack) {
    
39.         if (this == &stack) {
    
40.             return *this;
    
41.         }
    
42.         length = stack.length;
    
43.         sp = stack.sp;
    
44.         delete[] data;        // vc++6.0编译器报错，gcc竟然能过。这里如何改？
    
45.         data = new T[length];
    
46.         for (unsigned int i = 0; i < sp; i++) {
    
47.             data[i] = stack.data[i];
    
48.         }
    
49.         return *this;
    
50.         }
    
51. 
    
52. private:
    
53.     unsigned int length;// 最大长度
    
54.     unsigned int sp;    // 栈指针
    
55.     T *data;            // 内存数据
    
56. };
    
57. 
    
58. #endif
    
59. // stack.h代码结束
    
60. 
    
61. // 以下为main.cpp的代码
    
62. #include "stack.h"
    
63. 
    
64. int main()
    
65. {
    
66.     const unsigned short SIZE = 4;
    
67.     #define type int
    
68.     Stack<type> stack(4);
    
69.     Stack<type> stack2 = stack;        // 这里调用
    
70. 
    
71.         int i;
    
72.         try {
    
73.                 i = 0;
    
74.                 while (i < SIZE + 1) {
    
75.                         stack.push(i);
    
76.                         std::cout << "push\t" << i << std::endl;
    
77.                         i++;
    
78.                 }
    
79.         } catch (const char *e) {
    
80.                 std::cout << e;
    
81.         }
    
82. 
    
83.         try {
    
84.                 i = SIZE + 100;
    
85.                 while (i > 0) {
    
86.                         type value = stack.pop();
    
87.                         std::cout << "pop\teax(value = " << value << ")" << std::endl;
    
88.                         i--;
    
89.                 }
    
90.         } catch (const char *e) {
    
91.                 std::cout << e;
    
92.         }
    
93.     return 0;
    
94. }
    
95. // main.cpp代码结束

复制代码

@小甲鱼 @人造人 @lumber2388779 @ba21 @Krant5
求帮忙解决

Krant5

你的 Stack<type> stack2 = stack; 调用        

1.        Stack(const Stack &stack) {
   
2.                 *this = stack;
   
3.         }

复制代码

随后调用
      

1. Stack &operator=(const Stack &stack) {
   
2.         if (this == &stack) {
   
3.             return *this;
   
4.         }
   
5.         length = stack.length;
   
6.         sp = stack.sp;
   
7.         delete[] data;        // vc++6.0编译器报错，gcc竟然能过。这里如何改？
   
8.         data = new T[length];
   
9.         for (unsigned int i = 0; i < sp; i++) {
   
10.             data[i] = stack.data[i];
    
11.         }
    
12.         return *this;
    
13.         }

复制代码

但是事实上,T * data只是被定义了而已,但是并没有初始化,这时候delete[] data;就等于对野指针指向的空间删除,原则上一定会报错的.至于解决这个问题,可以在拷贝构造函数中添加 data=0;
即为:

1.         Stack(const Stack &stack) {
   
2.                 *this.data=0;
   
3.                 *this = stack;
   
4.         }

复制代码

mandala

Krant5 发表于 2017-8-19 16:35
你的 Stack stack2 = stack; 调用        

随后调用

你的方法很赞。想不到vc6编译器下delete的地址如果为0，直接跳过后面操作。