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

mandala

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

最佳答案

月排行榜 / 总排行榜

Krant5

2017-8-19 16:35:29

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

       Stack(const Stack &stack) {
                *this = stack;
        }

随后调用
     

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

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

        Stack(const Stack &stack) {
                *this.data=0;
                *this = stack;
        }

跳转到最佳答案楼层

Krant5

你初始化为空呗?

mandala

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

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

Krant5

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

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

mandala

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

// 以下为stack.h里的代码
#ifndef STACK_H
#define STACK_H

#include <iostream>

template <typename T>
class Stack {
public:
    Stack(unsigned int length = 4) {
        this->length = length;
        this->sp = length;
        this->data = new T[length];
    }

        Stack(const Stack &stack) {
                *this = stack;
        }

    ~Stack() {
        delete []data;
    }

    void push(T value) throw (const char *) {
        if (!sp) {
            throw "Stack overflow!\n";
        }
        data[--sp] = value;
    }

    T pop() throw (const char *) {
        if (sp == length) {
            throw  "Stack out of bounds!\n";
        }
        return data[sp++];
    }

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

private:
    unsigned int length;// 最大长度
    unsigned int sp;    // 栈指针
    T *data;            // 内存数据
};

#endif
// stack.h代码结束

// 以下为main.cpp的代码
#include "stack.h"

int main()
{
    const unsigned short SIZE = 4;
    #define type int
    Stack<type> stack(4);
    Stack<type> stack2 = stack;        // 这里调用

        int i;
        try {
                i = 0;
                while (i < SIZE + 1) {
                        stack.push(i);
                        std::cout << "push\t" << i << std::endl;
                        i++;
                }
        } catch (const char *e) {
                std::cout << e;
        }

        try {
                i = SIZE + 100;
                while (i > 0) {
                        type value = stack.pop();
                        std::cout << "pop\teax(value = " << value << ")" << std::endl;
                        i--;
                }
        } catch (const char *e) {
                std::cout << e;
        }
    return 0;
}
// main.cpp代码结束

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

Krant5

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

       Stack(const Stack &stack) {
                *this = stack;
        }

随后调用
     

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

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

        Stack(const Stack &stack) {
                *this.data=0;
                *this = stack;
        }

mandala

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

随后调用

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