C++ 代码未按预期运行(类和容器)

Question

这是我的代码插入...

主要.cpp

void addStuff(Journey& journey)
{
  journey.addPerson("John Doe", "USA");
}

void demo()
{
  Journey journey("Sweden");
  addStuff(journey);
  std::cout << journey;
}

int main(int argc, char* argv[])
{
  demo();
  return 0;
}

旅程.cpp

void Journey::addPerson(const char* name, const char* nationality)
{
  add(Person(name, nationality));
}

void Journey::add(Person person)
{
  persons_.push_back(person);
}

std::ostream& operator<<(std::ostream& out, const Journey& journey)
{
  out << "Journey: " << journey.name_ << std::endl;

  out << "  Persons attending:" << std::endl;
  for(Journey::PersonList::const_iterator person_it =  journey.persons_.begin();
      person_it != journey.persons_.end();
      person_it++)
  {
    out << "    " << *person_it;
  }

  return out;
}

人.cpp

Person::Person(){}

Person::Person(const char* name, const char* nationality) : name_(0), 
                                                            nationality_(0)
{
  copyString(&name_, name);
  copyString(&nationality_, nationality);
}

Person::Person(const Person& other): name_( other.name_), 
                                     nationality_( other.nationality_) {}

void Person::copyString(char** dest, const char* source)
{
  unsigned int str_len = strlen(source);
  char* str = new char[str_len+1];
  strncpy(str, source, str_len);
  str[str_len] = '\0';
  *dest = str;
}

std::ostream& operator<<(std::ostream& out, const Person& person)
{
  out << person.name_ << " (" << person.nationality_ << ")" << std::endl;
  return out;
}

但是，当我尝试执行代码时，结果是:

参加人员: P�� ()

我不太确定我做错了什么。问题可能是变量的范围和生命周期吗？据我了解，列表容器会复制每个条目，因此范围和生命周期不应该成为问题。我还在某处看到，为了将类实例存储在列表中，该类必须具有默认构造函数、复制构造函数和重载的 = 运算符。我的类(class) Person 具有所有这些特征。我发布的代码只是我发现与此问题相关的插入内容。如果有人能给我丝毫提示，我将不胜感激。

问候

Answer 1

你确实有一个复制构造函数，这很好，因为你做了很多复制。但是，您只进行浅拷贝，即只复制指针而不是实际内容。

这意味着如果你复制一个 Person 对象(就像调用 Journey::add(Person person) 时所做的那样)，那么你将拥有 两个对象都使用相同的指针指向相同的内存。如果您的析构函数(如果有的话)释放了内存，那么两个对象的内存都被释放了，但是其中一个对象仍然有指向现在空闲内存的指针，导致 em>未定义的行为 当您尝试取消引用这些指针时。

您需要进行深度 复制，换句话说，分配新内存并复制内容。或者做一些明智的事情并使用 std::string .