HEAP CORRUPTION DETECTED」エラーの原因と対処方法について

HEAP CORRUPTION DETECTED"エラーの原因と対処法

最近、典型的なクラスの動的メモリ割り当ての問題に遭遇し、ウェブで見つけた回答から、通常は新しいアプリケーションからのメモリオーバーフローであると結論付けました。
newで特定のサイズのメモリを要求するが、後からコピーしたものがそのメモリのサイズを超えてしまい、削除を行うとエラーが発生することがある。としています。

char* p=new char[5];
strcpy(p,"aaaaaa");
delete[] p;

//Cow.h
#ifndef COW_H
#define COW_H
class Cow
{
	char name[20];
	char* hobby;
	double weight;
public:
	Cow();
	Cow(const char* nm, const char* bo, double wt);
	Cow(const Cow& c);
	~Cow();
	Cow& operator=(const Cow& c);
	void showCow()const;//display all cow data;
};
#endif

//Cow.cpp
#include "Cow.h"
#include 

#include 

using namespace std;



Cow::Cow()
{
	strcpy(name, "none");
	hobby = new char[4];
	strcpy(hobby, "cow");
	weight = 0.0;
	
}

Cow::Cow(const char* nm, const char* bo, double wt)
{
	strcpy(name, nm);
	hobby = new char[strlen(bo) + 1];
	strcpy(hobby, bo);
	weight = wt;
}

Cow::Cow(const Cow& c)
{
	//delete[] hobby;
	strcpy(name, c.name);
	hobby = new char[strlen(c.hobby) + 1];// just compile error hobby = new char(strlen(c.hobby) + 1)
	strcpy(hobby, c.hobby);
	weight = c.weight;

}


Cow::~Cow()
{
	delete[] hobby;
}

Cow& Cow::operator=(const Cow& c)
{
	if (this == &c)
		return *this;
	else
	{
		delete[] hobby;
		std::strcpy(name, c.name);
		hobby = new char[strlen(c.hobby) + 1];
		strcpy(hobby, c.hobby);
		weight = c.weight;
		return *this;
	}

	// TODO: insert the return statement here
}

void Cow::showCow() const
{
	std::cout << "Name: " << name << std::endl;
	std::cout < < "Hobby: " < < < hobby < < std::endl;
	std::cout << "Weight: " << weight << std::endl;
	
}


//usecow.cpp
#include 

#include "Cow.h"
using namespace std;

int main()
{
	Cow cow1;
	cow1.showCow();
	Cow cow2("yellow","grass",120);
	cow2.showCow();
	Cow cow3(cow2);
	cow3.showCow();
	cow1 = cow2;
	cow1.showCow();
	//system("pause");
	return 0;
	
}

<イグ

//Cow.h
#ifndef COW_H
#define COW_H
class Cow
{
	char name[20];
	char* hobby;
	double weight;
public:
	Cow();
	Cow(const char* nm, const char* bo, double wt);
	Cow(const Cow& c);
	~Cow();
	Cow& operator=(const Cow& c);
	void showCow()const;//display all cow data;
};
#endif

//Cow.cpp
#include "Cow.h"
#include 

#include 

using namespace std;



Cow::Cow()
{
	strcpy(name, "none");
	hobby = new char[4];
	strcpy(hobby, "cow");
	weight = 0.0;
	
}

Cow::Cow(const char* nm, const char* bo, double wt)
{
	strcpy(name, nm);
	hobby = new char[strlen(bo) + 1];
	strcpy(hobby, bo);
	weight = wt;
}

Cow::Cow(const Cow& c)
{
	//delete[] hobby;
	strcpy(name, c.name);
	hobby = new char[strlen(c.hobby) + 1];// just compile error hobby = new char(strlen(c.hobby) + 1)
	strcpy(hobby, c.hobby);
	weight = c.weight;

}


Cow::~Cow()
{
	delete[] hobby;
}

Cow& Cow::operator=(const Cow& c)
{
	if (this == &c)
		return *this;
	else
	{
		delete[] hobby;
		std::strcpy(name, c.name);
		hobby = new char[strlen(c.hobby) + 1];
		strcpy(hobby, c.hobby);
		weight = c.weight;
		return *this;
	}

	// TODO: insert the return statement here
}

void Cow::showCow() const
{
	std::cout << "Name: " << name << std::endl;
	std::cout < < "Hobby: " < < < hobby < < std::endl;
	std::cout << "Weight: " << weight << std::endl;
	
}

//usecow.cpp
#include 

#include "Cow.h"
using namespace std;

int main()
{
	Cow cow1;
	cow1.showCow();
	Cow cow2("yellow","grass",120);
	cow2.showCow();
	Cow cow3(cow2);
	cow3.showCow();
	cow1 = cow2;
	cow1.showCow();
	//system("pause");
	return 0;
	
}

cow.cppファイルの中で、hobby = new char[strlen(c.hobby) + 1];/は、hobby = new char(strlen(c.hobby) + 1) と書かれているので、実際には正しいサイズのメモリを確保できず、デストラクタ delete []hobby でエラーになっています。