关于vs写c++的常见错误（一）

​

记录下我用Visual Studio2019写C/C++时遇到的典型错误与解决方案

一、数组指针做参数

这是一个顺序表，构造函数传参时直接将指针进行赋值，再对数组进行操作就会一下错误。

class SeqList {
private:
	int* list;
	int maxSize;
	int size;
public:
    SeqList(int* arr, int size) {
		maxSize = 1000;
		this->size = size;
        list=arr;
	}

错误信息：

​

问题解释：CRT检测到应用程序在堆缓冲区结束后写入内存。

解决：

1、对数组空间进行分配

2、遍历赋值数组元素

	SeqList(int* arr, int size) {
		maxSize = 1000;
		this->size = size;
		list = new int[maxSize];
		for (int i = 0; i < size; i++)
		{
			list[i] = arr[i];
		}
	}

二、浅拷贝与深拷贝

class SeqList {
private:
	int* list;
	int maxSize;
	int size;
public:
	SeqList() {
		maxSize = 1000;
		size = 0;
		list = new int[maxSize];
	}
	SeqList(int* arr, int size) {
		maxSize = 1000;
		list = new int[maxSize];
		this->size = size;
		for (int i = 0; i < size; i++)
		{
			this->list[i] = arr[i];
		}
	}
    // 析构函数
	~SeqList() {    
		delete[] list;
		//cout << "Distructor "<< endl;
	}
    // 合并顺序表
	SeqList list_concat(SeqList& e) {  
		SeqList newlist;
		int len1 = list_size(), len2 = e.list_size();
		int i1 = 0, i2 = 0;
		int t = 1;
		while (i1 < len1 || i2 < len2) {
            // 双指针合并两个有序数组
		}
		return newlist;
	}
};
int main() {
	SeqList s1(arr1, n1);
	SeqList s2(arr2, n2);

	SeqList s3 = s1.list_concat(s2);
	s3.list_display();

	return 0;
}

 错误信息：

​

 

如果把析构函数换成 cout<<"Distructor"<<endl; 

可以看到输出结果：s3被重复析构了两次

明明  SeqList list_concat(SeqList& e) 参数类型是引用，为什么还会析构两次呢？

​

问题出现在这行代码       SeqList s3 = s1.list_concat(s2);

传参是引用类型，避免了一次浅拷贝（仅仅是拷贝指针，指向的内容是同一个）

但s3 = s1.list_concat(s2); “=”赋值时又进行了一次浅拷贝（因为我没有实现拷贝构造，vs使用默认构造函数），实际执行的是

SeqList s3 = s1.list_concat(s2);
SeqList temp = s1.list_concat(s2);
s3 = temp;
delete temp;

所以临时对象temp也释放了一次list，实际对象s3也执行析构函数，导致同一块内存被重复释放。

（如果传参是值传递，那么会重复释放内存3次）

解决：

1、增加拷贝构造函数

    SeqList(const SeqList& e) {
        size = e.size;
        maxSize = e.maxSize;
        list = new int[e.size];
        for (int i = 0; i < e.size; i++)
        {
            list[i] = e.list[i];
        }
    }