Question

以下是我的代码插入......

的main.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;
}

Journy.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;
}

Person.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��（）

我不确定我做错了什么。问题可能是变量的范围和寿命吗？据我所知，列表容器会复制每个条目，因此范围和生命周期不应成为问题。我还看到某个地方，为了将一个类实例存储在列表中，该类必须具有默认构造函数，复制构造函数和一个=运算符重载。我的班级人物具有所有这些特征。我发布的代码只是我发现与此问题相关的插入内容。如果有人能给我一点点暗示问题可能是什么，那将非常感激。

关于

Answer 1

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

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

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

C ++代码表现不尽如人意（类和容器）

1 个答案: