c++代码的行为不符合预期(类和容器)

C++ code not behaving as expected (classes and containers)

本文关键字：代码 不符合 c++      更新时间：2023-10-16

这是我的代码的插入…

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] = '';
  *dest = str;
}
std::ostream& operator<<(std::ostream& out, const Person& person)
{
  out << person.name_ << " (" << person.nationality_ << ")" << std::endl;
  return out;
}

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

人员参加:P��()

我真的不知道我做错了什么。问题是变量的作用域和生存期吗?据我所知，列表容器会复制每个条目，所以作用域和生命周期不应该是问题。我还看到，为了将类实例存储在列表中，类必须具有重载的默认构造函数、复制构造函数和=操作符。我的班级Person具有所有这些特点。我张贴的代码只是插入，我发现相关的这个问题。如果有人能给我一点提示，我将不胜感激。

与

您确实有一个复制构造函数，这很好，因为您要进行大量复制。然而，你只做浅层复制，即只复制指针而不复制实际内容。

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

你需要做深复制，换句话说，分配新的内存和复制内容。或者明智地使用std::string

相关文章：