Телефонная книга, не работает редактирование записи по номеру

@Le_Mony · Регистрация: 17.11.2014

Author24 — интернет-сервис помощи студентам

Ребята, понимаю, как всех уже достало одно и то же, но я в упор не вижу в чем ошибка.
Не работает функция EditRecord, которая должна редактировать запись по номеру (case 8 в main). Без нее все идеально работает. Подскажите, с чем я намудрила?

C

#include <iostream>
#include <string.h>
using namespace std;
 
struct Record
{
    char Name[20];
    char LastName[20];
    unsigned int PhoneNumber;
    char Address[50];
};
 
Record *AddStruct(Record* Obj, const int size);
void AddRecord(Record* Obj, const int size);// - добавление записи в книгу
void PrintBook(Record* Obj, const int size);// - печать всей книги
void PrintRecord(Record* Obj, int num);// - печать одной записи
void PrintMenu(); //- печать меню
int FindRecordByName(Record* Obj, const int size);// - поиск записи по имени, возвращает номер элемента
int FindRecordByLastName(Record* Obj, const int size);// - поиск записи по фамилии, возвращает номер элемента
int FindRecordByNumber(Record* Obj, const int size);// - поиск записи по номеру телефона, возвращает номер элемента
Record *AfterDel(Record* Obj, const int size, int num);// - удаление записи по номеру элемента, возвращает массив структур
void EditRecord(Record* Obj, const int size, int num);// - редактирование записи по номеру
 
void main()
{
    Record* AllRecords = NULL;
    int NumRec = 0;
    bool AddMore = 0;
    do
    {
        AllRecords = AddStruct(AllRecords, NumRec);
        AddRecord(AllRecords, NumRec);
        NumRec++;
        cout << "Add more records to the Address Book?\n\tPress \"1\" to add more,\n\tPress \"0\" to finish;\n";
        cin >> AddMore;
    } while (AddMore != 0);
    system("cls");
    PrintBook(AllRecords, NumRec);
    PrintMenu();
 
    int Menu;
    do
    {
        cin >> Menu;
        cout << endl;
        int Num = -1;
        switch (Menu)
        {
        default:
            system("cls");
            cout << "Option not found! Try again!\n\n";
            PrintMenu();
            break;
        case 1://Печать всей книги
        {
                  system("cls");
                  PrintBook(AllRecords, NumRec);
                  PrintMenu();
        }
            break;
 
        case 2://Поиск по имени
        {
                  system("cls");
                  cout << "\n\tSEARCH BY NAME:\n";
                  int sRes = FindRecordByName(AllRecords, NumRec);
                  if (sRes >= 0)
                      PrintRecord(AllRecords, sRes);
                  else
                      cout << "Record not Found!\n\n";
                  PrintMenu();
        }
            break;
 
        case 3://Поиск по фамилии
        {
                  system("cls");
                  cout << "\n\tSEARCH BY LAST NAME:\n";
                  int sRes = FindRecordByLastName(AllRecords, NumRec);
                  if (sRes >= 0)
                      PrintRecord(AllRecords, sRes);
                  else
                      cout << "Record not Found!\n\n";
                  PrintMenu();
        }
            break;
 
        case 4://Поиск по номеру телефона
        {
                  system("cls");
                  cout << "\n\tSEARCH BY PHONE NUMBER:\n";
                  int sRes = FindRecordByNumber(AllRecords, NumRec);
                  if (sRes >= 0)
                      PrintRecord(AllRecords, sRes);
                  else
                      cout << "Record not Found!\n\n";
                  PrintMenu();
        }
            break;
 
        case 5: //Сортировка по Имени
            system("cls");
            cout << "sort by name\n";
            PrintMenu();
            break;
 
        case 6://Сортировка по фамилии
            system("cls");
            cout << "sort byLastName\n";
            PrintMenu();
            break;
 
        case 7://Удаление записи по имени
        {
            system("cls");
            cout << "\n\tSearch for record to delete:\n";
            int sRes = FindRecordByName(AllRecords, NumRec);
            if (sRes >= 0)
            {
                 PrintRecord(AllRecords, sRes);
                 AllRecords = AfterDel(AllRecords, NumRec, sRes);
                 NumRec--;
                 cout << "Record has been deleted!\n\n\n";
            }
            else
                cout << "Record not Found!\n\n";
        }
            PrintMenu();
            break;
 
        case 8://Редактирование записи
        {
                   system("cls");
                   cout << "\n\tSEARCH BY NAME:\n";
                   int sRes = FindRecordByName(AllRecords, NumRec);
 
                   if (sRes >= 0)
                   {
                       PrintRecord(AllRecords, sRes);
                       EditRecord(AllRecords, NumRec, sRes); //Ломает все!
                       PrintRecord(AllRecords, sRes);
                   }
                   else
                       cout << "Record not Found!\n\n";
                   PrintMenu();
        }
        break;
        case 9:
        {
                  bool AddMore = 0;
                  do
                  {
                      AllRecords = AddStruct(AllRecords, NumRec);
                      AddRecord(AllRecords, NumRec);
                      NumRec++;
                      cout << "\n\tPress \"1\" to add more records,\n\tPress \"0\" to finish;\n";
                      cin >> AddMore;
                  } while (AddMore != 0);
                  system("cls");
                  PrintBook(AllRecords, NumRec);
                  PrintMenu();
        }
                break;
        case 10:
            system("cls");
            cout << "\n\t\t\tThank You!\n\n\n";
            break;
 
        
        }
    } while (Menu != 10);
}
 
Record *AddStruct(Record* Obj, const int size)
{
    if (size == 0)
    {
        Obj = new Record[size + 1];
    }
    else
    {
        Record* tempObj = new Record[size + 1];
        for (int i = 0; i < size; i++)
        {
            tempObj[i] = Obj[i];
        }
        delete[] Obj;
        Obj = tempObj;
    }
    return Obj;
}
 
void AddRecord(Record* Obj, int size)
{
    char name[20];
    char lastName[20];
    unsigned int phoneNumber;
    char address[50];
    cout << "New Record \n~~~~~~~~~~~~~" << endl;
    cout << "Name:\t\t";
    cin >> name;
    cout << "Last Name:\t";
    cin >> lastName;
    cout << "Phone Number:\t";
    cin >> phoneNumber;
    cout << "Address:\t";
    cin >> address;
    strcpy_s(Obj[size].Name, 19, name);
    strcpy_s(Obj[size].LastName, 19, lastName);
    Obj[size].PhoneNumber = phoneNumber;
    strcpy_s(Obj[size].Address, 19, address);
    cout << "_________________________________________________________" << endl;
}
void PrintBook(Record* Obj, int size)
{
 
    cout << "\n\tYOUR ADDRESS BOOK\n\t***************" << endl;
    if (size==0)
        cout << "Nothing to Show!";
    else
    {
        for (int i = 0; i < size; i++)
        {
            cout << "Record  " << i + 1 << ':' << endl;
            cout << "\tName:\t\t" << Obj[i].Name << endl;
            cout << "\tLast Name:\t" << Obj[i].LastName << endl;
            cout << "\tPhone Number:\t" << Obj[i].PhoneNumber << endl;
            cout << "\tAddress:\t" << Obj[i].Address << endl;
            cout << "\t_________________________________________________________" << endl << endl;
        }
    }
}
 
void PrintRecord(Record* Obj, int num)
{
    cout << "Record you have searched for:" << endl;
    cout << "\tName:\t\t" << Obj[num].Name << endl;
    cout << "\tLast Name:\t" << Obj[num].LastName << endl;
    cout << "\tPhone Number:\t" << Obj[num].PhoneNumber << endl;
    cout << "\tAddress:\t" << Obj[num].Address << endl;
    cout << "\t_________________________________________________________" << endl << endl;
}
void PrintMenu()
{
    cout << "\tMENU\n\t~~~~~" << endl;
    cout << "1\tTo display all book - \tpress 1;\n";
    cout << "2\tTo find record by Name - \tpress 2;\n";
    cout << "3\tTo find record by Last Name - \tpress 3;\n";
    cout << "4\tTo find record by Phone Number - \tpress 4;\n";
    cout << "5\tTo sort records by Name and display - \tpress 5;\n";
    cout << "6\tTo sort records by Last Name and display - \tpress 6;\n";
    cout << "7\tTo find a record by Name and delete it - \tpress 7;\n";
    cout << "8\tTo find a record by Name and to edit it - \tpress 8;\n";
    cout << "9\tTo add more records to AddressBook - \tpress 9;\n";
    cout << "10\tTo finish work with AddressBook - \tpress 10;\n\n\n";
}
int FindRecordByName(Record* Obj, const int size)
{
    cout << "Enter the Name for search:\n";
    char sName[20];
    int NumFound = -1;
    cin >> sName;
    for (int i = 0; i < size; i++)
    {
        if (strcmp(Obj[i].Name, sName) == 0)
        {
            NumFound = i;
            break;
        }
        else continue;
    }
    return NumFound;
}
int FindRecordByLastName(Record* Obj, const int size)
{
    cout << "Enter the Last Name for search:\n";
    char lName[20];
    int NumFound = -1;
    cin >> lName;
    for (int i = 0; i < size; i++)
    {
        if (strcmp(Obj[i].LastName, lName) == 0)
        {
            NumFound = i;
            break;
        }
        else continue;
    }
    return NumFound;
}
int FindRecordByNumber(Record* Obj, const int size)
{
    cout << "Enter the Phone Number for search:\n";
    unsigned int PhNum;
    int NumFound = -1;
    cin >> PhNum;
    for (int i = 0; i < size; i++)
    {
        if (Obj[i].PhoneNumber == PhNum)
        {
            NumFound = i;
            break;
        }
        else continue;
    }
    return NumFound;
}
Record *AfterDel(Record* Obj, const int size, int num)
{
        Record* tempObj = new Record[size -1];
        for (int i = 0; i < num; i++)
        {
            tempObj[i] = Obj[i];
        }
        for (int i = num; i < size-1; i++)
        {
            tempObj[i] = Obj[i+1];
        }
        delete[] Obj;
        Obj = tempObj;
    return Obj;
};
void EditRecord(Record* Obj, const int size, int num)
{
    char name[20]="0";
    char lastName[20] = "0";
    unsigned int phoneNumber;
    char address[50] = "0";
    cout << "Insert New Name if you want to change it (or insert '0'):\n";
    cin>>name;
    if (name != "0")
    
    {
        strcpy_s(Obj[size].Name, 19, name);
    }
    cout << "\nInsert New Last Name if you want to change it (or insert '0'):\n";
    cin >> lastName;
    if (lastName !="0" )
    {
        strcpy_s(Obj[size].LastName, 19, lastName);
    }
    cout << "\nInsert New Phone Number if you want to change it (or insert '0'):\n";
    cin >> phoneNumber;
    if (phoneNumber !=0)
    {
        Obj[size].PhoneNumber = phoneNumber;
    }
    cout << "\nInsert New Address if you want to change it (or insert '0'):\n";
    cin >> address;
    if (address !="0")
    {
        strcpy_s(Obj[size].Address, 19, address);
    }
}

@Max Dark · 17.11.2014, 23:15

строки в Си сравниваются функцией strcmp

Добавлено через 1 минуту

Сообщение от Le_Mony

Obj[size]

Вы наверно хотели

C

Obj[num]

Иначе происходит выход за границы массива

Добавлено через 15 минут

Сообщение от Le_Mony

cout <<

в Си нет cout и др, они только в С++

@Le_Mony · 18.11.2014, 07:47 **[ТС]**

Сообщение от Cra3y

Вы наверно хотели

C

Obj[num]

Спасибо огромное! Конечно же Obj[num]. И как это я прохлопала?

Сообщение от Cra3y

в Си нет cout и др, они только в С++

В месте, где я учу программирование, нам всегда говорят "вы изучаете не Си, а основы процедурного программирования!" и поэтому начиная с первого занятия мы использовали cout и все прочее практически без оговорок и у меня как-то размыто понимание границы. Наверное, для общекультурного уровня нужно сесть, и разобраться, где Си, а где С++

Еще раз большое вам спасибо за уделенное время! Наверное, мой код - не самый приятный для прочтения

так что спасибо вдвойне.

	18.11.2014, 07:47