Нужен совет по возврату экземпляра объекта с использованием указателя — bad_alloc в ячейке памяти?
У меня возникли проблемы с попыткой вернуть экземпляр из списка экземпляров функции.
У меня есть набор, в котором я храню экземпляры своего класса Student. Я пытаюсь вернуть экземпляр из функции «найти», чтобы иметь возможность найти этот экземпляр по имени ученика. Я вызываю эту функцию в другом классе/файле. Я хочу иметь возможность получить этот экземпляр объекта в другом файле, отредактировать определенные свойства и обновить его в этом списке экземпляров.
Моя функция поиска в Student.cpp:
static set<Student, StudentCmp> studentInstances;
Student* Student::find(int studentId) {
Student *foundStudent = new Student();
for (Student s: studentInstances) {
if (s.getStudentID() == studentId) {
foundStudent = &s;
}
}
return foundStudent;
}
В другом классе:
*Student::find(1).setStudentName("John");
Да, этот студенческий билет существует в списке, и внутри функции поиска все работает нормально. Он находит идентификатор и соответствующий адрес, однако, когда я пытаюсь указать на эту переменную, используя адрес, указанный мне моей функцией поиска, я получаю эту ошибку:
Unhandled exception at 0x7622DFD8 in Driver.exe: Microsoft C++ exception: std::bad_alloc at memory location 0x006EF044
Я предполагаю, что адрес выходит за пределы? Я не знаю почему, но похоже, что адрес больше не указывает на мой объект, когда он выходит из функции. Я посмотрел это, чтобы найти ответы, и люди сказали, что нужно использовать ключевое слово «новое», чтобы сохранить объект в куче. но это, похоже, не работает для меня.
Какой-нибудь совет, как добиться того, чего я хочу?
Вы возвращаете адрес локальной переменной. Разве ваш компилятор не кричит на вас? Измените цикл for на for (Student& s :...), и теперь &s будет принимать адрес объекта Student в наборе.
Здесь указатели не нужны. Определять find точно не нужно. См.: en.cppreference.com/w/cpp/container/set/find
Ответы 2
Проблема здесь в том, что вы возвращаете указатель не на объект в наборе или выделенный вами указатель, а вместо этого вы возвращаете указатель на локальный объект функции, который уничтожается при выходе из функции.
for (Student s: studentInstances)
Делает копию каждого Student в studentInstances. Что вам нужно, это ссылка типа
for (Student& s: studentInstances)
так что когда вы возвращаете указатель, вы возвращаете указатель на объект, который живет в studentInstances
Обратите внимание, что у вас также произойдет утечка памяти, если вы найдете объект, потому что вы никогда не освобождаете foundStudent, если нашли ученика. Ваш оператор if должен быть
if (s.getStudentID() == studentId) {
delete foundStudent;
foundStudent = &s;
}
Лично я бы реорганизовал функцию на
Student* Student::find(int studentId) {
for (Student& s: studentInstances) {
if (s.getStudentID() == studentId) {
return &s;
}
}
return nullptr;
}
Теперь вы возвращаете либо указатель на действительный объект, либо нулевой указатель. Вы можете легко проверить это на сайте вызова, здесь нет ручного управления памятью, а также нет возможности вызвать утечку. Вам просто нужно убедиться, что вы всегда проверяете результат нулевого указателя после вызова функции.
С этой функцией что-то не так. Частично используются неуправляемые/тупые указатели, частично — new.
Вероятно, фундаментальная проблема заключается в том, где должны жить Student объекты. На первый взгляд, имеет смысл жить в studentInstances, std::set Студентах. Но вы пытаетесь изменить объекты Student на std::set, а это, как правило, плохая идея. Поскольку порядок std::set определяется его элементами, изменение этих элементов может нарушить порядок набора, а это нарушит весь набор.
Тогда проще всего было бы использовать набор интеллектуальных указателей. И это решит большинство ваших проблем — ну, кроме for (Student s: studentInstances). Это создает копию каждого студента. Надо знать, как в С++ по умолчанию все копируется.
Идентификаторы студентов, как правило, остаются неизменными на протяжении всей карьеры студента, поэтому, возможно, карта студенческих билетов со студентами также будет хорошей альтернативой. Особенно, если студенческий билет был изготовлен const.
ваш источник изучения C++ очень плох, я предлагаю вам вместо этого просмотреть хорошую книгу по C++.