为什么这会产生 bad_alloc 错误？

Question

目前，我正在尝试为 Student 设置一个成员函数，该函数从 cin 读取字符串，用作该函数的参数，然后使用数据创建一个 Student 对象。但是，它是否给我一个 bad_alloc 错误？我知道该函数正在获取字符串，但在创建新对象后它给出了此错误。

错误：

./a.out

Please insert name for student:
Bob_Russel
terminate called after throwing an instance of 'std::bad_alloc'
  what():  std::bad_alloc
Aborted

构造函数：

Student::Student(string tname){

  name = tname;

}

函数：

Student Student::readStudent(istream &i){

  Student *stud;

  string y;

  i >> y;

  stud = new Student(y);

  return  *stud;

}

testStudent.cpp：

 #include "Student.h"

int main(){

  Student *stud3;
  cout << "\nPlease insert name for student:\n";

  stud3->readStudent(cin);


return 0;

}

Answer 1

代码不仅会泄漏内存（在 readStudent 中创建一个永远不会被删除的新 Student），而且在 main 中，您使用了一个未初始化的指针调用readStudent。这可能会损坏您的堆，从而导致对 new 的调用引发 std::bad_alloc。

再看一下 C++ 内存管理和对象生命周期。这里实际上根本不需要使用指针。作为起点，您的 main 可以修改为：

int main() {
    Student stud3;
    std::cout << "Please insert name for student:" << std::endl;
    stud3.readStudent(std::cin);
}

如果您在 main 中读取名称（作为 std:: string），然后将名称直接传递给 Student 构造函数：

int main() {
    std::cout << "Please insert name for student:" << std::endl;
    // Read in the name.
    std::string name;
    std::cin >> name;
    // Create the student with the input name.
    Student stud3(name);
 }

Answer 2

看起来您正在尝试实现工厂方法。如果是这种情况，那么您缺少 static 关键字和 readStudent 调用的正确语法。

class Student{
public:
    Student(std::string tname);
    static Student* readStudent(std::istream &i);
private:
    std::string name
};

Student::Student(std::string tname) {
    name = tname;
}

Student* Student::readStudent(std::istream &i){
    std::string y;
    i >> y;
    return new Student(y);
}

int main(int argc, char* argv[]){
    Student *stud3 = NULL;

    std::cout << "\nPlease insert name for student:\n";

    stud3 = Student::readStudent(cin);

    return 0;
}

Answer 3

您使用 new 在堆上分配并且从不释放它，因此您耗尽了内存并得到了 bad_alloc。对于每个新都应该有一个删除。

这不会抛出 bad_alloc：

Student Student::readStudent(std::istream& i)
{        
   std::string y;    
   i >> y;    
   return Student(y);    
}