Я не могу получить функцию удаления, чтобы удалить элементы из списка

Question

Когда я добавляю человека, он добавляется правильно, но когда я иду, чтобы удалить имя, он говорит, что имя не найдено.Эта программа позволяет пользователю создавать телефонную книгу и редактировать информацию в телефонной книге

#include <iostream>
#include <string>
#include "PhoneBook.h"
using namespace std;
char MenuSelection ();

int main() {
    char Selection;
    PhoneBook myList;
    do{ 
        Selection = MenuSelection ();
        switch (Selection){
            case 'a':
                myList.AddEntry ();
                break;
            case 'd':
                myList.DisplayNamesAndNumbers( );
                break;
            case 's': 
                myList.FindEntry ();
                break;
            case 'r':
                myList.DeleteFunction ();
                break;
            case 'q': 

                break;
            default : 
                cout << "\n\nNot a command choice\n";
                cout << "Press enter to continue";
                cin.get();
                cin.get();
                system ("clear");
        }
    }while (Selection != 'q');
    myList.MakeFile ();
    cout << "Press enter to continue";
    return 0;
}

//This function prints out the opening menu and allws users to enter a command.
char MenuSelection (){
    char Response;
    cout << "\n                 MENU\n";
    cout << "a - add a name and phone number\n";
    cout << "d - display names and phone number\n";
    cout << "r - remove a name and phone number\n";
    cout << "s - search for a name and return the phone number\n";
    cout << "q - quit program\n\n";
    cout << "Enter your choice: ";
    cin >> Response;
    return Response;
}

Заголовочный файл для вышеуказанной программы

Это класс, который позволяет пользователю манипулировать телефонной книгой.

#include <iostream>
#include <fstream>
#include <string>
using namespace std;
const int GROUPSIZE = 100;
struct contact {string NameEntry; string PhoneNumber;};

class PhoneBook {
private:
    int Size;
    contact ContactNumber [GROUPSIZE];
public:
    PhoneBook ();
    void MakeFile ();
    void AddEntry ();
    void FindEntry ();
    string DisplayNamesAndNumbers ();
    string DeleteFunction ();
};

//opens the file and adds names and numbers to the text file
PhoneBook::PhoneBook (){
    string PersonsName, PhoneNum;
    int i = 0;
    ifstream infile;
    infile.open ("phonebook.txt");
    if (!infile){
        cout << "File does not exist";
    }
    else{
        while (!infile.eof ()){
            infile >> PersonsName >> PhoneNum;
            ContactNumber [i].NameEntry = PersonsName;
            ContactNumber [i++].PhoneNumber = PhoneNum;
        }
        Size = i;
        infile.close ();
    }
}

//adds name and phone number to the list
void PhoneBook::AddEntry (){
    string PersonsName, PhoneNum;
    cout << "\n\nEnter the name to be added: ";
    cin >> PersonsName;
    cout << "Enter the phone number for " << PersonsName << ": ";
    cin >> PhoneNum;
    ContactNumber [Size].NameEntry = PersonsName;
    ContactNumber [Size].PhoneNumber = PhoneNum;
    Size++;
    cout << "Press enter to continue";
    cin.get ();
    cin.get ();
    system ("clear");
}

//finds names and numbers in the list
void PhoneBook::FindEntry (){
    int Location, Counter;
    string Contact;
    cout << "\n\nName to find: ";
    cin >> Contact;
    Counter = 0;
    while (Counter < Size){
        Counter++;
    //for (int i = 0; i < size; i++){
        if (strcmp(Contact.c_str (),ContactNumber [Counter].NameEntry.c_str ()))
            Location = Counter;
        else 
            Location = -1;
    }
    if (Location != -1)
        cout << "The phone number for " << Contact << " is " << ContactNumber [Location].PhoneNumber << endl;
    else
        cout << Contact << " not in phonebook\n";
    cout << "Press enter to continue";
    cin.get();
    cin.get();
    system ("clear");
}

//displays all information in the list
string PhoneBook::DisplayNamesAndNumbers (){
    string PersonsName, PhoneNum;
    int Check = 1;
    cout << "\n\nList is being sorted\n";
    while (Check == 1){
        cout << "Name\t\tTelephone Number";
        for (int i = 0; i < Size; i++){
            cout << ContactNumber[i].NameEntry << "\t\t" << ContactNumber[i].PhoneNumber << "\n";
        }
        break;
    }
    cout << "Press enter to continue";
    cin.get();
    cin.get();
    system("clear");
    return "";
}

//deletes information from the list
string PhoneBook::DeleteFunction (){
    char Answer;
    string PersonsName;
    int Location;
    cout << "\n\nName to remove: ";
    cin >> PersonsName;
    for (int i = 0; i < GROUPSIZE; i++){
        if (!strcmp(PersonsName.c_str (),ContactNumber[i].NameEntry.c_str ()))
            Location = i;
        else 
            Location = -1;
    }
    if (Location != -1){
        ContactNumber[Location].NameEntry = ContactNumber[Size].NameEntry;
        ContactNumber[Location].PhoneNumber = ContactNumber[Size].PhoneNumber;
        cout << PersonsName <<" removed from phonebook\n";
        cout << "Press enter to continue";
        cin.get();
        cin.get();
        system("clear");
        return"";
    }
    cout << "Name not found in phonebook\n";
    cout << "Press enter to continue";
    cin.get();
    cin.get();
    system ("clear");
    return"";
}

//closes the file at the end of the program
void PhoneBook::MakeFile (){
    ofstream outfile;
    outfile.open("phonebook.txt");
    for (int i = 0; i < GROUPSIZE; i++){
        outfile << ContactNumber[i].NameEntry << " " << ContactNumber[i].PhoneNumber<<"\n";
    }
    outfile.close();
}

Answer 1

Похоже, вы забыли "прервать" в своем удалении после нахождения совпадения.

Answer 2

Я думаю, что вы используете strcmp неправильно. Возвращает 0, если обе строки совпадают, поэтому вам нужно strcmp() == 0. Посмотрите здесь для возможных возвращаемых значений.
Далее, поскольку вы уже используете std::string s, просто сравните их напрямую, они поддерживают это:

if(Contact == ContactNumber[Counter].NameEntry)

Кроме того, вы не выходите из цикла for после нахождения записи, которую хотите удалить.

---

Еще один дополнительный совет: ваша телефонная книга может содержать повторяющиеся записи, так как вы не проверяете, существует ли человек в вашей функции AddEntry.
И, наконец, после того, как вы прошли через всю эту сложность поддержания того, что по сути является отображением из одной строки в другую, для дальнейшего использования такой структуры данных рассмотрите возможность использования std::map. :)

#include <map>

int main(){
  map<string /*name*/, string /*number*/> phonebook;
  phonebook["Meyers"] = "03024233";
  string number = phonebook.find("Meyers");
}

Answer 3

contact ContactNumber [GROUPSIZE];

В текущей реализации я думаю, что она не удалит запись из массива объектов. У вас есть GROUPSIZE количество contact объектов (т. Е. Массив объектов). Массив нельзя изменить по размеру. Если вам нужно удалить элемент из массива, сначала вам нужно иметь ссылку типа contact.

contact *ContactNumber ;  // new it with GROUPSIZE number of objects in PhoneBook constructor.

Если вам нужно удалить элемент в позиции m, вам необходимо выполнить GROUPSIZE - m переводы с использованием правила копирования-обмена. то есть, m+1 объект позиции должен быть в m позиции, m+2 объект позиции должен быть в m+1 позиции, .... это правильный способ реализации, я думаю. И помните delete[] ContactNumber; в деструкторе PhoneBook, чтобы избежать утечек памяти.

Чтобы избежать боли, используйте вместо этого std::vector.