奇怪的 this-> 行为

Question

所以我有以下课程

class Community
{
private:
  char* Name;
  char foundationDate[11];
  Person* founder;
  int maxMembersCount;
  int membersCount;
  Person* members;
  static int communitiesCount;

.....

我想实现一个复制构造函数：

Community::Community(const Community& other)
{
    this->Name = new char[strlen(other.Name)+1];
    strcpy(this->Name,other.Name);
    strcpy(this->foundationDate,other.foundationDate);
    this->founder = other.founder;
    this->maxMembersCount = other.maxMembersCount;
    this->membersCount = other.membersCount;
    this->members = new Person[this->maxMembersCount];
    this->members = other.members;
    communitiesCount++;
}

但是每当我说 Community A=B; 时，这段代码就会崩溃； 所以对我来说，这段代码似乎是合法的，但是当我开始调试时，会出现消息：this->“无法读取内存”。如果您需要更多代码示例，请帮助我，请告诉我。

---

Community::Community(const char* name , char foundDate[],Person* founder,int maxMembers) {

    this->Name = new char[strlen(name)+1];
    strcpy(this->Name,name);
    strcpy(this->foundationDate,foundDate);
    this->founder = new Person(founder->getName(),founder->getEGN(),founder->getAddress());
    this->maxMembersCount = maxMembers;
    this->membersCount = 2;
    this->members = new Person[this->maxMembersCount];
    communitiesCount++;

}

这是类的主要构造函数，它工作得很好......

Answer 1

这里有多个问题，其中任何一个都可能是问题的一部分或全部。

• 如果 Name 或 foundationDate 在右侧不是以空值结尾的，它将跑掉并复制坏内存。
• 如果 founder 或 members 归对象所有，如果你不在析构函数中删除它们，你要么会泄漏内存，要么在浅拷贝然后删除两次等。

要解决此问题，只需将 Name 和 foundationDate std::string，然后将 founder 和 members 设置为值而不是指针。如果您绝对必须在堆上分配它们，请使用诸如shared_ptr 之类的智能指针来保存它，而不是使用容易出错的原始指针。

Answer 2

首先，检查other.Name 是否填充了指向以空字符结尾的字符串的指针，other.foundationDate 是否包含以空字符结尾的字符串。也就是说，您将良好的指针传递给strlen 和strcpy。

如果是这样，请检查作业中的 B 是否可以完全访问。

如果这也是真的，printf 一切。并调试发生异常的确切位置。或者发布可编译并重现错误的整个代码。

还要注意这里：

this->members = new Person[this->maxMembersCount];
this->members = other.members;

第一个赋值没有做任何事情（实际上是泄漏内存），而第二个 double 会在对象销毁时删除你的内存（如果你正确地delete[] members）。