このリークメモリはありますか？

Question

addStudent関数とmainMenu関数は、重要な2つです。

#include <iostream> 
#include <string> 

using namespace std; 

struct Student 
{ 
    string name; 
    string major; 
    Student *p_next; 
}; 

Student *addStudent(Student *p_students) 
{ 
    Student *p_new_student = new Student; 
    cout << "Student name: \n"; 
    cin >> p_new_student->name; 
    cout << p_new_student->name << "'s major: \n"; 
    cin >> p_new_student->major; 

    Student *p_place_holder = new Student; 
    Student *p_all_students = new Student; 
    p_all_students = p_students; 
    p_place_holder = NULL; 

    if (p_students == NULL) // Adds the first element to the linked list which was initialized to NULL 
    { 
     p_new_student->p_next = p_students; 
     delete p_place_holder; 
     delete p_all_students; 
     return p_new_student; 
    } 

    else // Adds elements beyond the first element to the linked list 
    { 
     while (p_all_students != NULL) 
     { 
     if (p_new_student->name.compare(p_all_students->name) <= 0) 
     { 
      if (p_place_holder == NULL) /* if p_new_student->name is before 
      p_all_students->name and p_all_students is still the first element in the list*/ 
      { 
       p_new_student->p_next = p_all_students; 
       p_all_students = p_new_student; 
       delete p_place_holder; 
       return p_all_students; 
      } 

      else 
      { 
       p_new_student->p_next = p_all_students; 
       p_place_holder->p_next = p_new_student; 
       return p_students; 
      } 

     } 

     else if (p_new_student->name.compare(p_all_students->name) > 0) 
     { 
      p_place_holder = p_all_students; 
      p_all_students = p_all_students->p_next; 
     } 
     } 
    } 
} 

void mainMenu(Student *p_students) 
{ 
    int response = 0; 
    cout << "1. Add Student\n"; 
    cout << "2. View Students\n"; 
    cin >> response; 

    if (response == 1) // calls addStudent 4 times and then mainMenu 
    { 
     p_students = addStudent(p_students); 
     p_students = addStudent(p_students); 
     p_students = addStudent(p_students); 
     p_students = addStudent(p_students); 
     mainMenu(p_students); 
    } 

    else if (response == 2) // lists the students and their majors and then exits 
    { 
     while (p_students != NULL) 
     { 
     cout << p_students->name << '\n'; 
     cout << p_students->major << "\n\n"; 
     p_students = p_students->p_next; 
     } 
    } 

    delete p_students; // hopefully deletes all allocated memory 
} 

int main() 
{ 
    Student *p_students = new Student; 
    p_students = NULL; 

    mainMenu(p_students); 
} 

"delete p_students;"プログラムがメモリをリークしないように、mainMenu関数内のすべての割り当てられたメモリを適切に削除します。どんな助けもありがとう、ありがとう。

Answer 1

これらの行は、学生のインスタンスを漏れます

Student *p_place_holder = new Student; 
Student *p_all_students = new Student; 
p_all_students = p_students; 
p_place_holder = NULL; 

あなたは、後でインスタンスを削除せずにNULLでポインタを上書きし、p_place_holderに割り当て、生徒を割り当てます。

また、持っている：

古い値でメモリを解放しようとせずに p_students 3回上書きします

p_students = addStudent(p_students); 
p_students = addStudent(p_students); 
p_students = addStudent(p_students); 
p_students = addStudent(p_students); 

。

大きな問題はdelete p_studentsで終わりますが、Student構造体を調べて、その中のポインタを見つけたり、削除したりするなど、再帰的には望みどおりに行いません。割り当てたメモリのほとんどがすべてリークしています。

私は新しいものと削除するペアを一緒に誤解していると思います。

Answer 2

メモリリークを防ぐには、強制所有権ポインタを使用する必要があります。たとえば、unique_ptrおよびshared_ptrです。 deleteで漏れを効果的に防ぐことはできません。